Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1303903 > unrolled thread

[PATCH 0/2] dt-bindings: refactor i2c eeprom bindings, step 1

Started byWolfram Sang <wsa@the-dreams.de>
First post2016-01-07 22:00 +0100
Last post2016-01-10 09:50 +0100
Articles 4 — 2 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH 0/2] dt-bindings: refactor i2c eeprom bindings, step 1 Wolfram Sang <wsa@the-dreams.de> - 2016-01-07 22:00 +0100
    [PATCH 1/2] dt-bindings: move I2C eeprom descriptions to the proper file Wolfram Sang <wsa@the-dreams.de> - 2016-01-07 22:00 +0100
      Re: [PATCH 1/2] dt-bindings: move I2C eeprom descriptions to the  proper file Rob Herring <robh@kernel.org> - 2016-01-08 00:10 +0100
        Re: [PATCH 1/2] dt-bindings: move I2C eeprom descriptions to the  proper file Wolfram Sang <wsa@the-dreams.de> - 2016-01-10 09:50 +0100

#1303903 — [PATCH 0/2] dt-bindings: refactor i2c eeprom bindings, step 1

FromWolfram Sang <wsa@the-dreams.de>
Date2016-01-07 22:00 +0100
Subject[PATCH 0/2] dt-bindings: refactor i2c eeprom bindings, step 1
Message-ID<qOpP6-4VR-33@gated-at.bofh.it>
A first step to clean up the bindings for i2c eeproms. Please see the patch
descriptions for further details.

I suggest I take these via my i2c tree. D'accord?

Thanks,

   Wolfram


Wolfram Sang (2):
  dt-bindings: move I2C eeprom descriptions to the proper file
  dt-bindings: i2c: eeprom: add another EEPROM device

 Documentation/devicetree/bindings/eeprom/eeprom.txt | 21 ++++++++++++++++-----
 .../devicetree/bindings/i2c/trivial-devices.txt     | 13 -------------
 2 files changed, 16 insertions(+), 18 deletions(-)

-- 
2.1.4

[toc] | [next] | [standalone]


#1303904 — [PATCH 1/2] dt-bindings: move I2C eeprom descriptions to the proper file

FromWolfram Sang <wsa@the-dreams.de>
Date2016-01-07 22:00 +0100
Subject[PATCH 1/2] dt-bindings: move I2C eeprom descriptions to the proper file
Message-ID<qOpP6-4VR-41@gated-at.bofh.it>
In reply to#1303903
From: Wolfram Sang <wsa+renesas@sang-engineering.com>

EEPROMs can have additional properties, so they are not suitable for
trivial-devices.txt. Move most bindings to the designated eeprom.txt.
Add the missing "atmel,24c08" while doing that. Note that the remaining
ones in trivial-devices need to be dealt with seperately because of
improper manufacturer names.

Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
---
 Documentation/devicetree/bindings/eeprom/eeprom.txt   | 19 ++++++++++++++-----
 .../devicetree/bindings/i2c/trivial-devices.txt       | 13 -------------
 2 files changed, 14 insertions(+), 18 deletions(-)

diff --git a/Documentation/devicetree/bindings/eeprom/eeprom.txt b/Documentation/devicetree/bindings/eeprom/eeprom.txt
index 4342c10de1bf8d..acb600399c3d94 100644
--- a/Documentation/devicetree/bindings/eeprom/eeprom.txt
+++ b/Documentation/devicetree/bindings/eeprom/eeprom.txt
@@ -2,11 +2,20 @@ EEPROMs (I2C)
 
 Required properties:
 
-  - compatible : should be "<manufacturer>,<type>"
-		 If there is no specific driver for <manufacturer>, a generic
-		 driver based on <type> is selected. Possible types are:
-		 24c00, 24c01, 24c02, 24c04, 24c08, 24c16, 24c32, 24c64,
-		 24c128, 24c256, 24c512, 24c1024, spd
+  - compatible : should be "<manufacturer>,<type>", like these:
+
+	"atmel,24c00", "atmel,24c01", "atmel,24c02", "atmel,24c04",
+	"atmel,24c08", "atmel,24c16", "atmel,24c32", "atmel,24c64",
+	"atmel,24c128", "atmel,24c256", "atmel,24c512", "atmel,24c1024"
+
+	"catalyst,24c32"
+
+	"ramtron,24c64"
+
+	 If there is no specific driver for <manufacturer>, a generic
+	 driver based on <type> is selected. Possible types are:
+	 "24c00", "24c01", "24c02", "24c04", "24c08", "24c16", "24c32", "24c64",
+	 "24c128", "24c256", "24c512", "24c1024", "spd"
 
   - reg : the I2C address of the EEPROM
 
diff --git a/Documentation/devicetree/bindings/i2c/trivial-devices.txt b/Documentation/devicetree/bindings/i2c/trivial-devices.txt
index c50cf13c852e51..7e41304e45f81e 100644
--- a/Documentation/devicetree/bindings/i2c/trivial-devices.txt
+++ b/Documentation/devicetree/bindings/i2c/trivial-devices.txt
@@ -21,21 +21,9 @@ adi,adt7490		+/-1C TDM Extended Temp Range I.C
 adi,adxl345		Three-Axis Digital Accelerometer
 adi,adxl346		Three-Axis Digital Accelerometer (backward-compatibility value "adi,adxl345" must be listed too)
 at,24c08		i2c serial eeprom  (24cxx)
-atmel,24c00		i2c serial eeprom  (24cxx)
-atmel,24c01		i2c serial eeprom  (24cxx)
-atmel,24c02		i2c serial eeprom  (24cxx)
-atmel,24c04		i2c serial eeprom  (24cxx)
-atmel,24c16		i2c serial eeprom  (24cxx)
-atmel,24c32		i2c serial eeprom  (24cxx)
-atmel,24c64		i2c serial eeprom  (24cxx)
-atmel,24c128		i2c serial eeprom  (24cxx)
-atmel,24c256		i2c serial eeprom  (24cxx)
-atmel,24c512		i2c serial eeprom  (24cxx)
-atmel,24c1024		i2c serial eeprom  (24cxx)
 atmel,at97sc3204t	i2c trusted platform module (TPM)
 capella,cm32181		CM32181: Ambient Light Sensor
 capella,cm3232		CM3232: Ambient Light Sensor
-catalyst,24c32		i2c serial eeprom
 cirrus,cs42l51		Cirrus Logic CS42L51 audio codec
 dallas,ds1307		64 x 8, Serial, I2C Real-Time Clock
 dallas,ds1338		I2C RTC with 56-Byte NV RAM
@@ -80,7 +68,6 @@ ovti,ov5642		OV5642: Color CMOS QSXGA (5-megapixel) Image Sensor with OmniBSI an
 pericom,pt7c4338	Real-time Clock Module
 plx,pex8648		48-Lane, 12-Port PCI Express Gen 2 (5.0 GT/s) Switch
 pulsedlight,lidar-lite-v2	Pulsedlight LIDAR range-finding sensor
-ramtron,24c64		i2c serial eeprom  (24cxx)
 ricoh,r2025sd		I2C bus SERIAL INTERFACE REAL-TIME CLOCK IC
 ricoh,r2221tl		I2C bus SERIAL INTERFACE REAL-TIME CLOCK IC
 ricoh,rs5c372a		I2C bus SERIAL INTERFACE REAL-TIME CLOCK IC
-- 
2.1.4

[toc] | [prev] | [next] | [standalone]


#1303993 — Re: [PATCH 1/2] dt-bindings: move I2C eeprom descriptions to the proper file

FromRob Herring <robh@kernel.org>
Date2016-01-08 00:10 +0100
SubjectRe: [PATCH 1/2] dt-bindings: move I2C eeprom descriptions to the proper file
Message-ID<qOrQT-6ud-29@gated-at.bofh.it>
In reply to#1303904
On Thu, Jan 07, 2016 at 09:50:59PM +0100, Wolfram Sang wrote:
> From: Wolfram Sang <wsa+renesas@sang-engineering.com>
> 
> EEPROMs can have additional properties, so they are not suitable for
> trivial-devices.txt. Move most bindings to the designated eeprom.txt.
> Add the missing "atmel,24c08" while doing that. Note that the remaining
> ones in trivial-devices need to be dealt with seperately because of

s/seperately/separately/

> improper manufacturer names.
> 
> Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
> ---
>  Documentation/devicetree/bindings/eeprom/eeprom.txt   | 19 ++++++++++++++-----
>  .../devicetree/bindings/i2c/trivial-devices.txt       | 13 -------------
>  2 files changed, 14 insertions(+), 18 deletions(-)

Acked-by: Rob Herring <robh@kernel.org>

Fine to take these thru your tree.

Rob

[toc] | [prev] | [next] | [standalone]


#1305514 — Re: [PATCH 1/2] dt-bindings: move I2C eeprom descriptions to the proper file

FromWolfram Sang <wsa@the-dreams.de>
Date2016-01-10 09:50 +0100
SubjectRe: [PATCH 1/2] dt-bindings: move I2C eeprom descriptions to the proper file
Message-ID<qPjRg-1N4-3@gated-at.bofh.it>
In reply to#1303993

[Multipart message — attachments visible in raw view] — view raw

On Thu, Jan 07, 2016 at 05:00:12PM -0600, Rob Herring wrote:
> On Thu, Jan 07, 2016 at 09:50:59PM +0100, Wolfram Sang wrote:
> > From: Wolfram Sang <wsa+renesas@sang-engineering.com>
> > 
> > EEPROMs can have additional properties, so they are not suitable for
> > trivial-devices.txt. Move most bindings to the designated eeprom.txt.
> > Add the missing "atmel,24c08" while doing that. Note that the remaining
> > ones in trivial-devices need to be dealt with seperately because of
> 
> s/seperately/separately/
> 
> > improper manufacturer names.
> > 
> > Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
> > ---
> >  Documentation/devicetree/bindings/eeprom/eeprom.txt   | 19 ++++++++++++++-----
> >  .../devicetree/bindings/i2c/trivial-devices.txt       | 13 -------------
> >  2 files changed, 14 insertions(+), 18 deletions(-)
> 
> Acked-by: Rob Herring <robh@kernel.org>
> 
> Fine to take these thru your tree.

Fixed the typo and applied both to for-next, thanks for the acks, Rob!


[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web