[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <aRF_APEKXpkOc7z6@kernel.org>
Date: Mon, 10 Nov 2025 13:58:24 +0800
From: Troy Mitchell <troy.mitchell@...ux.dev>
To: Encrow Thorne <jyc0019@...il.com>,
Philipp Zabel <p.zabel@...gutronix.de>
Cc: Troy Mitchell <troy.mitchell@...ux.dev>, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2] reset: fix BIT macro reference
On Thu, Nov 06, 2025 at 01:00:06AM +0800, Encrow Thorne wrote:
> On Wed, Nov 05, 2025 at 04:51:33PM +0100, Philipp Zabel wrote:
> > On Mi, 2025-11-05 at 23:40 +0800, Encrow Thorne wrote:
> > > RESET_CONTROL_FLAGS_BIT_* macros use BIT(), but reset.h does not
> > > include bits.h or bitops.h. This causes compilation errors when
> > ^^^^^^^^^^^
> > Troy suggested "or bitops.h" can be dropped.
> >
> > > including reset.h standalone.
> > >
> > > Include bits.h to make reset.h self-contained.
> > >
> > > Suggested-by: Troy Mitchell <troy.mitchell@...ux.dev>
> > > Reviewed-by: Troy Mitchell <troy.mitchell@...ux.dev>
> > > Signed-off-by: Encrow Thorne <jyc0019@...il.com>
> > > ---
> > > Changes in v2:
> > > - EDITME: describe what is new in this series revision.
> > > - EDITME: use bulletpoints and terse descriptions.
> >
> > These should be replaced with a short description of the changes
> > relative to v1.
> >
> >
> Sorry for the oversight on my side regarding the “Changes in v2” placeholders.
> Do you recommend I resend a corrected v2 now, or wait and include it in a v3?
> Thanks for your guidance.
Send v3 when you are ready.
- Troy
Powered by blists - more mailing lists