* [RESEND PATCH v5 00/16] eeprom: at24: Add OF device ID table
@ 2017-06-15 18:54 ` Javier Martinez Canillas
0 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-06-15 18:54 UTC (permalink / raw)
To: linux-arm-kernel
Hello Wolfram,
This series is a follow-up to patch [0] that added an OF device ID table
to the at24 EEPROM driver. As you suggested [1], this version instead of
adding entries for every used <vendor,device> tuple, only adds a single
entry for each chip type using the "atmel" vendor as a generic fallback.
The first patch documents in the DT binding what's the correct vendor to
use and what are the ones that are being deprecated. The second one adds
the OF device ID table for the at24 driver and the next patches use this
vendor in the compatible string to each DTS that defines a compatible I2C
EEPROM device node.
Patches can be applied independently since the DTS changes without driver
changes are no-op and the OF table won't be used without the DTS changes.
[0]: https://lkml.org/lkml/2017/3/14/589
[1]: https://lkml.org/lkml/2017/3/15/99
Best regards,
Javier
Changes in v5:
- Only deprecate the atmel variants at25 and at (Geert Uytterhoeven).
- Only replace atmel variant but keep other EEPROM vendors (Geert Uytterhoeven).
- Only replace atmel variant but keep other EEPROM vendors (Geert Uytterhoeven).
- Only replace atmel variant but keep other EEPROM vendors (Geert Uytterhoeven).
- Only replace atmel variant but keep other EEPROM vendors (Geert Uytterhoeven).
- Only replace atmel variant but keep other EEPROM vendors (Geert Uytterhoeven).
- Only replace atmel variant but keep other EEPROM vendors (Geert Uytterhoeven).
- Only replace atmel variant but keep other EEPROM vendors (Geert Uytterhoeven).
- Only replace atmel variant but keep other EEPROM vendors (Geert Uytterhoeven).
Changes in v4:
- Document the manufacturers that have been deprecated (Rob Herring).
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
Changes in v3:
- Fix wrong .data values for "atmel,24c02" and "atmel,24c64" entries.
- Add Geert Uytterhoeven reviewed-by tag.
- Add Geert Uytterhoeven reviewed-by tag.
Changes in v2:
- Only add a single OF device ID entry for each device type (Wolfram Sang).
Javier Martinez Canillas (16):
dt-bindings: i2c: eeprom: Document vendor to be used and deprecated
ones
eeprom: at24: Add OF device ID table
ARM: dts: efm32: Add generic compatible string for I2C EEPROM
ARM: dts: keystone: Add generic compatible string for I2C EEPROM
ARM: dts: lpc18xx: Add generic compatible string for I2C EEPROM
ARM: dts: r7s72100: Add generic compatible string for I2C EEPROM
ARM: dts: koelsch: Add generic compatible string for I2C EEPROM
ARM: dts: socfpga: Add generic compatible string for I2C EEPROM
ARM: dts: uniphier: Add generic compatible string for I2C EEPROM
ARM: dts: zynq: Add generic compatible string for I2C EEPROM
arm64: zynqmp: Add generic compatible string for I2C EEPROM
powerpc/5200: Add generic compatible string for I2C EEPROM
powerpc/fsl: Add generic compatible string for I2C EEPROM
powerpc/512x: Add generic compatible string for I2C EEPROM
powerpc/83xx: Add generic compatible string for I2C EEPROM
powerpc/44x: Add generic compatible string for I2C EEPROM
.../devicetree/bindings/eeprom/eeprom.txt | 6 +-
arch/arm/boot/dts/efm32gg-dk3750.dts | 2 +-
arch/arm/boot/dts/keystone-k2e-evm.dts | 2 +-
arch/arm/boot/dts/keystone-k2hk-evm.dts | 2 +-
arch/arm/boot/dts/keystone-k2l-evm.dts | 2 +-
arch/arm/boot/dts/lpc4337-ciaa.dts | 6 +-
arch/arm/boot/dts/lpc4350-hitex-eval.dts | 2 +-
arch/arm/boot/dts/lpc4357-ea4357-devkit.dts | 2 +-
arch/arm/boot/dts/r7s72100-genmai.dts | 2 +-
arch/arm/boot/dts/r8a7791-koelsch.dts | 2 +-
arch/arm/boot/dts/socfpga_cyclone5_vining_fpga.dts | 2 +-
arch/arm/boot/dts/uniphier-pro4-ace.dts | 2 +-
arch/arm/boot/dts/uniphier-pro4-sanji.dts | 2 +-
arch/arm/boot/dts/uniphier-pxs2-gentil.dts | 2 +-
arch/arm/boot/dts/zynq-zc702.dts | 2 +-
arch/arm/boot/dts/zynq-zc706.dts | 2 +-
arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts | 4 +-
arch/powerpc/boot/dts/digsy_mtc.dts | 2 +-
arch/powerpc/boot/dts/fsl/b4qds.dtsi | 8 +--
arch/powerpc/boot/dts/fsl/c293pcie.dts | 2 +-
arch/powerpc/boot/dts/fsl/p1010rdb.dtsi | 2 +-
arch/powerpc/boot/dts/fsl/p1023rdb.dts | 2 +-
arch/powerpc/boot/dts/fsl/p2041rdb.dts | 4 +-
arch/powerpc/boot/dts/fsl/p3041ds.dts | 4 +-
arch/powerpc/boot/dts/fsl/p4080ds.dts | 4 +-
arch/powerpc/boot/dts/fsl/p5020ds.dts | 4 +-
arch/powerpc/boot/dts/fsl/p5040ds.dts | 4 +-
arch/powerpc/boot/dts/fsl/t208xqds.dtsi | 8 +--
arch/powerpc/boot/dts/fsl/t4240qds.dts | 12 ++--
arch/powerpc/boot/dts/fsl/t4240rdb.dts | 6 +-
arch/powerpc/boot/dts/mpc5121ads.dts | 2 +-
arch/powerpc/boot/dts/mpc8308_p1m.dts | 2 +-
arch/powerpc/boot/dts/mpc8349emitx.dts | 4 +-
arch/powerpc/boot/dts/mpc8377_rdb.dts | 2 +-
arch/powerpc/boot/dts/mpc8377_wlan.dts | 2 +-
arch/powerpc/boot/dts/mpc8378_rdb.dts | 2 +-
arch/powerpc/boot/dts/mpc8379_rdb.dts | 2 +-
arch/powerpc/boot/dts/pcm030.dts | 2 +-
arch/powerpc/boot/dts/pcm032.dts | 2 +-
arch/powerpc/boot/dts/warp.dts | 2 +-
drivers/misc/eeprom/at24.c | 65 +++++++++++++++++++++-
41 files changed, 130 insertions(+), 63 deletions(-)
--
2.9.3
^ permalink raw reply [flat|nested] 115+ messages in thread* [RESEND PATCH v5 00/16] eeprom: at24: Add OF device ID table
@ 2017-06-15 18:54 ` Javier Martinez Canillas
0 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-06-15 18:54 UTC (permalink / raw)
To: linux-kernel
Cc: Wolfram Sang, Rob Herring, Javier Martinez Canillas,
Florian Larysch, David Lechner, Rob Herring, Andy Shevchenko,
Catalin Marinas, Sören Brinkmann, Simon Horman,
Michal Simek, Dinh Nguyen, Russell King, Will Deacon, devicetree,
Sekhar Nori, Scott Wood, Benjamin Herrenschmidt,
Joachim Eastwood, Mark Rutland
Hello Wolfram,
This series is a follow-up to patch [0] that added an OF device ID table
to the at24 EEPROM driver. As you suggested [1], this version instead of
adding entries for every used <vendor,device> tuple, only adds a single
entry for each chip type using the "atmel" vendor as a generic fallback.
The first patch documents in the DT binding what's the correct vendor to
use and what are the ones that are being deprecated. The second one adds
the OF device ID table for the at24 driver and the next patches use this
vendor in the compatible string to each DTS that defines a compatible I2C
EEPROM device node.
Patches can be applied independently since the DTS changes without driver
changes are no-op and the OF table won't be used without the DTS changes.
[0]: https://lkml.org/lkml/2017/3/14/589
[1]: https://lkml.org/lkml/2017/3/15/99
Best regards,
Javier
Changes in v5:
- Only deprecate the atmel variants at25 and at (Geert Uytterhoeven).
- Only replace atmel variant but keep other EEPROM vendors (Geert Uytterhoeven).
- Only replace atmel variant but keep other EEPROM vendors (Geert Uytterhoeven).
- Only replace atmel variant but keep other EEPROM vendors (Geert Uytterhoeven).
- Only replace atmel variant but keep other EEPROM vendors (Geert Uytterhoeven).
- Only replace atmel variant but keep other EEPROM vendors (Geert Uytterhoeven).
- Only replace atmel variant but keep other EEPROM vendors (Geert Uytterhoeven).
- Only replace atmel variant but keep other EEPROM vendors (Geert Uytterhoeven).
- Only replace atmel variant but keep other EEPROM vendors (Geert Uytterhoeven).
Changes in v4:
- Document the manufacturers that have been deprecated (Rob Herring).
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
Changes in v3:
- Fix wrong .data values for "atmel,24c02" and "atmel,24c64" entries.
- Add Geert Uytterhoeven reviewed-by tag.
- Add Geert Uytterhoeven reviewed-by tag.
Changes in v2:
- Only add a single OF device ID entry for each device type (Wolfram Sang).
Javier Martinez Canillas (16):
dt-bindings: i2c: eeprom: Document vendor to be used and deprecated
ones
eeprom: at24: Add OF device ID table
ARM: dts: efm32: Add generic compatible string for I2C EEPROM
ARM: dts: keystone: Add generic compatible string for I2C EEPROM
ARM: dts: lpc18xx: Add generic compatible string for I2C EEPROM
ARM: dts: r7s72100: Add generic compatible string for I2C EEPROM
ARM: dts: koelsch: Add generic compatible string for I2C EEPROM
ARM: dts: socfpga: Add generic compatible string for I2C EEPROM
ARM: dts: uniphier: Add generic compatible string for I2C EEPROM
ARM: dts: zynq: Add generic compatible string for I2C EEPROM
arm64: zynqmp: Add generic compatible string for I2C EEPROM
powerpc/5200: Add generic compatible string for I2C EEPROM
powerpc/fsl: Add generic compatible string for I2C EEPROM
powerpc/512x: Add generic compatible string for I2C EEPROM
powerpc/83xx: Add generic compatible string for I2C EEPROM
powerpc/44x: Add generic compatible string for I2C EEPROM
.../devicetree/bindings/eeprom/eeprom.txt | 6 +-
arch/arm/boot/dts/efm32gg-dk3750.dts | 2 +-
arch/arm/boot/dts/keystone-k2e-evm.dts | 2 +-
arch/arm/boot/dts/keystone-k2hk-evm.dts | 2 +-
arch/arm/boot/dts/keystone-k2l-evm.dts | 2 +-
arch/arm/boot/dts/lpc4337-ciaa.dts | 6 +-
arch/arm/boot/dts/lpc4350-hitex-eval.dts | 2 +-
arch/arm/boot/dts/lpc4357-ea4357-devkit.dts | 2 +-
arch/arm/boot/dts/r7s72100-genmai.dts | 2 +-
arch/arm/boot/dts/r8a7791-koelsch.dts | 2 +-
arch/arm/boot/dts/socfpga_cyclone5_vining_fpga.dts | 2 +-
arch/arm/boot/dts/uniphier-pro4-ace.dts | 2 +-
arch/arm/boot/dts/uniphier-pro4-sanji.dts | 2 +-
arch/arm/boot/dts/uniphier-pxs2-gentil.dts | 2 +-
arch/arm/boot/dts/zynq-zc702.dts | 2 +-
arch/arm/boot/dts/zynq-zc706.dts | 2 +-
arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts | 4 +-
arch/powerpc/boot/dts/digsy_mtc.dts | 2 +-
arch/powerpc/boot/dts/fsl/b4qds.dtsi | 8 +--
arch/powerpc/boot/dts/fsl/c293pcie.dts | 2 +-
arch/powerpc/boot/dts/fsl/p1010rdb.dtsi | 2 +-
arch/powerpc/boot/dts/fsl/p1023rdb.dts | 2 +-
arch/powerpc/boot/dts/fsl/p2041rdb.dts | 4 +-
arch/powerpc/boot/dts/fsl/p3041ds.dts | 4 +-
arch/powerpc/boot/dts/fsl/p4080ds.dts | 4 +-
arch/powerpc/boot/dts/fsl/p5020ds.dts | 4 +-
arch/powerpc/boot/dts/fsl/p5040ds.dts | 4 +-
arch/powerpc/boot/dts/fsl/t208xqds.dtsi | 8 +--
arch/powerpc/boot/dts/fsl/t4240qds.dts | 12 ++--
arch/powerpc/boot/dts/fsl/t4240rdb.dts | 6 +-
arch/powerpc/boot/dts/mpc5121ads.dts | 2 +-
arch/powerpc/boot/dts/mpc8308_p1m.dts | 2 +-
arch/powerpc/boot/dts/mpc8349emitx.dts | 4 +-
arch/powerpc/boot/dts/mpc8377_rdb.dts | 2 +-
arch/powerpc/boot/dts/mpc8377_wlan.dts | 2 +-
arch/powerpc/boot/dts/mpc8378_rdb.dts | 2 +-
arch/powerpc/boot/dts/mpc8379_rdb.dts | 2 +-
arch/powerpc/boot/dts/pcm030.dts | 2 +-
arch/powerpc/boot/dts/pcm032.dts | 2 +-
arch/powerpc/boot/dts/warp.dts | 2 +-
drivers/misc/eeprom/at24.c | 65 +++++++++++++++++++++-
41 files changed, 130 insertions(+), 63 deletions(-)
--
2.9.3
^ permalink raw reply [flat|nested] 115+ messages in thread* [RESEND PATCH v5 01/16] dt-bindings: i2c: eeprom: Document vendor to be used and deprecated ones
2017-06-15 18:54 ` Javier Martinez Canillas
(?)
(?)
@ 2017-06-15 18:54 ` Javier Martinez Canillas
2017-06-23 18:17 ` Rob Herring
2017-08-28 16:02 ` Wolfram Sang
-1 siblings, 2 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-06-15 18:54 UTC (permalink / raw)
To: linux-kernel
Cc: Wolfram Sang, Rob Herring, Javier Martinez Canillas,
Simon Horman, Sekhar Nori, David Lechner, devicetree,
Rob Herring, Mark Rutland
The at24 driver allows to register I2C EEPROM chips using different vendor
and devices, but the I2C subsystem does not take the vendor into account
when matching using the I2C table since it only has device entries.
But when matching using an OF table, both the vendor and device has to be
taken into account so the driver defines only a set of compatible strings
using the "atmel" vendor as a generic fallback for compatible I2C devices.
Document in the Device Tree binding document that this manufacturer should
be used as the generic fallback. Also document the deprecated vendors.
Suggested-by: Wolfram Sang <wsa@the-dreams.de>
Suggested-by: Rob Herring <robh@kernel.org>
Signed-off-by: Javier Martinez Canillas <javier@dowhile0.org>
---
Changes in v5:
- Only deprecate the atmel variants at25 and at (Geert Uytterhoeven).
Changes in v4:
- Document the manufacturers that have been deprecated (Rob Herring).
Changes in v3: None
Changes in v2: None
Documentation/devicetree/bindings/eeprom/eeprom.txt | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/Documentation/devicetree/bindings/eeprom/eeprom.txt b/Documentation/devicetree/bindings/eeprom/eeprom.txt
index 5696eb508e95..afc04589eadf 100644
--- a/Documentation/devicetree/bindings/eeprom/eeprom.txt
+++ b/Documentation/devicetree/bindings/eeprom/eeprom.txt
@@ -16,8 +16,12 @@ Required properties:
"renesas,r1ex24002"
+ The following manufacturers values have been deprecated:
+ "at", "at24"
+
If there is no specific driver for <manufacturer>, a generic
- driver based on <type> is selected. Possible types are:
+ device with <type> and manufacturer "atmel" should be used.
+ Possible types are:
"24c00", "24c01", "24c02", "24c04", "24c08", "24c16", "24c32", "24c64",
"24c128", "24c256", "24c512", "24c1024", "spd"
--
2.9.3
^ permalink raw reply related [flat|nested] 115+ messages in thread
* Re: [RESEND PATCH v5 01/16] dt-bindings: i2c: eeprom: Document vendor to be used and deprecated ones
@ 2017-06-23 18:17 ` Rob Herring
0 siblings, 0 replies; 115+ messages in thread
From: Rob Herring @ 2017-06-23 18:17 UTC (permalink / raw)
To: Javier Martinez Canillas
Cc: linux-kernel, Wolfram Sang, Simon Horman, Sekhar Nori,
David Lechner, devicetree, Mark Rutland
On Thu, Jun 15, 2017 at 08:54:03PM +0200, Javier Martinez Canillas wrote:
> The at24 driver allows to register I2C EEPROM chips using different vendor
> and devices, but the I2C subsystem does not take the vendor into account
> when matching using the I2C table since it only has device entries.
>
> But when matching using an OF table, both the vendor and device has to be
> taken into account so the driver defines only a set of compatible strings
> using the "atmel" vendor as a generic fallback for compatible I2C devices.
>
> Document in the Device Tree binding document that this manufacturer should
> be used as the generic fallback. Also document the deprecated vendors.
>
> Suggested-by: Wolfram Sang <wsa@the-dreams.de>
> Suggested-by: Rob Herring <robh@kernel.org>
> Signed-off-by: Javier Martinez Canillas <javier@dowhile0.org>
>
> ---
>
> Changes in v5:
> - Only deprecate the atmel variants at25 and at (Geert Uytterhoeven).
>
> Changes in v4:
> - Document the manufacturers that have been deprecated (Rob Herring).
>
> Changes in v3: None
> Changes in v2: None
>
> Documentation/devicetree/bindings/eeprom/eeprom.txt | 6 +++++-
> 1 file changed, 5 insertions(+), 1 deletion(-)
Acked-by: Rob Herring <robh@kernel.org>
^ permalink raw reply [flat|nested] 115+ messages in thread
* Re: [RESEND PATCH v5 01/16] dt-bindings: i2c: eeprom: Document vendor to be used and deprecated ones
@ 2017-06-23 18:17 ` Rob Herring
0 siblings, 0 replies; 115+ messages in thread
From: Rob Herring @ 2017-06-23 18:17 UTC (permalink / raw)
To: Javier Martinez Canillas
Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA, Wolfram Sang, Simon Horman,
Sekhar Nori, David Lechner, devicetree-u79uwXL29TY76Z2rM5mHXA,
Mark Rutland
On Thu, Jun 15, 2017 at 08:54:03PM +0200, Javier Martinez Canillas wrote:
> The at24 driver allows to register I2C EEPROM chips using different vendor
> and devices, but the I2C subsystem does not take the vendor into account
> when matching using the I2C table since it only has device entries.
>
> But when matching using an OF table, both the vendor and device has to be
> taken into account so the driver defines only a set of compatible strings
> using the "atmel" vendor as a generic fallback for compatible I2C devices.
>
> Document in the Device Tree binding document that this manufacturer should
> be used as the generic fallback. Also document the deprecated vendors.
>
> Suggested-by: Wolfram Sang <wsa-z923LK4zBo2bacvFa/9K2g@public.gmane.org>
> Suggested-by: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
> Signed-off-by: Javier Martinez Canillas <javier-0uQlZySMnqxg9hUCZPvPmw@public.gmane.org>
>
> ---
>
> Changes in v5:
> - Only deprecate the atmel variants at25 and at (Geert Uytterhoeven).
>
> Changes in v4:
> - Document the manufacturers that have been deprecated (Rob Herring).
>
> Changes in v3: None
> Changes in v2: None
>
> Documentation/devicetree/bindings/eeprom/eeprom.txt | 6 +++++-
> 1 file changed, 5 insertions(+), 1 deletion(-)
Acked-by: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 115+ messages in thread
* Re: [RESEND PATCH v5 01/16] dt-bindings: i2c: eeprom: Document vendor to be used and deprecated ones
2017-06-15 18:54 ` [RESEND PATCH v5 01/16] dt-bindings: i2c: eeprom: Document vendor to be used and deprecated ones Javier Martinez Canillas
2017-06-23 18:17 ` Rob Herring
@ 2017-08-28 16:02 ` Wolfram Sang
1 sibling, 0 replies; 115+ messages in thread
From: Wolfram Sang @ 2017-08-28 16:02 UTC (permalink / raw)
To: Javier Martinez Canillas
Cc: linux-kernel, Rob Herring, Simon Horman, Sekhar Nori,
David Lechner, devicetree, Rob Herring, Mark Rutland
[-- Attachment #1: Type: text/plain, Size: 896 bytes --]
On Thu, Jun 15, 2017 at 08:54:03PM +0200, Javier Martinez Canillas wrote:
> The at24 driver allows to register I2C EEPROM chips using different vendor
> and devices, but the I2C subsystem does not take the vendor into account
> when matching using the I2C table since it only has device entries.
>
> But when matching using an OF table, both the vendor and device has to be
> taken into account so the driver defines only a set of compatible strings
> using the "atmel" vendor as a generic fallback for compatible I2C devices.
>
> Document in the Device Tree binding document that this manufacturer should
> be used as the generic fallback. Also document the deprecated vendors.
>
> Suggested-by: Wolfram Sang <wsa@the-dreams.de>
> Suggested-by: Rob Herring <robh@kernel.org>
> Signed-off-by: Javier Martinez Canillas <javier@dowhile0.org>
>
Applied to for-next, thanks!
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 115+ messages in thread
* [RESEND PATCH v5 02/16] eeprom: at24: Add OF device ID table
2017-06-15 18:54 ` Javier Martinez Canillas
` (2 preceding siblings ...)
(?)
@ 2017-06-15 18:54 ` Javier Martinez Canillas
-1 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-06-15 18:54 UTC (permalink / raw)
To: linux-kernel
Cc: Wolfram Sang, Rob Herring, Javier Martinez Canillas,
Simon Horman, Andy Shevchenko, linux-i2c
The driver doesn't have a struct of_device_id table but supported devices
are registered via Device Trees. This is working on the assumption that a
I2C device registered via OF will always match a legacy I2C device ID and
that the MODALIAS reported will always be of the form i2c:<device>.
But this could change in the future so the correct approach is to have an
OF device ID table if the devices are registered via OF.
Suggested-by: Wolfram Sang <wsa@the-dreams.de>
Signed-off-by: Javier Martinez Canillas <javier@dowhile0.org>
---
Changes in v5: None
Changes in v4: None
Changes in v3:
- Fix wrong .data values for "atmel,24c02" and "atmel,24c64" entries.
Changes in v2:
- Only add a single OF device ID entry for each device type (Wolfram Sang).
drivers/misc/eeprom/at24.c | 65 +++++++++++++++++++++++++++++++++++++++++++++-
1 file changed, 64 insertions(+), 1 deletion(-)
diff --git a/drivers/misc/eeprom/at24.c b/drivers/misc/eeprom/at24.c
index 764ff5df0dbc..79c5c39be29c 100644
--- a/drivers/misc/eeprom/at24.c
+++ b/drivers/misc/eeprom/at24.c
@@ -12,6 +12,7 @@
#include <linux/kernel.h>
#include <linux/init.h>
#include <linux/module.h>
+#include <linux/of_device.h>
#include <linux/slab.h>
#include <linux/delay.h>
#include <linux/mutex.h>
@@ -175,6 +176,64 @@ static const struct i2c_device_id at24_ids[] = {
};
MODULE_DEVICE_TABLE(i2c, at24_ids);
+static const struct of_device_id at24_of_match[] = {
+ {
+ .compatible = "atmel,24c00",
+ .data = (void *)AT24_DEVICE_MAGIC(128 / 8, AT24_FLAG_TAKE8ADDR)
+ },
+ {
+ .compatible = "atmel,24c01",
+ .data = (void *)AT24_DEVICE_MAGIC(1024 / 8, 0)
+ },
+ {
+ .compatible = "atmel,24c02",
+ .data = (void *)AT24_DEVICE_MAGIC(2048 / 8, 0)
+ },
+ {
+ .compatible = "atmel,spd",
+ .data = (void *)AT24_DEVICE_MAGIC(2048 / 8,
+ AT24_FLAG_READONLY | AT24_FLAG_IRUGO)
+ },
+ {
+ .compatible = "atmel,24c04",
+ .data = (void *)AT24_DEVICE_MAGIC(4096 / 8, 0)
+ },
+ {
+ .compatible = "atmel,24c08",
+ .data = (void *)AT24_DEVICE_MAGIC(8192 / 8, 0)
+ },
+ {
+ .compatible = "atmel,24c16",
+ .data = (void *)AT24_DEVICE_MAGIC(16384 / 8, 0)
+ },
+ {
+ .compatible = "atmel,24c32",
+ .data = (void *)AT24_DEVICE_MAGIC(32768 / 8, AT24_FLAG_ADDR16)
+ },
+ {
+ .compatible = "atmel,24c64",
+ .data = (void *)AT24_DEVICE_MAGIC(65536 / 8, AT24_FLAG_ADDR16)
+ },
+ {
+ .compatible = "atmel,24c128",
+ .data = (void *)AT24_DEVICE_MAGIC(131072 / 8, AT24_FLAG_ADDR16)
+ },
+ {
+ .compatible = "atmel,24c256",
+ .data = (void *)AT24_DEVICE_MAGIC(262144 / 8, AT24_FLAG_ADDR16)
+ },
+ {
+ .compatible = "atmel,24c512",
+ .data = (void *)AT24_DEVICE_MAGIC(524288 / 8, AT24_FLAG_ADDR16)
+ },
+ {
+ .compatible = "atmel,24c1024",
+ .data = (void *)AT24_DEVICE_MAGIC(1048576 / 8, AT24_FLAG_ADDR16)
+ },
+ { },
+};
+MODULE_DEVICE_TABLE(of, at24_of_match);
+
static const struct acpi_device_id at24_acpi_ids[] = {
{ "INT3499", AT24_DEVICE_MAGIC(8192 / 8, 0) },
{ }
@@ -598,7 +657,10 @@ static int at24_probe(struct i2c_client *client, const struct i2c_device_id *id)
if (client->dev.platform_data) {
chip = *(struct at24_platform_data *)client->dev.platform_data;
} else {
- if (id) {
+ if (client->dev.of_node) {
+ magic = (kernel_ulong_t)
+ of_device_get_match_data(&client->dev);
+ } else if (id) {
magic = id->driver_data;
} else {
const struct acpi_device_id *aid;
@@ -814,6 +876,7 @@ static int at24_remove(struct i2c_client *client)
static struct i2c_driver at24_driver = {
.driver = {
.name = "at24",
+ .of_match_table = at24_of_match,
.acpi_match_table = ACPI_PTR(at24_acpi_ids),
},
.probe = at24_probe,
--
2.9.3
^ permalink raw reply related [flat|nested] 115+ messages in thread* [RESEND PATCH v5 03/16] ARM: dts: efm32: Add generic compatible string for I2C EEPROM
@ 2017-06-15 18:54 ` Javier Martinez Canillas
0 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-06-15 18:54 UTC (permalink / raw)
To: linux-kernel
Cc: Wolfram Sang, Rob Herring, Javier Martinez Canillas, devicetree,
Rob Herring, Uwe Kleine-König, Mark Rutland, Russell King,
linux-arm-kernel
The at24 driver allows to register I2C EEPROM chips using different vendor
and devices, but the I2C subsystem does not take the vendor into account
when matching using the I2C table since it only has device entries.
But when matching using an OF table, both the vendor and device has to be
taken into account so the driver defines only a set of compatible strings
using the "atmel" vendor as a generic fallback for compatible I2C devices.
So add this generic fallback to the device node compatible string to make
the device to match the driver using the OF device ID table.
Signed-off-by: Javier Martinez Canillas <javier@dowhile0.org>
---
Changes in v5:
- Only replace atmel variant but keep other EEPROM vendors (Geert Uytterhoeven).
Changes in v4:
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
Changes in v3: None
Changes in v2: None
arch/arm/boot/dts/efm32gg-dk3750.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/efm32gg-dk3750.dts b/arch/arm/boot/dts/efm32gg-dk3750.dts
index 98fc667d22c7..84f0a9abc290 100644
--- a/arch/arm/boot/dts/efm32gg-dk3750.dts
+++ b/arch/arm/boot/dts/efm32gg-dk3750.dts
@@ -36,7 +36,7 @@
};
eeprom@50 {
- compatible = "microchip,24c02";
+ compatible = "microchip,24c02", "atmel,24c02";
reg = <0x50>;
pagesize = <16>;
};
--
2.9.3
^ permalink raw reply related [flat|nested] 115+ messages in thread* [RESEND PATCH v5 03/16] ARM: dts: efm32: Add generic compatible string for I2C EEPROM
@ 2017-06-15 18:54 ` Javier Martinez Canillas
0 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-06-15 18:54 UTC (permalink / raw)
To: linux-arm-kernel
The at24 driver allows to register I2C EEPROM chips using different vendor
and devices, but the I2C subsystem does not take the vendor into account
when matching using the I2C table since it only has device entries.
But when matching using an OF table, both the vendor and device has to be
taken into account so the driver defines only a set of compatible strings
using the "atmel" vendor as a generic fallback for compatible I2C devices.
So add this generic fallback to the device node compatible string to make
the device to match the driver using the OF device ID table.
Signed-off-by: Javier Martinez Canillas <javier@dowhile0.org>
---
Changes in v5:
- Only replace atmel variant but keep other EEPROM vendors (Geert Uytterhoeven).
Changes in v4:
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
Changes in v3: None
Changes in v2: None
arch/arm/boot/dts/efm32gg-dk3750.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/efm32gg-dk3750.dts b/arch/arm/boot/dts/efm32gg-dk3750.dts
index 98fc667d22c7..84f0a9abc290 100644
--- a/arch/arm/boot/dts/efm32gg-dk3750.dts
+++ b/arch/arm/boot/dts/efm32gg-dk3750.dts
@@ -36,7 +36,7 @@
};
eeprom at 50 {
- compatible = "microchip,24c02";
+ compatible = "microchip,24c02", "atmel,24c02";
reg = <0x50>;
pagesize = <16>;
};
--
2.9.3
^ permalink raw reply related [flat|nested] 115+ messages in thread* [RESEND PATCH v5 03/16] ARM: dts: efm32: Add generic compatible string for I2C EEPROM
@ 2017-06-15 18:54 ` Javier Martinez Canillas
0 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-06-15 18:54 UTC (permalink / raw)
To: linux-kernel-u79uwXL29TY76Z2rM5mHXA
Cc: Wolfram Sang, Rob Herring, Javier Martinez Canillas,
devicetree-u79uwXL29TY76Z2rM5mHXA, Rob Herring,
Uwe Kleine-König, Mark Rutland, Russell King,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
The at24 driver allows to register I2C EEPROM chips using different vendor
and devices, but the I2C subsystem does not take the vendor into account
when matching using the I2C table since it only has device entries.
But when matching using an OF table, both the vendor and device has to be
taken into account so the driver defines only a set of compatible strings
using the "atmel" vendor as a generic fallback for compatible I2C devices.
So add this generic fallback to the device node compatible string to make
the device to match the driver using the OF device ID table.
Signed-off-by: Javier Martinez Canillas <javier-0uQlZySMnqxg9hUCZPvPmw@public.gmane.org>
---
Changes in v5:
- Only replace atmel variant but keep other EEPROM vendors (Geert Uytterhoeven).
Changes in v4:
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
Changes in v3: None
Changes in v2: None
arch/arm/boot/dts/efm32gg-dk3750.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/efm32gg-dk3750.dts b/arch/arm/boot/dts/efm32gg-dk3750.dts
index 98fc667d22c7..84f0a9abc290 100644
--- a/arch/arm/boot/dts/efm32gg-dk3750.dts
+++ b/arch/arm/boot/dts/efm32gg-dk3750.dts
@@ -36,7 +36,7 @@
};
eeprom@50 {
- compatible = "microchip,24c02";
+ compatible = "microchip,24c02", "atmel,24c02";
reg = <0x50>;
pagesize = <16>;
};
--
2.9.3
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply related [flat|nested] 115+ messages in thread* Re: [RESEND PATCH v5 03/16] ARM: dts: efm32: Add generic compatible string for I2C EEPROM
@ 2017-08-16 14:21 ` Javier Martinez Canillas
0 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-08-16 14:21 UTC (permalink / raw)
To: Linux Kernel
Cc: Wolfram Sang, Rob Herring, Javier Martinez Canillas, devicetree,
Rob Herring, Uwe Kleine-König, Mark Rutland, Russell King,
linux-arm-kernel
On Thu, Jun 15, 2017 at 8:54 PM, Javier Martinez Canillas
<javier@dowhile0.org> wrote:
> The at24 driver allows to register I2C EEPROM chips using different vendor
> and devices, but the I2C subsystem does not take the vendor into account
> when matching using the I2C table since it only has device entries.
>
> But when matching using an OF table, both the vendor and device has to be
> taken into account so the driver defines only a set of compatible strings
> using the "atmel" vendor as a generic fallback for compatible I2C devices.
>
> So add this generic fallback to the device node compatible string to make
> the device to match the driver using the OF device ID table.
>
> Signed-off-by: Javier Martinez Canillas <javier@dowhile0.org>
>
Any comment on this?
Wolfram, maybe you can ack the pending DTS patches so I can post them
to the arm-soc maintainers directly for they to pick them?
Best regards,
Javier
^ permalink raw reply [flat|nested] 115+ messages in thread
* [RESEND PATCH v5 03/16] ARM: dts: efm32: Add generic compatible string for I2C EEPROM
@ 2017-08-16 14:21 ` Javier Martinez Canillas
0 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-08-16 14:21 UTC (permalink / raw)
To: linux-arm-kernel
On Thu, Jun 15, 2017 at 8:54 PM, Javier Martinez Canillas
<javier@dowhile0.org> wrote:
> The at24 driver allows to register I2C EEPROM chips using different vendor
> and devices, but the I2C subsystem does not take the vendor into account
> when matching using the I2C table since it only has device entries.
>
> But when matching using an OF table, both the vendor and device has to be
> taken into account so the driver defines only a set of compatible strings
> using the "atmel" vendor as a generic fallback for compatible I2C devices.
>
> So add this generic fallback to the device node compatible string to make
> the device to match the driver using the OF device ID table.
>
> Signed-off-by: Javier Martinez Canillas <javier@dowhile0.org>
>
Any comment on this?
Wolfram, maybe you can ack the pending DTS patches so I can post them
to the arm-soc maintainers directly for they to pick them?
Best regards,
Javier
^ permalink raw reply [flat|nested] 115+ messages in thread
* Re: [RESEND PATCH v5 03/16] ARM: dts: efm32: Add generic compatible string for I2C EEPROM
@ 2017-08-16 14:21 ` Javier Martinez Canillas
0 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-08-16 14:21 UTC (permalink / raw)
To: Linux Kernel
Cc: Wolfram Sang, Rob Herring, Javier Martinez Canillas,
devicetree-u79uwXL29TY76Z2rM5mHXA, Rob Herring,
Uwe Kleine-König, Mark Rutland, Russell King,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
On Thu, Jun 15, 2017 at 8:54 PM, Javier Martinez Canillas
<javier-0uQlZySMnqxg9hUCZPvPmw@public.gmane.org> wrote:
> The at24 driver allows to register I2C EEPROM chips using different vendor
> and devices, but the I2C subsystem does not take the vendor into account
> when matching using the I2C table since it only has device entries.
>
> But when matching using an OF table, both the vendor and device has to be
> taken into account so the driver defines only a set of compatible strings
> using the "atmel" vendor as a generic fallback for compatible I2C devices.
>
> So add this generic fallback to the device node compatible string to make
> the device to match the driver using the OF device ID table.
>
> Signed-off-by: Javier Martinez Canillas <javier-0uQlZySMnqxg9hUCZPvPmw@public.gmane.org>
>
Any comment on this?
Wolfram, maybe you can ack the pending DTS patches so I can post them
to the arm-soc maintainers directly for they to pick them?
Best regards,
Javier
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 115+ messages in thread
* [RESEND PATCH v5 04/16] ARM: dts: keystone: Add generic compatible string for I2C EEPROM
@ 2017-06-15 18:54 ` Javier Martinez Canillas
0 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-06-15 18:54 UTC (permalink / raw)
To: linux-kernel
Cc: Wolfram Sang, Rob Herring, Javier Martinez Canillas, devicetree,
Santosh Shilimkar, Rob Herring, Mark Rutland, Russell King,
linux-arm-kernel
The at24 driver allows to register I2C EEPROM chips using different vendor
and devices, but the I2C subsystem does not take the vendor into account
when matching using the I2C table since it only has device entries.
But when matching using an OF table, both the vendor and device has to be
taken into account so the driver defines only a set of compatible strings
using the "atmel" vendor as a generic fallback for compatible I2C devices.
So add this generic fallback to the device node compatible string to make
the device to match the driver using the OF device ID table.
Signed-off-by: Javier Martinez Canillas <javier@dowhile0.org>
---
Changes in v5: None
Changes in v4:
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
Changes in v3: None
Changes in v2: None
arch/arm/boot/dts/keystone-k2e-evm.dts | 2 +-
arch/arm/boot/dts/keystone-k2hk-evm.dts | 2 +-
arch/arm/boot/dts/keystone-k2l-evm.dts | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/arch/arm/boot/dts/keystone-k2e-evm.dts b/arch/arm/boot/dts/keystone-k2e-evm.dts
index ae1ebe7ee021..c8d45e6cd9e6 100644
--- a/arch/arm/boot/dts/keystone-k2e-evm.dts
+++ b/arch/arm/boot/dts/keystone-k2e-evm.dts
@@ -69,7 +69,7 @@
&i2c0 {
dtt@50 {
- compatible = "at,24c1024";
+ compatible = "atmel,24c1024";
reg = <0x50>;
};
};
diff --git a/arch/arm/boot/dts/keystone-k2hk-evm.dts b/arch/arm/boot/dts/keystone-k2hk-evm.dts
index 2156ff92d08f..69af3a950da6 100644
--- a/arch/arm/boot/dts/keystone-k2hk-evm.dts
+++ b/arch/arm/boot/dts/keystone-k2hk-evm.dts
@@ -145,7 +145,7 @@
&i2c0 {
dtt@50 {
- compatible = "at,24c1024";
+ compatible = "atmel,24c1024";
reg = <0x50>;
};
};
diff --git a/arch/arm/boot/dts/keystone-k2l-evm.dts b/arch/arm/boot/dts/keystone-k2l-evm.dts
index 056b42f99d7a..402190e55e40 100644
--- a/arch/arm/boot/dts/keystone-k2l-evm.dts
+++ b/arch/arm/boot/dts/keystone-k2l-evm.dts
@@ -42,7 +42,7 @@
&i2c0 {
dtt@50 {
- compatible = "at,24c1024";
+ compatible = "atmel,24c1024";
reg = <0x50>;
};
};
--
2.9.3
^ permalink raw reply related [flat|nested] 115+ messages in thread* [RESEND PATCH v5 04/16] ARM: dts: keystone: Add generic compatible string for I2C EEPROM
@ 2017-06-15 18:54 ` Javier Martinez Canillas
0 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-06-15 18:54 UTC (permalink / raw)
To: linux-arm-kernel
The at24 driver allows to register I2C EEPROM chips using different vendor
and devices, but the I2C subsystem does not take the vendor into account
when matching using the I2C table since it only has device entries.
But when matching using an OF table, both the vendor and device has to be
taken into account so the driver defines only a set of compatible strings
using the "atmel" vendor as a generic fallback for compatible I2C devices.
So add this generic fallback to the device node compatible string to make
the device to match the driver using the OF device ID table.
Signed-off-by: Javier Martinez Canillas <javier@dowhile0.org>
---
Changes in v5: None
Changes in v4:
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
Changes in v3: None
Changes in v2: None
arch/arm/boot/dts/keystone-k2e-evm.dts | 2 +-
arch/arm/boot/dts/keystone-k2hk-evm.dts | 2 +-
arch/arm/boot/dts/keystone-k2l-evm.dts | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/arch/arm/boot/dts/keystone-k2e-evm.dts b/arch/arm/boot/dts/keystone-k2e-evm.dts
index ae1ebe7ee021..c8d45e6cd9e6 100644
--- a/arch/arm/boot/dts/keystone-k2e-evm.dts
+++ b/arch/arm/boot/dts/keystone-k2e-evm.dts
@@ -69,7 +69,7 @@
&i2c0 {
dtt at 50 {
- compatible = "at,24c1024";
+ compatible = "atmel,24c1024";
reg = <0x50>;
};
};
diff --git a/arch/arm/boot/dts/keystone-k2hk-evm.dts b/arch/arm/boot/dts/keystone-k2hk-evm.dts
index 2156ff92d08f..69af3a950da6 100644
--- a/arch/arm/boot/dts/keystone-k2hk-evm.dts
+++ b/arch/arm/boot/dts/keystone-k2hk-evm.dts
@@ -145,7 +145,7 @@
&i2c0 {
dtt at 50 {
- compatible = "at,24c1024";
+ compatible = "atmel,24c1024";
reg = <0x50>;
};
};
diff --git a/arch/arm/boot/dts/keystone-k2l-evm.dts b/arch/arm/boot/dts/keystone-k2l-evm.dts
index 056b42f99d7a..402190e55e40 100644
--- a/arch/arm/boot/dts/keystone-k2l-evm.dts
+++ b/arch/arm/boot/dts/keystone-k2l-evm.dts
@@ -42,7 +42,7 @@
&i2c0 {
dtt at 50 {
- compatible = "at,24c1024";
+ compatible = "atmel,24c1024";
reg = <0x50>;
};
};
--
2.9.3
^ permalink raw reply related [flat|nested] 115+ messages in thread* [RESEND PATCH v5 04/16] ARM: dts: keystone: Add generic compatible string for I2C EEPROM
@ 2017-06-15 18:54 ` Javier Martinez Canillas
0 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-06-15 18:54 UTC (permalink / raw)
To: linux-kernel-u79uwXL29TY76Z2rM5mHXA
Cc: Wolfram Sang, Rob Herring, Javier Martinez Canillas,
devicetree-u79uwXL29TY76Z2rM5mHXA, Santosh Shilimkar,
Rob Herring, Mark Rutland, Russell King,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
The at24 driver allows to register I2C EEPROM chips using different vendor
and devices, but the I2C subsystem does not take the vendor into account
when matching using the I2C table since it only has device entries.
But when matching using an OF table, both the vendor and device has to be
taken into account so the driver defines only a set of compatible strings
using the "atmel" vendor as a generic fallback for compatible I2C devices.
So add this generic fallback to the device node compatible string to make
the device to match the driver using the OF device ID table.
Signed-off-by: Javier Martinez Canillas <javier-0uQlZySMnqxg9hUCZPvPmw@public.gmane.org>
---
Changes in v5: None
Changes in v4:
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
Changes in v3: None
Changes in v2: None
arch/arm/boot/dts/keystone-k2e-evm.dts | 2 +-
arch/arm/boot/dts/keystone-k2hk-evm.dts | 2 +-
arch/arm/boot/dts/keystone-k2l-evm.dts | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/arch/arm/boot/dts/keystone-k2e-evm.dts b/arch/arm/boot/dts/keystone-k2e-evm.dts
index ae1ebe7ee021..c8d45e6cd9e6 100644
--- a/arch/arm/boot/dts/keystone-k2e-evm.dts
+++ b/arch/arm/boot/dts/keystone-k2e-evm.dts
@@ -69,7 +69,7 @@
&i2c0 {
dtt@50 {
- compatible = "at,24c1024";
+ compatible = "atmel,24c1024";
reg = <0x50>;
};
};
diff --git a/arch/arm/boot/dts/keystone-k2hk-evm.dts b/arch/arm/boot/dts/keystone-k2hk-evm.dts
index 2156ff92d08f..69af3a950da6 100644
--- a/arch/arm/boot/dts/keystone-k2hk-evm.dts
+++ b/arch/arm/boot/dts/keystone-k2hk-evm.dts
@@ -145,7 +145,7 @@
&i2c0 {
dtt@50 {
- compatible = "at,24c1024";
+ compatible = "atmel,24c1024";
reg = <0x50>;
};
};
diff --git a/arch/arm/boot/dts/keystone-k2l-evm.dts b/arch/arm/boot/dts/keystone-k2l-evm.dts
index 056b42f99d7a..402190e55e40 100644
--- a/arch/arm/boot/dts/keystone-k2l-evm.dts
+++ b/arch/arm/boot/dts/keystone-k2l-evm.dts
@@ -42,7 +42,7 @@
&i2c0 {
dtt@50 {
- compatible = "at,24c1024";
+ compatible = "atmel,24c1024";
reg = <0x50>;
};
};
--
2.9.3
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply related [flat|nested] 115+ messages in thread
* [RESEND PATCH v5 05/16] ARM: dts: lpc18xx: Add generic compatible string for I2C EEPROM
2017-06-15 18:54 ` Javier Martinez Canillas
@ 2017-06-15 18:54 ` Javier Martinez Canillas
-1 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-06-15 18:54 UTC (permalink / raw)
To: linux-kernel
Cc: Wolfram Sang, Rob Herring, Javier Martinez Canillas, devicetree,
Joachim Eastwood, Rob Herring, Mark Rutland, Russell King,
linux-arm-kernel
The at24 driver allows to register I2C EEPROM chips using different vendor
and devices, but the I2C subsystem does not take the vendor into account
when matching using the I2C table since it only has device entries.
But when matching using an OF table, both the vendor and device has to be
taken into account so the driver defines only a set of compatible strings
using the "atmel" vendor as a generic fallback for compatible I2C devices.
So add this generic fallback to the device node compatible string to make
the device to match the driver using the OF device ID table.
Signed-off-by: Javier Martinez Canillas <javier@dowhile0.org>
---
Changes in v5:
- Only replace atmel variant but keep other EEPROM vendors (Geert Uytterhoeven).
Changes in v4:
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
Changes in v3: None
Changes in v2: None
arch/arm/boot/dts/lpc4337-ciaa.dts | 6 +++---
arch/arm/boot/dts/lpc4350-hitex-eval.dts | 2 +-
arch/arm/boot/dts/lpc4357-ea4357-devkit.dts | 2 +-
3 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/arch/arm/boot/dts/lpc4337-ciaa.dts b/arch/arm/boot/dts/lpc4337-ciaa.dts
index 7c16d639a1b4..beddaba85393 100644
--- a/arch/arm/boot/dts/lpc4337-ciaa.dts
+++ b/arch/arm/boot/dts/lpc4337-ciaa.dts
@@ -174,17 +174,17 @@
clock-frequency = <400000>;
eeprom@50 {
- compatible = "microchip,24c512";
+ compatible = "microchip,24c512", "atmel,24c512";
reg = <0x50>;
};
eeprom@51 {
- compatible = "microchip,24c02";
+ compatible = "microchip,24c02", "atmel,24c02";
reg = <0x51>;
};
eeprom@54 {
- compatible = "microchip,24c512";
+ compatible = "microchip,24c512", "atmel,24c512";
reg = <0x54>;
};
};
diff --git a/arch/arm/boot/dts/lpc4350-hitex-eval.dts b/arch/arm/boot/dts/lpc4350-hitex-eval.dts
index 874c75d44013..8b973f537d3a 100644
--- a/arch/arm/boot/dts/lpc4350-hitex-eval.dts
+++ b/arch/arm/boot/dts/lpc4350-hitex-eval.dts
@@ -429,7 +429,7 @@
};
eeprom@50 {
- compatible = "nxp,24c02";
+ compatible = "nxp,24c02", "atmel,24c02";
reg = <0x50>;
};
diff --git a/arch/arm/boot/dts/lpc4357-ea4357-devkit.dts b/arch/arm/boot/dts/lpc4357-ea4357-devkit.dts
index 9b5fad622522..02b23fa29d75 100644
--- a/arch/arm/boot/dts/lpc4357-ea4357-devkit.dts
+++ b/arch/arm/boot/dts/lpc4357-ea4357-devkit.dts
@@ -490,7 +490,7 @@
};
eeprom@57 {
- compatible = "microchip,24c64";
+ compatible = "microchip,24c64", "atmel,24c64";
reg = <0x57>;
};
};
--
2.9.3
^ permalink raw reply related [flat|nested] 115+ messages in thread* [RESEND PATCH v5 05/16] ARM: dts: lpc18xx: Add generic compatible string for I2C EEPROM
@ 2017-06-15 18:54 ` Javier Martinez Canillas
0 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-06-15 18:54 UTC (permalink / raw)
To: linux-arm-kernel
The at24 driver allows to register I2C EEPROM chips using different vendor
and devices, but the I2C subsystem does not take the vendor into account
when matching using the I2C table since it only has device entries.
But when matching using an OF table, both the vendor and device has to be
taken into account so the driver defines only a set of compatible strings
using the "atmel" vendor as a generic fallback for compatible I2C devices.
So add this generic fallback to the device node compatible string to make
the device to match the driver using the OF device ID table.
Signed-off-by: Javier Martinez Canillas <javier@dowhile0.org>
---
Changes in v5:
- Only replace atmel variant but keep other EEPROM vendors (Geert Uytterhoeven).
Changes in v4:
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
Changes in v3: None
Changes in v2: None
arch/arm/boot/dts/lpc4337-ciaa.dts | 6 +++---
arch/arm/boot/dts/lpc4350-hitex-eval.dts | 2 +-
arch/arm/boot/dts/lpc4357-ea4357-devkit.dts | 2 +-
3 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/arch/arm/boot/dts/lpc4337-ciaa.dts b/arch/arm/boot/dts/lpc4337-ciaa.dts
index 7c16d639a1b4..beddaba85393 100644
--- a/arch/arm/boot/dts/lpc4337-ciaa.dts
+++ b/arch/arm/boot/dts/lpc4337-ciaa.dts
@@ -174,17 +174,17 @@
clock-frequency = <400000>;
eeprom at 50 {
- compatible = "microchip,24c512";
+ compatible = "microchip,24c512", "atmel,24c512";
reg = <0x50>;
};
eeprom at 51 {
- compatible = "microchip,24c02";
+ compatible = "microchip,24c02", "atmel,24c02";
reg = <0x51>;
};
eeprom at 54 {
- compatible = "microchip,24c512";
+ compatible = "microchip,24c512", "atmel,24c512";
reg = <0x54>;
};
};
diff --git a/arch/arm/boot/dts/lpc4350-hitex-eval.dts b/arch/arm/boot/dts/lpc4350-hitex-eval.dts
index 874c75d44013..8b973f537d3a 100644
--- a/arch/arm/boot/dts/lpc4350-hitex-eval.dts
+++ b/arch/arm/boot/dts/lpc4350-hitex-eval.dts
@@ -429,7 +429,7 @@
};
eeprom at 50 {
- compatible = "nxp,24c02";
+ compatible = "nxp,24c02", "atmel,24c02";
reg = <0x50>;
};
diff --git a/arch/arm/boot/dts/lpc4357-ea4357-devkit.dts b/arch/arm/boot/dts/lpc4357-ea4357-devkit.dts
index 9b5fad622522..02b23fa29d75 100644
--- a/arch/arm/boot/dts/lpc4357-ea4357-devkit.dts
+++ b/arch/arm/boot/dts/lpc4357-ea4357-devkit.dts
@@ -490,7 +490,7 @@
};
eeprom at 57 {
- compatible = "microchip,24c64";
+ compatible = "microchip,24c64", "atmel,24c64";
reg = <0x57>;
};
};
--
2.9.3
^ permalink raw reply related [flat|nested] 115+ messages in thread
* [RESEND PATCH v5 06/16] ARM: dts: r7s72100: Add generic compatible string for I2C EEPROM
2017-06-15 18:54 ` Javier Martinez Canillas
@ 2017-06-15 18:54 ` Javier Martinez Canillas
-1 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-06-15 18:54 UTC (permalink / raw)
To: linux-kernel
Cc: Wolfram Sang, Rob Herring, Javier Martinez Canillas,
Simon Horman, devicetree, Russell King, linux-renesas-soc,
Rob Herring, Magnus Damm, Mark Rutland, linux-arm-kernel
The at24 driver allows to register I2C EEPROM chips using different vendor
and devices, but the I2C subsystem does not take the vendor into account
when matching using the I2C table since it only has device entries.
But when matching using an OF table, both the vendor and device has to be
taken into account so the driver defines only a set of compatible strings
using the "atmel" vendor as a generic fallback for compatible I2C devices.
So add this generic fallback to the device node compatible string to make
the device to match the driver using the OF device ID table.
Signed-off-by: Javier Martinez Canillas <javier@dowhile0.org>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
Changes in v5:
- Only replace atmel variant but keep other EEPROM vendors (Geert Uytterhoeven).
Changes in v4:
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
Changes in v3:
- Add Geert Uytterhoeven reviewed-by tag.
Changes in v2: None
arch/arm/boot/dts/r7s72100-genmai.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/r7s72100-genmai.dts b/arch/arm/boot/dts/r7s72100-genmai.dts
index 52a7b586bac7..b0db621ff176 100644
--- a/arch/arm/boot/dts/r7s72100-genmai.dts
+++ b/arch/arm/boot/dts/r7s72100-genmai.dts
@@ -57,7 +57,7 @@
clock-frequency = <400000>;
eeprom@50 {
- compatible = "renesas,24c128";
+ compatible = "renesas,24c128", "atmel,24c128";
reg = <0x50>;
pagesize = <64>;
};
--
2.9.3
^ permalink raw reply related [flat|nested] 115+ messages in thread* [RESEND PATCH v5 06/16] ARM: dts: r7s72100: Add generic compatible string for I2C EEPROM
@ 2017-06-15 18:54 ` Javier Martinez Canillas
0 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-06-15 18:54 UTC (permalink / raw)
To: linux-arm-kernel
The at24 driver allows to register I2C EEPROM chips using different vendor
and devices, but the I2C subsystem does not take the vendor into account
when matching using the I2C table since it only has device entries.
But when matching using an OF table, both the vendor and device has to be
taken into account so the driver defines only a set of compatible strings
using the "atmel" vendor as a generic fallback for compatible I2C devices.
So add this generic fallback to the device node compatible string to make
the device to match the driver using the OF device ID table.
Signed-off-by: Javier Martinez Canillas <javier@dowhile0.org>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
Changes in v5:
- Only replace atmel variant but keep other EEPROM vendors (Geert Uytterhoeven).
Changes in v4:
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
Changes in v3:
- Add Geert Uytterhoeven reviewed-by tag.
Changes in v2: None
arch/arm/boot/dts/r7s72100-genmai.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/r7s72100-genmai.dts b/arch/arm/boot/dts/r7s72100-genmai.dts
index 52a7b586bac7..b0db621ff176 100644
--- a/arch/arm/boot/dts/r7s72100-genmai.dts
+++ b/arch/arm/boot/dts/r7s72100-genmai.dts
@@ -57,7 +57,7 @@
clock-frequency = <400000>;
eeprom at 50 {
- compatible = "renesas,24c128";
+ compatible = "renesas,24c128", "atmel,24c128";
reg = <0x50>;
pagesize = <64>;
};
--
2.9.3
^ permalink raw reply related [flat|nested] 115+ messages in thread* Re: [RESEND PATCH v5 06/16] ARM: dts: r7s72100: Add generic compatible string for I2C EEPROM
2017-06-15 18:54 ` Javier Martinez Canillas
@ 2017-06-16 13:36 ` Simon Horman
-1 siblings, 0 replies; 115+ messages in thread
From: Simon Horman @ 2017-06-16 13:36 UTC (permalink / raw)
To: Javier Martinez Canillas
Cc: linux-kernel, Wolfram Sang, Rob Herring, devicetree,
Russell King, linux-renesas-soc, Rob Herring, Magnus Damm,
Mark Rutland, linux-arm-kernel
On Thu, Jun 15, 2017 at 08:54:08PM +0200, Javier Martinez Canillas wrote:
> The at24 driver allows to register I2C EEPROM chips using different vendor
> and devices, but the I2C subsystem does not take the vendor into account
> when matching using the I2C table since it only has device entries.
>
> But when matching using an OF table, both the vendor and device has to be
> taken into account so the driver defines only a set of compatible strings
> using the "atmel" vendor as a generic fallback for compatible I2C devices.
>
> So add this generic fallback to the device node compatible string to make
> the device to match the driver using the OF device ID table.
>
> Signed-off-by: Javier Martinez Canillas <javier@dowhile0.org>
> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
Thanks, I have queued this up in the renesas tree for v4.14.
^ permalink raw reply [flat|nested] 115+ messages in thread
* [RESEND PATCH v5 06/16] ARM: dts: r7s72100: Add generic compatible string for I2C EEPROM
@ 2017-06-16 13:36 ` Simon Horman
0 siblings, 0 replies; 115+ messages in thread
From: Simon Horman @ 2017-06-16 13:36 UTC (permalink / raw)
To: linux-arm-kernel
On Thu, Jun 15, 2017 at 08:54:08PM +0200, Javier Martinez Canillas wrote:
> The at24 driver allows to register I2C EEPROM chips using different vendor
> and devices, but the I2C subsystem does not take the vendor into account
> when matching using the I2C table since it only has device entries.
>
> But when matching using an OF table, both the vendor and device has to be
> taken into account so the driver defines only a set of compatible strings
> using the "atmel" vendor as a generic fallback for compatible I2C devices.
>
> So add this generic fallback to the device node compatible string to make
> the device to match the driver using the OF device ID table.
>
> Signed-off-by: Javier Martinez Canillas <javier@dowhile0.org>
> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
Thanks, I have queued this up in the renesas tree for v4.14.
^ permalink raw reply [flat|nested] 115+ messages in thread
* [RESEND PATCH v5 07/16] ARM: dts: koelsch: Add generic compatible string for I2C EEPROM
@ 2017-06-15 18:54 ` Javier Martinez Canillas
0 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-06-15 18:54 UTC (permalink / raw)
To: linux-kernel
Cc: Wolfram Sang, Rob Herring, Javier Martinez Canillas,
Simon Horman, devicetree, Russell King, linux-renesas-soc,
Rob Herring, Magnus Damm, Mark Rutland, linux-arm-kernel
The at24 driver allows to register I2C EEPROM chips using different vendor
and devices, but the I2C subsystem does not take the vendor into account
when matching using the I2C table since it only has device entries.
But when matching using an OF table, both the vendor and device has to be
taken into account so the driver defines only a set of compatible strings
using the "atmel" vendor as a generic fallback for compatible I2C devices.
So add this generic fallback to the device node compatible string to make
the device to match the driver using the OF device ID table.
Signed-off-by: Javier Martinez Canillas <javier@dowhile0.org>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
Changes in v5:
- Only replace atmel variant but keep other EEPROM vendors (Geert Uytterhoeven).
Changes in v4:
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
Changes in v3:
- Add Geert Uytterhoeven reviewed-by tag.
Changes in v2: None
arch/arm/boot/dts/r8a7791-koelsch.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/r8a7791-koelsch.dts b/arch/arm/boot/dts/r8a7791-koelsch.dts
index 001e6116c47c..dc8ed02de707 100644
--- a/arch/arm/boot/dts/r8a7791-koelsch.dts
+++ b/arch/arm/boot/dts/r8a7791-koelsch.dts
@@ -702,7 +702,7 @@
};
eeprom@50 {
- compatible = "renesas,24c02";
+ compatible = "renesas,24c02", "atmel,24c02";
reg = <0x50>;
pagesize = <16>;
};
--
2.9.3
^ permalink raw reply related [flat|nested] 115+ messages in thread* [RESEND PATCH v5 07/16] ARM: dts: koelsch: Add generic compatible string for I2C EEPROM
@ 2017-06-15 18:54 ` Javier Martinez Canillas
0 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-06-15 18:54 UTC (permalink / raw)
To: linux-arm-kernel
The at24 driver allows to register I2C EEPROM chips using different vendor
and devices, but the I2C subsystem does not take the vendor into account
when matching using the I2C table since it only has device entries.
But when matching using an OF table, both the vendor and device has to be
taken into account so the driver defines only a set of compatible strings
using the "atmel" vendor as a generic fallback for compatible I2C devices.
So add this generic fallback to the device node compatible string to make
the device to match the driver using the OF device ID table.
Signed-off-by: Javier Martinez Canillas <javier@dowhile0.org>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
Changes in v5:
- Only replace atmel variant but keep other EEPROM vendors (Geert Uytterhoeven).
Changes in v4:
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
Changes in v3:
- Add Geert Uytterhoeven reviewed-by tag.
Changes in v2: None
arch/arm/boot/dts/r8a7791-koelsch.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/r8a7791-koelsch.dts b/arch/arm/boot/dts/r8a7791-koelsch.dts
index 001e6116c47c..dc8ed02de707 100644
--- a/arch/arm/boot/dts/r8a7791-koelsch.dts
+++ b/arch/arm/boot/dts/r8a7791-koelsch.dts
@@ -702,7 +702,7 @@
};
eeprom at 50 {
- compatible = "renesas,24c02";
+ compatible = "renesas,24c02", "atmel,24c02";
reg = <0x50>;
pagesize = <16>;
};
--
2.9.3
^ permalink raw reply related [flat|nested] 115+ messages in thread* [RESEND PATCH v5 07/16] ARM: dts: koelsch: Add generic compatible string for I2C EEPROM
@ 2017-06-15 18:54 ` Javier Martinez Canillas
0 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-06-15 18:54 UTC (permalink / raw)
To: linux-kernel-u79uwXL29TY76Z2rM5mHXA
Cc: Wolfram Sang, Rob Herring, Javier Martinez Canillas,
Simon Horman, devicetree-u79uwXL29TY76Z2rM5mHXA, Russell King,
linux-renesas-soc-u79uwXL29TY76Z2rM5mHXA, Rob Herring,
Magnus Damm, Mark Rutland,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
The at24 driver allows to register I2C EEPROM chips using different vendor
and devices, but the I2C subsystem does not take the vendor into account
when matching using the I2C table since it only has device entries.
But when matching using an OF table, both the vendor and device has to be
taken into account so the driver defines only a set of compatible strings
using the "atmel" vendor as a generic fallback for compatible I2C devices.
So add this generic fallback to the device node compatible string to make
the device to match the driver using the OF device ID table.
Signed-off-by: Javier Martinez Canillas <javier-0uQlZySMnqxg9hUCZPvPmw@public.gmane.org>
Reviewed-by: Geert Uytterhoeven <geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org>
---
Changes in v5:
- Only replace atmel variant but keep other EEPROM vendors (Geert Uytterhoeven).
Changes in v4:
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
Changes in v3:
- Add Geert Uytterhoeven reviewed-by tag.
Changes in v2: None
arch/arm/boot/dts/r8a7791-koelsch.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/r8a7791-koelsch.dts b/arch/arm/boot/dts/r8a7791-koelsch.dts
index 001e6116c47c..dc8ed02de707 100644
--- a/arch/arm/boot/dts/r8a7791-koelsch.dts
+++ b/arch/arm/boot/dts/r8a7791-koelsch.dts
@@ -702,7 +702,7 @@
};
eeprom@50 {
- compatible = "renesas,24c02";
+ compatible = "renesas,24c02", "atmel,24c02";
reg = <0x50>;
pagesize = <16>;
};
--
2.9.3
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply related [flat|nested] 115+ messages in thread* Re: [RESEND PATCH v5 07/16] ARM: dts: koelsch: Add generic compatible string for I2C EEPROM
2017-06-15 18:54 ` Javier Martinez Canillas
@ 2017-06-16 13:36 ` Simon Horman
-1 siblings, 0 replies; 115+ messages in thread
From: Simon Horman @ 2017-06-16 13:36 UTC (permalink / raw)
To: Javier Martinez Canillas
Cc: linux-kernel, Wolfram Sang, Rob Herring, devicetree,
Russell King, linux-renesas-soc, Rob Herring, Magnus Damm,
Mark Rutland, linux-arm-kernel
On Thu, Jun 15, 2017 at 08:54:09PM +0200, Javier Martinez Canillas wrote:
> The at24 driver allows to register I2C EEPROM chips using different vendor
> and devices, but the I2C subsystem does not take the vendor into account
> when matching using the I2C table since it only has device entries.
>
> But when matching using an OF table, both the vendor and device has to be
> taken into account so the driver defines only a set of compatible strings
> using the "atmel" vendor as a generic fallback for compatible I2C devices.
>
> So add this generic fallback to the device node compatible string to make
> the device to match the driver using the OF device ID table.
>
> Signed-off-by: Javier Martinez Canillas <javier@dowhile0.org>
> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
Thanks, I have queued this up in the renesas tree for v4.14.
^ permalink raw reply [flat|nested] 115+ messages in thread
* [RESEND PATCH v5 07/16] ARM: dts: koelsch: Add generic compatible string for I2C EEPROM
@ 2017-06-16 13:36 ` Simon Horman
0 siblings, 0 replies; 115+ messages in thread
From: Simon Horman @ 2017-06-16 13:36 UTC (permalink / raw)
To: linux-arm-kernel
On Thu, Jun 15, 2017 at 08:54:09PM +0200, Javier Martinez Canillas wrote:
> The at24 driver allows to register I2C EEPROM chips using different vendor
> and devices, but the I2C subsystem does not take the vendor into account
> when matching using the I2C table since it only has device entries.
>
> But when matching using an OF table, both the vendor and device has to be
> taken into account so the driver defines only a set of compatible strings
> using the "atmel" vendor as a generic fallback for compatible I2C devices.
>
> So add this generic fallback to the device node compatible string to make
> the device to match the driver using the OF device ID table.
>
> Signed-off-by: Javier Martinez Canillas <javier@dowhile0.org>
> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
Thanks, I have queued this up in the renesas tree for v4.14.
^ permalink raw reply [flat|nested] 115+ messages in thread
* [RESEND PATCH v5 08/16] ARM: dts: socfpga: Add generic compatible string for I2C EEPROM
@ 2017-06-15 18:54 ` Javier Martinez Canillas
0 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-06-15 18:54 UTC (permalink / raw)
To: linux-kernel
Cc: Wolfram Sang, Rob Herring, Javier Martinez Canillas, devicetree,
Dinh Nguyen, Rob Herring, Mark Rutland, Russell King,
linux-arm-kernel
The at24 driver allows to register I2C EEPROM chips using different vendor
and devices, but the I2C subsystem does not take the vendor into account
when matching using the I2C table since it only has device entries.
But when matching using an OF table, both the vendor and device has to be
taken into account so the driver defines only a set of compatible strings
using the "atmel" vendor as a generic fallback for compatible I2C devices.
So add this generic fallback to the device node compatible string to make
the device to match the driver using the OF device ID table.
Signed-off-by: Javier Martinez Canillas <javier@dowhile0.org>
---
Changes in v5: None
Changes in v4:
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
Changes in v3: None
Changes in v2: None
arch/arm/boot/dts/socfpga_cyclone5_vining_fpga.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/socfpga_cyclone5_vining_fpga.dts b/arch/arm/boot/dts/socfpga_cyclone5_vining_fpga.dts
index 893198049397..4bb2791ab04a 100644
--- a/arch/arm/boot/dts/socfpga_cyclone5_vining_fpga.dts
+++ b/arch/arm/boot/dts/socfpga_cyclone5_vining_fpga.dts
@@ -294,7 +294,7 @@
clock-frequency = <100000>;
at24@50 {
- compatible = "at24,24c02";
+ compatible = "atmel,24c02";
pagesize = <8>;
reg = <0x50>;
};
--
2.9.3
^ permalink raw reply related [flat|nested] 115+ messages in thread* [RESEND PATCH v5 08/16] ARM: dts: socfpga: Add generic compatible string for I2C EEPROM
@ 2017-06-15 18:54 ` Javier Martinez Canillas
0 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-06-15 18:54 UTC (permalink / raw)
To: linux-arm-kernel
The at24 driver allows to register I2C EEPROM chips using different vendor
and devices, but the I2C subsystem does not take the vendor into account
when matching using the I2C table since it only has device entries.
But when matching using an OF table, both the vendor and device has to be
taken into account so the driver defines only a set of compatible strings
using the "atmel" vendor as a generic fallback for compatible I2C devices.
So add this generic fallback to the device node compatible string to make
the device to match the driver using the OF device ID table.
Signed-off-by: Javier Martinez Canillas <javier@dowhile0.org>
---
Changes in v5: None
Changes in v4:
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
Changes in v3: None
Changes in v2: None
arch/arm/boot/dts/socfpga_cyclone5_vining_fpga.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/socfpga_cyclone5_vining_fpga.dts b/arch/arm/boot/dts/socfpga_cyclone5_vining_fpga.dts
index 893198049397..4bb2791ab04a 100644
--- a/arch/arm/boot/dts/socfpga_cyclone5_vining_fpga.dts
+++ b/arch/arm/boot/dts/socfpga_cyclone5_vining_fpga.dts
@@ -294,7 +294,7 @@
clock-frequency = <100000>;
at24 at 50 {
- compatible = "at24,24c02";
+ compatible = "atmel,24c02";
pagesize = <8>;
reg = <0x50>;
};
--
2.9.3
^ permalink raw reply related [flat|nested] 115+ messages in thread* [RESEND PATCH v5 08/16] ARM: dts: socfpga: Add generic compatible string for I2C EEPROM
@ 2017-06-15 18:54 ` Javier Martinez Canillas
0 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-06-15 18:54 UTC (permalink / raw)
To: linux-kernel-u79uwXL29TY76Z2rM5mHXA
Cc: Wolfram Sang, Rob Herring, Javier Martinez Canillas,
devicetree-u79uwXL29TY76Z2rM5mHXA, Dinh Nguyen, Rob Herring,
Mark Rutland, Russell King,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
The at24 driver allows to register I2C EEPROM chips using different vendor
and devices, but the I2C subsystem does not take the vendor into account
when matching using the I2C table since it only has device entries.
But when matching using an OF table, both the vendor and device has to be
taken into account so the driver defines only a set of compatible strings
using the "atmel" vendor as a generic fallback for compatible I2C devices.
So add this generic fallback to the device node compatible string to make
the device to match the driver using the OF device ID table.
Signed-off-by: Javier Martinez Canillas <javier-0uQlZySMnqxg9hUCZPvPmw@public.gmane.org>
---
Changes in v5: None
Changes in v4:
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
Changes in v3: None
Changes in v2: None
arch/arm/boot/dts/socfpga_cyclone5_vining_fpga.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/socfpga_cyclone5_vining_fpga.dts b/arch/arm/boot/dts/socfpga_cyclone5_vining_fpga.dts
index 893198049397..4bb2791ab04a 100644
--- a/arch/arm/boot/dts/socfpga_cyclone5_vining_fpga.dts
+++ b/arch/arm/boot/dts/socfpga_cyclone5_vining_fpga.dts
@@ -294,7 +294,7 @@
clock-frequency = <100000>;
at24@50 {
- compatible = "at24,24c02";
+ compatible = "atmel,24c02";
pagesize = <8>;
reg = <0x50>;
};
--
2.9.3
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply related [flat|nested] 115+ messages in thread* Re: [RESEND PATCH v5 08/16] ARM: dts: socfpga: Add generic compatible string for I2C EEPROM
@ 2017-11-07 8:20 ` Javier Martinez Canillas
0 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-11-07 8:20 UTC (permalink / raw)
To: Dinh Nguyen
Cc: Wolfram Sang, Rob Herring, Javier Martinez Canillas, devicetree,
Rob Herring, Mark Rutland, Russell King, linux-arm-kernel,
Linux Kernel
Hello Dinh,
On Thu, Jun 15, 2017 at 8:54 PM, Javier Martinez Canillas
<javier@dowhile0.org> wrote:
> The at24 driver allows to register I2C EEPROM chips using different vendor
> and devices, but the I2C subsystem does not take the vendor into account
> when matching using the I2C table since it only has device entries.
>
> But when matching using an OF table, both the vendor and device has to be
> taken into account so the driver defines only a set of compatible strings
> using the "atmel" vendor as a generic fallback for compatible I2C devices.
>
> So add this generic fallback to the device node compatible string to make
> the device to match the driver using the OF device ID table.
>
> Signed-off-by: Javier Martinez Canillas <javier@dowhile0.org>
>
> ---
>
> Changes in v5: None
> Changes in v4:
> - Only use the atmel manufacturer in the compatible string instead of
> keeping the deprecated ones (Rob Herring).
>
> Changes in v3: None
> Changes in v2: None
>
> arch/arm/boot/dts/socfpga_cyclone5_vining_fpga.dts | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/socfpga_cyclone5_vining_fpga.dts b/arch/arm/boot/dts/socfpga_cyclone5_vining_fpga.dts
> index 893198049397..4bb2791ab04a 100644
> --- a/arch/arm/boot/dts/socfpga_cyclone5_vining_fpga.dts
> +++ b/arch/arm/boot/dts/socfpga_cyclone5_vining_fpga.dts
> @@ -294,7 +294,7 @@
> clock-frequency = <100000>;
>
> at24@50 {
> - compatible = "at24,24c02";
> + compatible = "atmel,24c02";
> pagesize = <8>;
> reg = <0x50>;
> };
> --
Any comments about this patch?
Best regards,
Javier
^ permalink raw reply [flat|nested] 115+ messages in thread* [RESEND PATCH v5 08/16] ARM: dts: socfpga: Add generic compatible string for I2C EEPROM
@ 2017-11-07 8:20 ` Javier Martinez Canillas
0 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-11-07 8:20 UTC (permalink / raw)
To: linux-arm-kernel
Hello Dinh,
On Thu, Jun 15, 2017 at 8:54 PM, Javier Martinez Canillas
<javier@dowhile0.org> wrote:
> The at24 driver allows to register I2C EEPROM chips using different vendor
> and devices, but the I2C subsystem does not take the vendor into account
> when matching using the I2C table since it only has device entries.
>
> But when matching using an OF table, both the vendor and device has to be
> taken into account so the driver defines only a set of compatible strings
> using the "atmel" vendor as a generic fallback for compatible I2C devices.
>
> So add this generic fallback to the device node compatible string to make
> the device to match the driver using the OF device ID table.
>
> Signed-off-by: Javier Martinez Canillas <javier@dowhile0.org>
>
> ---
>
> Changes in v5: None
> Changes in v4:
> - Only use the atmel manufacturer in the compatible string instead of
> keeping the deprecated ones (Rob Herring).
>
> Changes in v3: None
> Changes in v2: None
>
> arch/arm/boot/dts/socfpga_cyclone5_vining_fpga.dts | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/socfpga_cyclone5_vining_fpga.dts b/arch/arm/boot/dts/socfpga_cyclone5_vining_fpga.dts
> index 893198049397..4bb2791ab04a 100644
> --- a/arch/arm/boot/dts/socfpga_cyclone5_vining_fpga.dts
> +++ b/arch/arm/boot/dts/socfpga_cyclone5_vining_fpga.dts
> @@ -294,7 +294,7 @@
> clock-frequency = <100000>;
>
> at24 at 50 {
> - compatible = "at24,24c02";
> + compatible = "atmel,24c02";
> pagesize = <8>;
> reg = <0x50>;
> };
> --
Any comments about this patch?
Best regards,
Javier
^ permalink raw reply [flat|nested] 115+ messages in thread* Re: [RESEND PATCH v5 08/16] ARM: dts: socfpga: Add generic compatible string for I2C EEPROM
@ 2017-11-07 8:20 ` Javier Martinez Canillas
0 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-11-07 8:20 UTC (permalink / raw)
To: Dinh Nguyen
Cc: Wolfram Sang, Rob Herring, Javier Martinez Canillas,
devicetree-u79uwXL29TY76Z2rM5mHXA, Rob Herring, Mark Rutland,
Russell King, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
Linux Kernel
Hello Dinh,
On Thu, Jun 15, 2017 at 8:54 PM, Javier Martinez Canillas
<javier-0uQlZySMnqxg9hUCZPvPmw@public.gmane.org> wrote:
> The at24 driver allows to register I2C EEPROM chips using different vendor
> and devices, but the I2C subsystem does not take the vendor into account
> when matching using the I2C table since it only has device entries.
>
> But when matching using an OF table, both the vendor and device has to be
> taken into account so the driver defines only a set of compatible strings
> using the "atmel" vendor as a generic fallback for compatible I2C devices.
>
> So add this generic fallback to the device node compatible string to make
> the device to match the driver using the OF device ID table.
>
> Signed-off-by: Javier Martinez Canillas <javier-0uQlZySMnqxg9hUCZPvPmw@public.gmane.org>
>
> ---
>
> Changes in v5: None
> Changes in v4:
> - Only use the atmel manufacturer in the compatible string instead of
> keeping the deprecated ones (Rob Herring).
>
> Changes in v3: None
> Changes in v2: None
>
> arch/arm/boot/dts/socfpga_cyclone5_vining_fpga.dts | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/socfpga_cyclone5_vining_fpga.dts b/arch/arm/boot/dts/socfpga_cyclone5_vining_fpga.dts
> index 893198049397..4bb2791ab04a 100644
> --- a/arch/arm/boot/dts/socfpga_cyclone5_vining_fpga.dts
> +++ b/arch/arm/boot/dts/socfpga_cyclone5_vining_fpga.dts
> @@ -294,7 +294,7 @@
> clock-frequency = <100000>;
>
> at24@50 {
> - compatible = "at24,24c02";
> + compatible = "atmel,24c02";
> pagesize = <8>;
> reg = <0x50>;
> };
> --
Any comments about this patch?
Best regards,
Javier
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 115+ messages in thread
* [RESEND PATCH v5 09/16] ARM: dts: uniphier: Add generic compatible string for I2C EEPROM
2017-06-15 18:54 ` Javier Martinez Canillas
@ 2017-06-15 18:54 ` Javier Martinez Canillas
-1 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-06-15 18:54 UTC (permalink / raw)
To: linux-kernel
Cc: Wolfram Sang, Rob Herring, Javier Martinez Canillas, devicetree,
Masahiro Yamada, Rob Herring, Mark Rutland, Russell King,
linux-arm-kernel
The at24 driver allows to register I2C EEPROM chips using different vendor
and devices, but the I2C subsystem does not take the vendor into account
when matching using the I2C table since it only has device entries.
But when matching using an OF table, both the vendor and device has to be
taken into account so the driver defines only a set of compatible strings
using the "atmel" vendor as a generic fallback for compatible I2C devices.
So add this generic fallback to the device node compatible string to make
the device to match the driver using the OF device ID table.
Signed-off-by: Javier Martinez Canillas <javier@dowhile0.org>
---
Changes in v5:
- Only replace atmel variant but keep other EEPROM vendors (Geert Uytterhoeven).
Changes in v4:
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
Changes in v3: None
Changes in v2: None
arch/arm/boot/dts/uniphier-pro4-ace.dts | 2 +-
arch/arm/boot/dts/uniphier-pro4-sanji.dts | 2 +-
arch/arm/boot/dts/uniphier-pxs2-gentil.dts | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/arch/arm/boot/dts/uniphier-pro4-ace.dts b/arch/arm/boot/dts/uniphier-pro4-ace.dts
index fefc89149234..b9a4919d7c93 100644
--- a/arch/arm/boot/dts/uniphier-pro4-ace.dts
+++ b/arch/arm/boot/dts/uniphier-pro4-ace.dts
@@ -88,7 +88,7 @@
status = "okay";
eeprom@54 {
- compatible = "st,24c64";
+ compatible = "st,24c64", "atmel,24c64";
reg = <0x54>;
pagesize = <32>;
};
diff --git a/arch/arm/boot/dts/uniphier-pro4-sanji.dts b/arch/arm/boot/dts/uniphier-pro4-sanji.dts
index 6c63c8bad825..3950baa8d55d 100644
--- a/arch/arm/boot/dts/uniphier-pro4-sanji.dts
+++ b/arch/arm/boot/dts/uniphier-pro4-sanji.dts
@@ -83,7 +83,7 @@
status = "okay";
eeprom@54 {
- compatible = "st,24c64";
+ compatible = "st,24c64", "atmel,24c64";
reg = <0x54>;
pagesize = <32>;
};
diff --git a/arch/arm/boot/dts/uniphier-pxs2-gentil.dts b/arch/arm/boot/dts/uniphier-pxs2-gentil.dts
index cccc86658d20..afba587ffbbc 100644
--- a/arch/arm/boot/dts/uniphier-pxs2-gentil.dts
+++ b/arch/arm/boot/dts/uniphier-pxs2-gentil.dts
@@ -80,7 +80,7 @@
status = "okay";
eeprom@54 {
- compatible = "st,24c64";
+ compatible = "st,24c64", "atmel,24c64";
reg = <0x54>;
pagesize = <32>;
};
--
2.9.3
^ permalink raw reply related [flat|nested] 115+ messages in thread* [RESEND PATCH v5 09/16] ARM: dts: uniphier: Add generic compatible string for I2C EEPROM
@ 2017-06-15 18:54 ` Javier Martinez Canillas
0 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-06-15 18:54 UTC (permalink / raw)
To: linux-arm-kernel
The at24 driver allows to register I2C EEPROM chips using different vendor
and devices, but the I2C subsystem does not take the vendor into account
when matching using the I2C table since it only has device entries.
But when matching using an OF table, both the vendor and device has to be
taken into account so the driver defines only a set of compatible strings
using the "atmel" vendor as a generic fallback for compatible I2C devices.
So add this generic fallback to the device node compatible string to make
the device to match the driver using the OF device ID table.
Signed-off-by: Javier Martinez Canillas <javier@dowhile0.org>
---
Changes in v5:
- Only replace atmel variant but keep other EEPROM vendors (Geert Uytterhoeven).
Changes in v4:
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
Changes in v3: None
Changes in v2: None
arch/arm/boot/dts/uniphier-pro4-ace.dts | 2 +-
arch/arm/boot/dts/uniphier-pro4-sanji.dts | 2 +-
arch/arm/boot/dts/uniphier-pxs2-gentil.dts | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/arch/arm/boot/dts/uniphier-pro4-ace.dts b/arch/arm/boot/dts/uniphier-pro4-ace.dts
index fefc89149234..b9a4919d7c93 100644
--- a/arch/arm/boot/dts/uniphier-pro4-ace.dts
+++ b/arch/arm/boot/dts/uniphier-pro4-ace.dts
@@ -88,7 +88,7 @@
status = "okay";
eeprom at 54 {
- compatible = "st,24c64";
+ compatible = "st,24c64", "atmel,24c64";
reg = <0x54>;
pagesize = <32>;
};
diff --git a/arch/arm/boot/dts/uniphier-pro4-sanji.dts b/arch/arm/boot/dts/uniphier-pro4-sanji.dts
index 6c63c8bad825..3950baa8d55d 100644
--- a/arch/arm/boot/dts/uniphier-pro4-sanji.dts
+++ b/arch/arm/boot/dts/uniphier-pro4-sanji.dts
@@ -83,7 +83,7 @@
status = "okay";
eeprom at 54 {
- compatible = "st,24c64";
+ compatible = "st,24c64", "atmel,24c64";
reg = <0x54>;
pagesize = <32>;
};
diff --git a/arch/arm/boot/dts/uniphier-pxs2-gentil.dts b/arch/arm/boot/dts/uniphier-pxs2-gentil.dts
index cccc86658d20..afba587ffbbc 100644
--- a/arch/arm/boot/dts/uniphier-pxs2-gentil.dts
+++ b/arch/arm/boot/dts/uniphier-pxs2-gentil.dts
@@ -80,7 +80,7 @@
status = "okay";
eeprom at 54 {
- compatible = "st,24c64";
+ compatible = "st,24c64", "atmel,24c64";
reg = <0x54>;
pagesize = <32>;
};
--
2.9.3
^ permalink raw reply related [flat|nested] 115+ messages in thread
* [RESEND PATCH v5 10/16] ARM: dts: zynq: Add generic compatible string for I2C EEPROM
@ 2017-06-15 18:54 ` Javier Martinez Canillas
0 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-06-15 18:54 UTC (permalink / raw)
To: linux-kernel
Cc: Wolfram Sang, Rob Herring, Javier Martinez Canillas,
Sören Brinkmann, devicetree, Michal Simek, Rob Herring,
Mark Rutland, Russell King, linux-arm-kernel
The at24 driver allows to register I2C EEPROM chips using different vendor
and devices, but the I2C subsystem does not take the vendor into account
when matching using the I2C table since it only has device entries.
But when matching using an OF table, both the vendor and device has to be
taken into account so the driver defines only a set of compatible strings
using the "atmel" vendor as a generic fallback for compatible I2C devices.
So add this generic fallback to the device node compatible string to make
the device to match the driver using the OF device ID table.
Signed-off-by: Javier Martinez Canillas <javier@dowhile0.org>
---
Changes in v5: None
Changes in v4:
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
Changes in v3: None
Changes in v2: None
arch/arm/boot/dts/zynq-zc702.dts | 2 +-
arch/arm/boot/dts/zynq-zc706.dts | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/arm/boot/dts/zynq-zc702.dts b/arch/arm/boot/dts/zynq-zc702.dts
index 0cdad2cc8b78..f8c840f7118f 100644
--- a/arch/arm/boot/dts/zynq-zc702.dts
+++ b/arch/arm/boot/dts/zynq-zc702.dts
@@ -136,7 +136,7 @@
#size-cells = <0>;
reg = <2>;
eeprom@54 {
- compatible = "at,24c08";
+ compatible = "atmel,24c08";
reg = <0x54>;
};
};
diff --git a/arch/arm/boot/dts/zynq-zc706.dts b/arch/arm/boot/dts/zynq-zc706.dts
index ad4bb06dba25..76492be4a912 100644
--- a/arch/arm/boot/dts/zynq-zc706.dts
+++ b/arch/arm/boot/dts/zynq-zc706.dts
@@ -92,7 +92,7 @@
#size-cells = <0>;
reg = <2>;
eeprom@54 {
- compatible = "at,24c08";
+ compatible = "atmel,24c08";
reg = <0x54>;
};
};
--
2.9.3
^ permalink raw reply related [flat|nested] 115+ messages in thread* [RESEND PATCH v5 10/16] ARM: dts: zynq: Add generic compatible string for I2C EEPROM
@ 2017-06-15 18:54 ` Javier Martinez Canillas
0 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-06-15 18:54 UTC (permalink / raw)
To: linux-arm-kernel
The at24 driver allows to register I2C EEPROM chips using different vendor
and devices, but the I2C subsystem does not take the vendor into account
when matching using the I2C table since it only has device entries.
But when matching using an OF table, both the vendor and device has to be
taken into account so the driver defines only a set of compatible strings
using the "atmel" vendor as a generic fallback for compatible I2C devices.
So add this generic fallback to the device node compatible string to make
the device to match the driver using the OF device ID table.
Signed-off-by: Javier Martinez Canillas <javier@dowhile0.org>
---
Changes in v5: None
Changes in v4:
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
Changes in v3: None
Changes in v2: None
arch/arm/boot/dts/zynq-zc702.dts | 2 +-
arch/arm/boot/dts/zynq-zc706.dts | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/arm/boot/dts/zynq-zc702.dts b/arch/arm/boot/dts/zynq-zc702.dts
index 0cdad2cc8b78..f8c840f7118f 100644
--- a/arch/arm/boot/dts/zynq-zc702.dts
+++ b/arch/arm/boot/dts/zynq-zc702.dts
@@ -136,7 +136,7 @@
#size-cells = <0>;
reg = <2>;
eeprom at 54 {
- compatible = "at,24c08";
+ compatible = "atmel,24c08";
reg = <0x54>;
};
};
diff --git a/arch/arm/boot/dts/zynq-zc706.dts b/arch/arm/boot/dts/zynq-zc706.dts
index ad4bb06dba25..76492be4a912 100644
--- a/arch/arm/boot/dts/zynq-zc706.dts
+++ b/arch/arm/boot/dts/zynq-zc706.dts
@@ -92,7 +92,7 @@
#size-cells = <0>;
reg = <2>;
eeprom at 54 {
- compatible = "at,24c08";
+ compatible = "atmel,24c08";
reg = <0x54>;
};
};
--
2.9.3
^ permalink raw reply related [flat|nested] 115+ messages in thread* [RESEND PATCH v5 10/16] ARM: dts: zynq: Add generic compatible string for I2C EEPROM
@ 2017-06-15 18:54 ` Javier Martinez Canillas
0 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-06-15 18:54 UTC (permalink / raw)
To: linux-kernel-u79uwXL29TY76Z2rM5mHXA
Cc: Wolfram Sang, Rob Herring, Javier Martinez Canillas,
Sören Brinkmann, devicetree-u79uwXL29TY76Z2rM5mHXA,
Michal Simek, Rob Herring, Mark Rutland, Russell King,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
The at24 driver allows to register I2C EEPROM chips using different vendor
and devices, but the I2C subsystem does not take the vendor into account
when matching using the I2C table since it only has device entries.
But when matching using an OF table, both the vendor and device has to be
taken into account so the driver defines only a set of compatible strings
using the "atmel" vendor as a generic fallback for compatible I2C devices.
So add this generic fallback to the device node compatible string to make
the device to match the driver using the OF device ID table.
Signed-off-by: Javier Martinez Canillas <javier-0uQlZySMnqxg9hUCZPvPmw@public.gmane.org>
---
Changes in v5: None
Changes in v4:
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
Changes in v3: None
Changes in v2: None
arch/arm/boot/dts/zynq-zc702.dts | 2 +-
arch/arm/boot/dts/zynq-zc706.dts | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/arm/boot/dts/zynq-zc702.dts b/arch/arm/boot/dts/zynq-zc702.dts
index 0cdad2cc8b78..f8c840f7118f 100644
--- a/arch/arm/boot/dts/zynq-zc702.dts
+++ b/arch/arm/boot/dts/zynq-zc702.dts
@@ -136,7 +136,7 @@
#size-cells = <0>;
reg = <2>;
eeprom@54 {
- compatible = "at,24c08";
+ compatible = "atmel,24c08";
reg = <0x54>;
};
};
diff --git a/arch/arm/boot/dts/zynq-zc706.dts b/arch/arm/boot/dts/zynq-zc706.dts
index ad4bb06dba25..76492be4a912 100644
--- a/arch/arm/boot/dts/zynq-zc706.dts
+++ b/arch/arm/boot/dts/zynq-zc706.dts
@@ -92,7 +92,7 @@
#size-cells = <0>;
reg = <2>;
eeprom@54 {
- compatible = "at,24c08";
+ compatible = "atmel,24c08";
reg = <0x54>;
};
};
--
2.9.3
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply related [flat|nested] 115+ messages in thread* Re: [RESEND PATCH v5 10/16] ARM: dts: zynq: Add generic compatible string for I2C EEPROM
@ 2017-06-21 11:43 ` Michal Simek
0 siblings, 0 replies; 115+ messages in thread
From: Michal Simek @ 2017-06-21 11:43 UTC (permalink / raw)
To: Javier Martinez Canillas, linux-kernel
Cc: Wolfram Sang, Rob Herring, Sören Brinkmann, devicetree,
Michal Simek, Rob Herring, Mark Rutland, Russell King,
linux-arm-kernel
On 15.6.2017 20:54, Javier Martinez Canillas wrote:
> The at24 driver allows to register I2C EEPROM chips using different vendor
> and devices, but the I2C subsystem does not take the vendor into account
> when matching using the I2C table since it only has device entries.
>
> But when matching using an OF table, both the vendor and device has to be
> taken into account so the driver defines only a set of compatible strings
> using the "atmel" vendor as a generic fallback for compatible I2C devices.
>
> So add this generic fallback to the device node compatible string to make
> the device to match the driver using the OF device ID table.
>
> Signed-off-by: Javier Martinez Canillas <javier@dowhile0.org>
>
> ---
>
> Changes in v5: None
> Changes in v4:
> - Only use the atmel manufacturer in the compatible string instead of
> keeping the deprecated ones (Rob Herring).
>
> Changes in v3: None
> Changes in v2: None
>
> arch/arm/boot/dts/zynq-zc702.dts | 2 +-
> arch/arm/boot/dts/zynq-zc706.dts | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm/boot/dts/zynq-zc702.dts b/arch/arm/boot/dts/zynq-zc702.dts
> index 0cdad2cc8b78..f8c840f7118f 100644
> --- a/arch/arm/boot/dts/zynq-zc702.dts
> +++ b/arch/arm/boot/dts/zynq-zc702.dts
> @@ -136,7 +136,7 @@
> #size-cells = <0>;
> reg = <2>;
> eeprom@54 {
> - compatible = "at,24c08";
> + compatible = "atmel,24c08";
> reg = <0x54>;
> };
> };
> diff --git a/arch/arm/boot/dts/zynq-zc706.dts b/arch/arm/boot/dts/zynq-zc706.dts
> index ad4bb06dba25..76492be4a912 100644
> --- a/arch/arm/boot/dts/zynq-zc706.dts
> +++ b/arch/arm/boot/dts/zynq-zc706.dts
> @@ -92,7 +92,7 @@
> #size-cells = <0>;
> reg = <2>;
> eeprom@54 {
> - compatible = "at,24c08";
> + compatible = "atmel,24c08";
> reg = <0x54>;
> };
> };
>
Acked-by: Michal Simek <michal.simek@xilinx.com>
Thanks,
Michal
^ permalink raw reply [flat|nested] 115+ messages in thread* [RESEND PATCH v5 10/16] ARM: dts: zynq: Add generic compatible string for I2C EEPROM
@ 2017-06-21 11:43 ` Michal Simek
0 siblings, 0 replies; 115+ messages in thread
From: Michal Simek @ 2017-06-21 11:43 UTC (permalink / raw)
To: linux-arm-kernel
On 15.6.2017 20:54, Javier Martinez Canillas wrote:
> The at24 driver allows to register I2C EEPROM chips using different vendor
> and devices, but the I2C subsystem does not take the vendor into account
> when matching using the I2C table since it only has device entries.
>
> But when matching using an OF table, both the vendor and device has to be
> taken into account so the driver defines only a set of compatible strings
> using the "atmel" vendor as a generic fallback for compatible I2C devices.
>
> So add this generic fallback to the device node compatible string to make
> the device to match the driver using the OF device ID table.
>
> Signed-off-by: Javier Martinez Canillas <javier@dowhile0.org>
>
> ---
>
> Changes in v5: None
> Changes in v4:
> - Only use the atmel manufacturer in the compatible string instead of
> keeping the deprecated ones (Rob Herring).
>
> Changes in v3: None
> Changes in v2: None
>
> arch/arm/boot/dts/zynq-zc702.dts | 2 +-
> arch/arm/boot/dts/zynq-zc706.dts | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm/boot/dts/zynq-zc702.dts b/arch/arm/boot/dts/zynq-zc702.dts
> index 0cdad2cc8b78..f8c840f7118f 100644
> --- a/arch/arm/boot/dts/zynq-zc702.dts
> +++ b/arch/arm/boot/dts/zynq-zc702.dts
> @@ -136,7 +136,7 @@
> #size-cells = <0>;
> reg = <2>;
> eeprom at 54 {
> - compatible = "at,24c08";
> + compatible = "atmel,24c08";
> reg = <0x54>;
> };
> };
> diff --git a/arch/arm/boot/dts/zynq-zc706.dts b/arch/arm/boot/dts/zynq-zc706.dts
> index ad4bb06dba25..76492be4a912 100644
> --- a/arch/arm/boot/dts/zynq-zc706.dts
> +++ b/arch/arm/boot/dts/zynq-zc706.dts
> @@ -92,7 +92,7 @@
> #size-cells = <0>;
> reg = <2>;
> eeprom at 54 {
> - compatible = "at,24c08";
> + compatible = "atmel,24c08";
> reg = <0x54>;
> };
> };
>
Acked-by: Michal Simek <michal.simek@xilinx.com>
Thanks,
Michal
^ permalink raw reply [flat|nested] 115+ messages in thread* Re: [RESEND PATCH v5 10/16] ARM: dts: zynq: Add generic compatible string for I2C EEPROM
@ 2017-06-21 11:43 ` Michal Simek
0 siblings, 0 replies; 115+ messages in thread
From: Michal Simek @ 2017-06-21 11:43 UTC (permalink / raw)
To: Javier Martinez Canillas, linux-kernel-u79uwXL29TY76Z2rM5mHXA
Cc: Wolfram Sang, Rob Herring, Sören Brinkmann,
devicetree-u79uwXL29TY76Z2rM5mHXA, Michal Simek, Rob Herring,
Mark Rutland, Russell King,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
On 15.6.2017 20:54, Javier Martinez Canillas wrote:
> The at24 driver allows to register I2C EEPROM chips using different vendor
> and devices, but the I2C subsystem does not take the vendor into account
> when matching using the I2C table since it only has device entries.
>
> But when matching using an OF table, both the vendor and device has to be
> taken into account so the driver defines only a set of compatible strings
> using the "atmel" vendor as a generic fallback for compatible I2C devices.
>
> So add this generic fallback to the device node compatible string to make
> the device to match the driver using the OF device ID table.
>
> Signed-off-by: Javier Martinez Canillas <javier-0uQlZySMnqxg9hUCZPvPmw@public.gmane.org>
>
> ---
>
> Changes in v5: None
> Changes in v4:
> - Only use the atmel manufacturer in the compatible string instead of
> keeping the deprecated ones (Rob Herring).
>
> Changes in v3: None
> Changes in v2: None
>
> arch/arm/boot/dts/zynq-zc702.dts | 2 +-
> arch/arm/boot/dts/zynq-zc706.dts | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm/boot/dts/zynq-zc702.dts b/arch/arm/boot/dts/zynq-zc702.dts
> index 0cdad2cc8b78..f8c840f7118f 100644
> --- a/arch/arm/boot/dts/zynq-zc702.dts
> +++ b/arch/arm/boot/dts/zynq-zc702.dts
> @@ -136,7 +136,7 @@
> #size-cells = <0>;
> reg = <2>;
> eeprom@54 {
> - compatible = "at,24c08";
> + compatible = "atmel,24c08";
> reg = <0x54>;
> };
> };
> diff --git a/arch/arm/boot/dts/zynq-zc706.dts b/arch/arm/boot/dts/zynq-zc706.dts
> index ad4bb06dba25..76492be4a912 100644
> --- a/arch/arm/boot/dts/zynq-zc706.dts
> +++ b/arch/arm/boot/dts/zynq-zc706.dts
> @@ -92,7 +92,7 @@
> #size-cells = <0>;
> reg = <2>;
> eeprom@54 {
> - compatible = "at,24c08";
> + compatible = "atmel,24c08";
> reg = <0x54>;
> };
> };
>
Acked-by: Michal Simek <michal.simek-gjFFaj9aHVfQT0dZR+AlfA@public.gmane.org>
Thanks,
Michal
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 115+ messages in thread* Re: [RESEND PATCH v5 10/16] ARM: dts: zynq: Add generic compatible string for I2C EEPROM
@ 2017-08-23 14:28 ` Javier Martinez Canillas
0 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-08-23 14:28 UTC (permalink / raw)
To: Michal Simek
Cc: Linux Kernel, Wolfram Sang, Rob Herring, Sören Brinkmann,
devicetree, Rob Herring, Mark Rutland, Russell King,
linux-arm-kernel
Hello Michal,
On Wed, Jun 21, 2017 at 1:43 PM, Michal Simek <michal.simek@xilinx.com> wrote:
> On 15.6.2017 20:54, Javier Martinez Canillas wrote:
>> The at24 driver allows to register I2C EEPROM chips using different vendor
>> and devices, but the I2C subsystem does not take the vendor into account
>> when matching using the I2C table since it only has device entries.
>>
>> But when matching using an OF table, both the vendor and device has to be
>> taken into account so the driver defines only a set of compatible strings
>> using the "atmel" vendor as a generic fallback for compatible I2C devices.
>>
>> So add this generic fallback to the device node compatible string to make
>> the device to match the driver using the OF device ID table.
>>
>> Signed-off-by: Javier Martinez Canillas <javier@dowhile0.org>
>>
[snip]
>> --- a/arch/arm/boot/dts/zynq-zc706.dts
>> +++ b/arch/arm/boot/dts/zynq-zc706.dts
>> @@ -92,7 +92,7 @@
>> #size-cells = <0>;
>> reg = <2>;
>> eeprom@54 {
>> - compatible = "at,24c08";
>> + compatible = "atmel,24c08";
>> reg = <0x54>;
>> };
>> };
>>
>
> Acked-by: Michal Simek <michal.simek@xilinx.com>
>
Same question than for the other patch, could you please also apply this?
Sorry for not being clear in the cover letter that the patches could
be applied independently.
Best regards,
Javier
^ permalink raw reply [flat|nested] 115+ messages in thread* [RESEND PATCH v5 10/16] ARM: dts: zynq: Add generic compatible string for I2C EEPROM
@ 2017-08-23 14:28 ` Javier Martinez Canillas
0 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-08-23 14:28 UTC (permalink / raw)
To: linux-arm-kernel
Hello Michal,
On Wed, Jun 21, 2017 at 1:43 PM, Michal Simek <michal.simek@xilinx.com> wrote:
> On 15.6.2017 20:54, Javier Martinez Canillas wrote:
>> The at24 driver allows to register I2C EEPROM chips using different vendor
>> and devices, but the I2C subsystem does not take the vendor into account
>> when matching using the I2C table since it only has device entries.
>>
>> But when matching using an OF table, both the vendor and device has to be
>> taken into account so the driver defines only a set of compatible strings
>> using the "atmel" vendor as a generic fallback for compatible I2C devices.
>>
>> So add this generic fallback to the device node compatible string to make
>> the device to match the driver using the OF device ID table.
>>
>> Signed-off-by: Javier Martinez Canillas <javier@dowhile0.org>
>>
[snip]
>> --- a/arch/arm/boot/dts/zynq-zc706.dts
>> +++ b/arch/arm/boot/dts/zynq-zc706.dts
>> @@ -92,7 +92,7 @@
>> #size-cells = <0>;
>> reg = <2>;
>> eeprom at 54 {
>> - compatible = "at,24c08";
>> + compatible = "atmel,24c08";
>> reg = <0x54>;
>> };
>> };
>>
>
> Acked-by: Michal Simek <michal.simek@xilinx.com>
>
Same question than for the other patch, could you please also apply this?
Sorry for not being clear in the cover letter that the patches could
be applied independently.
Best regards,
Javier
^ permalink raw reply [flat|nested] 115+ messages in thread* Re: [RESEND PATCH v5 10/16] ARM: dts: zynq: Add generic compatible string for I2C EEPROM
@ 2017-08-23 14:28 ` Javier Martinez Canillas
0 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-08-23 14:28 UTC (permalink / raw)
To: Michal Simek
Cc: Linux Kernel, Wolfram Sang, Rob Herring, Sören Brinkmann,
devicetree-u79uwXL29TY76Z2rM5mHXA, Rob Herring, Mark Rutland,
Russell King, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
Hello Michal,
On Wed, Jun 21, 2017 at 1:43 PM, Michal Simek <michal.simek-gjFFaj9aHVfQT0dZR+AlfA@public.gmane.org> wrote:
> On 15.6.2017 20:54, Javier Martinez Canillas wrote:
>> The at24 driver allows to register I2C EEPROM chips using different vendor
>> and devices, but the I2C subsystem does not take the vendor into account
>> when matching using the I2C table since it only has device entries.
>>
>> But when matching using an OF table, both the vendor and device has to be
>> taken into account so the driver defines only a set of compatible strings
>> using the "atmel" vendor as a generic fallback for compatible I2C devices.
>>
>> So add this generic fallback to the device node compatible string to make
>> the device to match the driver using the OF device ID table.
>>
>> Signed-off-by: Javier Martinez Canillas <javier-0uQlZySMnqxg9hUCZPvPmw@public.gmane.org>
>>
[snip]
>> --- a/arch/arm/boot/dts/zynq-zc706.dts
>> +++ b/arch/arm/boot/dts/zynq-zc706.dts
>> @@ -92,7 +92,7 @@
>> #size-cells = <0>;
>> reg = <2>;
>> eeprom@54 {
>> - compatible = "at,24c08";
>> + compatible = "atmel,24c08";
>> reg = <0x54>;
>> };
>> };
>>
>
> Acked-by: Michal Simek <michal.simek-gjFFaj9aHVfQT0dZR+AlfA@public.gmane.org>
>
Same question than for the other patch, could you please also apply this?
Sorry for not being clear in the cover letter that the patches could
be applied independently.
Best regards,
Javier
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 115+ messages in thread* Re: [RESEND PATCH v5 10/16] ARM: dts: zynq: Add generic compatible string for I2C EEPROM
2017-08-23 14:28 ` Javier Martinez Canillas
@ 2017-08-24 13:44 ` Michal Simek
-1 siblings, 0 replies; 115+ messages in thread
From: Michal Simek @ 2017-08-24 13:44 UTC (permalink / raw)
To: Javier Martinez Canillas, Michal Simek, Arnd Bergmann
Cc: Linux Kernel, Wolfram Sang, Rob Herring, Sören Brinkmann,
devicetree, Rob Herring, Mark Rutland, Russell King,
linux-arm-kernel
Hi,
On 23.8.2017 16:28, Javier Martinez Canillas wrote:
> Hello Michal,
>
> On Wed, Jun 21, 2017 at 1:43 PM, Michal Simek <michal.simek@xilinx.com> wrote:
>> On 15.6.2017 20:54, Javier Martinez Canillas wrote:
>>> The at24 driver allows to register I2C EEPROM chips using different vendor
>>> and devices, but the I2C subsystem does not take the vendor into account
>>> when matching using the I2C table since it only has device entries.
>>>
>>> But when matching using an OF table, both the vendor and device has to be
>>> taken into account so the driver defines only a set of compatible strings
>>> using the "atmel" vendor as a generic fallback for compatible I2C devices.
>>>
>>> So add this generic fallback to the device node compatible string to make
>>> the device to match the driver using the OF device ID table.
>>>
>>> Signed-off-by: Javier Martinez Canillas <javier@dowhile0.org>
>>>
>
> [snip]
>
>>> --- a/arch/arm/boot/dts/zynq-zc706.dts
>>> +++ b/arch/arm/boot/dts/zynq-zc706.dts
>>> @@ -92,7 +92,7 @@
>>> #size-cells = <0>;
>>> reg = <2>;
>>> eeprom@54 {
>>> - compatible = "at,24c08";
>>> + compatible = "atmel,24c08";
>>> reg = <0x54>;
>>> };
>>> };
>>>
>>
>> Acked-by: Michal Simek <michal.simek@xilinx.com>
>>
>
> Same question than for the other patch, could you please also apply this?
>
> Sorry for not being clear in the cover letter that the patches could
> be applied independently.
Ok. Applied to
https://github.com/Xilinx/linux-xlnx/commits/zynq/dt
Will wait if something else pop up.
Arnd: If you want to take this directly, feel free to do it.
Thanks,
Michal
^ permalink raw reply [flat|nested] 115+ messages in thread* [RESEND PATCH v5 10/16] ARM: dts: zynq: Add generic compatible string for I2C EEPROM
@ 2017-08-24 13:44 ` Michal Simek
0 siblings, 0 replies; 115+ messages in thread
From: Michal Simek @ 2017-08-24 13:44 UTC (permalink / raw)
To: linux-arm-kernel
Hi,
On 23.8.2017 16:28, Javier Martinez Canillas wrote:
> Hello Michal,
>
> On Wed, Jun 21, 2017 at 1:43 PM, Michal Simek <michal.simek@xilinx.com> wrote:
>> On 15.6.2017 20:54, Javier Martinez Canillas wrote:
>>> The at24 driver allows to register I2C EEPROM chips using different vendor
>>> and devices, but the I2C subsystem does not take the vendor into account
>>> when matching using the I2C table since it only has device entries.
>>>
>>> But when matching using an OF table, both the vendor and device has to be
>>> taken into account so the driver defines only a set of compatible strings
>>> using the "atmel" vendor as a generic fallback for compatible I2C devices.
>>>
>>> So add this generic fallback to the device node compatible string to make
>>> the device to match the driver using the OF device ID table.
>>>
>>> Signed-off-by: Javier Martinez Canillas <javier@dowhile0.org>
>>>
>
> [snip]
>
>>> --- a/arch/arm/boot/dts/zynq-zc706.dts
>>> +++ b/arch/arm/boot/dts/zynq-zc706.dts
>>> @@ -92,7 +92,7 @@
>>> #size-cells = <0>;
>>> reg = <2>;
>>> eeprom at 54 {
>>> - compatible = "at,24c08";
>>> + compatible = "atmel,24c08";
>>> reg = <0x54>;
>>> };
>>> };
>>>
>>
>> Acked-by: Michal Simek <michal.simek@xilinx.com>
>>
>
> Same question than for the other patch, could you please also apply this?
>
> Sorry for not being clear in the cover letter that the patches could
> be applied independently.
Ok. Applied to
https://github.com/Xilinx/linux-xlnx/commits/zynq/dt
Will wait if something else pop up.
Arnd: If you want to take this directly, feel free to do it.
Thanks,
Michal
^ permalink raw reply [flat|nested] 115+ messages in thread* Re: [RESEND PATCH v5 10/16] ARM: dts: zynq: Add generic compatible string for I2C EEPROM
@ 2017-09-21 9:41 ` Javier Martinez Canillas
0 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-09-21 9:41 UTC (permalink / raw)
To: Michal Simek
Cc: Arnd Bergmann, Linux Kernel, Wolfram Sang, Rob Herring,
Sören Brinkmann, devicetree, Rob Herring, Mark Rutland,
Russell King, linux-arm-kernel
Hello Michal,
On Thu, Aug 24, 2017 at 3:44 PM, Michal Simek <michal.simek@xilinx.com> wrote:
[snip]
>
> Ok. Applied to
> https://github.com/Xilinx/linux-xlnx/commits/zynq/dt
>
> Will wait if something else pop up.
>
I see the patch applied to that repository but I don't see it in
neither mainline nor linux-next.
> Arnd: If you want to take this directly, feel free to do it.
>
> Thanks,
> Michal
Best regards,
Javier
^ permalink raw reply [flat|nested] 115+ messages in thread
* [RESEND PATCH v5 10/16] ARM: dts: zynq: Add generic compatible string for I2C EEPROM
@ 2017-09-21 9:41 ` Javier Martinez Canillas
0 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-09-21 9:41 UTC (permalink / raw)
To: linux-arm-kernel
Hello Michal,
On Thu, Aug 24, 2017 at 3:44 PM, Michal Simek <michal.simek@xilinx.com> wrote:
[snip]
>
> Ok. Applied to
> https://github.com/Xilinx/linux-xlnx/commits/zynq/dt
>
> Will wait if something else pop up.
>
I see the patch applied to that repository but I don't see it in
neither mainline nor linux-next.
> Arnd: If you want to take this directly, feel free to do it.
>
> Thanks,
> Michal
Best regards,
Javier
^ permalink raw reply [flat|nested] 115+ messages in thread
* Re: [RESEND PATCH v5 10/16] ARM: dts: zynq: Add generic compatible string for I2C EEPROM
@ 2017-09-21 9:41 ` Javier Martinez Canillas
0 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-09-21 9:41 UTC (permalink / raw)
To: Michal Simek
Cc: Arnd Bergmann, Linux Kernel, Wolfram Sang, Rob Herring,
Sören Brinkmann, devicetree-u79uwXL29TY76Z2rM5mHXA,
Rob Herring, Mark Rutland, Russell King,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
Hello Michal,
On Thu, Aug 24, 2017 at 3:44 PM, Michal Simek <michal.simek-gjFFaj9aHVfQT0dZR+AlfA@public.gmane.org> wrote:
[snip]
>
> Ok. Applied to
> https://github.com/Xilinx/linux-xlnx/commits/zynq/dt
>
> Will wait if something else pop up.
>
I see the patch applied to that repository but I don't see it in
neither mainline nor linux-next.
> Arnd: If you want to take this directly, feel free to do it.
>
> Thanks,
> Michal
Best regards,
Javier
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 115+ messages in thread
* Re: [RESEND PATCH v5 10/16] ARM: dts: zynq: Add generic compatible string for I2C EEPROM
2017-09-21 9:41 ` Javier Martinez Canillas
@ 2017-09-25 8:06 ` Michal Simek
-1 siblings, 0 replies; 115+ messages in thread
From: Michal Simek @ 2017-09-25 8:06 UTC (permalink / raw)
To: Javier Martinez Canillas, Michal Simek
Cc: Arnd Bergmann, Linux Kernel, Wolfram Sang, Rob Herring,
Sören Brinkmann, devicetree, Rob Herring, Mark Rutland,
Russell King, linux-arm-kernel
Hi,
On 21.9.2017 11:41, Javier Martinez Canillas wrote:
> Hello Michal,
>
> On Thu, Aug 24, 2017 at 3:44 PM, Michal Simek <michal.simek@xilinx.com> wrote:
>
> [snip]
>
>>
>> Ok. Applied to
>> https://github.com/Xilinx/linux-xlnx/commits/zynq/dt
>>
>> Will wait if something else pop up.
>>
>
> I see the patch applied to that repository but I don't see it in
> neither mainline nor linux-next.
xilinx tree was merging to linux-next but there were conflicts with arm
soc tree that's why it was removed.
>
>> Arnd: If you want to take this directly, feel free to do it.
>>
>> Thanks,
>> Michal
It is in queue for v4.15.
FYI: There are still 3 more in doc.
Documentation/devicetree/bindings/i2c/i2c-mux-pca954x.txt:50:
compatible = "at,24c08";
Documentation/devicetree/bindings/i2c/nxp,pca9541.txt:25:
compatible = "at,24c08";
Documentation/devicetree/bindings/trivial-devices.txt:34:at,24c08
i2c serial eeprom (24cxx)
Thanks,
Michal
^ permalink raw reply [flat|nested] 115+ messages in thread* [RESEND PATCH v5 10/16] ARM: dts: zynq: Add generic compatible string for I2C EEPROM
@ 2017-09-25 8:06 ` Michal Simek
0 siblings, 0 replies; 115+ messages in thread
From: Michal Simek @ 2017-09-25 8:06 UTC (permalink / raw)
To: linux-arm-kernel
Hi,
On 21.9.2017 11:41, Javier Martinez Canillas wrote:
> Hello Michal,
>
> On Thu, Aug 24, 2017 at 3:44 PM, Michal Simek <michal.simek@xilinx.com> wrote:
>
> [snip]
>
>>
>> Ok. Applied to
>> https://github.com/Xilinx/linux-xlnx/commits/zynq/dt
>>
>> Will wait if something else pop up.
>>
>
> I see the patch applied to that repository but I don't see it in
> neither mainline nor linux-next.
xilinx tree was merging to linux-next but there were conflicts with arm
soc tree that's why it was removed.
>
>> Arnd: If you want to take this directly, feel free to do it.
>>
>> Thanks,
>> Michal
It is in queue for v4.15.
FYI: There are still 3 more in doc.
Documentation/devicetree/bindings/i2c/i2c-mux-pca954x.txt:50:
compatible = "at,24c08";
Documentation/devicetree/bindings/i2c/nxp,pca9541.txt:25:
compatible = "at,24c08";
Documentation/devicetree/bindings/trivial-devices.txt:34:at,24c08
i2c serial eeprom (24cxx)
Thanks,
Michal
^ permalink raw reply [flat|nested] 115+ messages in thread* Re: [RESEND PATCH v5 10/16] ARM: dts: zynq: Add generic compatible string for I2C EEPROM
@ 2017-09-26 8:00 ` Javier Martinez Canillas
0 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-09-26 8:00 UTC (permalink / raw)
To: Michal Simek
Cc: Arnd Bergmann, Linux Kernel, Wolfram Sang, Rob Herring,
Sören Brinkmann, devicetree, Rob Herring, Mark Rutland,
Russell King, linux-arm-kernel
On Mon, Sep 25, 2017 at 10:06 AM, Michal Simek <michal.simek@xilinx.com> wrote:
> Hi,
>
> On 21.9.2017 11:41, Javier Martinez Canillas wrote:
>> Hello Michal,
>>
>> On Thu, Aug 24, 2017 at 3:44 PM, Michal Simek <michal.simek@xilinx.com> wrote:
>>
>> [snip]
>>
>>>
>>> Ok. Applied to
>>> https://github.com/Xilinx/linux-xlnx/commits/zynq/dt
>>>
>>> Will wait if something else pop up.
>>>
>>
>> I see the patch applied to that repository but I don't see it in
>> neither mainline nor linux-next.
>
> xilinx tree was merging to linux-next but there were conflicts with arm
> soc tree that's why it was removed.
>
>>
>>> Arnd: If you want to take this directly, feel free to do it.
>>>
>>> Thanks,
>>> Michal
>
> It is in queue for v4.15.
>
Got it, thanks a lot for the info.
> FYI: There are still 3 more in doc.
>
> Documentation/devicetree/bindings/i2c/i2c-mux-pca954x.txt:50:
> compatible = " ";
> Documentation/devicetree/bindings/i2c/nxp,pca9541.txt:25:
> compatible = "at,24c08";
> Documentation/devicetree/bindings/trivial-devices.txt:34:at,24c08
> i2c serial eeprom (24cxx)
>
Indeed, thanks for pointing out. I'll take a look to these as well.
>
> Thanks,
> Michal
>
Best regards,
Javier
^ permalink raw reply [flat|nested] 115+ messages in thread
* [RESEND PATCH v5 10/16] ARM: dts: zynq: Add generic compatible string for I2C EEPROM
@ 2017-09-26 8:00 ` Javier Martinez Canillas
0 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-09-26 8:00 UTC (permalink / raw)
To: linux-arm-kernel
On Mon, Sep 25, 2017 at 10:06 AM, Michal Simek <michal.simek@xilinx.com> wrote:
> Hi,
>
> On 21.9.2017 11:41, Javier Martinez Canillas wrote:
>> Hello Michal,
>>
>> On Thu, Aug 24, 2017 at 3:44 PM, Michal Simek <michal.simek@xilinx.com> wrote:
>>
>> [snip]
>>
>>>
>>> Ok. Applied to
>>> https://github.com/Xilinx/linux-xlnx/commits/zynq/dt
>>>
>>> Will wait if something else pop up.
>>>
>>
>> I see the patch applied to that repository but I don't see it in
>> neither mainline nor linux-next.
>
> xilinx tree was merging to linux-next but there were conflicts with arm
> soc tree that's why it was removed.
>
>>
>>> Arnd: If you want to take this directly, feel free to do it.
>>>
>>> Thanks,
>>> Michal
>
> It is in queue for v4.15.
>
Got it, thanks a lot for the info.
> FYI: There are still 3 more in doc.
>
> Documentation/devicetree/bindings/i2c/i2c-mux-pca954x.txt:50:
> compatible = " ";
> Documentation/devicetree/bindings/i2c/nxp,pca9541.txt:25:
> compatible = "at,24c08";
> Documentation/devicetree/bindings/trivial-devices.txt:34:at,24c08
> i2c serial eeprom (24cxx)
>
Indeed, thanks for pointing out. I'll take a look to these as well.
>
> Thanks,
> Michal
>
Best regards,
Javier
^ permalink raw reply [flat|nested] 115+ messages in thread
* Re: [RESEND PATCH v5 10/16] ARM: dts: zynq: Add generic compatible string for I2C EEPROM
@ 2017-09-26 8:00 ` Javier Martinez Canillas
0 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-09-26 8:00 UTC (permalink / raw)
To: Michal Simek
Cc: Arnd Bergmann, Linux Kernel, Wolfram Sang, Rob Herring,
Sören Brinkmann, devicetree-u79uwXL29TY76Z2rM5mHXA,
Rob Herring, Mark Rutland, Russell King,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
On Mon, Sep 25, 2017 at 10:06 AM, Michal Simek <michal.simek-gjFFaj9aHVfQT0dZR+AlfA@public.gmane.org> wrote:
> Hi,
>
> On 21.9.2017 11:41, Javier Martinez Canillas wrote:
>> Hello Michal,
>>
>> On Thu, Aug 24, 2017 at 3:44 PM, Michal Simek <michal.simek-gjFFaj9aHVfQT0dZR+AlfA@public.gmane.org> wrote:
>>
>> [snip]
>>
>>>
>>> Ok. Applied to
>>> https://github.com/Xilinx/linux-xlnx/commits/zynq/dt
>>>
>>> Will wait if something else pop up.
>>>
>>
>> I see the patch applied to that repository but I don't see it in
>> neither mainline nor linux-next.
>
> xilinx tree was merging to linux-next but there were conflicts with arm
> soc tree that's why it was removed.
>
>>
>>> Arnd: If you want to take this directly, feel free to do it.
>>>
>>> Thanks,
>>> Michal
>
> It is in queue for v4.15.
>
Got it, thanks a lot for the info.
> FYI: There are still 3 more in doc.
>
> Documentation/devicetree/bindings/i2c/i2c-mux-pca954x.txt:50:
> compatible = " ";
> Documentation/devicetree/bindings/i2c/nxp,pca9541.txt:25:
> compatible = "at,24c08";
> Documentation/devicetree/bindings/trivial-devices.txt:34:at,24c08
> i2c serial eeprom (24cxx)
>
Indeed, thanks for pointing out. I'll take a look to these as well.
>
> Thanks,
> Michal
>
Best regards,
Javier
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 115+ messages in thread
* [RESEND PATCH v5 11/16] arm64: zynqmp: Add generic compatible string for I2C EEPROM
@ 2017-06-15 18:54 ` Javier Martinez Canillas
0 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-06-15 18:54 UTC (permalink / raw)
To: linux-kernel
Cc: Wolfram Sang, Rob Herring, Javier Martinez Canillas,
Sören Brinkmann, devicetree, Michal Simek, Rob Herring,
Will Deacon, Mark Rutland, Catalin Marinas, linux-arm-kernel
The at24 driver allows to register I2C EEPROM chips using different vendor
and devices, but the I2C subsystem does not take the vendor into account
when matching using the I2C table since it only has device entries.
But when matching using an OF table, both the vendor and device has to be
taken into account so the driver defines only a set of compatible strings
using the "atmel" vendor as a generic fallback for compatible I2C devices.
So add this generic fallback to the device node compatible string to make
the device to match the driver using the OF device ID table.
Signed-off-by: Javier Martinez Canillas <javier@dowhile0.org>
---
Changes in v5: None
Changes in v4:
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
Changes in v3: None
Changes in v2: None
arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts b/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts
index ef1b9e573af0..53907b7a7fd9 100644
--- a/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts
+++ b/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts
@@ -55,7 +55,7 @@
status = "okay";
clock-frequency = <400000>;
eeprom@54 {
- compatible = "at,24c64";
+ compatible = "atmel,24c64";
reg = <0x54>;
};
};
@@ -64,7 +64,7 @@
status = "okay";
clock-frequency = <400000>;
eeprom@55 {
- compatible = "at,24c64";
+ compatible = "atmel,24c64";
reg = <0x55>;
};
};
--
2.9.3
^ permalink raw reply related [flat|nested] 115+ messages in thread* [RESEND PATCH v5 11/16] arm64: zynqmp: Add generic compatible string for I2C EEPROM
@ 2017-06-15 18:54 ` Javier Martinez Canillas
0 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-06-15 18:54 UTC (permalink / raw)
To: linux-arm-kernel
The at24 driver allows to register I2C EEPROM chips using different vendor
and devices, but the I2C subsystem does not take the vendor into account
when matching using the I2C table since it only has device entries.
But when matching using an OF table, both the vendor and device has to be
taken into account so the driver defines only a set of compatible strings
using the "atmel" vendor as a generic fallback for compatible I2C devices.
So add this generic fallback to the device node compatible string to make
the device to match the driver using the OF device ID table.
Signed-off-by: Javier Martinez Canillas <javier@dowhile0.org>
---
Changes in v5: None
Changes in v4:
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
Changes in v3: None
Changes in v2: None
arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts b/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts
index ef1b9e573af0..53907b7a7fd9 100644
--- a/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts
+++ b/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts
@@ -55,7 +55,7 @@
status = "okay";
clock-frequency = <400000>;
eeprom at 54 {
- compatible = "at,24c64";
+ compatible = "atmel,24c64";
reg = <0x54>;
};
};
@@ -64,7 +64,7 @@
status = "okay";
clock-frequency = <400000>;
eeprom at 55 {
- compatible = "at,24c64";
+ compatible = "atmel,24c64";
reg = <0x55>;
};
};
--
2.9.3
^ permalink raw reply related [flat|nested] 115+ messages in thread* [RESEND PATCH v5 11/16] arm64: zynqmp: Add generic compatible string for I2C EEPROM
@ 2017-06-15 18:54 ` Javier Martinez Canillas
0 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-06-15 18:54 UTC (permalink / raw)
To: linux-kernel-u79uwXL29TY76Z2rM5mHXA
Cc: Wolfram Sang, Rob Herring, Javier Martinez Canillas,
Sören Brinkmann, devicetree-u79uwXL29TY76Z2rM5mHXA,
Michal Simek, Rob Herring, Will Deacon, Mark Rutland,
Catalin Marinas,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
The at24 driver allows to register I2C EEPROM chips using different vendor
and devices, but the I2C subsystem does not take the vendor into account
when matching using the I2C table since it only has device entries.
But when matching using an OF table, both the vendor and device has to be
taken into account so the driver defines only a set of compatible strings
using the "atmel" vendor as a generic fallback for compatible I2C devices.
So add this generic fallback to the device node compatible string to make
the device to match the driver using the OF device ID table.
Signed-off-by: Javier Martinez Canillas <javier-0uQlZySMnqxg9hUCZPvPmw@public.gmane.org>
---
Changes in v5: None
Changes in v4:
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
Changes in v3: None
Changes in v2: None
arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts b/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts
index ef1b9e573af0..53907b7a7fd9 100644
--- a/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts
+++ b/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts
@@ -55,7 +55,7 @@
status = "okay";
clock-frequency = <400000>;
eeprom@54 {
- compatible = "at,24c64";
+ compatible = "atmel,24c64";
reg = <0x54>;
};
};
@@ -64,7 +64,7 @@
status = "okay";
clock-frequency = <400000>;
eeprom@55 {
- compatible = "at,24c64";
+ compatible = "atmel,24c64";
reg = <0x55>;
};
};
--
2.9.3
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply related [flat|nested] 115+ messages in thread* Re: [RESEND PATCH v5 11/16] arm64: zynqmp: Add generic compatible string for I2C EEPROM
@ 2017-06-21 11:43 ` Michal Simek
0 siblings, 0 replies; 115+ messages in thread
From: Michal Simek @ 2017-06-21 11:43 UTC (permalink / raw)
To: Javier Martinez Canillas, linux-kernel
Cc: Wolfram Sang, Rob Herring, Sören Brinkmann, devicetree,
Michal Simek, Rob Herring, Will Deacon, Mark Rutland,
Catalin Marinas, linux-arm-kernel
On 15.6.2017 20:54, Javier Martinez Canillas wrote:
> The at24 driver allows to register I2C EEPROM chips using different vendor
> and devices, but the I2C subsystem does not take the vendor into account
> when matching using the I2C table since it only has device entries.
>
> But when matching using an OF table, both the vendor and device has to be
> taken into account so the driver defines only a set of compatible strings
> using the "atmel" vendor as a generic fallback for compatible I2C devices.
>
> So add this generic fallback to the device node compatible string to make
> the device to match the driver using the OF device ID table.
>
> Signed-off-by: Javier Martinez Canillas <javier@dowhile0.org>
>
> ---
>
> Changes in v5: None
> Changes in v4:
> - Only use the atmel manufacturer in the compatible string instead of
> keeping the deprecated ones (Rob Herring).
>
> Changes in v3: None
> Changes in v2: None
>
> arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts b/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts
> index ef1b9e573af0..53907b7a7fd9 100644
> --- a/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts
> +++ b/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts
> @@ -55,7 +55,7 @@
> status = "okay";
> clock-frequency = <400000>;
> eeprom@54 {
> - compatible = "at,24c64";
> + compatible = "atmel,24c64";
> reg = <0x54>;
> };
> };
> @@ -64,7 +64,7 @@
> status = "okay";
> clock-frequency = <400000>;
> eeprom@55 {
> - compatible = "at,24c64";
> + compatible = "atmel,24c64";
> reg = <0x55>;
> };
> };
>
Acked-by: Michal Simek <michal.simek@xilinx.com>
Thanks,
Michal
^ permalink raw reply [flat|nested] 115+ messages in thread* [RESEND PATCH v5 11/16] arm64: zynqmp: Add generic compatible string for I2C EEPROM
@ 2017-06-21 11:43 ` Michal Simek
0 siblings, 0 replies; 115+ messages in thread
From: Michal Simek @ 2017-06-21 11:43 UTC (permalink / raw)
To: linux-arm-kernel
On 15.6.2017 20:54, Javier Martinez Canillas wrote:
> The at24 driver allows to register I2C EEPROM chips using different vendor
> and devices, but the I2C subsystem does not take the vendor into account
> when matching using the I2C table since it only has device entries.
>
> But when matching using an OF table, both the vendor and device has to be
> taken into account so the driver defines only a set of compatible strings
> using the "atmel" vendor as a generic fallback for compatible I2C devices.
>
> So add this generic fallback to the device node compatible string to make
> the device to match the driver using the OF device ID table.
>
> Signed-off-by: Javier Martinez Canillas <javier@dowhile0.org>
>
> ---
>
> Changes in v5: None
> Changes in v4:
> - Only use the atmel manufacturer in the compatible string instead of
> keeping the deprecated ones (Rob Herring).
>
> Changes in v3: None
> Changes in v2: None
>
> arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts b/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts
> index ef1b9e573af0..53907b7a7fd9 100644
> --- a/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts
> +++ b/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts
> @@ -55,7 +55,7 @@
> status = "okay";
> clock-frequency = <400000>;
> eeprom at 54 {
> - compatible = "at,24c64";
> + compatible = "atmel,24c64";
> reg = <0x54>;
> };
> };
> @@ -64,7 +64,7 @@
> status = "okay";
> clock-frequency = <400000>;
> eeprom at 55 {
> - compatible = "at,24c64";
> + compatible = "atmel,24c64";
> reg = <0x55>;
> };
> };
>
Acked-by: Michal Simek <michal.simek@xilinx.com>
Thanks,
Michal
^ permalink raw reply [flat|nested] 115+ messages in thread* Re: [RESEND PATCH v5 11/16] arm64: zynqmp: Add generic compatible string for I2C EEPROM
@ 2017-06-21 11:43 ` Michal Simek
0 siblings, 0 replies; 115+ messages in thread
From: Michal Simek @ 2017-06-21 11:43 UTC (permalink / raw)
To: Javier Martinez Canillas, linux-kernel-u79uwXL29TY76Z2rM5mHXA
Cc: Wolfram Sang, Rob Herring, Sören Brinkmann,
devicetree-u79uwXL29TY76Z2rM5mHXA, Michal Simek, Rob Herring,
Will Deacon, Mark Rutland, Catalin Marinas,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
On 15.6.2017 20:54, Javier Martinez Canillas wrote:
> The at24 driver allows to register I2C EEPROM chips using different vendor
> and devices, but the I2C subsystem does not take the vendor into account
> when matching using the I2C table since it only has device entries.
>
> But when matching using an OF table, both the vendor and device has to be
> taken into account so the driver defines only a set of compatible strings
> using the "atmel" vendor as a generic fallback for compatible I2C devices.
>
> So add this generic fallback to the device node compatible string to make
> the device to match the driver using the OF device ID table.
>
> Signed-off-by: Javier Martinez Canillas <javier-0uQlZySMnqxg9hUCZPvPmw@public.gmane.org>
>
> ---
>
> Changes in v5: None
> Changes in v4:
> - Only use the atmel manufacturer in the compatible string instead of
> keeping the deprecated ones (Rob Herring).
>
> Changes in v3: None
> Changes in v2: None
>
> arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts b/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts
> index ef1b9e573af0..53907b7a7fd9 100644
> --- a/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts
> +++ b/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts
> @@ -55,7 +55,7 @@
> status = "okay";
> clock-frequency = <400000>;
> eeprom@54 {
> - compatible = "at,24c64";
> + compatible = "atmel,24c64";
> reg = <0x54>;
> };
> };
> @@ -64,7 +64,7 @@
> status = "okay";
> clock-frequency = <400000>;
> eeprom@55 {
> - compatible = "at,24c64";
> + compatible = "atmel,24c64";
> reg = <0x55>;
> };
> };
>
Acked-by: Michal Simek <michal.simek-gjFFaj9aHVfQT0dZR+AlfA@public.gmane.org>
Thanks,
Michal
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 115+ messages in thread* Re: [RESEND PATCH v5 11/16] arm64: zynqmp: Add generic compatible string for I2C EEPROM
2017-06-21 11:43 ` Michal Simek
@ 2017-08-21 11:27 ` Javier Martinez Canillas
-1 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-08-21 11:27 UTC (permalink / raw)
To: Michal Simek, Will Deacon, Mark Rutland, Catalin Marinas
Cc: Linux Kernel, Wolfram Sang, Rob Herring, Sören Brinkmann,
devicetree, Rob Herring, linux-arm-kernel
[snip]
>>
>> diff --git a/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts b/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts
>> index ef1b9e573af0..53907b7a7fd9 100644
>> --- a/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts
>> +++ b/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts
>> @@ -55,7 +55,7 @@
>> status = "okay";
>> clock-frequency = <400000>;
>> eeprom@54 {
>> - compatible = "at,24c64";
>> + compatible = "atmel,24c64";
>> reg = <0x54>;
>> };
>> };
>> @@ -64,7 +64,7 @@
>> status = "okay";
>> clock-frequency = <400000>;
>> eeprom@55 {
>> - compatible = "at,24c64";
>> + compatible = "atmel,24c64";
>> reg = <0x55>;
>> };
>> };
>>
>
> Acked-by: Michal Simek <michal.simek@xilinx.com>
>
Any reasons why this patch was never picked? It was acked two months ago.
Best regards,
Javier
^ permalink raw reply [flat|nested] 115+ messages in thread* [RESEND PATCH v5 11/16] arm64: zynqmp: Add generic compatible string for I2C EEPROM
@ 2017-08-21 11:27 ` Javier Martinez Canillas
0 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-08-21 11:27 UTC (permalink / raw)
To: linux-arm-kernel
[snip]
>>
>> diff --git a/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts b/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts
>> index ef1b9e573af0..53907b7a7fd9 100644
>> --- a/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts
>> +++ b/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts
>> @@ -55,7 +55,7 @@
>> status = "okay";
>> clock-frequency = <400000>;
>> eeprom at 54 {
>> - compatible = "at,24c64";
>> + compatible = "atmel,24c64";
>> reg = <0x54>;
>> };
>> };
>> @@ -64,7 +64,7 @@
>> status = "okay";
>> clock-frequency = <400000>;
>> eeprom at 55 {
>> - compatible = "at,24c64";
>> + compatible = "atmel,24c64";
>> reg = <0x55>;
>> };
>> };
>>
>
> Acked-by: Michal Simek <michal.simek@xilinx.com>
>
Any reasons why this patch was never picked? It was acked two months ago.
Best regards,
Javier
^ permalink raw reply [flat|nested] 115+ messages in thread* Re: [RESEND PATCH v5 11/16] arm64: zynqmp: Add generic compatible string for I2C EEPROM
2017-08-21 11:27 ` Javier Martinez Canillas
@ 2017-08-21 12:04 ` Michal Simek
-1 siblings, 0 replies; 115+ messages in thread
From: Michal Simek @ 2017-08-21 12:04 UTC (permalink / raw)
To: Javier Martinez Canillas, Michal Simek, Will Deacon,
Mark Rutland, Catalin Marinas
Cc: Linux Kernel, Wolfram Sang, Rob Herring, Sören Brinkmann,
devicetree, Rob Herring, linux-arm-kernel
On 21.8.2017 13:27, Javier Martinez Canillas wrote:
> [snip]
>
>>>
>>> diff --git a/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts b/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts
>>> index ef1b9e573af0..53907b7a7fd9 100644
>>> --- a/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts
>>> +++ b/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts
>>> @@ -55,7 +55,7 @@
>>> status = "okay";
>>> clock-frequency = <400000>;
>>> eeprom@54 {
>>> - compatible = "at,24c64";
>>> + compatible = "atmel,24c64";
>>> reg = <0x54>;
>>> };
>>> };
>>> @@ -64,7 +64,7 @@
>>> status = "okay";
>>> clock-frequency = <400000>;
>>> eeprom@55 {
>>> - compatible = "at,24c64";
>>> + compatible = "atmel,24c64";
>>> reg = <0x55>;
>>> };
>>> };
>>>
>>
>> Acked-by: Michal Simek <michal.simek@xilinx.com>
>>
>
> Any reasons why this patch was never picked? It was acked two months ago.
Normally when you send series then it should be applied as the whole
series not just one piece of that.
Anyway I see that changes are applied and this one is missing that's why
I will add to zynqmp/dt branch and going to send pull request today.
Thanks,
Michal
^ permalink raw reply [flat|nested] 115+ messages in thread* [RESEND PATCH v5 11/16] arm64: zynqmp: Add generic compatible string for I2C EEPROM
@ 2017-08-21 12:04 ` Michal Simek
0 siblings, 0 replies; 115+ messages in thread
From: Michal Simek @ 2017-08-21 12:04 UTC (permalink / raw)
To: linux-arm-kernel
On 21.8.2017 13:27, Javier Martinez Canillas wrote:
> [snip]
>
>>>
>>> diff --git a/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts b/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts
>>> index ef1b9e573af0..53907b7a7fd9 100644
>>> --- a/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts
>>> +++ b/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts
>>> @@ -55,7 +55,7 @@
>>> status = "okay";
>>> clock-frequency = <400000>;
>>> eeprom at 54 {
>>> - compatible = "at,24c64";
>>> + compatible = "atmel,24c64";
>>> reg = <0x54>;
>>> };
>>> };
>>> @@ -64,7 +64,7 @@
>>> status = "okay";
>>> clock-frequency = <400000>;
>>> eeprom at 55 {
>>> - compatible = "at,24c64";
>>> + compatible = "atmel,24c64";
>>> reg = <0x55>;
>>> };
>>> };
>>>
>>
>> Acked-by: Michal Simek <michal.simek@xilinx.com>
>>
>
> Any reasons why this patch was never picked? It was acked two months ago.
Normally when you send series then it should be applied as the whole
series not just one piece of that.
Anyway I see that changes are applied and this one is missing that's why
I will add to zynqmp/dt branch and going to send pull request today.
Thanks,
Michal
^ permalink raw reply [flat|nested] 115+ messages in thread* Re: [RESEND PATCH v5 11/16] arm64: zynqmp: Add generic compatible string for I2C EEPROM
2017-08-21 12:04 ` Michal Simek
@ 2017-08-21 12:58 ` Javier Martinez Canillas
-1 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-08-21 12:58 UTC (permalink / raw)
To: Michal Simek
Cc: Will Deacon, Mark Rutland, Catalin Marinas, Linux Kernel,
Wolfram Sang, Rob Herring, Sören Brinkmann, devicetree,
Rob Herring, linux-arm-kernel
Hello Michal,
On Mon, Aug 21, 2017 at 2:04 PM, Michal Simek <michal.simek@xilinx.com> wrote:
> On 21.8.2017 13:27, Javier Martinez Canillas wrote:
>> [snip]
>>
>>>>
>>>> diff --git a/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts b/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts
>>>> index ef1b9e573af0..53907b7a7fd9 100644
>>>> --- a/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts
>>>> +++ b/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts
>>>> @@ -55,7 +55,7 @@
>>>> status = "okay";
>>>> clock-frequency = <400000>;
>>>> eeprom@54 {
>>>> - compatible = "at,24c64";
>>>> + compatible = "atmel,24c64";
>>>> reg = <0x54>;
>>>> };
>>>> };
>>>> @@ -64,7 +64,7 @@
>>>> status = "okay";
>>>> clock-frequency = <400000>;
>>>> eeprom@55 {
>>>> - compatible = "at,24c64";
>>>> + compatible = "atmel,24c64";
>>>> reg = <0x55>;
>>>> };
>>>> };
>>>>
>>>
>>> Acked-by: Michal Simek <michal.simek@xilinx.com>
>>>
>>
>> Any reasons why this patch was never picked? It was acked two months ago.
>
> Normally when you send series then it should be applied as the whole
> series not just one piece of that.
>
Right, I mentioned in the cover letter that the patches could be
applied independently. Sorry for the confusion.
> Anyway I see that changes are applied and this one is missing that's why
> I will add to zynqmp/dt branch and going to send pull request today.
>
Great, thanks a lot for your help.
> Thanks,
> Michal
Best regards,
Javier
^ permalink raw reply [flat|nested] 115+ messages in thread* [RESEND PATCH v5 11/16] arm64: zynqmp: Add generic compatible string for I2C EEPROM
@ 2017-08-21 12:58 ` Javier Martinez Canillas
0 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-08-21 12:58 UTC (permalink / raw)
To: linux-arm-kernel
Hello Michal,
On Mon, Aug 21, 2017 at 2:04 PM, Michal Simek <michal.simek@xilinx.com> wrote:
> On 21.8.2017 13:27, Javier Martinez Canillas wrote:
>> [snip]
>>
>>>>
>>>> diff --git a/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts b/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts
>>>> index ef1b9e573af0..53907b7a7fd9 100644
>>>> --- a/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts
>>>> +++ b/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts
>>>> @@ -55,7 +55,7 @@
>>>> status = "okay";
>>>> clock-frequency = <400000>;
>>>> eeprom at 54 {
>>>> - compatible = "at,24c64";
>>>> + compatible = "atmel,24c64";
>>>> reg = <0x54>;
>>>> };
>>>> };
>>>> @@ -64,7 +64,7 @@
>>>> status = "okay";
>>>> clock-frequency = <400000>;
>>>> eeprom at 55 {
>>>> - compatible = "at,24c64";
>>>> + compatible = "atmel,24c64";
>>>> reg = <0x55>;
>>>> };
>>>> };
>>>>
>>>
>>> Acked-by: Michal Simek <michal.simek@xilinx.com>
>>>
>>
>> Any reasons why this patch was never picked? It was acked two months ago.
>
> Normally when you send series then it should be applied as the whole
> series not just one piece of that.
>
Right, I mentioned in the cover letter that the patches could be
applied independently. Sorry for the confusion.
> Anyway I see that changes are applied and this one is missing that's why
> I will add to zynqmp/dt branch and going to send pull request today.
>
Great, thanks a lot for your help.
> Thanks,
> Michal
Best regards,
Javier
^ permalink raw reply [flat|nested] 115+ messages in thread
* [RESEND PATCH v5 12/16] powerpc/5200: Add generic compatible string for I2C EEPROM
2017-06-15 18:54 ` Javier Martinez Canillas
` (12 preceding siblings ...)
(?)
@ 2017-06-15 18:54 ` Javier Martinez Canillas
2017-06-29 12:21 ` Michael Ellerman
-1 siblings, 1 reply; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-06-15 18:54 UTC (permalink / raw)
To: linux-kernel
Cc: Wolfram Sang, Rob Herring, Javier Martinez Canillas, devicetree,
Michael Ellerman, Benjamin Herrenschmidt, Paul Mackerras,
Rob Herring, Mark Rutland, linuxppc-dev
The at24 driver allows to register I2C EEPROM chips using different vendor
and devices, but the I2C subsystem does not take the vendor into account
when matching using the I2C table since it only has device entries.
But when matching using an OF table, both the vendor and device has to be
taken into account so the driver defines only a set of compatible strings
using the "atmel" vendor as a generic fallback for compatible I2C devices.
So add this generic fallback to the device node compatible string to make
the device to match the driver using the OF device ID table.
Signed-off-by: Javier Martinez Canillas <javier@dowhile0.org>
---
Changes in v5:
- Only replace atmel variant but keep other EEPROM vendors (Geert Uytterhoeven).
Changes in v4:
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
Changes in v3: None
Changes in v2: None
arch/powerpc/boot/dts/digsy_mtc.dts | 2 +-
arch/powerpc/boot/dts/pcm030.dts | 2 +-
arch/powerpc/boot/dts/pcm032.dts | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/arch/powerpc/boot/dts/digsy_mtc.dts b/arch/powerpc/boot/dts/digsy_mtc.dts
index 955bff629df3..c280e75c86bf 100644
--- a/arch/powerpc/boot/dts/digsy_mtc.dts
+++ b/arch/powerpc/boot/dts/digsy_mtc.dts
@@ -73,7 +73,7 @@
i2c@3d00 {
eeprom@50 {
- compatible = "at,24c08";
+ compatible = "atmel,24c08";
reg = <0x50>;
};
diff --git a/arch/powerpc/boot/dts/pcm030.dts b/arch/powerpc/boot/dts/pcm030.dts
index 192e66af0001..836e47cc4bed 100644
--- a/arch/powerpc/boot/dts/pcm030.dts
+++ b/arch/powerpc/boot/dts/pcm030.dts
@@ -71,7 +71,7 @@
reg = <0x51>;
};
eeprom@52 {
- compatible = "catalyst,24c32";
+ compatible = "catalyst,24c32", "atmel,24c32";
reg = <0x52>;
pagesize = <32>;
};
diff --git a/arch/powerpc/boot/dts/pcm032.dts b/arch/powerpc/boot/dts/pcm032.dts
index 96b139bf50e9..576249bf2fb9 100644
--- a/arch/powerpc/boot/dts/pcm032.dts
+++ b/arch/powerpc/boot/dts/pcm032.dts
@@ -75,7 +75,7 @@
reg = <0x51>;
};
eeprom@52 {
- compatible = "catalyst,24c32";
+ compatible = "catalyst,24c32", "atmel,24c32";
reg = <0x52>;
pagesize = <32>;
};
--
2.9.3
^ permalink raw reply related [flat|nested] 115+ messages in thread* Re: [RESEND, v5, 12/16] powerpc/5200: Add generic compatible string for I2C EEPROM
@ 2017-06-29 12:21 ` Michael Ellerman
0 siblings, 0 replies; 115+ messages in thread
From: Michael Ellerman @ 2017-06-29 12:21 UTC (permalink / raw)
To: Javier Martinez Canillas, linux-kernel
Cc: Mark Rutland, Rob Herring, devicetree, Wolfram Sang, Rob Herring,
Paul Mackerras, Javier Martinez Canillas, linuxppc-dev
On Thu, 2017-06-15 at 18:54:14 UTC, Javier Martinez Canillas wrote:
> The at24 driver allows to register I2C EEPROM chips using different vendor
> and devices, but the I2C subsystem does not take the vendor into account
> when matching using the I2C table since it only has device entries.
>
> But when matching using an OF table, both the vendor and device has to be
> taken into account so the driver defines only a set of compatible strings
> using the "atmel" vendor as a generic fallback for compatible I2C devices.
>
> So add this generic fallback to the device node compatible string to make
> the device to match the driver using the OF device ID table.
>
> Signed-off-by: Javier Martinez Canillas <javier@dowhile0.org>
Applied to powerpc next, thanks.
https://git.kernel.org/powerpc/c/fd393188064e3d01a91a21160751f6
cheers
^ permalink raw reply [flat|nested] 115+ messages in thread
* Re: [RESEND, v5, 12/16] powerpc/5200: Add generic compatible string for I2C EEPROM
@ 2017-06-29 12:21 ` Michael Ellerman
0 siblings, 0 replies; 115+ messages in thread
From: Michael Ellerman @ 2017-06-29 12:21 UTC (permalink / raw)
To: linux-kernel-u79uwXL29TY76Z2rM5mHXA
Cc: Mark Rutland, Rob Herring, devicetree-u79uwXL29TY76Z2rM5mHXA,
Wolfram Sang, Rob Herring, Paul Mackerras,
Javier Martinez Canillas, linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ
On Thu, 2017-06-15 at 18:54:14 UTC, Javier Martinez Canillas wrote:
> The at24 driver allows to register I2C EEPROM chips using different vendor
> and devices, but the I2C subsystem does not take the vendor into account
> when matching using the I2C table since it only has device entries.
>
> But when matching using an OF table, both the vendor and device has to be
> taken into account so the driver defines only a set of compatible strings
> using the "atmel" vendor as a generic fallback for compatible I2C devices.
>
> So add this generic fallback to the device node compatible string to make
> the device to match the driver using the OF device ID table.
>
> Signed-off-by: Javier Martinez Canillas <javier-0uQlZySMnqxg9hUCZPvPmw@public.gmane.org>
Applied to powerpc next, thanks.
https://git.kernel.org/powerpc/c/fd393188064e3d01a91a21160751f6
cheers
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 115+ messages in thread
* [RESEND PATCH v5 13/16] powerpc/fsl: Add generic compatible string for I2C EEPROM
2017-06-15 18:54 ` Javier Martinez Canillas
` (13 preceding siblings ...)
(?)
@ 2017-06-15 18:54 ` Javier Martinez Canillas
2017-06-29 12:21 ` Michael Ellerman
-1 siblings, 1 reply; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-06-15 18:54 UTC (permalink / raw)
To: linux-kernel
Cc: Wolfram Sang, Rob Herring, Javier Martinez Canillas, devicetree,
Florian Larysch, Michael Ellerman, Scott Wood,
Benjamin Herrenschmidt, Paul Mackerras, Rob Herring,
Mark Rutland, linuxppc-dev
The at24 driver allows to register I2C EEPROM chips using different vendor
and devices, but the I2C subsystem does not take the vendor into account
when matching using the I2C table since it only has device entries.
But when matching using an OF table, both the vendor and device has to be
taken into account so the driver defines only a set of compatible strings
using the "atmel" vendor as a generic fallback for compatible I2C devices.
So add this generic fallback to the device node compatible string to make
the device to match the driver using the OF device ID table.
Signed-off-by: Javier Martinez Canillas <javier@dowhile0.org>
---
Changes in v5:
- Only replace atmel variant but keep other EEPROM vendors (Geert Uytterhoeven).
Changes in v4:
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
Changes in v3: None
Changes in v2: None
arch/powerpc/boot/dts/fsl/b4qds.dtsi | 8 ++++----
arch/powerpc/boot/dts/fsl/c293pcie.dts | 2 +-
arch/powerpc/boot/dts/fsl/p1010rdb.dtsi | 2 +-
arch/powerpc/boot/dts/fsl/p1023rdb.dts | 2 +-
arch/powerpc/boot/dts/fsl/p2041rdb.dts | 4 ++--
arch/powerpc/boot/dts/fsl/p3041ds.dts | 4 ++--
arch/powerpc/boot/dts/fsl/p4080ds.dts | 4 ++--
arch/powerpc/boot/dts/fsl/p5020ds.dts | 4 ++--
arch/powerpc/boot/dts/fsl/p5040ds.dts | 4 ++--
arch/powerpc/boot/dts/fsl/t208xqds.dtsi | 8 ++++----
arch/powerpc/boot/dts/fsl/t4240qds.dts | 12 ++++++------
arch/powerpc/boot/dts/fsl/t4240rdb.dts | 6 +++---
12 files changed, 30 insertions(+), 30 deletions(-)
diff --git a/arch/powerpc/boot/dts/fsl/b4qds.dtsi b/arch/powerpc/boot/dts/fsl/b4qds.dtsi
index 3785ef826d07..999efd3bc167 100644
--- a/arch/powerpc/boot/dts/fsl/b4qds.dtsi
+++ b/arch/powerpc/boot/dts/fsl/b4qds.dtsi
@@ -166,19 +166,19 @@
reg = <0>;
eeprom@50 {
- compatible = "at24,24c64";
+ compatible = "atmel,24c64";
reg = <0x50>;
};
eeprom@51 {
- compatible = "at24,24c256";
+ compatible = "atmel,24c256";
reg = <0x51>;
};
eeprom@53 {
- compatible = "at24,24c256";
+ compatible = "atmel,24c256";
reg = <0x53>;
};
eeprom@57 {
- compatible = "at24,24c256";
+ compatible = "atmel,24c256";
reg = <0x57>;
};
rtc@68 {
diff --git a/arch/powerpc/boot/dts/fsl/c293pcie.dts b/arch/powerpc/boot/dts/fsl/c293pcie.dts
index 66709788429d..5e905e0857cf 100644
--- a/arch/powerpc/boot/dts/fsl/c293pcie.dts
+++ b/arch/powerpc/boot/dts/fsl/c293pcie.dts
@@ -153,7 +153,7 @@
&soc {
i2c@3000 {
eeprom@50 {
- compatible = "st,24c1024";
+ compatible = "st,24c1024", "atmel,24c1024";
reg = <0x50>;
};
diff --git a/arch/powerpc/boot/dts/fsl/p1010rdb.dtsi b/arch/powerpc/boot/dts/fsl/p1010rdb.dtsi
index a8e4ba070104..2ca9cee2ddeb 100644
--- a/arch/powerpc/boot/dts/fsl/p1010rdb.dtsi
+++ b/arch/powerpc/boot/dts/fsl/p1010rdb.dtsi
@@ -89,7 +89,7 @@
&board_soc {
i2c@3000 {
eeprom@50 {
- compatible = "st,24c256";
+ compatible = "st,24c256", "atmel,24c256";
reg = <0x50>;
};
diff --git a/arch/powerpc/boot/dts/fsl/p1023rdb.dts b/arch/powerpc/boot/dts/fsl/p1023rdb.dts
index 9716ca64651c..ead928364beb 100644
--- a/arch/powerpc/boot/dts/fsl/p1023rdb.dts
+++ b/arch/powerpc/boot/dts/fsl/p1023rdb.dts
@@ -79,7 +79,7 @@
i2c@3000 {
eeprom@53 {
- compatible = "at24,24c04";
+ compatible = "atmel,24c04";
reg = <0x53>;
};
diff --git a/arch/powerpc/boot/dts/fsl/p2041rdb.dts b/arch/powerpc/boot/dts/fsl/p2041rdb.dts
index e50fea95a853..950816b9d6e1 100644
--- a/arch/powerpc/boot/dts/fsl/p2041rdb.dts
+++ b/arch/powerpc/boot/dts/fsl/p2041rdb.dts
@@ -127,7 +127,7 @@
reg = <0x48>;
};
eeprom@50 {
- compatible = "at24,24c256";
+ compatible = "atmel,24c256";
reg = <0x50>;
};
rtc@68 {
@@ -142,7 +142,7 @@
i2c@118100 {
eeprom@50 {
- compatible = "at24,24c256";
+ compatible = "atmel,24c256";
reg = <0x50>;
};
};
diff --git a/arch/powerpc/boot/dts/fsl/p3041ds.dts b/arch/powerpc/boot/dts/fsl/p3041ds.dts
index 40748e415adb..6f5f7283c533 100644
--- a/arch/powerpc/boot/dts/fsl/p3041ds.dts
+++ b/arch/powerpc/boot/dts/fsl/p3041ds.dts
@@ -124,11 +124,11 @@
i2c@118100 {
eeprom@51 {
- compatible = "at24,24c256";
+ compatible = "atmel,24c256";
reg = <0x51>;
};
eeprom@52 {
- compatible = "at24,24c256";
+ compatible = "atmel,24c256";
reg = <0x52>;
};
};
diff --git a/arch/powerpc/boot/dts/fsl/p4080ds.dts b/arch/powerpc/boot/dts/fsl/p4080ds.dts
index 816b9788d5f6..65e20152e22f 100644
--- a/arch/powerpc/boot/dts/fsl/p4080ds.dts
+++ b/arch/powerpc/boot/dts/fsl/p4080ds.dts
@@ -125,11 +125,11 @@
i2c@118100 {
eeprom@51 {
- compatible = "at24,24c256";
+ compatible = "atmel,24c256";
reg = <0x51>;
};
eeprom@52 {
- compatible = "at24,24c256";
+ compatible = "atmel,24c256";
reg = <0x52>;
};
rtc@68 {
diff --git a/arch/powerpc/boot/dts/fsl/p5020ds.dts b/arch/powerpc/boot/dts/fsl/p5020ds.dts
index cd6f37386111..b24adf902d8d 100644
--- a/arch/powerpc/boot/dts/fsl/p5020ds.dts
+++ b/arch/powerpc/boot/dts/fsl/p5020ds.dts
@@ -124,11 +124,11 @@
i2c@118100 {
eeprom@51 {
- compatible = "at24,24c256";
+ compatible = "atmel,24c256";
reg = <0x51>;
};
eeprom@52 {
- compatible = "at24,24c256";
+ compatible = "atmel,24c256";
reg = <0x52>;
};
};
diff --git a/arch/powerpc/boot/dts/fsl/p5040ds.dts b/arch/powerpc/boot/dts/fsl/p5040ds.dts
index 45084738cf4e..30850b3228e0 100644
--- a/arch/powerpc/boot/dts/fsl/p5040ds.dts
+++ b/arch/powerpc/boot/dts/fsl/p5040ds.dts
@@ -133,11 +133,11 @@
i2c@118100 {
eeprom@51 {
- compatible = "at24,24c256";
+ compatible = "atmel,24c256";
reg = <0x51>;
};
eeprom@52 {
- compatible = "at24,24c256";
+ compatible = "atmel,24c256";
reg = <0x52>;
};
};
diff --git a/arch/powerpc/boot/dts/fsl/t208xqds.dtsi b/arch/powerpc/boot/dts/fsl/t208xqds.dtsi
index ec080bd01b09..db4139999b28 100644
--- a/arch/powerpc/boot/dts/fsl/t208xqds.dtsi
+++ b/arch/powerpc/boot/dts/fsl/t208xqds.dtsi
@@ -147,17 +147,17 @@
reg = <0x0>;
eeprom@50 {
- compatible = "at24,24c512";
+ compatible = "atmel,24c512";
reg = <0x50>;
};
eeprom@51 {
- compatible = "at24,24c02";
+ compatible = "atmel,24c02";
reg = <0x51>;
};
eeprom@57 {
- compatible = "at24,24c02";
+ compatible = "atmel,24c02";
reg = <0x57>;
};
@@ -174,7 +174,7 @@
reg = <0x1>;
eeprom@55 {
- compatible = "at24,24c02";
+ compatible = "atmel,24c02";
reg = <0x55>;
};
};
diff --git a/arch/powerpc/boot/dts/fsl/t4240qds.dts b/arch/powerpc/boot/dts/fsl/t4240qds.dts
index 9573ceada07c..c0913ac5aaad 100644
--- a/arch/powerpc/boot/dts/fsl/t4240qds.dts
+++ b/arch/powerpc/boot/dts/fsl/t4240qds.dts
@@ -377,27 +377,27 @@
reg = <0>;
eeprom@51 {
- compatible = "at24,24c256";
+ compatible = "atmel,24c256";
reg = <0x51>;
};
eeprom@52 {
- compatible = "at24,24c256";
+ compatible = "atmel,24c256";
reg = <0x52>;
};
eeprom@53 {
- compatible = "at24,24c256";
+ compatible = "atmel,24c256";
reg = <0x53>;
};
eeprom@54 {
- compatible = "at24,24c256";
+ compatible = "atmel,24c256";
reg = <0x54>;
};
eeprom@55 {
- compatible = "at24,24c256";
+ compatible = "atmel,24c256";
reg = <0x55>;
};
eeprom@56 {
- compatible = "at24,24c256";
+ compatible = "atmel,24c256";
reg = <0x56>;
};
rtc@68 {
diff --git a/arch/powerpc/boot/dts/fsl/t4240rdb.dts b/arch/powerpc/boot/dts/fsl/t4240rdb.dts
index 8166c660712a..15eb0a3f7290 100644
--- a/arch/powerpc/boot/dts/fsl/t4240rdb.dts
+++ b/arch/powerpc/boot/dts/fsl/t4240rdb.dts
@@ -130,15 +130,15 @@
reg = <0x2f>;
};
eeprom@52 {
- compatible = "at24,24c256";
+ compatible = "atmel,24c256";
reg = <0x52>;
};
eeprom@54 {
- compatible = "at24,24c256";
+ compatible = "atmel,24c256";
reg = <0x54>;
};
eeprom@56 {
- compatible = "at24,24c256";
+ compatible = "atmel,24c256";
reg = <0x56>;
};
rtc@68 {
--
2.9.3
^ permalink raw reply related [flat|nested] 115+ messages in thread* Re: [RESEND, v5, 13/16] powerpc/fsl: Add generic compatible string for I2C EEPROM
2017-06-15 18:54 ` [RESEND PATCH v5 13/16] powerpc/fsl: " Javier Martinez Canillas
@ 2017-06-29 12:21 ` Michael Ellerman
0 siblings, 0 replies; 115+ messages in thread
From: Michael Ellerman @ 2017-06-29 12:21 UTC (permalink / raw)
To: Javier Martinez Canillas, linux-kernel
Cc: Mark Rutland, Rob Herring, Florian Larysch, devicetree,
Wolfram Sang, Scott Wood, Rob Herring, Paul Mackerras,
Javier Martinez Canillas, linuxppc-dev
On Thu, 2017-06-15 at 18:54:15 UTC, Javier Martinez Canillas wrote:
> The at24 driver allows to register I2C EEPROM chips using different vendor
> and devices, but the I2C subsystem does not take the vendor into account
> when matching using the I2C table since it only has device entries.
>
> But when matching using an OF table, both the vendor and device has to be
> taken into account so the driver defines only a set of compatible strings
> using the "atmel" vendor as a generic fallback for compatible I2C devices.
>
> So add this generic fallback to the device node compatible string to make
> the device to match the driver using the OF device ID table.
>
> Signed-off-by: Javier Martinez Canillas <javier@dowhile0.org>
Applied to powerpc next, thanks.
https://git.kernel.org/powerpc/c/226b9391d1a4e91fe08627f75ebad1
cheers
^ permalink raw reply [flat|nested] 115+ messages in thread
* Re: [RESEND, v5, 13/16] powerpc/fsl: Add generic compatible string for I2C EEPROM
@ 2017-06-29 12:21 ` Michael Ellerman
0 siblings, 0 replies; 115+ messages in thread
From: Michael Ellerman @ 2017-06-29 12:21 UTC (permalink / raw)
To: linux-kernel
Cc: Mark Rutland, Rob Herring, Florian Larysch, devicetree,
Wolfram Sang, Scott Wood, Rob Herring, Paul Mackerras,
Javier Martinez Canillas, linuxppc-dev
On Thu, 2017-06-15 at 18:54:15 UTC, Javier Martinez Canillas wrote:
> The at24 driver allows to register I2C EEPROM chips using different vendor
> and devices, but the I2C subsystem does not take the vendor into account
> when matching using the I2C table since it only has device entries.
>
> But when matching using an OF table, both the vendor and device has to be
> taken into account so the driver defines only a set of compatible strings
> using the "atmel" vendor as a generic fallback for compatible I2C devices.
>
> So add this generic fallback to the device node compatible string to make
> the device to match the driver using the OF device ID table.
>
> Signed-off-by: Javier Martinez Canillas <javier@dowhile0.org>
Applied to powerpc next, thanks.
https://git.kernel.org/powerpc/c/226b9391d1a4e91fe08627f75ebad1
cheers
^ permalink raw reply [flat|nested] 115+ messages in thread
* [RESEND PATCH v5 14/16] powerpc/512x: Add generic compatible string for I2C EEPROM
2017-06-15 18:54 ` Javier Martinez Canillas
` (14 preceding siblings ...)
(?)
@ 2017-06-15 18:54 ` Javier Martinez Canillas
2017-06-29 12:21 ` Michael Ellerman
-1 siblings, 1 reply; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-06-15 18:54 UTC (permalink / raw)
To: linux-kernel
Cc: Wolfram Sang, Rob Herring, Javier Martinez Canillas, devicetree,
Michael Ellerman, Benjamin Herrenschmidt, Paul Mackerras,
Rob Herring, Mark Rutland, linuxppc-dev
The at24 driver allows to register I2C EEPROM chips using different vendor
and devices, but the I2C subsystem does not take the vendor into account
when matching using the I2C table since it only has device entries.
But when matching using an OF table, both the vendor and device has to be
taken into account so the driver defines only a set of compatible strings
using the "atmel" vendor as a generic fallback for compatible I2C devices.
So add this generic fallback to the device node compatible string to make
the device to match the driver using the OF device ID table.
Signed-off-by: Javier Martinez Canillas <javier@dowhile0.org>
---
Changes in v5: None
Changes in v4:
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
Changes in v3: None
Changes in v2: None
arch/powerpc/boot/dts/mpc5121ads.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/powerpc/boot/dts/mpc5121ads.dts b/arch/powerpc/boot/dts/mpc5121ads.dts
index 75888ce2c792..fcaa9bad4bda 100644
--- a/arch/powerpc/boot/dts/mpc5121ads.dts
+++ b/arch/powerpc/boot/dts/mpc5121ads.dts
@@ -94,7 +94,7 @@
};
eeprom@50 {
- compatible = "at,24c32";
+ compatible = "atmel,24c32";
reg = <0x50>;
};
--
2.9.3
^ permalink raw reply related [flat|nested] 115+ messages in thread* Re: [RESEND, v5, 14/16] powerpc/512x: Add generic compatible string for I2C EEPROM
@ 2017-06-29 12:21 ` Michael Ellerman
0 siblings, 0 replies; 115+ messages in thread
From: Michael Ellerman @ 2017-06-29 12:21 UTC (permalink / raw)
To: Javier Martinez Canillas, linux-kernel
Cc: Mark Rutland, Rob Herring, devicetree, Wolfram Sang, Rob Herring,
Paul Mackerras, Javier Martinez Canillas, linuxppc-dev
On Thu, 2017-06-15 at 18:54:16 UTC, Javier Martinez Canillas wrote:
> The at24 driver allows to register I2C EEPROM chips using different vendor
> and devices, but the I2C subsystem does not take the vendor into account
> when matching using the I2C table since it only has device entries.
>
> But when matching using an OF table, both the vendor and device has to be
> taken into account so the driver defines only a set of compatible strings
> using the "atmel" vendor as a generic fallback for compatible I2C devices.
>
> So add this generic fallback to the device node compatible string to make
> the device to match the driver using the OF device ID table.
>
> Signed-off-by: Javier Martinez Canillas <javier@dowhile0.org>
Applied to powerpc next, thanks.
https://git.kernel.org/powerpc/c/9b4091682700fb5909aed439fc0101
cheers
^ permalink raw reply [flat|nested] 115+ messages in thread
* Re: [RESEND, v5, 14/16] powerpc/512x: Add generic compatible string for I2C EEPROM
@ 2017-06-29 12:21 ` Michael Ellerman
0 siblings, 0 replies; 115+ messages in thread
From: Michael Ellerman @ 2017-06-29 12:21 UTC (permalink / raw)
To: linux-kernel-u79uwXL29TY76Z2rM5mHXA
Cc: Mark Rutland, Rob Herring, devicetree-u79uwXL29TY76Z2rM5mHXA,
Wolfram Sang, Rob Herring, Paul Mackerras,
Javier Martinez Canillas, linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ
On Thu, 2017-06-15 at 18:54:16 UTC, Javier Martinez Canillas wrote:
> The at24 driver allows to register I2C EEPROM chips using different vendor
> and devices, but the I2C subsystem does not take the vendor into account
> when matching using the I2C table since it only has device entries.
>
> But when matching using an OF table, both the vendor and device has to be
> taken into account so the driver defines only a set of compatible strings
> using the "atmel" vendor as a generic fallback for compatible I2C devices.
>
> So add this generic fallback to the device node compatible string to make
> the device to match the driver using the OF device ID table.
>
> Signed-off-by: Javier Martinez Canillas <javier-0uQlZySMnqxg9hUCZPvPmw@public.gmane.org>
Applied to powerpc next, thanks.
https://git.kernel.org/powerpc/c/9b4091682700fb5909aed439fc0101
cheers
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 115+ messages in thread
* [RESEND PATCH v5 15/16] powerpc/83xx: Add generic compatible string for I2C EEPROM
2017-06-15 18:54 ` Javier Martinez Canillas
` (15 preceding siblings ...)
(?)
@ 2017-06-15 18:54 ` Javier Martinez Canillas
2017-06-29 12:21 ` Michael Ellerman
-1 siblings, 1 reply; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-06-15 18:54 UTC (permalink / raw)
To: linux-kernel
Cc: Wolfram Sang, Rob Herring, Javier Martinez Canillas, devicetree,
Michael Ellerman, Benjamin Herrenschmidt, Paul Mackerras,
Rob Herring, Mark Rutland, linuxppc-dev
The at24 driver allows to register I2C EEPROM chips using different vendor
and devices, but the I2C subsystem does not take the vendor into account
when matching using the I2C table since it only has device entries.
But when matching using an OF table, both the vendor and device has to be
taken into account so the driver defines only a set of compatible strings
using the "atmel" vendor as a generic fallback for compatible I2C devices.
So add this generic fallback to the device node compatible string to make
the device to match the driver using the OF device ID table.
Signed-off-by: Javier Martinez Canillas <javier@dowhile0.org>
---
Changes in v5:
- Only replace atmel variant but keep other EEPROM vendors (Geert Uytterhoeven).
Changes in v4:
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
Changes in v3: None
Changes in v2: None
arch/powerpc/boot/dts/mpc8308_p1m.dts | 2 +-
arch/powerpc/boot/dts/mpc8349emitx.dts | 4 ++--
arch/powerpc/boot/dts/mpc8377_rdb.dts | 2 +-
arch/powerpc/boot/dts/mpc8377_wlan.dts | 2 +-
arch/powerpc/boot/dts/mpc8378_rdb.dts | 2 +-
arch/powerpc/boot/dts/mpc8379_rdb.dts | 2 +-
6 files changed, 7 insertions(+), 7 deletions(-)
diff --git a/arch/powerpc/boot/dts/mpc8308_p1m.dts b/arch/powerpc/boot/dts/mpc8308_p1m.dts
index 57f86cdf9f36..cab933b3957a 100644
--- a/arch/powerpc/boot/dts/mpc8308_p1m.dts
+++ b/arch/powerpc/boot/dts/mpc8308_p1m.dts
@@ -123,7 +123,7 @@
interrupt-parent = <&ipic>;
dfsrr;
fram@50 {
- compatible = "ramtron,24c64";
+ compatible = "ramtron,24c64", "atmel,24c64";
reg = <0x50>;
};
};
diff --git a/arch/powerpc/boot/dts/mpc8349emitx.dts b/arch/powerpc/boot/dts/mpc8349emitx.dts
index 90aed3ac2f69..648a85858eb5 100644
--- a/arch/powerpc/boot/dts/mpc8349emitx.dts
+++ b/arch/powerpc/boot/dts/mpc8349emitx.dts
@@ -92,7 +92,7 @@
dfsrr;
eeprom: at24@50 {
- compatible = "st,24c256";
+ compatible = "st,24c256", "atmel,24c256";
reg = <0x50>;
};
@@ -130,7 +130,7 @@
};
spd: at24@51 {
- compatible = "at24,spd";
+ compatible = "atmel,spd";
reg = <0x51>;
};
diff --git a/arch/powerpc/boot/dts/mpc8377_rdb.dts b/arch/powerpc/boot/dts/mpc8377_rdb.dts
index e32613963ab0..5e85d8c93bca 100644
--- a/arch/powerpc/boot/dts/mpc8377_rdb.dts
+++ b/arch/powerpc/boot/dts/mpc8377_rdb.dts
@@ -150,7 +150,7 @@
};
at24@50 {
- compatible = "at24,24c256";
+ compatible = "atmel,24c256";
reg = <0x50>;
};
diff --git a/arch/powerpc/boot/dts/mpc8377_wlan.dts b/arch/powerpc/boot/dts/mpc8377_wlan.dts
index c0c790168b96..fee15fcbb46f 100644
--- a/arch/powerpc/boot/dts/mpc8377_wlan.dts
+++ b/arch/powerpc/boot/dts/mpc8377_wlan.dts
@@ -135,7 +135,7 @@
dfsrr;
at24@50 {
- compatible = "at24,24c256";
+ compatible = "atmel,24c256";
reg = <0x50>;
};
diff --git a/arch/powerpc/boot/dts/mpc8378_rdb.dts b/arch/powerpc/boot/dts/mpc8378_rdb.dts
index 71842fcd621f..e973d61956b9 100644
--- a/arch/powerpc/boot/dts/mpc8378_rdb.dts
+++ b/arch/powerpc/boot/dts/mpc8378_rdb.dts
@@ -150,7 +150,7 @@
};
at24@50 {
- compatible = "at24,24c256";
+ compatible = "atmel,24c256";
reg = <0x50>;
};
diff --git a/arch/powerpc/boot/dts/mpc8379_rdb.dts b/arch/powerpc/boot/dts/mpc8379_rdb.dts
index e442a29b2fe0..ed5d12ff2ee0 100644
--- a/arch/powerpc/boot/dts/mpc8379_rdb.dts
+++ b/arch/powerpc/boot/dts/mpc8379_rdb.dts
@@ -148,7 +148,7 @@
};
at24@50 {
- compatible = "at24,24c256";
+ compatible = "atmel,24c256";
reg = <0x50>;
};
--
2.9.3
^ permalink raw reply related [flat|nested] 115+ messages in thread* Re: [RESEND, v5, 15/16] powerpc/83xx: Add generic compatible string for I2C EEPROM
@ 2017-06-29 12:21 ` Michael Ellerman
0 siblings, 0 replies; 115+ messages in thread
From: Michael Ellerman @ 2017-06-29 12:21 UTC (permalink / raw)
To: Javier Martinez Canillas, linux-kernel
Cc: Mark Rutland, Rob Herring, devicetree, Wolfram Sang, Rob Herring,
Paul Mackerras, Javier Martinez Canillas, linuxppc-dev
On Thu, 2017-06-15 at 18:54:17 UTC, Javier Martinez Canillas wrote:
> The at24 driver allows to register I2C EEPROM chips using different vendor
> and devices, but the I2C subsystem does not take the vendor into account
> when matching using the I2C table since it only has device entries.
>
> But when matching using an OF table, both the vendor and device has to be
> taken into account so the driver defines only a set of compatible strings
> using the "atmel" vendor as a generic fallback for compatible I2C devices.
>
> So add this generic fallback to the device node compatible string to make
> the device to match the driver using the OF device ID table.
>
> Signed-off-by: Javier Martinez Canillas <javier@dowhile0.org>
Applied to powerpc next, thanks.
https://git.kernel.org/powerpc/c/8d0590cefdc22bc5d6f4d30ea19c95
cheers
^ permalink raw reply [flat|nested] 115+ messages in thread
* Re: [RESEND, v5, 15/16] powerpc/83xx: Add generic compatible string for I2C EEPROM
@ 2017-06-29 12:21 ` Michael Ellerman
0 siblings, 0 replies; 115+ messages in thread
From: Michael Ellerman @ 2017-06-29 12:21 UTC (permalink / raw)
To: linux-kernel-u79uwXL29TY76Z2rM5mHXA
Cc: Mark Rutland, Rob Herring, devicetree-u79uwXL29TY76Z2rM5mHXA,
Wolfram Sang, Rob Herring, Paul Mackerras,
Javier Martinez Canillas, linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ
On Thu, 2017-06-15 at 18:54:17 UTC, Javier Martinez Canillas wrote:
> The at24 driver allows to register I2C EEPROM chips using different vendor
> and devices, but the I2C subsystem does not take the vendor into account
> when matching using the I2C table since it only has device entries.
>
> But when matching using an OF table, both the vendor and device has to be
> taken into account so the driver defines only a set of compatible strings
> using the "atmel" vendor as a generic fallback for compatible I2C devices.
>
> So add this generic fallback to the device node compatible string to make
> the device to match the driver using the OF device ID table.
>
> Signed-off-by: Javier Martinez Canillas <javier-0uQlZySMnqxg9hUCZPvPmw@public.gmane.org>
Applied to powerpc next, thanks.
https://git.kernel.org/powerpc/c/8d0590cefdc22bc5d6f4d30ea19c95
cheers
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 115+ messages in thread
* [RESEND PATCH v5 16/16] powerpc/44x: Add generic compatible string for I2C EEPROM
2017-06-15 18:54 ` Javier Martinez Canillas
` (16 preceding siblings ...)
(?)
@ 2017-06-15 18:54 ` Javier Martinez Canillas
2017-06-29 12:21 ` Michael Ellerman
-1 siblings, 1 reply; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-06-15 18:54 UTC (permalink / raw)
To: linux-kernel
Cc: Wolfram Sang, Rob Herring, Javier Martinez Canillas, devicetree,
Michael Ellerman, Benjamin Herrenschmidt, Paul Mackerras,
Rob Herring, Mark Rutland, linuxppc-dev
The at24 driver allows to register I2C EEPROM chips using different vendor
and devices, but the I2C subsystem does not take the vendor into account
when matching using the I2C table since it only has device entries.
But when matching using an OF table, both the vendor and device has to be
taken into account so the driver defines only a set of compatible strings
using the "atmel" vendor as a generic fallback for compatible I2C devices.
So add this generic fallback to the device node compatible string to make
the device to match the driver using the OF device ID table.
Signed-off-by: Javier Martinez Canillas <javier@dowhile0.org>
---
Changes in v5: None
Changes in v4:
- Only use the atmel manufacturer in the compatible string instead of
keeping the deprecated ones (Rob Herring).
Changes in v3: None
Changes in v2: None
arch/powerpc/boot/dts/warp.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/powerpc/boot/dts/warp.dts b/arch/powerpc/boot/dts/warp.dts
index e576ee85c42f..ea9053ef4819 100644
--- a/arch/powerpc/boot/dts/warp.dts
+++ b/arch/powerpc/boot/dts/warp.dts
@@ -238,7 +238,7 @@
/* This will create 52 and 53 */
at24@52 {
- compatible = "at,24c04";
+ compatible = "atmel,24c04";
reg = <0x52>;
};
};
--
2.9.3
^ permalink raw reply related [flat|nested] 115+ messages in thread* Re: [RESEND, v5, 16/16] powerpc/44x: Add generic compatible string for I2C EEPROM
2017-06-15 18:54 ` [RESEND PATCH v5 16/16] powerpc/44x: " Javier Martinez Canillas
@ 2017-06-29 12:21 ` Michael Ellerman
0 siblings, 0 replies; 115+ messages in thread
From: Michael Ellerman @ 2017-06-29 12:21 UTC (permalink / raw)
To: Javier Martinez Canillas, linux-kernel
Cc: Mark Rutland, Rob Herring, devicetree, Wolfram Sang, Rob Herring,
Paul Mackerras, Javier Martinez Canillas, linuxppc-dev
On Thu, 2017-06-15 at 18:54:18 UTC, Javier Martinez Canillas wrote:
> The at24 driver allows to register I2C EEPROM chips using different vendor
> and devices, but the I2C subsystem does not take the vendor into account
> when matching using the I2C table since it only has device entries.
>
> But when matching using an OF table, both the vendor and device has to be
> taken into account so the driver defines only a set of compatible strings
> using the "atmel" vendor as a generic fallback for compatible I2C devices.
>
> So add this generic fallback to the device node compatible string to make
> the device to match the driver using the OF device ID table.
>
> Signed-off-by: Javier Martinez Canillas <javier@dowhile0.org>
Applied to powerpc next, thanks.
https://git.kernel.org/powerpc/c/adeb8667eaf5f00b5a7f0cede680cd
cheers
^ permalink raw reply [flat|nested] 115+ messages in thread
* Re: [RESEND, v5, 16/16] powerpc/44x: Add generic compatible string for I2C EEPROM
@ 2017-06-29 12:21 ` Michael Ellerman
0 siblings, 0 replies; 115+ messages in thread
From: Michael Ellerman @ 2017-06-29 12:21 UTC (permalink / raw)
To: linux-kernel
Cc: Mark Rutland, Rob Herring, devicetree, Wolfram Sang, Rob Herring,
Paul Mackerras, Javier Martinez Canillas, linuxppc-dev
On Thu, 2017-06-15 at 18:54:18 UTC, Javier Martinez Canillas wrote:
> The at24 driver allows to register I2C EEPROM chips using different vendor
> and devices, but the I2C subsystem does not take the vendor into account
> when matching using the I2C table since it only has device entries.
>
> But when matching using an OF table, both the vendor and device has to be
> taken into account so the driver defines only a set of compatible strings
> using the "atmel" vendor as a generic fallback for compatible I2C devices.
>
> So add this generic fallback to the device node compatible string to make
> the device to match the driver using the OF device ID table.
>
> Signed-off-by: Javier Martinez Canillas <javier@dowhile0.org>
Applied to powerpc next, thanks.
https://git.kernel.org/powerpc/c/adeb8667eaf5f00b5a7f0cede680cd
cheers
^ permalink raw reply [flat|nested] 115+ messages in thread
* Re: [RESEND PATCH v5 00/16] eeprom: at24: Add OF device ID table
2017-06-15 18:54 ` Javier Martinez Canillas
(?)
@ 2017-07-10 7:47 ` Javier Martinez Canillas
-1 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-07-10 7:47 UTC (permalink / raw)
To: Wolfram Sang
Cc: Linux Kernel, Rob Herring, Javier Martinez Canillas,
Florian Larysch, David Lechner, Rob Herring, Andy Shevchenko,
Catalin Marinas, Sören Brinkmann, Simon Horman,
Michal Simek, Dinh Nguyen, Russell King, Will Deacon, devicetree,
Sekhar Nori, Scott Wood, Benjamin Herrenschmidt,
Joachim Eastwood, Mark Rutland, linux-arm-kernel,
Masahiro Yamada, Michael Ellerman, Santosh Shilimkar,
Linux-Renesas, Paul Mackerras, Magnus Damm, linuxppc-dev,
Uwe Kleine-König, Linux I2C
Hello Wolfram,
On Thu, Jun 15, 2017 at 8:54 PM, Javier Martinez Canillas
<javier@dowhile0.org> wrote:
>
> This series is a follow-up to patch [0] that added an OF device ID table
> to the at24 EEPROM driver. As you suggested [1], this version instead of
> adding entries for every used <vendor,device> tuple, only adds a single
> entry for each chip type using the "atmel" vendor as a generic fallback.
>
> The first patch documents in the DT binding what's the correct vendor to
> use and what are the ones that are being deprecated. The second one adds
> the OF device ID table for the at24 driver and the next patches use this
> vendor in the compatible string to each DTS that defines a compatible I2C
> EEPROM device node.
>
> Patches can be applied independently since the DTS changes without driver
> changes are no-op and the OF table won't be used without the DTS changes.
>
> [0]: https://lkml.org/lkml/2017/3/14/589
> [1]: https://lkml.org/lkml/2017/3/15/99
>
Are you planning to pick this series? It has been in the list for
months and were resent many times...
Best regards,
Javier
^ permalink raw reply [flat|nested] 115+ messages in thread
* [RESEND PATCH v5 00/16] eeprom: at24: Add OF device ID table
@ 2017-07-10 7:47 ` Javier Martinez Canillas
0 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-07-10 7:47 UTC (permalink / raw)
To: linux-arm-kernel
Hello Wolfram,
On Thu, Jun 15, 2017 at 8:54 PM, Javier Martinez Canillas
<javier@dowhile0.org> wrote:
>
> This series is a follow-up to patch [0] that added an OF device ID table
> to the at24 EEPROM driver. As you suggested [1], this version instead of
> adding entries for every used <vendor,device> tuple, only adds a single
> entry for each chip type using the "atmel" vendor as a generic fallback.
>
> The first patch documents in the DT binding what's the correct vendor to
> use and what are the ones that are being deprecated. The second one adds
> the OF device ID table for the at24 driver and the next patches use this
> vendor in the compatible string to each DTS that defines a compatible I2C
> EEPROM device node.
>
> Patches can be applied independently since the DTS changes without driver
> changes are no-op and the OF table won't be used without the DTS changes.
>
> [0]: https://lkml.org/lkml/2017/3/14/589
> [1]: https://lkml.org/lkml/2017/3/15/99
>
Are you planning to pick this series? It has been in the list for
months and were resent many times...
Best regards,
Javier
^ permalink raw reply [flat|nested] 115+ messages in thread
* Re: [RESEND PATCH v5 00/16] eeprom: at24: Add OF device ID table
@ 2017-07-10 7:47 ` Javier Martinez Canillas
0 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-07-10 7:47 UTC (permalink / raw)
To: Wolfram Sang
Cc: Linux Kernel, Rob Herring, Javier Martinez Canillas,
Florian Larysch, David Lechner, Rob Herring, Andy Shevchenko,
Catalin Marinas, Sören Brinkmann, Simon Horman,
Michal Simek, Dinh Nguyen, Russell King, Will Deacon, devicetree,
Sekhar Nori, Scott Wood, Benjamin Herrenschmidt,
Joachim Eastwood
Hello Wolfram,
On Thu, Jun 15, 2017 at 8:54 PM, Javier Martinez Canillas
<javier@dowhile0.org> wrote:
>
> This series is a follow-up to patch [0] that added an OF device ID table
> to the at24 EEPROM driver. As you suggested [1], this version instead of
> adding entries for every used <vendor,device> tuple, only adds a single
> entry for each chip type using the "atmel" vendor as a generic fallback.
>
> The first patch documents in the DT binding what's the correct vendor to
> use and what are the ones that are being deprecated. The second one adds
> the OF device ID table for the at24 driver and the next patches use this
> vendor in the compatible string to each DTS that defines a compatible I2C
> EEPROM device node.
>
> Patches can be applied independently since the DTS changes without driver
> changes are no-op and the OF table won't be used without the DTS changes.
>
> [0]: https://lkml.org/lkml/2017/3/14/589
> [1]: https://lkml.org/lkml/2017/3/15/99
>
Are you planning to pick this series? It has been in the list for
months and were resent many times...
Best regards,
Javier
^ permalink raw reply [flat|nested] 115+ messages in thread
* Re: [RESEND PATCH v5 00/16] eeprom: at24: Add OF device ID table
2017-06-15 18:54 ` Javier Martinez Canillas
(?)
@ 2017-07-31 15:30 ` Wolfram Sang
-1 siblings, 0 replies; 115+ messages in thread
From: Wolfram Sang @ 2017-07-31 15:30 UTC (permalink / raw)
To: Javier Martinez Canillas
Cc: linux-kernel, Rob Herring, Florian Larysch, David Lechner,
Rob Herring, Andy Shevchenko, Catalin Marinas,
Sören Brinkmann, Simon Horman, Michal Simek, Dinh Nguyen,
Russell King, Will Deacon, devicetree, Sekhar Nori, Scott Wood,
Benjamin Herrenschmidt, Joachim Eastwood, Mark Rutland,
linux-arm-kernel, Masahiro Yamada, Michael Ellerman,
Santosh Shilimkar, linux-renesas-soc, Paul Mackerras,
Magnus Damm, linuxppc-dev, Uwe Kleine-König, linux-i2c
[-- Attachment #1: Type: text/plain, Size: 397 bytes --]
> Patches can be applied independently since the DTS changes without driver
> changes are no-op and the OF table won't be used without the DTS changes.
But there is a dependency, no? If I apply the driver patch,
non-converted device trees will not find their eeproms anymore. So, I
need to wait until all DTS patches are upstream, right? I can pick patch
1, though. We can already document it.
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 115+ messages in thread
* [RESEND PATCH v5 00/16] eeprom: at24: Add OF device ID table
@ 2017-07-31 15:30 ` Wolfram Sang
0 siblings, 0 replies; 115+ messages in thread
From: Wolfram Sang @ 2017-07-31 15:30 UTC (permalink / raw)
To: linux-arm-kernel
> Patches can be applied independently since the DTS changes without driver
> changes are no-op and the OF table won't be used without the DTS changes.
But there is a dependency, no? If I apply the driver patch,
non-converted device trees will not find their eeproms anymore. So, I
need to wait until all DTS patches are upstream, right? I can pick patch
1, though. We can already document it.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20170731/696b547b/attachment-0001.sig>
^ permalink raw reply [flat|nested] 115+ messages in thread
* Re: [RESEND PATCH v5 00/16] eeprom: at24: Add OF device ID table
@ 2017-07-31 15:30 ` Wolfram Sang
0 siblings, 0 replies; 115+ messages in thread
From: Wolfram Sang @ 2017-07-31 15:30 UTC (permalink / raw)
To: Javier Martinez Canillas
Cc: linux-kernel, Rob Herring, Florian Larysch, David Lechner,
Rob Herring, Andy Shevchenko, Catalin Marinas,
Sören Brinkmann, Simon Horman, Michal Simek, Dinh Nguyen,
Russell King, Will Deacon, devicetree, Sekhar Nori, Scott Wood,
Benjamin Herrenschmidt, Joachim Eastwood, Mark Rutland,
linux-arm-kernel
[-- Attachment #1: Type: text/plain, Size: 397 bytes --]
> Patches can be applied independently since the DTS changes without driver
> changes are no-op and the OF table won't be used without the DTS changes.
But there is a dependency, no? If I apply the driver patch,
non-converted device trees will not find their eeproms anymore. So, I
need to wait until all DTS patches are upstream, right? I can pick patch
1, though. We can already document it.
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 115+ messages in thread
* Re: [RESEND PATCH v5 00/16] eeprom: at24: Add OF device ID table
2017-07-31 15:30 ` Wolfram Sang
(?)
@ 2017-07-31 16:17 ` Javier Martinez Canillas
-1 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-07-31 16:17 UTC (permalink / raw)
To: Wolfram Sang
Cc: Linux Kernel, Rob Herring, Florian Larysch, David Lechner,
Rob Herring, Andy Shevchenko, Catalin Marinas,
Sören Brinkmann, Simon Horman, Michal Simek, Dinh Nguyen,
Russell King, Will Deacon, devicetree, Sekhar Nori, Scott Wood,
Benjamin Herrenschmidt, Joachim Eastwood, Mark Rutland,
linux-arm-kernel, Masahiro Yamada, Michael Ellerman,
Santosh Shilimkar, Linux-Renesas, Paul Mackerras, Magnus Damm,
linuxppc-dev, Uwe Kleine-König, Linux I2C
Hello Wolfram,
On Mon, Jul 31, 2017 at 5:30 PM, Wolfram Sang <wsa@the-dreams.de> wrote:
>
>> Patches can be applied independently since the DTS changes without driver
>> changes are no-op and the OF table won't be used without the DTS changes.
>
> But there is a dependency, no? If I apply the driver patch,
> non-converted device trees will not find their eeproms anymore. So, I
I don't think that's correct. If you apply this patch before the DTS
changes, the driver will still match using the I2C device ID table
like it has been doing it until today.
IOW, this is what will happen:
1- an OF device is registered with the wrong compatible (not found in
the OF table)
2- the I2C core strips the vendor part and fills the struct i2c_client
.name with the device part.
3- i2c_device_match() will be called since a new device has been registered
4- i2c_of_match_device() will fail because there's no OF entry that
matches the device compatible
5- the I2C core fallbacks to i2c_match_id() and matches using the I2C
device ID table.
So no noticeable difference AFAICT in that case.
Best regards,
Javier
^ permalink raw reply [flat|nested] 115+ messages in thread
* [RESEND PATCH v5 00/16] eeprom: at24: Add OF device ID table
@ 2017-07-31 16:17 ` Javier Martinez Canillas
0 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-07-31 16:17 UTC (permalink / raw)
To: linux-arm-kernel
Hello Wolfram,
On Mon, Jul 31, 2017 at 5:30 PM, Wolfram Sang <wsa@the-dreams.de> wrote:
>
>> Patches can be applied independently since the DTS changes without driver
>> changes are no-op and the OF table won't be used without the DTS changes.
>
> But there is a dependency, no? If I apply the driver patch,
> non-converted device trees will not find their eeproms anymore. So, I
I don't think that's correct. If you apply this patch before the DTS
changes, the driver will still match using the I2C device ID table
like it has been doing it until today.
IOW, this is what will happen:
1- an OF device is registered with the wrong compatible (not found in
the OF table)
2- the I2C core strips the vendor part and fills the struct i2c_client
.name with the device part.
3- i2c_device_match() will be called since a new device has been registered
4- i2c_of_match_device() will fail because there's no OF entry that
matches the device compatible
5- the I2C core fallbacks to i2c_match_id() and matches using the I2C
device ID table.
So no noticeable difference AFAICT in that case.
Best regards,
Javier
^ permalink raw reply [flat|nested] 115+ messages in thread
* Re: [RESEND PATCH v5 00/16] eeprom: at24: Add OF device ID table
@ 2017-07-31 16:17 ` Javier Martinez Canillas
0 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-07-31 16:17 UTC (permalink / raw)
To: Wolfram Sang
Cc: Linux Kernel, Rob Herring, Florian Larysch, David Lechner,
Rob Herring, Andy Shevchenko, Catalin Marinas,
Sören Brinkmann, Simon Horman, Michal Simek, Dinh Nguyen,
Russell King, Will Deacon, devicetree-u79uwXL29TY76Z2rM5mHXA,
Sekhar Nori, Scott Wood, Benjamin Herrenschmidt,
Joachim Eastwood, Mark Rutland
Hello Wolfram,
On Mon, Jul 31, 2017 at 5:30 PM, Wolfram Sang <wsa-z923LK4zBo2bacvFa/9K2g@public.gmane.org> wrote:
>
>> Patches can be applied independently since the DTS changes without driver
>> changes are no-op and the OF table won't be used without the DTS changes.
>
> But there is a dependency, no? If I apply the driver patch,
> non-converted device trees will not find their eeproms anymore. So, I
I don't think that's correct. If you apply this patch before the DTS
changes, the driver will still match using the I2C device ID table
like it has been doing it until today.
IOW, this is what will happen:
1- an OF device is registered with the wrong compatible (not found in
the OF table)
2- the I2C core strips the vendor part and fills the struct i2c_client
.name with the device part.
3- i2c_device_match() will be called since a new device has been registered
4- i2c_of_match_device() will fail because there's no OF entry that
matches the device compatible
5- the I2C core fallbacks to i2c_match_id() and matches using the I2C
device ID table.
So no noticeable difference AFAICT in that case.
Best regards,
Javier
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 115+ messages in thread
* Re: [RESEND PATCH v5 00/16] eeprom: at24: Add OF device ID table
@ 2017-08-28 16:01 ` Wolfram Sang
0 siblings, 0 replies; 115+ messages in thread
From: Wolfram Sang @ 2017-08-28 16:01 UTC (permalink / raw)
To: Javier Martinez Canillas
Cc: Linux Kernel, Rob Herring, Florian Larysch, David Lechner,
Rob Herring, Andy Shevchenko, Catalin Marinas,
Sören Brinkmann, Simon Horman, Michal Simek, Dinh Nguyen,
Russell King, Will Deacon, devicetree, Sekhar Nori, Scott Wood,
Benjamin Herrenschmidt, Joachim Eastwood, Mark Rutland,
linux-arm-kernel, Masahiro Yamada, Michael Ellerman,
Santosh Shilimkar, Linux-Renesas, Paul Mackerras, Magnus Damm,
linuxppc-dev, Uwe Kleine-König, Linux I2C
[-- Attachment #1: Type: text/plain, Size: 679 bytes --]
> > But there is a dependency, no? If I apply the driver patch,
> > non-converted device trees will not find their eeproms anymore. So, I
>
> I don't think that's correct. If you apply this patch before the DTS
> changes, the driver will still match using the I2C device ID table
> like it has been doing it until today.
My tests do not confirm this. If I add a node with a "renesas,24c01"
compatible to my board, it works before your patch, but not after. If I
change it to "atmel,24c01" it works even after your patch. I haven't
looked into it, though, maybe i2c_of_match_device_sysfs() is stepping on
our foots here?
Did you test and did it work for you?
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 115+ messages in thread
* [RESEND PATCH v5 00/16] eeprom: at24: Add OF device ID table
@ 2017-08-28 16:01 ` Wolfram Sang
0 siblings, 0 replies; 115+ messages in thread
From: Wolfram Sang @ 2017-08-28 16:01 UTC (permalink / raw)
To: linux-arm-kernel
> > But there is a dependency, no? If I apply the driver patch,
> > non-converted device trees will not find their eeproms anymore. So, I
>
> I don't think that's correct. If you apply this patch before the DTS
> changes, the driver will still match using the I2C device ID table
> like it has been doing it until today.
My tests do not confirm this. If I add a node with a "renesas,24c01"
compatible to my board, it works before your patch, but not after. If I
change it to "atmel,24c01" it works even after your patch. I haven't
looked into it, though, maybe i2c_of_match_device_sysfs() is stepping on
our foots here?
Did you test and did it work for you?
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20170828/a9aba751/attachment.sig>
^ permalink raw reply [flat|nested] 115+ messages in thread
* Re: [RESEND PATCH v5 00/16] eeprom: at24: Add OF device ID table
@ 2017-08-28 16:01 ` Wolfram Sang
0 siblings, 0 replies; 115+ messages in thread
From: Wolfram Sang @ 2017-08-28 16:01 UTC (permalink / raw)
To: Javier Martinez Canillas
Cc: Linux Kernel, Rob Herring, Florian Larysch, David Lechner,
Rob Herring, Andy Shevchenko, Catalin Marinas,
Sören Brinkmann, Simon Horman, Michal Simek, Dinh Nguyen,
Russell King, Will Deacon, devicetree-u79uwXL29TY76Z2rM5mHXA,
Sekhar Nori, Scott Wood, Benjamin Herrenschmidt,
Joachim Eastwood, Mark Rutland
[-- Attachment #1: Type: text/plain, Size: 679 bytes --]
> > But there is a dependency, no? If I apply the driver patch,
> > non-converted device trees will not find their eeproms anymore. So, I
>
> I don't think that's correct. If you apply this patch before the DTS
> changes, the driver will still match using the I2C device ID table
> like it has been doing it until today.
My tests do not confirm this. If I add a node with a "renesas,24c01"
compatible to my board, it works before your patch, but not after. If I
change it to "atmel,24c01" it works even after your patch. I haven't
looked into it, though, maybe i2c_of_match_device_sysfs() is stepping on
our foots here?
Did you test and did it work for you?
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 115+ messages in thread
* Re: [RESEND PATCH v5 00/16] eeprom: at24: Add OF device ID table
2017-08-28 16:01 ` Wolfram Sang
(?)
@ 2017-08-29 8:44 ` Javier Martinez Canillas
-1 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-08-29 8:44 UTC (permalink / raw)
To: Wolfram Sang
Cc: Linux Kernel, Rob Herring, Florian Larysch, David Lechner,
Rob Herring, Andy Shevchenko, Catalin Marinas,
Sören Brinkmann, Simon Horman, Michal Simek, Dinh Nguyen,
Russell King, Will Deacon, devicetree, Sekhar Nori, Scott Wood,
Benjamin Herrenschmidt, Joachim Eastwood, Mark Rutland,
linux-arm-kernel, Masahiro Yamada, Michael Ellerman,
Santosh Shilimkar, Linux-Renesas, Paul Mackerras, Magnus Damm,
linuxppc-dev, Uwe Kleine-König, Linux I2C
Hello Wolfram,
On Mon, Aug 28, 2017 at 6:01 PM, Wolfram Sang <wsa@the-dreams.de> wrote:
>
>> > But there is a dependency, no? If I apply the driver patch,
>> > non-converted device trees will not find their eeproms anymore. So, I
>>
>> I don't think that's correct. If you apply this patch before the DTS
>> changes, the driver will still match using the I2C device ID table
>> like it has been doing it until today.
>
> My tests do not confirm this. If I add a node with a "renesas,24c01"
> compatible to my board, it works before your patch, but not after. If I
> change it to "atmel,24c01" it works even after your patch. I haven't
> looked into it, though, maybe i2c_of_match_device_sysfs() is stepping on
> our foots here?
>
> Did you test and did it work for you?
>
I would swear that I tested both combinations (driver patch without DT
changes and DTS changes without driver patch), but it was months ago
when I first posted the patches so I may misremembering.
I don't have a DT based system at hand now, but I'll test it again and
let you know probably tomorrow.
Best regards,
Javier
^ permalink raw reply [flat|nested] 115+ messages in thread
* [RESEND PATCH v5 00/16] eeprom: at24: Add OF device ID table
@ 2017-08-29 8:44 ` Javier Martinez Canillas
0 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-08-29 8:44 UTC (permalink / raw)
To: linux-arm-kernel
Hello Wolfram,
On Mon, Aug 28, 2017 at 6:01 PM, Wolfram Sang <wsa@the-dreams.de> wrote:
>
>> > But there is a dependency, no? If I apply the driver patch,
>> > non-converted device trees will not find their eeproms anymore. So, I
>>
>> I don't think that's correct. If you apply this patch before the DTS
>> changes, the driver will still match using the I2C device ID table
>> like it has been doing it until today.
>
> My tests do not confirm this. If I add a node with a "renesas,24c01"
> compatible to my board, it works before your patch, but not after. If I
> change it to "atmel,24c01" it works even after your patch. I haven't
> looked into it, though, maybe i2c_of_match_device_sysfs() is stepping on
> our foots here?
>
> Did you test and did it work for you?
>
I would swear that I tested both combinations (driver patch without DT
changes and DTS changes without driver patch), but it was months ago
when I first posted the patches so I may misremembering.
I don't have a DT based system at hand now, but I'll test it again and
let you know probably tomorrow.
Best regards,
Javier
^ permalink raw reply [flat|nested] 115+ messages in thread
* Re: [RESEND PATCH v5 00/16] eeprom: at24: Add OF device ID table
@ 2017-08-29 8:44 ` Javier Martinez Canillas
0 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-08-29 8:44 UTC (permalink / raw)
To: Wolfram Sang
Cc: Linux Kernel, Rob Herring, Florian Larysch, David Lechner,
Rob Herring, Andy Shevchenko, Catalin Marinas,
Sören Brinkmann, Simon Horman, Michal Simek, Dinh Nguyen,
Russell King, Will Deacon, devicetree, Sekhar Nori, Scott Wood,
Benjamin Herrenschmidt, Joachim Eastwood, Mark Rutland
Hello Wolfram,
On Mon, Aug 28, 2017 at 6:01 PM, Wolfram Sang <wsa@the-dreams.de> wrote:
>
>> > But there is a dependency, no? If I apply the driver patch,
>> > non-converted device trees will not find their eeproms anymore. So, I
>>
>> I don't think that's correct. If you apply this patch before the DTS
>> changes, the driver will still match using the I2C device ID table
>> like it has been doing it until today.
>
> My tests do not confirm this. If I add a node with a "renesas,24c01"
> compatible to my board, it works before your patch, but not after. If I
> change it to "atmel,24c01" it works even after your patch. I haven't
> looked into it, though, maybe i2c_of_match_device_sysfs() is stepping on
> our foots here?
>
> Did you test and did it work for you?
>
I would swear that I tested both combinations (driver patch without DT
changes and DTS changes without driver patch), but it was months ago
when I first posted the patches so I may misremembering.
I don't have a DT based system at hand now, but I'll test it again and
let you know probably tomorrow.
Best regards,
Javier
^ permalink raw reply [flat|nested] 115+ messages in thread
* Re: [RESEND PATCH v5 00/16] eeprom: at24: Add OF device ID table
2017-08-29 8:44 ` Javier Martinez Canillas
(?)
@ 2017-08-29 8:48 ` Wolfram Sang
-1 siblings, 0 replies; 115+ messages in thread
From: Wolfram Sang @ 2017-08-29 8:48 UTC (permalink / raw)
To: Javier Martinez Canillas
Cc: Linux Kernel, Rob Herring, Florian Larysch, David Lechner,
Rob Herring, Andy Shevchenko, Catalin Marinas,
Sören Brinkmann, Simon Horman, Michal Simek, Dinh Nguyen,
Russell King, Will Deacon, devicetree, Sekhar Nori, Scott Wood,
Benjamin Herrenschmidt, Joachim Eastwood, Mark Rutland,
linux-arm-kernel, Masahiro Yamada, Michael Ellerman,
Santosh Shilimkar, Linux-Renesas, Paul Mackerras, Magnus Damm,
linuxppc-dev, Uwe Kleine-König, Linux I2C
[-- Attachment #1: Type: text/plain, Size: 147 bytes --]
> I don't have a DT based system at hand now, but I'll test it again and
> let you know probably tomorrow.
I will try again today, too. Thanks!
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 115+ messages in thread
* [RESEND PATCH v5 00/16] eeprom: at24: Add OF device ID table
@ 2017-08-29 8:48 ` Wolfram Sang
0 siblings, 0 replies; 115+ messages in thread
From: Wolfram Sang @ 2017-08-29 8:48 UTC (permalink / raw)
To: linux-arm-kernel
> I don't have a DT based system at hand now, but I'll test it again and
> let you know probably tomorrow.
I will try again today, too. Thanks!
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20170829/5ad7fa63/attachment.sig>
^ permalink raw reply [flat|nested] 115+ messages in thread
* Re: [RESEND PATCH v5 00/16] eeprom: at24: Add OF device ID table
@ 2017-08-29 8:48 ` Wolfram Sang
0 siblings, 0 replies; 115+ messages in thread
From: Wolfram Sang @ 2017-08-29 8:48 UTC (permalink / raw)
To: Javier Martinez Canillas
Cc: Linux Kernel, Rob Herring, Florian Larysch, David Lechner,
Rob Herring, Andy Shevchenko, Catalin Marinas,
Sören Brinkmann, Simon Horman, Michal Simek, Dinh Nguyen,
Russell King, Will Deacon, devicetree, Sekhar Nori, Scott Wood,
Benjamin Herrenschmidt, Joachim Eastwood, Mark Rutland
[-- Attachment #1: Type: text/plain, Size: 147 bytes --]
> I don't have a DT based system at hand now, but I'll test it again and
> let you know probably tomorrow.
I will try again today, too. Thanks!
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 115+ messages in thread
* Re: [RESEND PATCH v5 00/16] eeprom: at24: Add OF device ID table
2017-08-29 8:48 ` Wolfram Sang
(?)
@ 2017-08-30 16:19 ` Javier Martinez Canillas
-1 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-08-30 16:19 UTC (permalink / raw)
To: Wolfram Sang
Cc: Linux Kernel, Rob Herring, Florian Larysch, David Lechner,
Rob Herring, Andy Shevchenko, Catalin Marinas,
Sören Brinkmann, Simon Horman, Michal Simek, Dinh Nguyen,
Russell King, Will Deacon, devicetree, Sekhar Nori, Scott Wood,
Benjamin Herrenschmidt, Joachim Eastwood, Mark Rutland,
linux-arm-kernel, Masahiro Yamada, Michael Ellerman,
Santosh Shilimkar, Linux-Renesas, Paul Mackerras, Magnus Damm,
linuxppc-dev, Uwe Kleine-König, Linux I2C
Hello Wolfram,
On Tue, Aug 29, 2017 at 10:48 AM, Wolfram Sang <wsa@the-dreams.de> wrote:
>
>> I don't have a DT based system at hand now, but I'll test it again and
>> let you know probably tomorrow.
>
> I will try again today, too. Thanks!
>
Ok, I had some time to do some tests again. I used an ARM Chromebook
(Exynos Peach Pi) that has an I2C touchpad (Atmel maXTouch).
Tested the following cases:
1) Driver without OF device ID table (only a I2C table with a
"maxtouch" entry) and DTS defining a device node with a
"atmel,maxtouch" compatible string. This is the case without any of
the patches in this series.
$ modinfo drivers/input/touchscreen/atmel_mxt_ts.ko | grep maxtouch
alias: i2c:maxtouch
$ grep maxtouch /sys/devices/platform/soc/12e00000.i2c/i2c-8/8-004b/uevent
OF_COMPATIBLE_0=atmel,maxtouch
MODALIAS=i2c:maxtouch
2) Driver without OF device ID table (only a I2C table with a
"maxtouch" entry) and DTS defining a device node with a
"atmel,maxtouch", "generic,maxtouch" compatible string. This is the
case when platform maintainers merge the DTS patches without the
driver patch.
$ modinfo drivers/input/touchscreen/atmel_mxt_ts.ko | grep maxtouch
alias: i2c:maxtouch
$ grep maxtouch /sys/devices/platform/soc/12e00000.i2c/i2c-8/8-004b/uevent
OF_COMPATIBLE_0=atmel,maxtouch
OF_COMPATIBLE_1=generic,maxtouch
MODALIAS=i2c:maxtouch
3) Driver with an OF device ID table (with a "generic,maxtouch" entry)
and DTS defining a device node with a "atmel,maxtouch" compatible
string. This is the case when the driver patch is merged without the
DTS patches.
$ modinfo drivers/input/touchscreen/atmel_mxt_ts.ko | grep maxtouch
alias: of:N*T*Cgeneric,maxtouchC*
alias: of:N*T*Cgeneric,maxtouch
alias: i2c:maxtouch
$ grep maxtouch /sys/devices/platform/soc/12e00000.i2c/i2c-8/8-004b/uevent
OF_COMPATIBLE_0=atmel,maxtouch
MODALIAS=i2c:maxtouch
4) Driver with an OF device ID table (with a "generic,maxtouch" entry)
and DTS defining a device node with a "atmel,maxtouch",
"generic,maxtouch" compatible string. This is the case when both the
DTS and driver patches are merged.
$ modinfo drivers/input/touchscreen/atmel_mxt_ts.ko | grep maxtouch
alias: of:N*T*Cgeneric,maxtouchC*
alias: of:N*T*Cgeneric,maxtouch
alias: i2c:maxtouch
$ grep maxtouch /sys/devices/platform/soc/12e00000.i2c/i2c-8/8-004b/uevent
OF_COMPATIBLE_0=atmel,maxtouch
OF_COMPATIBLE_1=generic,maxtouch
MODALIAS=i2c:maxtouch
For all cases module autoload, driver probe and evtest worked for me.
You said that (3) doesn't work but I don't understand why is failing
for you. Probably I'm missing something.
Best regards,
Javier
^ permalink raw reply [flat|nested] 115+ messages in thread
* [RESEND PATCH v5 00/16] eeprom: at24: Add OF device ID table
@ 2017-08-30 16:19 ` Javier Martinez Canillas
0 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-08-30 16:19 UTC (permalink / raw)
To: linux-arm-kernel
Hello Wolfram,
On Tue, Aug 29, 2017 at 10:48 AM, Wolfram Sang <wsa@the-dreams.de> wrote:
>
>> I don't have a DT based system at hand now, but I'll test it again and
>> let you know probably tomorrow.
>
> I will try again today, too. Thanks!
>
Ok, I had some time to do some tests again. I used an ARM Chromebook
(Exynos Peach Pi) that has an I2C touchpad (Atmel maXTouch).
Tested the following cases:
1) Driver without OF device ID table (only a I2C table with a
"maxtouch" entry) and DTS defining a device node with a
"atmel,maxtouch" compatible string. This is the case without any of
the patches in this series.
$ modinfo drivers/input/touchscreen/atmel_mxt_ts.ko | grep maxtouch
alias: i2c:maxtouch
$ grep maxtouch /sys/devices/platform/soc/12e00000.i2c/i2c-8/8-004b/uevent
OF_COMPATIBLE_0=atmel,maxtouch
MODALIAS=i2c:maxtouch
2) Driver without OF device ID table (only a I2C table with a
"maxtouch" entry) and DTS defining a device node with a
"atmel,maxtouch", "generic,maxtouch" compatible string. This is the
case when platform maintainers merge the DTS patches without the
driver patch.
$ modinfo drivers/input/touchscreen/atmel_mxt_ts.ko | grep maxtouch
alias: i2c:maxtouch
$ grep maxtouch /sys/devices/platform/soc/12e00000.i2c/i2c-8/8-004b/uevent
OF_COMPATIBLE_0=atmel,maxtouch
OF_COMPATIBLE_1=generic,maxtouch
MODALIAS=i2c:maxtouch
3) Driver with an OF device ID table (with a "generic,maxtouch" entry)
and DTS defining a device node with a "atmel,maxtouch" compatible
string. This is the case when the driver patch is merged without the
DTS patches.
$ modinfo drivers/input/touchscreen/atmel_mxt_ts.ko | grep maxtouch
alias: of:N*T*Cgeneric,maxtouchC*
alias: of:N*T*Cgeneric,maxtouch
alias: i2c:maxtouch
$ grep maxtouch /sys/devices/platform/soc/12e00000.i2c/i2c-8/8-004b/uevent
OF_COMPATIBLE_0=atmel,maxtouch
MODALIAS=i2c:maxtouch
4) Driver with an OF device ID table (with a "generic,maxtouch" entry)
and DTS defining a device node with a "atmel,maxtouch",
"generic,maxtouch" compatible string. This is the case when both the
DTS and driver patches are merged.
$ modinfo drivers/input/touchscreen/atmel_mxt_ts.ko | grep maxtouch
alias: of:N*T*Cgeneric,maxtouchC*
alias: of:N*T*Cgeneric,maxtouch
alias: i2c:maxtouch
$ grep maxtouch /sys/devices/platform/soc/12e00000.i2c/i2c-8/8-004b/uevent
OF_COMPATIBLE_0=atmel,maxtouch
OF_COMPATIBLE_1=generic,maxtouch
MODALIAS=i2c:maxtouch
For all cases module autoload, driver probe and evtest worked for me.
You said that (3) doesn't work but I don't understand why is failing
for you. Probably I'm missing something.
Best regards,
Javier
^ permalink raw reply [flat|nested] 115+ messages in thread
* Re: [RESEND PATCH v5 00/16] eeprom: at24: Add OF device ID table
@ 2017-08-30 16:19 ` Javier Martinez Canillas
0 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-08-30 16:19 UTC (permalink / raw)
To: Wolfram Sang
Cc: Linux Kernel, Rob Herring, Florian Larysch, David Lechner,
Rob Herring, Andy Shevchenko, Catalin Marinas,
Sören Brinkmann, Simon Horman, Michal Simek, Dinh Nguyen,
Russell King, Will Deacon, devicetree-u79uwXL29TY76Z2rM5mHXA,
Sekhar Nori, Scott Wood, Benjamin Herrenschmidt,
Joachim Eastwood, Mark Rutland
Hello Wolfram,
On Tue, Aug 29, 2017 at 10:48 AM, Wolfram Sang <wsa-z923LK4zBo2bacvFa/9K2g@public.gmane.org> wrote:
>
>> I don't have a DT based system at hand now, but I'll test it again and
>> let you know probably tomorrow.
>
> I will try again today, too. Thanks!
>
Ok, I had some time to do some tests again. I used an ARM Chromebook
(Exynos Peach Pi) that has an I2C touchpad (Atmel maXTouch).
Tested the following cases:
1) Driver without OF device ID table (only a I2C table with a
"maxtouch" entry) and DTS defining a device node with a
"atmel,maxtouch" compatible string. This is the case without any of
the patches in this series.
$ modinfo drivers/input/touchscreen/atmel_mxt_ts.ko | grep maxtouch
alias: i2c:maxtouch
$ grep maxtouch /sys/devices/platform/soc/12e00000.i2c/i2c-8/8-004b/uevent
OF_COMPATIBLE_0=atmel,maxtouch
MODALIAS=i2c:maxtouch
2) Driver without OF device ID table (only a I2C table with a
"maxtouch" entry) and DTS defining a device node with a
"atmel,maxtouch", "generic,maxtouch" compatible string. This is the
case when platform maintainers merge the DTS patches without the
driver patch.
$ modinfo drivers/input/touchscreen/atmel_mxt_ts.ko | grep maxtouch
alias: i2c:maxtouch
$ grep maxtouch /sys/devices/platform/soc/12e00000.i2c/i2c-8/8-004b/uevent
OF_COMPATIBLE_0=atmel,maxtouch
OF_COMPATIBLE_1=generic,maxtouch
MODALIAS=i2c:maxtouch
3) Driver with an OF device ID table (with a "generic,maxtouch" entry)
and DTS defining a device node with a "atmel,maxtouch" compatible
string. This is the case when the driver patch is merged without the
DTS patches.
$ modinfo drivers/input/touchscreen/atmel_mxt_ts.ko | grep maxtouch
alias: of:N*T*Cgeneric,maxtouchC*
alias: of:N*T*Cgeneric,maxtouch
alias: i2c:maxtouch
$ grep maxtouch /sys/devices/platform/soc/12e00000.i2c/i2c-8/8-004b/uevent
OF_COMPATIBLE_0=atmel,maxtouch
MODALIAS=i2c:maxtouch
4) Driver with an OF device ID table (with a "generic,maxtouch" entry)
and DTS defining a device node with a "atmel,maxtouch",
"generic,maxtouch" compatible string. This is the case when both the
DTS and driver patches are merged.
$ modinfo drivers/input/touchscreen/atmel_mxt_ts.ko | grep maxtouch
alias: of:N*T*Cgeneric,maxtouchC*
alias: of:N*T*Cgeneric,maxtouch
alias: i2c:maxtouch
$ grep maxtouch /sys/devices/platform/soc/12e00000.i2c/i2c-8/8-004b/uevent
OF_COMPATIBLE_0=atmel,maxtouch
OF_COMPATIBLE_1=generic,maxtouch
MODALIAS=i2c:maxtouch
For all cases module autoload, driver probe and evtest worked for me.
You said that (3) doesn't work but I don't understand why is failing
for you. Probably I'm missing something.
Best regards,
Javier
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 115+ messages in thread
* Re: [RESEND PATCH v5 00/16] eeprom: at24: Add OF device ID table
2017-08-30 16:19 ` Javier Martinez Canillas
(?)
@ 2017-08-30 17:42 ` Wolfram Sang
-1 siblings, 0 replies; 115+ messages in thread
From: Wolfram Sang @ 2017-08-30 17:42 UTC (permalink / raw)
To: Javier Martinez Canillas
Cc: Linux Kernel, Rob Herring, Florian Larysch, David Lechner,
Rob Herring, Andy Shevchenko, Catalin Marinas,
Sören Brinkmann, Simon Horman, Michal Simek, Dinh Nguyen,
Russell King, Will Deacon, devicetree, Sekhar Nori, Scott Wood,
Benjamin Herrenschmidt, Joachim Eastwood, Mark Rutland,
linux-arm-kernel, Masahiro Yamada, Michael Ellerman,
Santosh Shilimkar, Linux-Renesas, Paul Mackerras, Magnus Damm,
linuxppc-dev, Uwe Kleine-König, Linux I2C
[-- Attachment #1: Type: text/plain, Size: 1718 bytes --]
On Wed, Aug 30, 2017 at 06:19:02PM +0200, Javier Martinez Canillas wrote:
> Hello Wolfram,
>
> On Tue, Aug 29, 2017 at 10:48 AM, Wolfram Sang <wsa@the-dreams.de> wrote:
> >
> >> I don't have a DT based system at hand now, but I'll test it again and
> >> let you know probably tomorrow.
> >
> > I will try again today, too. Thanks!
> >
>
> Ok, I had some time to do some tests again. I used an ARM Chromebook
> (Exynos Peach Pi) that has an I2C touchpad (Atmel maXTouch).
I tried again as well and it still fails for me.
> Tested the following cases:
I think we should talk about the same case: Let me repeat what I did:
1) I added your patch "eeprom: at24: Add OF device ID table"
2) I added an EEPROM node to an I2C
+ eeprom@50 {
+ compatible = "renesas,24c01";
+ reg = <0x50>;
+ };
-> no at24 binding to the device
3) I revert your patch
-> at24 binding to the device
I think you should be able to test this DTS snipplet even without a real
eeprom. Especially after applying this to the at24 driver.
diff --git a/drivers/misc/eeprom/at24.c b/drivers/misc/eeprom/at24.c
index 79c5c39be29cac..f9f547680c53db 100644
--- a/drivers/misc/eeprom/at24.c
+++ b/drivers/misc/eeprom/at24.c
@@ -805,11 +805,6 @@ static int at24_probe(struct i2c_client *client, const struct i2c_device_id *id)
* Perform a one-byte test read to verify that the
* chip is functional.
*/
- err = at24_read(at24, 0, &test_byte, 1);
- if (err) {
- err = -ENODEV;
- goto err_clients;
- }
at24->nvmem_config.name = dev_name(&client->dev);
at24->nvmem_config.dev = &client->dev;
Can you check this?
Thanks,
Wolfram
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply related [flat|nested] 115+ messages in thread* [RESEND PATCH v5 00/16] eeprom: at24: Add OF device ID table
@ 2017-08-30 17:42 ` Wolfram Sang
0 siblings, 0 replies; 115+ messages in thread
From: Wolfram Sang @ 2017-08-30 17:42 UTC (permalink / raw)
To: linux-arm-kernel
On Wed, Aug 30, 2017 at 06:19:02PM +0200, Javier Martinez Canillas wrote:
> Hello Wolfram,
>
> On Tue, Aug 29, 2017 at 10:48 AM, Wolfram Sang <wsa@the-dreams.de> wrote:
> >
> >> I don't have a DT based system at hand now, but I'll test it again and
> >> let you know probably tomorrow.
> >
> > I will try again today, too. Thanks!
> >
>
> Ok, I had some time to do some tests again. I used an ARM Chromebook
> (Exynos Peach Pi) that has an I2C touchpad (Atmel maXTouch).
I tried again as well and it still fails for me.
> Tested the following cases:
I think we should talk about the same case: Let me repeat what I did:
1) I added your patch "eeprom: at24: Add OF device ID table"
2) I added an EEPROM node to an I2C
+ eeprom at 50 {
+ compatible = "renesas,24c01";
+ reg = <0x50>;
+ };
-> no at24 binding to the device
3) I revert your patch
-> at24 binding to the device
I think you should be able to test this DTS snipplet even without a real
eeprom. Especially after applying this to the at24 driver.
diff --git a/drivers/misc/eeprom/at24.c b/drivers/misc/eeprom/at24.c
index 79c5c39be29cac..f9f547680c53db 100644
--- a/drivers/misc/eeprom/at24.c
+++ b/drivers/misc/eeprom/at24.c
@@ -805,11 +805,6 @@ static int at24_probe(struct i2c_client *client, const struct i2c_device_id *id)
* Perform a one-byte test read to verify that the
* chip is functional.
*/
- err = at24_read(at24, 0, &test_byte, 1);
- if (err) {
- err = -ENODEV;
- goto err_clients;
- }
at24->nvmem_config.name = dev_name(&client->dev);
at24->nvmem_config.dev = &client->dev;
Can you check this?
Thanks,
Wolfram
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20170830/ff0f0039/attachment.sig>
^ permalink raw reply related [flat|nested] 115+ messages in thread* Re: [RESEND PATCH v5 00/16] eeprom: at24: Add OF device ID table
@ 2017-08-30 17:42 ` Wolfram Sang
0 siblings, 0 replies; 115+ messages in thread
From: Wolfram Sang @ 2017-08-30 17:42 UTC (permalink / raw)
To: Javier Martinez Canillas
Cc: Linux Kernel, Rob Herring, Florian Larysch, David Lechner,
Rob Herring, Andy Shevchenko, Catalin Marinas,
Sören Brinkmann, Simon Horman, Michal Simek, Dinh Nguyen,
Russell King, Will Deacon, devicetree, Sekhar Nori, Scott Wood,
Benjamin Herrenschmidt, Joachim Eastwood, Mark Rutland
[-- Attachment #1: Type: text/plain, Size: 1718 bytes --]
On Wed, Aug 30, 2017 at 06:19:02PM +0200, Javier Martinez Canillas wrote:
> Hello Wolfram,
>
> On Tue, Aug 29, 2017 at 10:48 AM, Wolfram Sang <wsa@the-dreams.de> wrote:
> >
> >> I don't have a DT based system at hand now, but I'll test it again and
> >> let you know probably tomorrow.
> >
> > I will try again today, too. Thanks!
> >
>
> Ok, I had some time to do some tests again. I used an ARM Chromebook
> (Exynos Peach Pi) that has an I2C touchpad (Atmel maXTouch).
I tried again as well and it still fails for me.
> Tested the following cases:
I think we should talk about the same case: Let me repeat what I did:
1) I added your patch "eeprom: at24: Add OF device ID table"
2) I added an EEPROM node to an I2C
+ eeprom@50 {
+ compatible = "renesas,24c01";
+ reg = <0x50>;
+ };
-> no at24 binding to the device
3) I revert your patch
-> at24 binding to the device
I think you should be able to test this DTS snipplet even without a real
eeprom. Especially after applying this to the at24 driver.
diff --git a/drivers/misc/eeprom/at24.c b/drivers/misc/eeprom/at24.c
index 79c5c39be29cac..f9f547680c53db 100644
--- a/drivers/misc/eeprom/at24.c
+++ b/drivers/misc/eeprom/at24.c
@@ -805,11 +805,6 @@ static int at24_probe(struct i2c_client *client, const struct i2c_device_id *id)
* Perform a one-byte test read to verify that the
* chip is functional.
*/
- err = at24_read(at24, 0, &test_byte, 1);
- if (err) {
- err = -ENODEV;
- goto err_clients;
- }
at24->nvmem_config.name = dev_name(&client->dev);
at24->nvmem_config.dev = &client->dev;
Can you check this?
Thanks,
Wolfram
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply related [flat|nested] 115+ messages in thread* Re: [RESEND PATCH v5 00/16] eeprom: at24: Add OF device ID table
2017-08-30 17:42 ` Wolfram Sang
(?)
@ 2017-08-30 19:57 ` Javier Martinez Canillas
-1 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-08-30 19:57 UTC (permalink / raw)
To: Wolfram Sang
Cc: Linux Kernel, Rob Herring, Florian Larysch, David Lechner,
Rob Herring, Andy Shevchenko, Catalin Marinas,
Sören Brinkmann, Simon Horman, Michal Simek, Dinh Nguyen,
Russell King, Will Deacon, devicetree, Sekhar Nori, Scott Wood,
Benjamin Herrenschmidt, Joachim Eastwood, Mark Rutland,
linux-arm-kernel, Masahiro Yamada, Michael Ellerman,
Santosh Shilimkar, Linux-Renesas, Paul Mackerras, Magnus Damm,
linuxppc-dev, Uwe Kleine-König, Linux I2C
>
> I think we should talk about the same case: Let me repeat what I did:
>
> 1) I added your patch "eeprom: at24: Add OF device ID table"
> 2) I added an EEPROM node to an I2C
>
> + eeprom@50 {
> + compatible = "renesas,24c01";
> + reg = <0x50>;
> + };
>
> -> no at24 binding to the device
>
> 3) I revert your patch
>
> -> at24 binding to the device
>
I've tested this and you are right, it fails...
The problem is that the patch also changes how the driver obtains the
EEPROM parameters (the magic value in the entry's data field).
So even when module autoload and device / driver matching works, the
driver probe function fails because if (client->dev.of_node) the
driver attempts to get the entry data using
of_device_get_match_data(), which is obviously wrong since the
compatible string in the dev node isn't present in the OF table.
The id->driver_data from the I2C table should be used instead since
that's the table that matches in this case.
One option is to fallback to id->driver_data if
of_device_get_match_data() fails, but that's just an (ugly)
workaround. So I agree with you that the best option is to wait for
the DTS patches to land first.
It worked for me on my previous tests because the tested drivers
didn't use a table entry data, I'm so sorry for missing this :(
Best regards,
Javier
^ permalink raw reply [flat|nested] 115+ messages in thread* [RESEND PATCH v5 00/16] eeprom: at24: Add OF device ID table
@ 2017-08-30 19:57 ` Javier Martinez Canillas
0 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-08-30 19:57 UTC (permalink / raw)
To: linux-arm-kernel
>
> I think we should talk about the same case: Let me repeat what I did:
>
> 1) I added your patch "eeprom: at24: Add OF device ID table"
> 2) I added an EEPROM node to an I2C
>
> + eeprom at 50 {
> + compatible = "renesas,24c01";
> + reg = <0x50>;
> + };
>
> -> no at24 binding to the device
>
> 3) I revert your patch
>
> -> at24 binding to the device
>
I've tested this and you are right, it fails...
The problem is that the patch also changes how the driver obtains the
EEPROM parameters (the magic value in the entry's data field).
So even when module autoload and device / driver matching works, the
driver probe function fails because if (client->dev.of_node) the
driver attempts to get the entry data using
of_device_get_match_data(), which is obviously wrong since the
compatible string in the dev node isn't present in the OF table.
The id->driver_data from the I2C table should be used instead since
that's the table that matches in this case.
One option is to fallback to id->driver_data if
of_device_get_match_data() fails, but that's just an (ugly)
workaround. So I agree with you that the best option is to wait for
the DTS patches to land first.
It worked for me on my previous tests because the tested drivers
didn't use a table entry data, I'm so sorry for missing this :(
Best regards,
Javier
^ permalink raw reply [flat|nested] 115+ messages in thread* Re: [RESEND PATCH v5 00/16] eeprom: at24: Add OF device ID table
@ 2017-08-30 19:57 ` Javier Martinez Canillas
0 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-08-30 19:57 UTC (permalink / raw)
To: Wolfram Sang
Cc: Linux Kernel, Rob Herring, Florian Larysch, David Lechner,
Rob Herring, Andy Shevchenko, Catalin Marinas,
Sören Brinkmann, Simon Horman, Michal Simek, Dinh Nguyen,
Russell King, Will Deacon, devicetree, Sekhar Nori, Scott Wood,
Benjamin Herrenschmidt, Joachim Eastwood, Mark Rutland
>
> I think we should talk about the same case: Let me repeat what I did:
>
> 1) I added your patch "eeprom: at24: Add OF device ID table"
> 2) I added an EEPROM node to an I2C
>
> + eeprom@50 {
> + compatible = "renesas,24c01";
> + reg = <0x50>;
> + };
>
> -> no at24 binding to the device
>
> 3) I revert your patch
>
> -> at24 binding to the device
>
I've tested this and you are right, it fails...
The problem is that the patch also changes how the driver obtains the
EEPROM parameters (the magic value in the entry's data field).
So even when module autoload and device / driver matching works, the
driver probe function fails because if (client->dev.of_node) the
driver attempts to get the entry data using
of_device_get_match_data(), which is obviously wrong since the
compatible string in the dev node isn't present in the OF table.
The id->driver_data from the I2C table should be used instead since
that's the table that matches in this case.
One option is to fallback to id->driver_data if
of_device_get_match_data() fails, but that's just an (ugly)
workaround. So I agree with you that the best option is to wait for
the DTS patches to land first.
It worked for me on my previous tests because the tested drivers
didn't use a table entry data, I'm so sorry for missing this :(
Best regards,
Javier
^ permalink raw reply [flat|nested] 115+ messages in thread* Re: [RESEND PATCH v5 00/16] eeprom: at24: Add OF device ID table
2017-08-30 19:57 ` Javier Martinez Canillas
(?)
@ 2017-08-30 20:15 ` Geert Uytterhoeven
-1 siblings, 0 replies; 115+ messages in thread
From: Geert Uytterhoeven @ 2017-08-30 20:15 UTC (permalink / raw)
To: Javier Martinez Canillas
Cc: Wolfram Sang, Linux Kernel, Rob Herring, Florian Larysch,
David Lechner, Rob Herring, Andy Shevchenko, Catalin Marinas,
Sören Brinkmann, Simon Horman, Michal Simek, Dinh Nguyen,
Russell King, Will Deacon, devicetree, Sekhar Nori, Scott Wood,
Benjamin Herrenschmidt, Joachim Eastwood, Mark Rutland,
linux-arm-kernel, Masahiro Yamada, Michael Ellerman,
Santosh Shilimkar, Linux-Renesas, Paul Mackerras, Magnus Damm,
linuxppc-dev, Uwe Kleine-König, Linux I2C
Hi Javier,
On Wed, Aug 30, 2017 at 9:57 PM, Javier Martinez Canillas
<javier@dowhile0.org> wrote:
>> I think we should talk about the same case: Let me repeat what I did:
>>
>> 1) I added your patch "eeprom: at24: Add OF device ID table"
>> 2) I added an EEPROM node to an I2C
>>
>> + eeprom@50 {
>> + compatible = "renesas,24c01";
>> + reg = <0x50>;
>> + };
>>
>> -> no at24 binding to the device
>>
>> 3) I revert your patch
>>
>> -> at24 binding to the device
>>
>
> I've tested this and you are right, it fails...
>
> The problem is that the patch also changes how the driver obtains the
> EEPROM parameters (the magic value in the entry's data field).
>
> So even when module autoload and device / driver matching works, the
> driver probe function fails because if (client->dev.of_node) the
> driver attempts to get the entry data using
> of_device_get_match_data(), which is obviously wrong since the
> compatible string in the dev node isn't present in the OF table.
>
> The id->driver_data from the I2C table should be used instead since
> that's the table that matches in this case.
>
> One option is to fallback to id->driver_data if
> of_device_get_match_data() fails, but that's just an (ugly)
> workaround. So I agree with you that the best option is to wait for
> the DTS patches to land first.
Which means new kernels won't work with old DTBs. Oops...
I'm afraid that needs to be fixed. People care about DTB backward
compatibility on many platforms.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply [flat|nested] 115+ messages in thread* [RESEND PATCH v5 00/16] eeprom: at24: Add OF device ID table
@ 2017-08-30 20:15 ` Geert Uytterhoeven
0 siblings, 0 replies; 115+ messages in thread
From: Geert Uytterhoeven @ 2017-08-30 20:15 UTC (permalink / raw)
To: linux-arm-kernel
Hi Javier,
On Wed, Aug 30, 2017 at 9:57 PM, Javier Martinez Canillas
<javier@dowhile0.org> wrote:
>> I think we should talk about the same case: Let me repeat what I did:
>>
>> 1) I added your patch "eeprom: at24: Add OF device ID table"
>> 2) I added an EEPROM node to an I2C
>>
>> + eeprom at 50 {
>> + compatible = "renesas,24c01";
>> + reg = <0x50>;
>> + };
>>
>> -> no at24 binding to the device
>>
>> 3) I revert your patch
>>
>> -> at24 binding to the device
>>
>
> I've tested this and you are right, it fails...
>
> The problem is that the patch also changes how the driver obtains the
> EEPROM parameters (the magic value in the entry's data field).
>
> So even when module autoload and device / driver matching works, the
> driver probe function fails because if (client->dev.of_node) the
> driver attempts to get the entry data using
> of_device_get_match_data(), which is obviously wrong since the
> compatible string in the dev node isn't present in the OF table.
>
> The id->driver_data from the I2C table should be used instead since
> that's the table that matches in this case.
>
> One option is to fallback to id->driver_data if
> of_device_get_match_data() fails, but that's just an (ugly)
> workaround. So I agree with you that the best option is to wait for
> the DTS patches to land first.
Which means new kernels won't work with old DTBs. Oops...
I'm afraid that needs to be fixed. People care about DTB backward
compatibility on many platforms.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert at linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply [flat|nested] 115+ messages in thread* Re: [RESEND PATCH v5 00/16] eeprom: at24: Add OF device ID table
@ 2017-08-30 20:15 ` Geert Uytterhoeven
0 siblings, 0 replies; 115+ messages in thread
From: Geert Uytterhoeven @ 2017-08-30 20:15 UTC (permalink / raw)
To: Javier Martinez Canillas
Cc: Wolfram Sang, Linux Kernel, Rob Herring, Florian Larysch,
David Lechner, Rob Herring, Andy Shevchenko, Catalin Marinas,
Sören Brinkmann, Simon Horman, Michal Simek, Dinh Nguyen,
Russell King, Will Deacon, devicetree, Sekhar Nori, Scott Wood,
Benjamin Herrenschmidt, Joachim Eastwood
Hi Javier,
On Wed, Aug 30, 2017 at 9:57 PM, Javier Martinez Canillas
<javier@dowhile0.org> wrote:
>> I think we should talk about the same case: Let me repeat what I did:
>>
>> 1) I added your patch "eeprom: at24: Add OF device ID table"
>> 2) I added an EEPROM node to an I2C
>>
>> + eeprom@50 {
>> + compatible = "renesas,24c01";
>> + reg = <0x50>;
>> + };
>>
>> -> no at24 binding to the device
>>
>> 3) I revert your patch
>>
>> -> at24 binding to the device
>>
>
> I've tested this and you are right, it fails...
>
> The problem is that the patch also changes how the driver obtains the
> EEPROM parameters (the magic value in the entry's data field).
>
> So even when module autoload and device / driver matching works, the
> driver probe function fails because if (client->dev.of_node) the
> driver attempts to get the entry data using
> of_device_get_match_data(), which is obviously wrong since the
> compatible string in the dev node isn't present in the OF table.
>
> The id->driver_data from the I2C table should be used instead since
> that's the table that matches in this case.
>
> One option is to fallback to id->driver_data if
> of_device_get_match_data() fails, but that's just an (ugly)
> workaround. So I agree with you that the best option is to wait for
> the DTS patches to land first.
Which means new kernels won't work with old DTBs. Oops...
I'm afraid that needs to be fixed. People care about DTB backward
compatibility on many platforms.
Gr{oetje,eeting}s,
Geert
^ permalink raw reply [flat|nested] 115+ messages in thread* Re: [RESEND PATCH v5 00/16] eeprom: at24: Add OF device ID table
2017-08-30 20:15 ` Geert Uytterhoeven
(?)
@ 2017-08-30 20:59 ` Javier Martinez Canillas
-1 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-08-30 20:59 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Wolfram Sang, Linux Kernel, Rob Herring, Florian Larysch,
David Lechner, Rob Herring, Andy Shevchenko, Catalin Marinas,
Sören Brinkmann, Simon Horman, Michal Simek, Dinh Nguyen,
Russell King, Will Deacon, devicetree, Sekhar Nori, Scott Wood,
Benjamin Herrenschmidt, Joachim Eastwood, Mark Rutland,
linux-arm-kernel, Masahiro Yamada, Michael Ellerman,
Santosh Shilimkar, Linux-Renesas, Paul Mackerras, Magnus Damm,
linuxppc-dev, Uwe Kleine-König, Linux I2C
Hello Geert,
On Wed, Aug 30, 2017 at 10:15 PM, Geert Uytterhoeven
<geert@linux-m68k.org> wrote:
> Hi Javier,
>
> On Wed, Aug 30, 2017 at 9:57 PM, Javier Martinez Canillas
> <javier@dowhile0.org> wrote:
>>> I think we should talk about the same case: Let me repeat what I did:
>>>
>>> 1) I added your patch "eeprom: at24: Add OF device ID table"
>>> 2) I added an EEPROM node to an I2C
>>>
>>> + eeprom@50 {
>>> + compatible = "renesas,24c01";
>>> + reg = <0x50>;
>>> + };
>>>
>>> -> no at24 binding to the device
>>>
>>> 3) I revert your patch
>>>
>>> -> at24 binding to the device
>>>
>>
>> I've tested this and you are right, it fails...
>>
>> The problem is that the patch also changes how the driver obtains the
>> EEPROM parameters (the magic value in the entry's data field).
>>
>> So even when module autoload and device / driver matching works, the
>> driver probe function fails because if (client->dev.of_node) the
>> driver attempts to get the entry data using
>> of_device_get_match_data(), which is obviously wrong since the
>> compatible string in the dev node isn't present in the OF table.
>>
>> The id->driver_data from the I2C table should be used instead since
>> that's the table that matches in this case.
>>
>> One option is to fallback to id->driver_data if
>> of_device_get_match_data() fails, but that's just an (ugly)
>> workaround. So I agree with you that the best option is to wait for
>> the DTS patches to land first.
>
> Which means new kernels won't work with old DTBs. Oops...
> I'm afraid that needs to be fixed. People care about DTB backward
> compatibility on many platforms.
>
Right, I've yet to find one of those mythical platforms that ship old
DTBs with new kernels, but I agree with you since people seem to care
about backward compatibility (at least on theory).
So I see two options then:
1) Use the workaround I mentioned and lookup the I2C device ID table
entry data if of_device_get_match_data() fails
2) Only call of_device_get_match_data() if (dev->of_node &&
of_match_device(dev->driver->of_match_table, dev))
Not sure what's the preferred idiom for these cases.
To good thing about keeping backward compatibility is that Wolfram
would be able to pick the driver patch even before the DTS patches
land.
Best regards,
Javier
^ permalink raw reply [flat|nested] 115+ messages in thread* [RESEND PATCH v5 00/16] eeprom: at24: Add OF device ID table
@ 2017-08-30 20:59 ` Javier Martinez Canillas
0 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-08-30 20:59 UTC (permalink / raw)
To: linux-arm-kernel
Hello Geert,
On Wed, Aug 30, 2017 at 10:15 PM, Geert Uytterhoeven
<geert@linux-m68k.org> wrote:
> Hi Javier,
>
> On Wed, Aug 30, 2017 at 9:57 PM, Javier Martinez Canillas
> <javier@dowhile0.org> wrote:
>>> I think we should talk about the same case: Let me repeat what I did:
>>>
>>> 1) I added your patch "eeprom: at24: Add OF device ID table"
>>> 2) I added an EEPROM node to an I2C
>>>
>>> + eeprom at 50 {
>>> + compatible = "renesas,24c01";
>>> + reg = <0x50>;
>>> + };
>>>
>>> -> no at24 binding to the device
>>>
>>> 3) I revert your patch
>>>
>>> -> at24 binding to the device
>>>
>>
>> I've tested this and you are right, it fails...
>>
>> The problem is that the patch also changes how the driver obtains the
>> EEPROM parameters (the magic value in the entry's data field).
>>
>> So even when module autoload and device / driver matching works, the
>> driver probe function fails because if (client->dev.of_node) the
>> driver attempts to get the entry data using
>> of_device_get_match_data(), which is obviously wrong since the
>> compatible string in the dev node isn't present in the OF table.
>>
>> The id->driver_data from the I2C table should be used instead since
>> that's the table that matches in this case.
>>
>> One option is to fallback to id->driver_data if
>> of_device_get_match_data() fails, but that's just an (ugly)
>> workaround. So I agree with you that the best option is to wait for
>> the DTS patches to land first.
>
> Which means new kernels won't work with old DTBs. Oops...
> I'm afraid that needs to be fixed. People care about DTB backward
> compatibility on many platforms.
>
Right, I've yet to find one of those mythical platforms that ship old
DTBs with new kernels, but I agree with you since people seem to care
about backward compatibility (at least on theory).
So I see two options then:
1) Use the workaround I mentioned and lookup the I2C device ID table
entry data if of_device_get_match_data() fails
2) Only call of_device_get_match_data() if (dev->of_node &&
of_match_device(dev->driver->of_match_table, dev))
Not sure what's the preferred idiom for these cases.
To good thing about keeping backward compatibility is that Wolfram
would be able to pick the driver patch even before the DTS patches
land.
Best regards,
Javier
^ permalink raw reply [flat|nested] 115+ messages in thread* Re: [RESEND PATCH v5 00/16] eeprom: at24: Add OF device ID table
@ 2017-08-30 20:59 ` Javier Martinez Canillas
0 siblings, 0 replies; 115+ messages in thread
From: Javier Martinez Canillas @ 2017-08-30 20:59 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Wolfram Sang, Linux Kernel, Rob Herring, Florian Larysch,
David Lechner, Rob Herring, Andy Shevchenko, Catalin Marinas,
Sören Brinkmann, Simon Horman, Michal Simek, Dinh Nguyen,
Russell King, Will Deacon, devicetree, Sekhar Nori, Scott Wood,
Benjamin Herrenschmidt, Joachim Eastwood
Hello Geert,
On Wed, Aug 30, 2017 at 10:15 PM, Geert Uytterhoeven
<geert@linux-m68k.org> wrote:
> Hi Javier,
>
> On Wed, Aug 30, 2017 at 9:57 PM, Javier Martinez Canillas
> <javier@dowhile0.org> wrote:
>>> I think we should talk about the same case: Let me repeat what I did:
>>>
>>> 1) I added your patch "eeprom: at24: Add OF device ID table"
>>> 2) I added an EEPROM node to an I2C
>>>
>>> + eeprom@50 {
>>> + compatible = "renesas,24c01";
>>> + reg = <0x50>;
>>> + };
>>>
>>> -> no at24 binding to the device
>>>
>>> 3) I revert your patch
>>>
>>> -> at24 binding to the device
>>>
>>
>> I've tested this and you are right, it fails...
>>
>> The problem is that the patch also changes how the driver obtains the
>> EEPROM parameters (the magic value in the entry's data field).
>>
>> So even when module autoload and device / driver matching works, the
>> driver probe function fails because if (client->dev.of_node) the
>> driver attempts to get the entry data using
>> of_device_get_match_data(), which is obviously wrong since the
>> compatible string in the dev node isn't present in the OF table.
>>
>> The id->driver_data from the I2C table should be used instead since
>> that's the table that matches in this case.
>>
>> One option is to fallback to id->driver_data if
>> of_device_get_match_data() fails, but that's just an (ugly)
>> workaround. So I agree with you that the best option is to wait for
>> the DTS patches to land first.
>
> Which means new kernels won't work with old DTBs. Oops...
> I'm afraid that needs to be fixed. People care about DTB backward
> compatibility on many platforms.
>
Right, I've yet to find one of those mythical platforms that ship old
DTBs with new kernels, but I agree with you since people seem to care
about backward compatibility (at least on theory).
So I see two options then:
1) Use the workaround I mentioned and lookup the I2C device ID table
entry data if of_device_get_match_data() fails
2) Only call of_device_get_match_data() if (dev->of_node &&
of_match_device(dev->driver->of_match_table, dev))
Not sure what's the preferred idiom for these cases.
To good thing about keeping backward compatibility is that Wolfram
would be able to pick the driver patch even before the DTS patches
land.
Best regards,
Javier
^ permalink raw reply [flat|nested] 115+ messages in thread