Message ID | 1502736119-32084-2-git-send-email-divagar.mohandass@intel.com |
---|---|
State | Superseded |
Headers | show |
On Mon, Aug 14, 2017 at 1:41 PM, Divagar Mohandass <divagar.mohandass@intel.com> wrote: > This adds eeprom "size" as optional property for i2c eeproms. > > "size" should be mentioned in byte and it should refer > to the eeprom size. This will be read by the driver and > used to calculating the number of bytes in read/write calls. > > Signed-off-by: Divagar Mohandass <divagar.mohandass@intel.com> > --- > Documentation/devicetree/bindings/eeprom/eeprom.txt | 2 ++ > 1 file changed, 2 insertions(+) Acked-by: Rob Herring <robh@kernel.org>
Hi Divagar, On Tue, Aug 15, 2017 at 12:11:57AM +0530, Divagar Mohandass wrote: > This adds eeprom "size" as optional property for i2c eeproms. > > "size" should be mentioned in byte and it should refer > to the eeprom size. This will be read by the driver and > used to calculating the number of bytes in read/write calls. Instead of the three lines above, how about: The "size" property allows explicitly specifying the size of the EEPROM chip in bytes. > > Signed-off-by: Divagar Mohandass <divagar.mohandass@intel.com> > --- > Documentation/devicetree/bindings/eeprom/eeprom.txt | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/Documentation/devicetree/bindings/eeprom/eeprom.txt b/Documentation/devicetree/bindings/eeprom/eeprom.txt > index 5696eb5..1436569 100644 > --- a/Documentation/devicetree/bindings/eeprom/eeprom.txt > +++ b/Documentation/devicetree/bindings/eeprom/eeprom.txt > @@ -32,6 +32,8 @@ Optional properties: > > - read-only: this parameterless property disables writes to the eeprom > > + - size: total eeprom size in bytes > + > Example: > > eeprom@52 { > -- > 1.9.1 >
Hi Sakari, Thanks for your review comments. Will address it in next patch version. --- ^Divagar >-----Original Message----- >From: Sakari Ailus [mailto:sakari.ailus@iki.fi] >Sent: Tuesday, August 22, 2017 2:41 PM >To: Mohandass, Divagar <divagar.mohandass@intel.com> >Cc: robh+dt@kernel.org; mark.rutland@arm.com; wsa@the-dreams.de; >devicetree@vger.kernel.org; linux-i2c@vger.kernel.org; linux- >kernel@vger.kernel.org; Mani, Rajmohan <rajmohan.mani@intel.com> >Subject: Re: [PATCH v2 1/3] dt-bindings: add eeprom "size" property > >Hi Divagar, > >On Tue, Aug 15, 2017 at 12:11:57AM +0530, Divagar Mohandass wrote: >> This adds eeprom "size" as optional property for i2c eeproms. >> >> "size" should be mentioned in byte and it should refer to the eeprom >> size. This will be read by the driver and used to calculating the >> number of bytes in read/write calls. > >Instead of the three lines above, how about: > >The "size" property allows explicitly specifying the size of the EEPROM chip in >bytes. > >> >> Signed-off-by: Divagar Mohandass <divagar.mohandass@intel.com> >> --- >> Documentation/devicetree/bindings/eeprom/eeprom.txt | 2 ++ >> 1 file changed, 2 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/eeprom/eeprom.txt >> b/Documentation/devicetree/bindings/eeprom/eeprom.txt >> index 5696eb5..1436569 100644 >> --- a/Documentation/devicetree/bindings/eeprom/eeprom.txt >> +++ b/Documentation/devicetree/bindings/eeprom/eeprom.txt >> @@ -32,6 +32,8 @@ Optional properties: >> >> - read-only: this parameterless property disables writes to the >> eeprom >> >> + - size: total eeprom size in bytes >> + >> Example: >> >> eeprom@52 { >> -- >> 1.9.1 >> > >-- >Regards, > >Sakari Ailus >e-mail: sakari.ailus@iki.fi
diff --git a/Documentation/devicetree/bindings/eeprom/eeprom.txt b/Documentation/devicetree/bindings/eeprom/eeprom.txt index 5696eb5..1436569 100644 --- a/Documentation/devicetree/bindings/eeprom/eeprom.txt +++ b/Documentation/devicetree/bindings/eeprom/eeprom.txt @@ -32,6 +32,8 @@ Optional properties: - read-only: this parameterless property disables writes to the eeprom + - size: total eeprom size in bytes + Example: eeprom@52 {
This adds eeprom "size" as optional property for i2c eeproms. "size" should be mentioned in byte and it should refer to the eeprom size. This will be read by the driver and used to calculating the number of bytes in read/write calls. Signed-off-by: Divagar Mohandass <divagar.mohandass@intel.com> --- Documentation/devicetree/bindings/eeprom/eeprom.txt | 2 ++ 1 file changed, 2 insertions(+)