[prev in list] [next in list] [prev in thread] [next in thread] 

List:       linux-i2c
Subject:    Re: [PATCH] i2c: brcmstb: Allow enabling the driver on DSL SoCs
From:       Florian Fainelli <f.fainelli () gmail ! com>
Date:       2018-09-29 20:37:13
Message-ID: 34253924-54D8-4CB1-822E-21123EB503CA () gmail ! com
[Download RAW message or body]



On September 29, 2018 12:55:08 AM PDT, Geert Uytterhoeven <geert@linux-m68k.org> wrote:
>Hi Florian,
>
>On Sat, Sep 29, 2018 at 12:40 AM Florian Fainelli
><f.fainelli@gmail.com> 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.
>>
>> Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
>
>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.
>>
>>  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.

Makes sense, v2 coming, thank you!
-- 
Florian
[prev in list] [next in list] [prev in thread] [next in thread]