From: Hyunki Koo <[email protected]>
There are many exynos device and not all exynos device does not have
HSI2C controllers. Thus remove select this by default
Signed-off-by: Hyunki Koo <[email protected]>
---
drivers/i2c/busses/Kconfig | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/i2c/busses/Kconfig b/drivers/i2c/busses/Kconfig
index 6a0aa76859f3..3955315f48c7 100644
--- a/drivers/i2c/busses/Kconfig
+++ b/drivers/i2c/busses/Kconfig
@@ -612,7 +612,6 @@ config I2C_EMEV2
config I2C_EXYNOS5
tristate "Exynos5 high-speed I2C driver"
depends on ARCH_EXYNOS && OF
- default y
help
High-speed I2C controller on Exynos5 based Samsung SoCs.
--
2.17.1
On Thu, 5 Dec 2019 at 16:05, Hyunki Koo <[email protected]> wrote:
>
> From: Hyunki Koo <[email protected]>
>
> There are many exynos device and not all exynos device does not have
> HSI2C controllers. Thus remove select this by default
>
> Signed-off-by: Hyunki Koo <[email protected]>
> ---
> drivers/i2c/busses/Kconfig | 1 -
> 1 file changed, 1 deletion(-)
Thanks for the patch but this is not a proper solution because:
1. Basically you disabled this driver on exynos, multi_v7 and ARMv8 defconfigs.
2. We provide one kernel for all ARMv7 Exynos and one for all ARMv8
platforms therefore this driver should be there. Having unnecessary
driver also does not hurt - it will not match by compatible.
So really no.
Best regards,
Krzysztof
> diff --git a/drivers/i2c/busses/Kconfig b/drivers/i2c/busses/Kconfig
> index 6a0aa76859f3..3955315f48c7 100644
> --- a/drivers/i2c/busses/Kconfig
> +++ b/drivers/i2c/busses/Kconfig
> @@ -612,7 +612,6 @@ config I2C_EMEV2
> config I2C_EXYNOS5
> tristate "Exynos5 high-speed I2C driver"
> depends on ARCH_EXYNOS && OF
> - default y
> help
> High-speed I2C controller on Exynos5 based Samsung SoCs.
>
> --
> 2.17.1
>