2021-11-08 16:53:39

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH 2/2] arm64: defconfig: Enable Samsung I2C driver

On 08/11/2021 14:49, Sam Protsenko wrote:
> i2c-s3c2410 driver is needed for some arm64 Exynos SoCs, e.g. Exynos850.
>
> Signed-off-by: Sam Protsenko <[email protected]>
> ---
> arch/arm64/configs/defconfig | 1 +
> 1 file changed, 1 insertion(+)

Hm, that's a little bit unexpected. I thought you will be also using the
HSI2C driver (where Jaewon is adding USI support).

Best regards,
Krzysztof


2021-11-08 20:57:28

by Sam Protsenko

[permalink] [raw]
Subject: Re: [PATCH 2/2] arm64: defconfig: Enable Samsung I2C driver

On Mon, 8 Nov 2021 at 16:18, Krzysztof Kozlowski
<[email protected]> wrote:
>
> On 08/11/2021 14:49, Sam Protsenko wrote:
> > i2c-s3c2410 driver is needed for some arm64 Exynos SoCs, e.g. Exynos850.
> >
> > Signed-off-by: Sam Protsenko <[email protected]>
> > ---
> > arch/arm64/configs/defconfig | 1 +
> > 1 file changed, 1 insertion(+)
>
> Hm, that's a little bit unexpected. I thought you will be also using the
> HSI2C driver (where Jaewon is adding USI support).
>

Thought you'd ask :) Exynos850 has both I2C and HSI2C blocks:
- I2C is a regular I2C, and uses s3c2410-i2c driver
- whilst HSI2C is a high-speed I2C (used for example for cameras),
and uses exynos5-hsi2c driver

I'll look into HSI2C enablement on Exynos850 soon as well, so Jaewon's
patches might be helpful for my case too -- didn't check that yet.

> Best regards,
> Krzysztof