From mboxrd@z Thu Jan 1 00:00:00 1970 From: nsekhar@ti.com (Sekhar Nori) Date: Fri, 28 Oct 2016 13:51:03 +0530 Subject: [PATCH v6 4/5] ARM: DTS: da850: Add cfgchip syscon node In-Reply-To: References: <1477451211-31979-1-git-send-email-david@lechnology.com> <1477451211-31979-5-git-send-email-david@lechnology.com> Message-ID: <09eb61b9-d66e-838e-2411-d787fda65096@ti.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Wednesday 26 October 2016 09:38 PM, David Lechner wrote: > On 10/25/2016 10:06 PM, David Lechner wrote: >> Add a syscon node for the SoC CFGCHIPn registers. This is needed for >> the new usb phy driver. >> >> Signed-off-by: David Lechner >> --- >> 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. > >> + 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