linux-kernel - Re: [EXT] Re: [PATCH v3 1/3] dt-bindings: usb: cdns-imx8qm: add imx8qm cdns3 glue bindings

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]

Message-ID: <835cda64-5d42-1ff3-aa8f-0802fe3d705f@linaro.org>
Date:   Wed, 22 Mar 2023 22:38:05 +0100
From:   Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
To:     Frank Li <frank.li@....com>
Cc:     "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
        "festevam@...il.com" <festevam@...il.com>,
        "imx@...ts.linux.dev" <imx@...ts.linux.dev>,
        "kernel@...gutronix.de" <kernel@...gutronix.de>,
        "krzysztof.kozlowski+dt@...aro.org" 
        <krzysztof.kozlowski+dt@...aro.org>,
        "linux-arm-kernel@...ts.infradead.org" 
        <linux-arm-kernel@...ts.infradead.org>,
        dl-linux-imx <linux-imx@....com>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "robh+dt@...nel.org" <robh+dt@...nel.org>,
        "s.hauer@...gutronix.de" <s.hauer@...gutronix.de>,
        "shawnguo@...nel.org" <shawnguo@...nel.org>
Subject: Re: [EXT] Re: [PATCH v3 1/3] dt-bindings: usb: cdns-imx8qm: add
 imx8qm cdns3 glue bindings

On 22/03/2023 22:36, Frank Li wrote:
> 
> 
>> -----Original Message-----
>> From: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
>> Sent: Wednesday, March 22, 2023 4:32 PM
>> To: Frank Li <frank.li@....com>
>> Cc: devicetree@...r.kernel.org; festevam@...il.com; imx@...ts.linux.dev;
>> kernel@...gutronix.de; krzysztof.kozlowski+dt@...aro.org; linux-arm-
>> kernel@...ts.infradead.org; dl-linux-imx <linux-imx@....com>; linux-
>> kernel@...r.kernel.org; robh+dt@...nel.org; s.hauer@...gutronix.de;
>> shawnguo@...nel.org
>> Subject: Re: [EXT] Re: [PATCH v3 1/3] dt-bindings: usb: cdns-imx8qm: add
>> imx8qm cdns3 glue bindings
>>
>> Caution: EXT Email
>>
>> On 22/03/2023 15:34, Frank Li wrote:
>>>
>>>
>>>> -----Original Message-----
>>>> From: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
>>>> Sent: Wednesday, March 22, 2023 2:32 AM
>>>> To: Frank Li <frank.li@....
>>>>> +      - const: usb3_aclk
>>>>> +      - const: usb3_ipg_clk
>>>>> +      - const: usb3_core_pclk
>>>>> +
>>>>> +  assigned-clocks:
>>>>> +    items:
>>>>> +      - description: Phandle and clock specifoer of
>>>> IMX_SC_PM_CLK_MST_BUS.
>>>>
>>>> Drop useless pieces so "Phandle and clock specifoer of " and name the
>>>> hardware, not the syntax.
>>>>
>>>>> +
>>>>> +  assigned-clock-rates:
>>>>> +    items:
>>>>> +      - description: Should be in Range 100 - 600 Mhz.
>>>>
>>>> That's better but I still do not understand why do you need it in the
>>>> bindings. You never actually answered this question.
>>>
>>> I am not sure 100% sure the reason.
>>> I think difference system target's  axi bus frequency is difference,
>>> And just one time work, needn't software to manage it.
>>> Following other driver's code style may be another reason.
>>
>> That's the reason of heaving it in DTS. But I am asking about bindings.
>> You do understand you define here interface?
> 
> I defined here is descript AXI frequency for usb controller. Supposed difference
> Platform will have difference working frequency.  

I don't understand how does this answer my concerns of having it in DT
bindings. If you do not add it, you "will have difference working
frequency", so what's the point?

Best regards,
Krzysztof