Le 03/04/2016 07:41, Simon Arlott a écrit :
> On 03/04/16 13:00, Álvaro Fernández Rojas wrote:
>> BCM63168 and BCM63268 are very similar and Broadcom refers to them as BCM63268
>
> They are practically the same but they both exist.
>
>> diff --git a/arch/mips/bmips/setup.c b/arch/mips/bmips/setup.c
>> index 38b5bd5..bfee6ea 100644
>> --- a/arch/mips/bmips/setup.c
>> +++ b/arch/mips/bmips/setup.c
>> @@ -112,10 +112,10 @@ static void bcm6368_quirks(void)
>> static const struct bmips_quirk bmips_quirk_list[] = {
>> { "brcm,bcm3384-viper", &bcm3384_viper_quirks },
>> { "brcm,bcm33843-viper", &bcm3384_viper_quirks },
>> + { "brcm,bcm63268", &bcm6368_quirks },
>> { "brcm,bcm6328", &bcm6328_quirks },
>> { "brcm,bcm6358", &bcm6358_quirks },
>> { "brcm,bcm6368", &bcm6368_quirks },
>> - { "brcm,bcm63168", &bcm6368_quirks },
>
> You can add "brcm,bcm63268" but you can't remove support for
> "brcm,bcm63168".
Agreed, this cannot be removed now.
--
Florian
