linux-kernel - Re: [PATCH] ARM: zynq: dts: Setting default i2c clock frequency to 400kHz

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: <e0b50ae1-b501-cb41-829f-736903e5794f@linaro.org>
Date:   Tue, 2 May 2023 22:13:45 +0200
From:   Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
To:     Michal Simek <michal.simek@....com>, linux-kernel@...r.kernel.org,
        monstr@...str.eu, michal.simek@...inx.com, git@...inx.com
Cc:     Varalaxmi Bingi <varalaxmi.bingi@....com>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
        Rob Herring <robh+dt@...nel.org>, devicetree@...r.kernel.org,
        linux-arm-kernel@...ts.infradead.org
Subject: Re: [PATCH] ARM: zynq: dts: Setting default i2c clock frequency to
 400kHz

On 02/05/2023 15:53, Michal Simek wrote:
> From: Varalaxmi Bingi <varalaxmi.bingi@....com>
> 
> Setting default i2c clock frequency for Zynq to maximum rate of 400kHz.
> Current default value is 100kHz.
> 
> Signed-off-by: Varalaxmi Bingi <varalaxmi.bingi@....com>
> Signed-off-by: Michal Simek <michal.simek@....com>
> ---
> 
>  arch/arm/boot/dts/zynq-7000.dtsi | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/zynq-7000.dtsi b/arch/arm/boot/dts/zynq-7000.dtsi
> index cd9931f6bcbd..a7db3f3009f2 100644
> --- a/arch/arm/boot/dts/zynq-7000.dtsi
> +++ b/arch/arm/boot/dts/zynq-7000.dtsi
> @@ -149,6 +149,7 @@ i2c0: i2c@...04000 {
>  			clocks = <&clkc 38>;
>  			interrupt-parent = <&intc>;
>  			interrupts = <0 25 4>;
> +			clock-frequency = <400000>;

I think this should be rather property of the board, because it depends
on I2C devices, right?

Best regards,
Krzysztof