Hi Sakari, Thanks for your review comments. Will address it in next patch version.
--- ^Divagar >-----Original Message----- >From: Sakari Ailus [mailto:sakari.ai...@iki.fi] >Sent: Tuesday, August 22, 2017 2:41 PM >To: Mohandass, Divagar <divagar.mohand...@intel.com> >Cc: robh...@kernel.org; mark.rutl...@arm.com; w...@the-dreams.de; >devicet...@vger.kernel.org; linux-...@vger.kernel.org; linux- >ker...@vger.kernel.org; Mani, Rajmohan <rajmohan.m...@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.mohand...@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.ai...@iki.fi
