On Wednesday 26 October 2016 09:38 PM, David Lechner wrote:

On 10/25/2016 10:06 PM, David Lechner wrote:

quoted

Add a syscon node for the SoC CFGCHIPn registers. This is needed for
the new usb phy driver.

Signed-off-by: David Lechner <david@lechnology.com>
---
 arch/arm/boot/dts/da850.dtsi | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/arch/arm/boot/dts/da850.dtsi b/arch/arm/boot/dts/da850.dtsi
index f79e1b9..6bbf20d 100644
--- a/arch/arm/boot/dts/da850.dtsi
+++ b/arch/arm/boot/dts/da850.dtsi

@@ -188,6 +188,10 @@
             };

         };
+        cfgchip: cfgchip at 1417c {

I wonder if there is a more generic name instead of cfgchip at . Is there a
preferred generic name for syscon nodes?

I did not find anything in ePAPR, but chip-controller might be more
appropriate.

quoted

+            compatible = "ti,da830-cfgchip", "syscon";

Looks like we need "simple-mfd" too in the compatible list?

I think we can also fold patch 5/5 into this patch and add the cfgchip
along with USB phy child node included.

If you respin the patch, I can drop 4/5 and 5/5 that I have queued and
included the updated patch instead.

Thanks,
Sekhar