[RFC 1/7] Make Hi3620 explicit, remove wildcards
Marty Plummer
netz.kernel at gmail.com
Thu Sep 8 13:12:26 PDT 2016
On 09/08/2016 03:05 PM, Arnd Bergmann wrote:
> On Thursday, September 8, 2016 12:23:40 PM CEST Marty Plummer wrote:
>
>> arch/arm/Kconfig.debug | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/arch/arm/Kconfig.debug b/arch/arm/Kconfig.debug
>> index a9693b6..9094ca6 100644
>> --- a/arch/arm/Kconfig.debug
>> +++ b/arch/arm/Kconfig.debug
>> @@ -280,7 +280,7 @@ choice
>>
>> config DEBUG_HI3620_UART
>> bool "Hisilicon HI3620 Debug UART"
>> - depends on ARCH_HI3xxx
>> + depends on ARCH_HI3620
>> select DEBUG_UART_PL01X
>> help
>> Say Y here if you want kernel low-level debugging support
>>
>
> While patches should normally be split up into per-subsystem
> changes, they also have to be done in an 'atomic' way:
> applying just the first patch without the second one must
> not introduce a regression.
>
> In this case, I'd suggest simply doing a larger patch for
> the global rename to collect 'Acked-by's and send that through
> the arm-soc tree.
>
> If you get in a similar situation with more complex changes,
> you should come up with a way to do it independently. Here
> you would first have to introduce a CONFIG_ARCH_HI3620
> symbol in one patch and make that always selected at
> the same time as CONFIG_ARCH_HI3xxx, then change all users
> of that symbol, and finally remove the original.
>
> Arnd
>
So, in this case it should be one single patch? Geeze this is hella
complicated to just get a name changed XD
More information about the linux-arm-kernel
mailing list