On 19.12.24 18:47, Frank Li wrote: > On Thu, Dec 19, 2024 at 08:25:34AM +0100, Ahmad Fatoum wrote: >> From: Oleksij Rempel <o.rempel@xxxxxxxxxxxxxx> >> >> The NXP PCF85063TP RTC we use is capable of up to 400 kHz of SCL clock >> frequency, so let's make use of this instead of the 100 kHz bus frequency >> we are currently using. > > Increase I2C frequency to 400khz from 100kHz because NXP PCF85063TP RTC > support it. Unlike your other suggestions, these is no information lost by rewriting the commit message as you suggest. I don't mind, but must admit it feels like bikeshedding. What is your concrete objection to my commit message? Thanks, Ahmad > > Frank > >> >> Signed-off-by: Oleksij Rempel <o.rempel@xxxxxxxxxxxxxx> >> Signed-off-by: Ahmad Fatoum <a.fatoum@xxxxxxxxxxxxxx> >> --- >> arch/arm64/boot/dts/freescale/imx8mp-skov-reva.dtsi | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/arch/arm64/boot/dts/freescale/imx8mp-skov-reva.dtsi b/arch/arm64/boot/dts/freescale/imx8mp-skov-reva.dtsi >> index a683f46fcbab..ec7857db7bf0 100644 >> --- a/arch/arm64/boot/dts/freescale/imx8mp-skov-reva.dtsi >> +++ b/arch/arm64/boot/dts/freescale/imx8mp-skov-reva.dtsi >> @@ -333,7 +333,7 @@ reg_nvcc_sd2: LDO5 { >> }; >> >> &i2c3 { >> - clock-frequency = <100000>; >> + clock-frequency = <400000>; >> pinctrl-names = "default"; >> pinctrl-0 = <&pinctrl_i2c3>; >> status = "okay"; >> >> -- >> 2.39.5 >> > -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |