linux-kernel - Re: [PATCH] ANDROID: i2c: exynos5: remove default enable in Kconfig

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] [day] [month] [year] [list]

Message-ID: <CAJKOXPemBE2AHW4aju+SzvHA-rRCNwYhpHpG8ThsY_W2v1tzoA@mail.gmail.com>
Date:   Thu, 5 Dec 2019 16:29:15 +0100
From:   Krzysztof Kozlowski <krzk@...nel.org>
To:     Hyunki Koo <hyunki00.koo@...il.com>
Cc:     wsa@...-dreams.de, Hyunki Koo <hyunki00.koo@...sung.com>,
        Jean Delvare <jdelvare@...e.de>,
        Jarkko Nikula <jarkko.nikula@...ux.intel.com>,
        Alexandre Belloni <alexandre.belloni@...tlin.com>,
        Max Staudt <max@...as.org>,
        Juergen Fitschen <jfi@...-embedded.de>,
        Elie Morisse <syniurge@...il.com>, Stefan Roese <sr@...x.de>,
        Baolin Wang <baolin.wang@...aro.org>,
        Arnd Bergmann <arnd@...db.de>,
        Mauro Carvalho Chehab <mchehab+samsung@...nel.org>,
        linux-i2c@...r.kernel.org,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] ANDROID: i2c: exynos5: remove default enable in Kconfig

On Thu, 5 Dec 2019 at 16:05, Hyunki Koo <hyunki00.koo@...il.com> wrote:
>
> From: Hyunki Koo <hyunki00.koo@...sung.com>
>
> 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 <hyunki00.koo@...sung.com>
> ---
>  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
>