Re: [PATCH v1 1/2] dt-bindings: i2c: ls2x-i2c: Add clock- related properties

[Krzysztof Kozlowski <krzk@kernel.org>]

On 26/03/2026 08:02, Krzysztof Kozlowski wrote:
> On 26/03/2026 03:12, Hongliang Wang wrote:
>> Hi Krzysztof,
>>
>> Consider the clock framework relies on the device tree, and can only be 
>> used on
>> Loongson 2K platform with dts parameter mechanism, It cannot be used on 
>> Loongson
> 
> Don't top post or request read receipts.
> 
>> 3A+7A platform with the acpi parameter mechanism.
> 
> And this patch is for ACPI? Then we finish discussion here, because
> dt-bindings is not for ACPI.
> 
>> The i2c-ls2x driver is compatible with both Loongson 2K and 3A+7A 
>> platform, parse
>> the same parameters regardless of dts or acpi parameter passing, So 
>> clock-input
>> and clock-div attributes are defined to describe input clock of i2c 
>> controller and
>> divisor of input clock. It can be used on both 2K and 3A+7A platform.
> 
> And you cannot use them in DTS.

I need to keep guessing what you want to achieve, because neither your
message nor commit text was explicit - if you need properties for ACPI
and you want to be sure that DTS does not have them, then you could
define them as "foo:false" with a comment why (you always explain WHY
you are doing things). We don't have such convention so far, but I think
it will be useful when Rob finishes the ABI checker.

*Otherwise* minimum would be a comment in the driver that these are not
allowed in DTS.

Best regards,
Krzysztof