On Fri, May 19, 2023 at 04:49:44PM +0200, Geert Uytterhoeven wrote: > On Fri, May 19, 2023 at 4:39 PM Biju Das <biju.das.jz@xxxxxxxxxxxxxx> wrote: > > > -----Original Message----- > > > From: Conor Dooley <conor.dooley@xxxxxxxxxxxxx> > > > On Fri, May 19, 2023 at 06:53:03AM +0000, Biju Das wrote: > > > > > Subject: Re: [PATCH v4 09/11] regulator: dt-bindings: Add Renesas > > > > > > +required: > > > > > > + - compatible > > > > > > + - reg > > > > > > + - reg-names > > > > > > > > > > Out of curiosity as much as anything else, why do you need reg-names > > > > > if the two registers are always required? > > OK, will drop reg-names from required property. > > Please don't, as i2c_new_ancillary_device() does rely on the name > to set the address when overriding from the default. That looks like my answer! Thanks Geert & sorry for the noise here. Modulo Geert's requested change, I think I owe you a Reviewed-by: Conor Dooley <conor.dooley@xxxxxxxxxxxxx> Biju. Thanks, Conor.
Attachment:
signature.asc
Description: PGP signature
