From mboxrd@z Thu Jan 1 00:00:00 1970 From: Krzysztof Kozlowski Subject: Re: [PATCH 5/5] i2c: Enable compile testing for some of drivers Date: Fri, 3 Jan 2020 14:42:31 +0100 Message-ID: <20200103134231.GA2095@pi3> References: <20191230172751.17985-1-krzk@kernel.org> <20191230172751.17985-5-krzk@kernel.org> <20191231090146.GA6872@pi3> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Geert Uytterhoeven Cc: Linux ARM , Wolfram Sang , Linus Walleij , Linux Kernel Mailing List , Vladimir Zapolskiy , Max Staudt , Elie Morisse , Jarkko Nikula , Linux I2C , Sylvain Lemieux , Juergen Fitschen , Jean Delvare List-Id: linux-i2c@vger.kernel.org On Tue, Dec 31, 2019 at 10:20:01AM +0100, Geert Uytterhoeven wrote: > Hi Krzysztof, > > On Tue, Dec 31, 2019 at 10:01 AM Krzysztof Kozlowski wrote: > > On Mon, Dec 30, 2019 at 08:11:03PM +0100, Geert Uytterhoeven wrote: > > > On Mon, Dec 30, 2019 at 6:28 PM Krzysztof Kozlowski wrote: > > > > Some of the I2C bus drivers can be compile tested to increase build > > > > coverage. This requires also: > > > > 1. Adding dependencies on COMMON_CLK for BCM2835 and Meson I2C > > > > controllers, > > > > 2. Adding 'if' conditional to 'default y' so they will not get enabled > > > > by default on all other architectures, > > > > 3. Limiting few compile test options to supported architectures (which > > > > provide the readsX()/writesX() primitives). > > > > > > > > Signed-off-by: Krzysztof Kozlowski > > > > > @@ -611,8 +612,8 @@ config I2C_EMEV2 > > > > > > > > config I2C_EXYNOS5 > > > > tristate "Exynos5 high-speed I2C driver" > > > > - depends on ARCH_EXYNOS && OF > > > > - default y > > > > + depends on (ARCH_EXYNOS && OF) || COMPILE_TEST > > > > > > This means it is only useful on DT-based Exynos platforms, but compiles > > > everywhere? > > > > Yes. The driver will proble only from DT. > > > > > > > > Do you still have support for non-DT Exynos platforms? > > > ARCH_EXYNOS depends on ARCH_MULTI_V7? > > > > No, only DT. I think dependency here is a left over from board times and > > optional OF. Actually many drivers depend on OF and some OF-like ARCH so > > it could be removed. > > > > Since driver uses OF, it's rather the choice whether to explicitly > > mention OF. > > So the !OF case is only for compile-testing. Then it can be simplified to: > > depends on ARCH_EXYNOS || COMPILE_TEST > Many other drivers do exactly the same: depend on ARCH_xxx (which is already OF only) and on OF. Since OF now has all (or almost all) stubs for !OF case, such "depends on OF" is purely from logical point of view: the driver will not bind without OF. It will not be useful. I do not have strong opinion about this but is there a point to change only this one, while in all other places drivers depend on OF? It will just bring inconsistency. Best regards, Krzysztof