Re: [PATCH v2] dt-bindings: i2c: renesas,riic: Drop clock-frequency from required

"Rob Herring (Arm)" <[email protected]> Tue, 11 Aug 2026 21:56:02 -0500
Newsgroups org.kernel.vger.linux-i2c,org.kernel.vger.linux-devicetree,org.kernel.vger.linux-kernel,org.kernel.vger.linux-renesas-soc
Message-ID <[email protected]>
On Thu, 06 Aug 2026 13:10:13 +0200, Tommaso Merciai wrote:
> clock-frequency is described as optional, with 100 kHz assumed when
> absent, yet it is also listed as required.
> 
> The description is correct: i2c-riic calls i2c_parse_fw_timings() with
> use_defaults set, so the core falls back to I2C_MAX_STANDARD_MODE_FREQ
> when the property is absent.
> 
> The stale requirement only produces dtbs_check noise on boards relying
> on the default:
> 
>   i2c@14400800 (renesas,riic-r9a09g047): 'clock-frequency' is a
>   required property
> 
> Drop it.
> 
> Signed-off-by: Tommaso Merciai <[email protected]>
> ---
> v1->v2:
>  - Dropped all v1 series, need a rework.
>  - Fixed dtbs_check failure by dropping clock-frequency from the required list,
>    instead of adding it to the dts, as the absence of this property
>    indicates the default frequency 100 kHz.
>    v1: https://lore.kernel.org/all/[email protected]/
> 
>  Documentation/devicetree/bindings/i2c/renesas,riic.yaml | 1 -
>  1 file changed, 1 deletion(-)
> 

Acked-by: Rob Herring (Arm) <[email protected]>