On Wed, 4 May 2022 18:36:33 +0000
Dmitry Rokosov <DDRokosov@xxxxxxxxxxxxxx> wrote:
> Hello Rob,
>
> I've missed one note below, please check if possible.
>
> On Tue, Apr 26, 2022 at 03:31:05PM -0500, Rob Herring wrote:
> > On Tue, Apr 19, 2022 at 03:45:58PM +0000, Rokosov Dmitry Dmitrievich wrote:
> > > + "#address-cells":
> > > + const: 1
> > > +
> > > + "#size-cells":
> > > + const: 0
> >
> > These apply to 'reg' in a child node, but you don't have child nodes so
> > drop them.
> >
>
> I'm afraid, I made a mistake in the previous reply. Here I applied
> address-cells and size-cells to show which type accelerometer's reg
> property has, like this one:
>
> i2c {
> #address-cells = <1>;
> #size-cells = <0>;
>
> accelerometer@62 {
> compatible = "memsensing,msa311";
> reg = <0x62>;
>
> Above accelerometer@62 is child node for i2c bus node, so address and
> size cells mean accelerometer reg format. Am I missing something?
They are characteristics of the i2c master and hence
are listed in those binding docs, not the one for the accelerometer.
Thanks,
Jonathan
>
