Hi Geert,
Am Dienstag, 3. Januar 2023, 17:08:36 CET schrieb Geert Uytterhoeven:
> Hi Marek,
>
> On Tue, Jan 3, 2023 at 4:45 PM Marek Vasut <marex@xxxxxxx> wrote:
> > On 1/3/23 13:31, Alexander Stein wrote:
> > > This is in preparation to support additional devices which have
> > > different
> > > IDs as well as a slightly different register layout.
> > >
> > > Signed-off-by: Alexander Stein <alexander.stein@xxxxxxxxxxxxxxx>
> > > ---
> > >
> > > drivers/clk/clk-renesas-pcie.c | 24 ++++++++++++++++++++++++
> > > 1 file changed, 24 insertions(+)
> > >
> > > diff --git a/drivers/clk/clk-renesas-pcie.c
> > > b/drivers/clk/clk-renesas-pcie.c index e6247141d0c0..0076ed8f11b0
> > > 100644
> > > --- a/drivers/clk/clk-renesas-pcie.c
> > > +++ b/drivers/clk/clk-renesas-pcie.c
> > > @@ -45,6 +45,13 @@
> > >
> > > #define RS9_REG_DID 0x6
> > > #define RS9_REG_BCP 0x7
> > >
> > > +#define RS9_REG_VID_IDT 0x01
> > > +
> > > +#define RS9_REG_DID_TYPE_FGV (0x0 <<
> > > RS9_REG_DID_TYPE_SHIFT) +#define RS9_REG_DID_TYPE_DBV
> > > (0x1 << RS9_REG_DID_TYPE_SHIFT) +#define RS9_REG_DID_TYPE_DMV
> > > (0x2 << RS9_REG_DID_TYPE_SHIFT)>
> > I'm not entirely sure whether this shouldn't be using the BIT() macro,
> > what do you think ?
>
> They're not one-bit values (which bit does RS9_REG_DID_TYPE_FGV set? ;-),
> but values in a bitfield.
>
> So using FIELD_PREP() and friends would make more sense to me.
FIELD_PREP() seems pretty nice, but unless I miss something it can't be used
for initializing struct members. See renesas_9fgv0241_info.
Best regards,
Alexander
> Gr{oetje,eeting}s,
>
> Geert
>
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 --
> geert@xxxxxxxxxxxxxx
>
> In personal conversations with technical people, I call myself a hacker. But
> when I'm talking to journalists I just say "programmer" or something like
> that. -- Linus Torvalds
