[U-Boot] [PATCH v2 07/15] qemu-malta: rename to just "malta"

[Gabor Juhos]

2013.10.24. 12:54 keltezéssel, Paul Burton írta:

<...>

>> I agree that it is related to the rename. However renaming the
>> mips_boot_qemu_malta constant is a simple cleanup which can be done in a
>> preceding patch. The same is true for the CONFIG_QEMU_MALTA->CONFIG_MALTA
>> conversion.
>>
>> In general, renaming a file and altering that within the same patch makes the
>> review harder.
>>
>> -Gabor
>>
>>
> 
> And for more complex changes I might agree, but only a single file is both
> renamed and modified by this patch: include/configs/qemu-malta.h =>
> include/configs/malta.h. Only a single line is modified within that file,
> changing CONFIG_QEMU_MALTA to CONFIG_MALTA. I don't think it's very difficult to
> understand and I think splitting it would be overkill. So if Daniel tells me (v3
> of) this patch isn't getting in as-is then I'll split it, but short of that I'm
> not changing it.

I don't say that it is hard to understand that. Simply the first two version of
the patch introduced a regression on older qemu version due to the missing
change in mips32/start.S, and it took some time to me to figure out why a rename
patch causes such thing. If it would have been a separate change, it would have
been obvious.

-Gabor