From mboxrd@z Thu Jan 1 00:00:00 1970 From: Diana Craciun Subject: Re: [PATCHv2 4/6] dt-bindings: arm: add Freescale LS1021A SoC device tree binding Date: Wed, 20 Aug 2014 15:34:57 +0300 Message-ID: <53F495F1.60800@freescale.com> References: <1407145148-29217-1-git-send-email-jingchang.lu@freescale.com> <1407145148-29217-5-git-send-email-jingchang.lu@freescale.com> Mime-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1407145148-29217-5-git-send-email-jingchang.lu-KZfg59tc24xl57MIdRCFDg@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Jingchang Lu Cc: shawn.guo-KZfg59tc24xl57MIdRCFDg@public.gmane.org, mark.rutland-5wv7dgnIgG8@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org List-Id: devicetree@vger.kernel.org On 08/04/2014 12:39 PM, Jingchang Lu wrote: > Signed-off-by: Jingchang Lu > --- > Documentation/devicetree/bindings/arm/fsl.txt | 37 +++++++++++++++++++++++++++ > 1 file changed, 37 insertions(+) > > diff --git a/Documentation/devicetree/bindings/arm/fsl.txt b/Documentation/devicetree/bindings/arm/fsl.txt > index e935d7d..c962124 100644 > --- a/Documentation/devicetree/bindings/arm/fsl.txt > +++ b/Documentation/devicetree/bindings/arm/fsl.txt > @@ -74,3 +74,40 @@ Required root node properties: > i.MX6q generic board > Required root node properties: > - compatible = "fsl,imx6q"; > + > + > +Freescale LS1021A Platform Device Tree Bindings > +------------------------------------------------ > + > +Required root node compatible properties: > + - compatible = "fsl,ls1021a"; > + > +SoC-specific device tree bindings for system configuration > +------------------------------------------- > + > +Required device node compatible properties: > + > + - compatible = "fsl,ls1021a-scfg": > + scfg is the supplemental configuration unit, provides SoC specific > + configuration and status registers for the chip. There is no dedicate > + driver for it, but for device whose configuration and status register > + locates in this space should operate on it. Such as getting PEX port > + status. > + > + - compatible = "fsl,ls1021a-dcfg": > + dcfg is the device configuration unit that provides general purpose > + configuration and status for the device, there is no dedicate driver > + for it, but for device whose configuration and status register locates > + in this space should operate on it. Such as setting the secondary core > + start address and release the secondary core from holdoff and startup. Is it that important to be mentioned in the binding that there is no driver for it? It seems to be just an implementation detail, the device tree describes the hardware not any particular implementation. > + > +Example: > + scfg: scfg@1570000 { > + compatible = "fsl,ls1021a-scfg"; > + reg = <0x0 0x1570000 0x0 0x10000>; The reg is not part of the description above. I think that each of these nodes should be described separate, maybe something like this: Freescale SCFG scfg is the supplemental configuration unit, provides SoC specific configuration and status registers for the chip. There is no dedicate driver for it, but for device whose configuration and status register locates in this space should operate on it. Such as getting PEX port status. Required properties: - compatible: Should be "fsl,ls1021a-scfg" - reg: should contain..... > + }; > + > + dcfg: dcfg@1ee0000 { > + compatible = "fsl,ls1021a-dcfg"; > + reg = <0x0 0x1ee0000 0x0 0x10000>; > + }; Diana -- 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 From mboxrd@z Thu Jan 1 00:00:00 1970 From: diana.craciun@freescale.com (Diana Craciun) Date: Wed, 20 Aug 2014 15:34:57 +0300 Subject: [PATCHv2 4/6] dt-bindings: arm: add Freescale LS1021A SoC device tree binding In-Reply-To: <1407145148-29217-5-git-send-email-jingchang.lu@freescale.com> References: <1407145148-29217-1-git-send-email-jingchang.lu@freescale.com> <1407145148-29217-5-git-send-email-jingchang.lu@freescale.com> Message-ID: <53F495F1.60800@freescale.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 08/04/2014 12:39 PM, Jingchang Lu wrote: > Signed-off-by: Jingchang Lu > --- > Documentation/devicetree/bindings/arm/fsl.txt | 37 +++++++++++++++++++++++++++ > 1 file changed, 37 insertions(+) > > diff --git a/Documentation/devicetree/bindings/arm/fsl.txt b/Documentation/devicetree/bindings/arm/fsl.txt > index e935d7d..c962124 100644 > --- a/Documentation/devicetree/bindings/arm/fsl.txt > +++ b/Documentation/devicetree/bindings/arm/fsl.txt > @@ -74,3 +74,40 @@ Required root node properties: > i.MX6q generic board > Required root node properties: > - compatible = "fsl,imx6q"; > + > + > +Freescale LS1021A Platform Device Tree Bindings > +------------------------------------------------ > + > +Required root node compatible properties: > + - compatible = "fsl,ls1021a"; > + > +SoC-specific device tree bindings for system configuration > +------------------------------------------- > + > +Required device node compatible properties: > + > + - compatible = "fsl,ls1021a-scfg": > + scfg is the supplemental configuration unit, provides SoC specific > + configuration and status registers for the chip. There is no dedicate > + driver for it, but for device whose configuration and status register > + locates in this space should operate on it. Such as getting PEX port > + status. > + > + - compatible = "fsl,ls1021a-dcfg": > + dcfg is the device configuration unit that provides general purpose > + configuration and status for the device, there is no dedicate driver > + for it, but for device whose configuration and status register locates > + in this space should operate on it. Such as setting the secondary core > + start address and release the secondary core from holdoff and startup. Is it that important to be mentioned in the binding that there is no driver for it? It seems to be just an implementation detail, the device tree describes the hardware not any particular implementation. > + > +Example: > + scfg: scfg at 1570000 { > + compatible = "fsl,ls1021a-scfg"; > + reg = <0x0 0x1570000 0x0 0x10000>; The reg is not part of the description above. I think that each of these nodes should be described separate, maybe something like this: Freescale SCFG scfg is the supplemental configuration unit, provides SoC specific configuration and status registers for the chip. There is no dedicate driver for it, but for device whose configuration and status register locates in this space should operate on it. Such as getting PEX port status. Required properties: - compatible: Should be "fsl,ls1021a-scfg" - reg: should contain..... > + }; > + > + dcfg: dcfg at 1ee0000 { > + compatible = "fsl,ls1021a-dcfg"; > + reg = <0x0 0x1ee0000 0x0 0x10000>; > + }; Diana