Re: [PATCH v5 01/17] spi: dt-bindings: add spi-max-post-config-frequency-hz property

Krzysztof Kozlowski <[email protected]>
Newsgroups org.kernel.vger.linux-spi,org.infradead.lists.linux-mtd,org.kernel.vger.linux-devicetree,org.kernel.vger.linux-kernel
Message-ID <[email protected]>
On 20/07/2026 09:38, Santhosh Kumar K wrote:
>>> diff --git a/Documentation/devicetree/bindings/spi/spi-peripheral-props.yaml b/Documentation/devicetree/bindings/spi/spi-peripheral-props.yaml
>>> index 880a9f624566..12211f8c9f7d 100644
>>> --- a/Documentation/devicetree/bindings/spi/spi-peripheral-props.yaml
>>> +++ b/Documentation/devicetree/bindings/spi/spi-peripheral-props.yaml
>>> @@ -45,6 +45,11 @@ properties:
>>>       description:
>>>         Maximum SPI clocking speed of the device in Hz.
>>>   
>>> +  spi-max-post-config-frequency-hz:
>>> +    description:
>>> +      Maximum SPI clock frequency in Hz achievable after controller-side
>>> +      configuration.
>>
>> How did you implement my feedback?
> 
> Three points from your feedback:
> 
> 1. -hz suffix: added
> 
> 2. Array property: I still lean towards keeping this as a scalar for
> now, as we don't have a concrete use case for multiple values today, and
> supporting two frequencies should be sufficient for the foreseeable
> future. I had responded to this in the v4 discussion as well, you may
> have missed that response.
> 
> 3. maxItems: Since the property remains a scalar, I have not added maxItems.

So how did you make this property scalar?

Best regards,
Krzysztof
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.