linux-kernel - Re: [EXT] Re: [PATCH V2 1/6] dt-bindings: usb: usbmisc-imx: convert to DT schema

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Tue, 18 Oct 2022 18:51:52 -0400
From:   Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
To:     Xu Yang <xu.yang_2@....com>,
        "Peng Fan (OSS)" <peng.fan@....nxp.com>,
        "gregkh@...uxfoundation.org" <gregkh@...uxfoundation.org>,
        "robh+dt@...nel.org" <robh+dt@...nel.org>,
        "krzysztof.kozlowski+dt@...aro.org" 
        <krzysztof.kozlowski+dt@...aro.org>,
        "shawnguo@...nel.org" <shawnguo@...nel.org>,
        "s.hauer@...gutronix.de" <s.hauer@...gutronix.de>
Cc:     "kernel@...gutronix.de" <kernel@...gutronix.de>,
        "festevam@...il.com" <festevam@...il.com>,
        dl-linux-imx <linux-imx@....com>,
        "linux-usb@...r.kernel.org" <linux-usb@...r.kernel.org>,
        "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "linux-arm-kernel@...ts.infradead.org" 
        <linux-arm-kernel@...ts.infradead.org>, Jun Li <jun.li@....com>,
        Peng Fan <peng.fan@....com>
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 <krzysztof.kozlowski@...aro.org>
>> Sent: Saturday, October 15, 2022 10:53 PM
>> To: Peng Fan (OSS) <peng.fan@....nxp.com>; gregkh@...uxfoundation.org; robh+dt@...nel.org;
>> krzysztof.kozlowski+dt@...aro.org; shawnguo@...nel.org; s.hauer@...gutronix.de; Xu Yang <xu.yang_2@....com>
>> Cc: kernel@...gutronix.de; festevam@...il.com; dl-linux-imx <linux-imx@....com>; linux-usb@...r.kernel.org;
>> devicetree@...r.kernel.org; linux-kernel@...r.kernel.org; linux-arm-kernel@...ts.infradead.org; Jun Li <jun.li@....com>;
>> Peng Fan <peng.fan@....com>
>> 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 <peng.fan@....com>
>>>
>>> Convert usbmisc-imx to DT schema format.
>>>
>>> Signed-off-by: Peng Fan <peng.fan@....com>
>>> ---
>>>  .../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&amp;data=05%7C01%7Cxu.yang_2%40nxp.com%7C24e82b830d9f47018ff408daaebcf83d%7C686ea1d3bc2b4c6
>> fa92cd99c5c301635%7C0%7C0%7C638014423898086337%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoi
>> V2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&amp;sdata=gNwMbGsiuMS1zgvbgATYAY70gv2nIhX%2F
>> 16IxiSYBf3s%3D&amp;reserved=0
>>> +$schema: https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fdevicetree.org%2Fmeta-
>> schemas%2Fcore.yaml%23&amp;data=05%7C01%7Cxu.yang_2%40nxp.com%7C24e82b830d9f47018ff408daaebcf83d%7C68
>> 6ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C638014423898242035%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjA
>> wMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&amp;sdata=jJUP5aKNkfkAefhNQL9almV
>> Gqzn8cTNyjcRlZMOZwRk%3D&amp;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