From: Wolfram Sang <hidden> Date: 2014-02-13 08:26:30
Why is the devicetree list not on CC? (Added now)
On Thu, Feb 06, 2014 at 10:51:25AM +0100, Maxime Ripard wrote:
quoted hunk
Switch the device tree to the new compatibles introduced in the i2c drivers
to have a common pattern accross all Allwinner SoCs.
Signed-off-by: Maxime Ripard <maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
---
arch/arm/boot/dts/sun4i-a10.dtsi | 6 +++---
arch/arm/boot/dts/sun5i-a10s.dtsi | 6 +++---
arch/arm/boot/dts/sun5i-a13.dtsi | 6 +++---
arch/arm/boot/dts/sun7i-a20.dtsi | 10 +++++-----
4 files changed, 14 insertions(+), 14 deletions(-)
Can't we have:
compatible = "allwinner,sun4i-a10-i2c", "allwinner,sun4i-i2c";
? And keep the old "allwinner,sun4i-i2c" and extend it with a SoC
specific a10 compatible entry when a distinction is needed?
For non-a10, That should be at least
compatible = "allwinner,sun4i-a13-i2c", "allwinner,sun4i-a10-i2c";
or
compatible = "allwinner,sun4i-a13-i2c", "allwinner,sun4i-i2c";
depending on the outcome above.
Or is my knowledge outdated already?
From: Maxime Ripard <hidden> Date: 2014-02-13 22:52:48
On Thu, Feb 13, 2014 at 09:26:30AM +0100, Wolfram Sang wrote:
Why is the devicetree list not on CC? (Added now)
On Thu, Feb 06, 2014 at 10:51:25AM +0100, Maxime Ripard wrote:
quoted
Switch the device tree to the new compatibles introduced in the i2c drivers
to have a common pattern accross all Allwinner SoCs.
Signed-off-by: Maxime Ripard <maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
---
arch/arm/boot/dts/sun4i-a10.dtsi | 6 +++---
arch/arm/boot/dts/sun5i-a10s.dtsi | 6 +++---
arch/arm/boot/dts/sun5i-a13.dtsi | 6 +++---
arch/arm/boot/dts/sun7i-a20.dtsi | 10 +++++-----
4 files changed, 14 insertions(+), 14 deletions(-)
Can't we have:
compatible = "allwinner,sun4i-a10-i2c", "allwinner,sun4i-i2c";
? And keep the old "allwinner,sun4i-i2c" and extend it with a SoC
specific a10 compatible entry when a distinction is needed?
Actually, the two are exactly equivalent. The point is that the
compatible naming scheme doesn't follow what we are using (which is
<family>-<soc>-i2c), so we wan't to get rid of the old naming scheme
all together.
For non-a10, That should be at least
compatible = "allwinner,sun4i-a13-i2c", "allwinner,sun4i-a10-i2c";
or
compatible = "allwinner,sun4i-a13-i2c", "allwinner,sun4i-i2c";
depending on the outcome above.
Or is my knowledge outdated already?
Since they are strictly compatible, we don't need to introduce any
different compatible string here.
--
Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
From: Wolfram Sang <hidden> Date: 2014-02-14 07:44:07
quoted
For non-a10, That should be at least
compatible = "allwinner,sun4i-a13-i2c", "allwinner,sun4i-a10-i2c";
or
compatible = "allwinner,sun4i-a13-i2c", "allwinner,sun4i-i2c";
depending on the outcome above.
Or is my knowledge outdated already?
Since they are strictly compatible, we don't need to introduce any
different compatible string here.
You never know all errata in advance. From what I know, one should
always use the specfic naming first, and then the generic fallback. So,
in case a distinction is needed later (think errata), then one doesn't
need to change the devicetrees.
From: Maxime Ripard <hidden> Date: 2014-02-14 20:36:29
On Fri, Feb 14, 2014 at 08:44:07AM +0100, Wolfram Sang wrote:
quoted
quoted
For non-a10, That should be at least
compatible = "allwinner,sun4i-a13-i2c", "allwinner,sun4i-a10-i2c";
or
compatible = "allwinner,sun4i-a13-i2c", "allwinner,sun4i-i2c";
depending on the outcome above.
Or is my knowledge outdated already?
Since they are strictly compatible, we don't need to introduce any
different compatible string here.
You never know all errata in advance. From what I know, one should
always use the specfic naming first, and then the generic fallback. So,
in case a distinction is needed later (think errata), then one doesn't
need to change the devicetrees.
And adding a A13-specific compatible wouldn't change anything, because
it does work on at least one revision of them, so if you'd have to
deal with an errata, you'd have to introduce a new compatible for this
revision only anyway.
--
Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com