From:   "Alim Akhtar" <alim.akhtar@samsung.com>
To:     "'Krzysztof Kozlowski'" <krzysztof.kozlowski@linaro.org>,
        "'Rob Herring'" <robh+dt@kernel.org>,
        "'Krzysztof Kozlowski'" <krzysztof.kozlowski+dt@linaro.org>,
        "'Inki Dae'" <inki.dae@samsung.com>,
        "'Kyungmin Park'" <kyungmin.park@samsung.com>,
        "'Kukjin Kim'" <kgene@kernel.org>, <devicetree@vger.kernel.org>,
        <linux-arm-kernel@lists.infradead.org>,
        <linux-samsung-soc@vger.kernel.org>, <linux-kernel@vger.kernel.org>
Cc:     <replicant@osuosl.org>, <phone-devel@vger.kernel.org>,
        <~postmarketos/upstreaming@lists.sr.ht>,
        =?utf-8?Q?'Martin_J=C3=BCcker'?= <martin.juecker@gmail.com>,
        "'Henrik Grimler'" <henrik@grimler.se>
In-Reply-To: <20230120155404.323386-12-krzysztof.kozlowski@linaro.org>
Subject: RE: [PATCH 11/11] ARM: dts: exynos: correct HSI2C properties in
 Exynos5410 Odroid XU
Date:   Sat, 28 Jan 2023 10:16:37 +0530
Message-ID: <000701d932d3$834cb170$89e61450$@samsung.com>
MIME-Version: 1.0
Content-Transfer-Encoding: quoted-printable
Thread-Index: AQHNuRkKafHieBmUHCY1mc35yYPTTQKvTNNCAsqGfBCunmYTcA==
Content-Language: en-us
Content-Type: text/plain; charset="utf-8"
CMS-TYPE: 105P
DLP-Filter: Pass
References: <20230120155404.323386-1-krzysztof.kozlowski@linaro.org>
        <CGME20230120155431epcas5p37731d63d93c501522c4b08515f174ce4@epcas5p3.samsung.com>
        <20230120155404.323386-12-krzysztof.kozlowski@linaro.org>
Precedence: bulk


> -----Original Message-----
> From: Krzysztof Kozlowski <krzysztof.kozlowski=40linaro.org>
> Sent: Friday, January 20, 2023 9:24 PM
> To: Rob Herring <robh+dt=40kernel.org>; Krzysztof Kozlowski
> <krzysztof.kozlowski+dt=40linaro.org>; Alim Akhtar
> <alim.akhtar=40samsung.com>; Inki Dae <inki.dae=40samsung.com>; Kyungmin
> Park <kyungmin.park=40samsung.com>; Kukjin Kim <kgene=40kernel.org>;
> devicetree=40vger.kernel.org; linux-arm-kernel=40lists.infradead.org; lin=
ux-
> samsung-soc=40vger.kernel.org; linux-kernel=40vger.kernel.org
> Cc: replicant=40osuosl.org; phone-devel=40vger.kernel.org;
> =7Epostmarketos/upstreaming=40lists.sr.ht; Martin J=C3=BCcker=0D=0A>=20<m=
artin.juecker=40gmail.com>;=20Henrik=20Grimler=20<henrik=40grimler.se>;=0D=
=0A>=20Krzysztof=20Kozlowski=20<krzysztof.kozlowski=40linaro.org>=0D=0A>=20=
Subject:=20=5BPATCH=2011/11=5D=20ARM:=20dts:=20exynos:=20correct=20HSI2C=20=
properties=20in=0D=0A>=20Exynos5410=20Odroid=20XU=0D=0A>=20=0D=0A>=20The=20=
HSI2C=20controller=20uses=20clock-frequency=20and=20does=20not=20have=20i2c=
-sda-delay.=0D=0A>=20The=20properties=20were=20copy-pasted=20from=20the=20s=
lower=20S3C-family=20I2C=20controller:=0D=0A>=20=0D=0A>=20=20=20exynos5410-=
odroidxu.dtb:=20i2c=4012ca0000:=20Unevaluated=20properties=20are=20not=0D=
=0A>=20allowed=20('samsung,i2c-max-bus-freq',=20'samsung,i2c-sda-delay'=20w=
ere=0D=0A>=20unexpected)=0D=0A>=20=0D=0A>=20Signed-off-by:=20Krzysztof=20Ko=
zlowski=20<krzysztof.kozlowski=40linaro.org>=0D=0A>=20---=0D=0A=0D=0AReview=
ed-by:=20Alim=20Akhtar=20<alim.akhtar=40samsung.com>=0D=0A=0D=0A>=20=20arch=
/arm/boot/dts/exynos5410-odroidxu.dts=20=7C=203=20+--=0D=0A>=20=201=20file=
=20changed,=201=20insertion(+),=202=20deletions(-)=0D=0A>=20=0D=0A>=20diff=
=20--git=20a/arch/arm/boot/dts/exynos5410-odroidxu.dts=0D=0A>=20b/arch/arm/=
boot/dts/exynos5410-odroidxu.dts=0D=0A>=20index=20d1cbc6b8a570..232561620da=
2=20100644=0D=0A>=20---=20a/arch/arm/boot/dts/exynos5410-odroidxu.dts=0D=0A=
>=20+++=20b/arch/arm/boot/dts/exynos5410-odroidxu.dts=0D=0A>=20=40=40=20-16=
4,8=20+164,7=20=40=40=20map2=20=7B=0D=0A>=20=20=7D;=0D=0A>=20=0D=0A>=20=20&=
hsi2c_4=20=7B=0D=0A>=20-=09samsung,i2c-sda-delay=20=3D=20<100>;=0D=0A>=20-=
=09samsung,i2c-max-bus-freq=20=3D=20<400000>;=0D=0A>=20+=09clock-frequency=
=20=3D=20<400000>;=0D=0A>=20=20=09status=20=3D=20=22okay=22;=0D=0A>=20=0D=
=0A>=20=20=09usb3503:=20usb-hub=408=20=7B=0D=0A>=20--=0D=0A>=202.34.1=0D=0A=
=0D=0A=0D=0A