linux-kernel - Re: [PATCH V4 1/4] ufs: dt-bindings: Document gear and rate limit properties

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] [day] [month] [year] [list]

Message-ID: <0107dcce-45cb-419f-814b-3eccbbbe628d@quicinc.com>
Date: Tue, 2 Sep 2025 22:22:13 +0530
From: Ram Kumar Dwivedi <quic_rdwivedi@...cinc.com>
To: Krzysztof Kozlowski <krzk@...nel.org>, <alim.akhtar@...sung.com>,
        <avri.altman@....com>, <bvanassche@....org>, <robh@...nel.org>,
        <krzk+dt@...nel.org>, <conor+dt@...nel.org>, <mani@...nel.org>,
        <James.Bottomley@...senPartnership.com>, <martin.petersen@...cle.com>
CC: <linux-scsi@...r.kernel.org>, <devicetree@...r.kernel.org>,
        <linux-kernel@...r.kernel.org>, <linux-arm-msm@...r.kernel.org>
Subject: Re: [PATCH V4 1/4] ufs: dt-bindings: Document gear and rate limit
 properties



On 02-Sep-25 11:42 AM, Krzysztof Kozlowski wrote:
> On 01/09/2025 17:57, Ram Kumar Dwivedi wrote:
>> Add optional "limit-hs-gear" and "limit-rate" properties to the
>> UFS controller common binding. These properties allow limiting
>> the maximum HS gear and rate.
>>
>> This is useful in cases where the customer board may have signal
>> integrity, clock configuration or layout issues that prevent reliable
>> operation at higher gears. Such limitations are especially critical in
>> those platforms, where stability is prioritized over peak performance.
>>
>> Signed-off-by: Ram Kumar Dwivedi <quic_rdwivedi@...cinc.com>
>> ---
>>  .../devicetree/bindings/ufs/ufs-common.yaml      | 16 ++++++++++++++++
>>  1 file changed, 16 insertions(+)
>>
>> diff --git a/Documentation/devicetree/bindings/ufs/ufs-common.yaml b/Documentation/devicetree/bindings/ufs/ufs-common.yaml
>> index 31fe7f30ff5b..b4c99fee552f 100644
>> --- a/Documentation/devicetree/bindings/ufs/ufs-common.yaml
>> +++ b/Documentation/devicetree/bindings/ufs/ufs-common.yaml
>> @@ -89,6 +89,22 @@ properties:
>>  
>>    msi-parent: true
>>  
>> +  limit-hs-gear:
>> +    $ref: /schemas/types.yaml#/definitions/uint32
>> +    minimum: 1
>> +    maximum: 5
> 
> No improvements.

Hi Krzysztof,

I have updated it in the next patchset.

Thanks,
Ram.> 
>> +    default: 5
>> +    description:
>> +      Restricts the maximum HS gear used in both TX and RX directions.
>> +
>> +  limit-rate:
>> +    $ref: /schemas/types.yaml#/definitions/string
>> +    enum: [Rate-A, Rate-B]
> 
> lowercase
I have updated it in the next patchset.

Thanks,
Ram.> 
>> +    default: Rate-B
> 
> 
> Best regards,
> Krzysztof