From mboxrd@z Thu Jan 1 00:00:00 1970 From: Florian Fainelli Subject: Re: [PATCH] i2c: brcmstb: Allow enabling the driver on DSL SoCs Date: Sat, 29 Sep 2018 13:37:13 -0700 Message-ID: <34253924-54D8-4CB1-822E-21123EB503CA@gmail.com> References: <20180928223947.20751-1-f.fainelli@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Return-path: In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: Geert Uytterhoeven Cc: Linux Kernel Mailing List , Kamal Dasu , Wolfram Sang , Jarkko Nikula , Jean Delvare , Andy Shevchenko , M'boumba Cedric Madianga , kramasub@codeaurora.org, Pierre-Yves MORDRET , eajames@linux.vnet.ibm.com, Ard Biesheuvel , manivannan.sadhasivam@linaro.org, Arnd Bergmann , Heikki Krogerus , Linux I2C List-Id: linux-i2c@vger.kernel.org On September 29, 2018 12:55:08 AM PDT, Geert Uytterhoeven wrote: >Hi Florian, > >On Sat, Sep 29, 2018 at 12:40 AM Florian Fainelli > wrote: >> ARCH_BCM_63XX which is used by ARM-based DSL SoCs from Broadcom uses >the >> same controller, make it possible to select the STB driver and update >> the Kconfig and help text a bit=2E >> >> Signed-off-by: Florian Fainelli > >Thanks for your patch! > >> --- a/drivers/i2c/busses/Kconfig >> +++ b/drivers/i2c/busses/Kconfig >> @@ -432,12 +432,13 @@ config I2C_BCM_KONA >> If you do not need KONA I2C interface, say N=2E >> >> config I2C_BRCMSTB >> - tristate "BRCM Settop I2C controller" >> - depends on ARCH_BRCMSTB || BMIPS_GENERIC || COMPILE_TEST >> + tristate "BRCM Settop/DSL I2C controller" >> + depends on ARCH_BRCMSTB || BMIPS_GENERIC || COMPILE_TEST || \ >> + ARCH_BCM_63XX > >I think it would be good to keep the "|| COMPILE_TEST" as the last >dependency=2E Makes sense, v2 coming, thank you! --=20 Florian