Thread (4 messages) 4 messages, 2 authors, 2016-06-25

Re: Re: [PATCH 01/14] ARM: dts: sun8i: Add SID node

From: Chen-Yu Tsai <hidden>
Date: 2016-06-25 01:09:57
Also in: linux-arm-kernel, lkml

Hi,

On Sat, Jun 25, 2016 at 3:58 AM, Ondřej Jirman [off-list ref] wrote:
Hello,

thank you for the review.

On 24.6.2016 04:41, Chen-Yu Tsai wrote:
quoted
On Fri, Jun 24, 2016 at 3:20 AM,  [off-list ref] wrote:
quoted
From: Josef Gajdusek <redacted>

Add a node describing the Security ID memory to the Allwinner H3 .dtsi file.

Signed-off-by: Josef Gajdusek <redacted>
---
 arch/arm/boot/dts/sun8i-h3.dtsi | 7 +++++++
 1 file changed, 7 insertions(+)
diff --git a/arch/arm/boot/dts/sun8i-h3.dtsi b/arch/arm/boot/dts/sun8i-h3.dtsi
index 4a4926b..172576d 100644
--- a/arch/arm/boot/dts/sun8i-h3.dtsi
+++ b/arch/arm/boot/dts/sun8i-h3.dtsi
@@ -389,6 +389,13 @@
                        #size-cells = <0>;
                };

+               sid: eeprom@01c14000 {
+                       compatible = "allwinner,sun4i-a10-sid";
This has been discussed before. The hardware is not compatible.
The write control registers are at different offsets.
I'm not sure what you mean by write control registers. Code in
sunxi_sid.c implements only read access to the nvram. Can you pelase
elaborate?
See http://lists.infradead.org/pipermail/linux-arm-kernel/2015-November/388022.html

Also, different compatibles are used for different hardware, regardless
of how close the drivers may be. The driver might only be compatible when
implementing a subset of the possible features. If one were to fully
implement it, they would become incompatible.

To put it another way, the compatible string designates the hardware,
and the driver implements support for that compatible string.

ChenYu
  Ondrej
quoted
ChenYu
quoted
+                       reg = <0x01c14000 0x400>;
+                       #address-cells = <1>;
+                       #size-cells = <1>;
+               };
+
                usbphy: phy@01c19400 {
                        compatible = "allwinner,sun8i-h3-usb-phy";
                        reg = <0x01c19400 0x2c>,
--
2.9.0
--
You received this message because you are subscribed to the Google Groups "linux-sunxi" group.
To unsubscribe from this group and stop receiving emails from it, send an email to linux-sunxi+unsubscribe-/JYPxA39Uh5TLH3MbocFF+G/Ez6ZCGd0@public.gmane.org
For more options, visit https://groups.google.com/d/optout.
-- 
You received this message because you are subscribed to the Google Groups "linux-sunxi" group.
To unsubscribe from this group and stop receiving emails from it, send an email to linux-sunxi+unsubscribe-/JYPxA39Uh5TLH3MbocFF+G/Ez6ZCGd0@public.gmane.org
For more options, visit https://groups.google.com/d/optout.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help