From: Peng Fan <[email protected]>
Convert usbmisc-imx to DT schema format.
Signed-off-by: Peng Fan <[email protected]>
---
.../devicetree/bindings/usb/fsl,usbmisc.yaml | 52 +++++++++++++++++++
.../devicetree/bindings/usb/usbmisc-imx.txt | 18 -------
2 files changed, 52 insertions(+), 18 deletions(-)
create mode 100644 Documentation/devicetree/bindings/usb/fsl,usbmisc.yaml
delete mode 100644 Documentation/devicetree/bindings/usb/usbmisc-imx.txt
diff --git a/Documentation/devicetree/bindings/usb/fsl,usbmisc.yaml b/Documentation/devicetree/bindings/usb/fsl,usbmisc.yaml
new file mode 100644
index 000000000000..c83ffb6729b5
--- /dev/null
+++ b/Documentation/devicetree/bindings/usb/fsl,usbmisc.yaml
@@ -0,0 +1,52 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/usb/fsl,usbmisc.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Freescale i.MX non-core registers
+
+maintainers:
+ - Xu Yang <[email protected]>
+
+properties:
+ compatible:
+ oneOf:
+ - enum:
+ - fsl,imx6q-usbmisc
+ - fsl,imx7ulp-usbmisc
+ - fsl,vf610-usbmisc
+ - items:
+ - enum:
+ - fsl,imx6ul-usbmisc
+ - fsl,imx6sx-usbmisc
+ - fsl,imx7d-usbmisc
+ - const: fsl,imx6q-usbmisc
+ - items:
+ - enum:
+ - fsl,imx7ulp-usbmisc
+ - const: fsl,imx7d-usbmisc
+
+ reg:
+ maxItems: 1
+
+ '#index-cells':
+ const: 1
+ description: Cells used to describe usb controller index.
+
+required:
+ - compatible
+ - reg
+ - '#index-cells'
+
+additionalProperties: false
+
+examples:
+ - |
+ usbmisc@2184800 {
+ #index-cells = <1>;
+ compatible = "fsl,imx6q-usbmisc";
+ reg = <0x02184800 0x200>;
+ };
+
+...
diff --git a/Documentation/devicetree/bindings/usb/usbmisc-imx.txt b/Documentation/devicetree/bindings/usb/usbmisc-imx.txt
deleted file mode 100644
index b796836d2ce7..000000000000
--- a/Documentation/devicetree/bindings/usb/usbmisc-imx.txt
+++ /dev/null
@@ -1,18 +0,0 @@
-* Freescale i.MX non-core registers
-
-Required properties:
-- #index-cells: Cells used to describe usb controller index. Should be <1>
-- compatible: Should be one of below:
- "fsl,imx6q-usbmisc" for imx6q
- "fsl,vf610-usbmisc" for Vybrid vf610
- "fsl,imx6sx-usbmisc" for imx6sx
- "fsl,imx7d-usbmisc" for imx7d
- "fsl,imx7ulp-usbmisc" for imx7ulp
-- reg: Should contain registers location and length
-
-Examples:
-usbmisc@2184800 {
- #index-cells = <1>;
- compatible = "fsl,imx6q-usbmisc";
- reg = <0x02184800 0x200>;
-};
--
2.37.1
On Fri, Oct 14, 2022 at 05:51:43PM +0800, Peng Fan (OSS) wrote:
> From: Peng Fan <[email protected]>
>
> Convert usbmisc-imx to DT schema format.
>
> Signed-off-by: Peng Fan <[email protected]>
> ---
> .../devicetree/bindings/usb/fsl,usbmisc.yaml | 52 +++++++++++++++++++
> .../devicetree/bindings/usb/usbmisc-imx.txt | 18 -------
> 2 files changed, 52 insertions(+), 18 deletions(-)
> create mode 100644 Documentation/devicetree/bindings/usb/fsl,usbmisc.yaml
> delete mode 100644 Documentation/devicetree/bindings/usb/usbmisc-imx.txt
>
> diff --git a/Documentation/devicetree/bindings/usb/fsl,usbmisc.yaml b/Documentation/devicetree/bindings/usb/fsl,usbmisc.yaml
> new file mode 100644
> index 000000000000..c83ffb6729b5
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/usb/fsl,usbmisc.yaml
> @@ -0,0 +1,52 @@
> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/usb/fsl,usbmisc.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Freescale i.MX non-core registers
> +
> +maintainers:
> + - Xu Yang <[email protected]>
Signing someone else up to be a maintainer requires them to sign off on
the patch to agree with this. Why not list yourself instead?
thanks,
greg k-h
On 14/10/2022 05:51, Peng Fan (OSS) wrote:
> From: Peng Fan <[email protected]>
>
> Convert usbmisc-imx to DT schema format.
>
> Signed-off-by: Peng Fan <[email protected]>
> ---
> .../devicetree/bindings/usb/fsl,usbmisc.yaml | 52 +++++++++++++++++++
> .../devicetree/bindings/usb/usbmisc-imx.txt | 18 -------
> 2 files changed, 52 insertions(+), 18 deletions(-)
> create mode 100644 Documentation/devicetree/bindings/usb/fsl,usbmisc.yaml
> delete mode 100644 Documentation/devicetree/bindings/usb/usbmisc-imx.txt
>
> diff --git a/Documentation/devicetree/bindings/usb/fsl,usbmisc.yaml b/Documentation/devicetree/bindings/usb/fsl,usbmisc.yaml
> new file mode 100644
> index 000000000000..c83ffb6729b5
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/usb/fsl,usbmisc.yaml
> @@ -0,0 +1,52 @@
> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/usb/fsl,usbmisc.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Freescale i.MX non-core registers
You sent v2 before we finished this topic. Is it correct? Even if TXT
had such title, let's don't keep the wrong name.
If it is non-core registers, why it is in USB? Why it is an usb-misc device?
> +
> +maintainers:
> + - Xu Yang <[email protected]>
> +
Best regards,
Krzysztof
Hi Greg,
> Subject: Re: [PATCH V2 1/6] dt-bindings: usb: usbmisc-imx: convert to DT
> schema
>
> On Fri, Oct 14, 2022 at 05:51:43PM +0800, Peng Fan (OSS) wrote:
> > From: Peng Fan <[email protected]>
> >
> > Convert usbmisc-imx to DT schema format.
> >
> > Signed-off-by: Peng Fan <[email protected]>
> > ---
> > .../devicetree/bindings/usb/fsl,usbmisc.yaml | 52 +++++++++++++++++++
> > .../devicetree/bindings/usb/usbmisc-imx.txt | 18 -------
> > 2 files changed, 52 insertions(+), 18 deletions(-) create mode
> > 100644 Documentation/devicetree/bindings/usb/fsl,usbmisc.yaml
> > delete mode 100644
> > Documentation/devicetree/bindings/usb/usbmisc-imx.txt
> >
> > diff --git a/Documentation/devicetree/bindings/usb/fsl,usbmisc.yaml
> > b/Documentation/devicetree/bindings/usb/fsl,usbmisc.yaml
> > new file mode 100644
> > index 000000000000..c83ffb6729b5
> > --- /dev/null
> > +++ b/Documentation/devicetree/bindings/usb/fsl,usbmisc.yaml
> > @@ -0,0 +1,52 @@
> > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) %YAML 1.2
> > +---
> > +$id:
> >
> +https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fdevi
> >
> +cetree.org%2Fschemas%2Fusb%2Ffsl%2Cusbmisc.yaml%23&data=05
> %7C01%7
> >
> +Cpeng.fan%40nxp.com%7C15ebc6c1040f46f3ecef08daaddafa7a%7C686ea
> 1d3bc2b
> >
> +4c6fa92cd99c5c301635%7C0%7C0%7C638013453273330209%7CUnknown
> %7CTWFpbGZ
> >
> +sb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6
> Mn0%
> >
> +3D%7C3000%7C%7C%7C&sdata=NZ4SFEve9DqdFVremHTDVGgrpISKyj
> yoEJKYE4sk
> > +7tA%3D&reserved=0
> > +$schema:
> >
> +https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fdevi
> > +cetree.org%2Fmeta-
> schemas%2Fcore.yaml%23&data=05%7C01%7Cpeng.fan%
> >
> +40nxp.com%7C15ebc6c1040f46f3ecef08daaddafa7a%7C686ea1d3bc2b4c6f
> a92cd9
> >
> +9c5c301635%7C0%7C0%7C638013453273330209%7CUnknown%7CTWFpb
> GZsb3d8eyJWI
> >
> +joiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7
> C3000%
> >
> +7C%7C%7C&sdata=SYlpPoSmOQxNYaJFAlvwPQRFq0Oaugwc%2F52w6
> tvgcrk%3D&a
> > +mp;reserved=0
> > +
> > +title: Freescale i.MX non-core registers
> > +
> > +maintainers:
> > + - Xu Yang <[email protected]>
>
> Signing someone else up to be a maintainer requires them to sign off on the
> patch to agree with this. Why not list yourself instead?
I am not expert in USB area. I am just convert the file to yaml to ease the upstream
of i.MX device tree.
Is it ok for for Xu Yang to Ack the patch or I must add:
Signed-off-by: Xu Yang <[email protected]> in the patch?
Thanks,
Peng.
>
> thanks,
>
> greg k-h
Hi Krzysztof,
> -----Original Message-----
> From: Krzysztof Kozlowski <[email protected]>
> Sent: Saturday, October 15, 2022 10:53 PM
> To: Peng Fan (OSS) <[email protected]>; [email protected]; [email protected];
> [email protected]; [email protected]; [email protected]; Xu Yang <[email protected]>
> Cc: [email protected]; [email protected]; dl-linux-imx <[email protected]>; [email protected];
> [email protected]; [email protected]; [email protected]; Jun Li <[email protected]>;
> Peng Fan <[email protected]>
> Subject: [EXT] Re: [PATCH V2 1/6] dt-bindings: usb: usbmisc-imx: convert to DT schema
>
> Caution: EXT Email
>
> On 14/10/2022 05:51, Peng Fan (OSS) wrote:
> > From: Peng Fan <[email protected]>
> >
> > Convert usbmisc-imx to DT schema format.
> >
> > Signed-off-by: Peng Fan <[email protected]>
> > ---
> > .../devicetree/bindings/usb/fsl,usbmisc.yaml | 52 +++++++++++++++++++
> > .../devicetree/bindings/usb/usbmisc-imx.txt | 18 -------
> > 2 files changed, 52 insertions(+), 18 deletions(-)
> > create mode 100644 Documentation/devicetree/bindings/usb/fsl,usbmisc.yaml
> > delete mode 100644 Documentation/devicetree/bindings/usb/usbmisc-imx.txt
> >
> > diff --git a/Documentation/devicetree/bindings/usb/fsl,usbmisc.yaml
> b/Documentation/devicetree/bindings/usb/fsl,usbmisc.yaml
> > new file mode 100644
> > index 000000000000..c83ffb6729b5
> > --- /dev/null
> > +++ b/Documentation/devicetree/bindings/usb/fsl,usbmisc.yaml
> > @@ -0,0 +1,52 @@
> > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> > +%YAML 1.2
> > +---
> > +$id:
> https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fdevicetree.org%2Fschemas%2Fusb%2Ffsl%2Cusbmi
> sc.yaml%23&data=05%7C01%7Cxu.yang_2%40nxp.com%7C24e82b830d9f47018ff408daaebcf83d%7C686ea1d3bc2b4c6
> fa92cd99c5c301635%7C0%7C0%7C638014423898086337%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoi
> V2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=gNwMbGsiuMS1zgvbgATYAY70gv2nIhX%2F
> 16IxiSYBf3s%3D&reserved=0
> > +$schema: https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fdevicetree.org%2Fmeta-
> schemas%2Fcore.yaml%23&data=05%7C01%7Cxu.yang_2%40nxp.com%7C24e82b830d9f47018ff408daaebcf83d%7C68
> 6ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C638014423898242035%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjA
> wMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=jJUP5aKNkfkAefhNQL9almV
> Gqzn8cTNyjcRlZMOZwRk%3D&reserved=0
> > +
> > +title: Freescale i.MX non-core registers
>
> You sent v2 before we finished this topic. Is it correct? Even if TXT
> had such title, let's don't keep the wrong name.
>
> If it is non-core registers, why it is in USB? Why it is an usb-misc device?
The chipidea's doc has a more clear explanation about core and non-core
registers as follow:
"There are two kinds of registers in the USB module: USB core registers
and USB non-core registers. USB core registers are used to control USB
core functions, and more independent of USB features. Each USB controller
core has its own core registers. USB non-core registers are additional
to USB core registers, and more dependent on USB features. i.MX series
products vary in non-core registers."
So we have named it like this.
Thanks,
Xu Yang
>
> > +
> > +maintainers:
> > + - Xu Yang <[email protected]>
> > +
>
> Best regards,
> Krzysztof
On 16/10/2022 23:17, Xu Yang wrote:
> Hi Krzysztof,
>
>> -----Original Message-----
>> From: Krzysztof Kozlowski <[email protected]>
>> Sent: Saturday, October 15, 2022 10:53 PM
>> To: Peng Fan (OSS) <[email protected]>; [email protected]; [email protected];
>> [email protected]; [email protected]; [email protected]; Xu Yang <[email protected]>
>> Cc: [email protected]; [email protected]; dl-linux-imx <[email protected]>; [email protected];
>> [email protected]; [email protected]; [email protected]; Jun Li <[email protected]>;
>> Peng Fan <[email protected]>
>> Subject: [EXT] Re: [PATCH V2 1/6] dt-bindings: usb: usbmisc-imx: convert to DT schema
>>
>> Caution: EXT Email
>>
>> On 14/10/2022 05:51, Peng Fan (OSS) wrote:
>>> From: Peng Fan <[email protected]>
>>>
>>> Convert usbmisc-imx to DT schema format.
>>>
>>> Signed-off-by: Peng Fan <[email protected]>
>>> ---
>>> .../devicetree/bindings/usb/fsl,usbmisc.yaml | 52 +++++++++++++++++++
>>> .../devicetree/bindings/usb/usbmisc-imx.txt | 18 -------
>>> 2 files changed, 52 insertions(+), 18 deletions(-)
>>> create mode 100644 Documentation/devicetree/bindings/usb/fsl,usbmisc.yaml
>>> delete mode 100644 Documentation/devicetree/bindings/usb/usbmisc-imx.txt
>>>
>>> diff --git a/Documentation/devicetree/bindings/usb/fsl,usbmisc.yaml
>> b/Documentation/devicetree/bindings/usb/fsl,usbmisc.yaml
>>> new file mode 100644
>>> index 000000000000..c83ffb6729b5
>>> --- /dev/null
>>> +++ b/Documentation/devicetree/bindings/usb/fsl,usbmisc.yaml
>>> @@ -0,0 +1,52 @@
>>> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
>>> +%YAML 1.2
>>> +---
>>> +$id:
>> https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fdevicetree.org%2Fschemas%2Fusb%2Ffsl%2Cusbmi
>> sc.yaml%23&data=05%7C01%7Cxu.yang_2%40nxp.com%7C24e82b830d9f47018ff408daaebcf83d%7C686ea1d3bc2b4c6
>> fa92cd99c5c301635%7C0%7C0%7C638014423898086337%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoi
>> V2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=gNwMbGsiuMS1zgvbgATYAY70gv2nIhX%2F
>> 16IxiSYBf3s%3D&reserved=0
>>> +$schema: https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fdevicetree.org%2Fmeta-
>> schemas%2Fcore.yaml%23&data=05%7C01%7Cxu.yang_2%40nxp.com%7C24e82b830d9f47018ff408daaebcf83d%7C68
>> 6ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C638014423898242035%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjA
>> wMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=jJUP5aKNkfkAefhNQL9almV
>> Gqzn8cTNyjcRlZMOZwRk%3D&reserved=0
>>> +
>>> +title: Freescale i.MX non-core registers
>>
>> You sent v2 before we finished this topic. Is it correct? Even if TXT
>> had such title, let's don't keep the wrong name.
>>
>> If it is non-core registers, why it is in USB? Why it is an usb-misc device?
>
> The chipidea's doc has a more clear explanation about core and non-core
> registers as follow:
>
> "There are two kinds of registers in the USB module: USB core registers
> and USB non-core registers. USB core registers are used to control USB
> core functions, and more independent of USB features. Each USB controller
> core has its own core registers. USB non-core registers are additional
> to USB core registers, and more dependent on USB features. i.MX series
> products vary in non-core registers."
>
> So we have named it like this.
"Freescale i.MX non-core registers" implies this hardware is about i.MX
registers. i.MX like for all SoCs.
What you wrote now, about USB registers - quite different.
However if you describe here registers, not some device, usually it's
place is not USB...
And actually all devices are bunch of registers, right? So everything
should be called registers?
Name is poor choice, so please find more appropriate one matching real
purpose of this block.
Best regards,
Krzysztof
Hi,
> -----Original Message-----
> From: Krzysztof Kozlowski <[email protected]>
> Sent: Wednesday, October 19, 2022 6:52 AM
> To: Xu Yang <[email protected]>; Peng Fan (OSS) <[email protected]>;
> [email protected]; [email protected];
> [email protected]; [email protected];
> [email protected]
> Cc: [email protected]; [email protected]; dl-linux-imx
> <[email protected]>; [email protected];
> [email protected]; [email protected];
> [email protected]; Jun Li <[email protected]>; Peng Fan
> <[email protected]>
> Subject: Re: [EXT] Re: [PATCH V2 1/6] dt-bindings: usb: usbmisc-imx: convert
> to DT schema
>
> On 16/10/2022 23:17, Xu Yang wrote:
> > Hi Krzysztof,
> >
> >> -----Original Message-----
> >> From: Krzysztof Kozlowski <[email protected]>
> >> Sent: Saturday, October 15, 2022 10:53 PM
> >> To: Peng Fan (OSS) <[email protected]>;
> >> [email protected]; [email protected];
> >> [email protected]; [email protected];
> >> [email protected]; Xu Yang <[email protected]>
> >> Cc: [email protected]; [email protected]; dl-linux-imx
> >> <[email protected]>; [email protected];
> >> [email protected]; [email protected];
> >> [email protected]; Jun Li <[email protected]>; Peng
> >> Fan <[email protected]>
> >> Subject: [EXT] Re: [PATCH V2 1/6] dt-bindings: usb: usbmisc-imx:
> >> convert to DT schema
> >>
> >> Caution: EXT Email
> >>
> >> On 14/10/2022 05:51, Peng Fan (OSS) wrote:
> >>> From: Peng Fan <[email protected]>
> >>>
> >>> Convert usbmisc-imx to DT schema format.
> >>>
> >>> Signed-off-by: Peng Fan <[email protected]>
> >>> ---
> >>> .../devicetree/bindings/usb/fsl,usbmisc.yaml | 52
> +++++++++++++++++++
> >>> .../devicetree/bindings/usb/usbmisc-imx.txt | 18 -------
> >>> 2 files changed, 52 insertions(+), 18 deletions(-) create mode
> >>> 100644 Documentation/devicetree/bindings/usb/fsl,usbmisc.yaml
> >>> delete mode 100644
> >>> Documentation/devicetree/bindings/usb/usbmisc-imx.txt
> >>>
> >>> diff --git a/Documentation/devicetree/bindings/usb/fsl,usbmisc.yaml
> >> b/Documentation/devicetree/bindings/usb/fsl,usbmisc.yaml
> >>> new file mode 100644
> >>> index 000000000000..c83ffb6729b5
> >>> --- /dev/null
> >>> +++ b/Documentation/devicetree/bindings/usb/fsl,usbmisc.yaml
> >>> @@ -0,0 +1,52 @@
> >>> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) %YAML 1.2
> >>> +---
> >>> +$id:
> >>
> https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fdevi
> >>
> cetree.org%2Fschemas%2Fusb%2Ffsl%2Cusbmi&data=05%7C01%7Cjun.li%40
> >>
> nxp.com%7C15ee8a29acf04f95ff4108dab15b5b88%7C686ea1d3bc2b4c6fa92cd99c
> >>
> 5c301635%7C0%7C0%7C638017303177794862%7CUnknown%7CTWFpbGZsb3d8eyJWIjo
> >>
> iMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C
> >> %7C%7C&sdata=tovO%2F6fnxAMSUu9hTFv1Kgarmo%2BwxKCxS%2FeJAbt3e0k%3
> D
> >> &reserved=0
> >>
> sc.yaml%23&data=05%7C01%7Cxu.yang_2%40nxp.com%7C24e82b830d9f47018
> >> ff408daaebcf83d%7C686ea1d3bc2b4c6
> >>
> fa92cd99c5c301635%7C0%7C0%7C638014423898086337%7CUnknown%7CTWFpbGZsb3
> >> d8eyJWIjoiMC4wLjAwMDAiLCJQIjoi
> >>
> V2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=gNwM
> >> bGsiuMS1zgvbgATYAY70gv2nIhX%2F
> >> 16IxiSYBf3s%3D&reserved=0
> >>> +$schema:
> >>>
> +https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fde
> >>>
> +vicetree.org%2Fmeta-&data=05%7C01%7Cjun.li%40nxp.com%7C15ee8a29
> >>>
> +acf04f95ff4108dab15b5b88%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0
> >>>
> +%7C638017303177794862%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiL
> >>>
> +CJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&s
> >>>
> +data=NxKdgHlJ9q%2F82XtXwWxHf5UFJR3v08L83jomQxg3Blo%3D&reserved=
> >>> +0
> >>
> schemas%2Fcore.yaml%23&data=05%7C01%7Cxu.yang_2%40nxp.com%7C24e82
> >> b830d9f47018ff408daaebcf83d%7C68
> >>
> 6ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C638014423898242035%7CUnknown
> >> %7CTWFpbGZsb3d8eyJWIjoiMC4wLjA
> >>
> wMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&a
> >> mp;sdata=jJUP5aKNkfkAefhNQL9almV
> >> Gqzn8cTNyjcRlZMOZwRk%3D&reserved=0
> >>> +
> >>> +title: Freescale i.MX non-core registers
> >>
> >> You sent v2 before we finished this topic. Is it correct? Even if TXT
> >> had such title, let's don't keep the wrong name.
> >>
> >> If it is non-core registers, why it is in USB? Why it is an usb-misc device?
> >
> > The chipidea's doc has a more clear explanation about core and
> > non-core registers as follow:
> >
> > "There are two kinds of registers in the USB module: USB core
> > registers and USB non-core registers. USB core registers are used to
> > control USB core functions, and more independent of USB features. Each
> > USB controller core has its own core registers. USB non-core registers
> > are additional to USB core registers, and more dependent on USB
> > features. i.MX series products vary in non-core registers."
> >
> > So we have named it like this.
>
> "Freescale i.MX non-core registers" implies this hardware is about i.MX
> registers. i.MX like for all SoCs.
>
> What you wrote now, about USB registers - quite different.
>
> However if you describe here registers, not some device, usually it's place
> is not USB...
>
> And actually all devices are bunch of registers, right? So everything should
> be called registers?
>
> Name is poor choice, so please find more appropriate one matching real purpose
> of this block.
How about "Freescale i.MX wrapper module for Chipidea USB2 controller"
Thanks
Li Jun
>
>
> Best regards,
> Krzysztof