[PATCH 2/6] arm: mvebu: Enable USB controllers on Armada 370 evaluation board

Ezequiel Garcia ezequiel.garcia at free-electrons.com
Wed Jan 16 00:58:30 EST 2013


Hi Arnd,

On 01/15/2013 10:07 AM, Arnd Bergmann wrote:
> On Tuesday 15 January 2013, Ezequiel Garcia wrote:
>> Cc: Lior Amsalem <alior at marvell.com>
>> Cc: Thomas Petazzoni <thomas.petazzoni at free-electrons.com>
>> Cc: Gregory CLEMENT <gregory.clement at free-electrons.com>
>> Signed-off-by: Ezequiel Garcia <ezequiel.garcia at free-electrons.com>
> 
> The patches look good, but when you have four trivial patches
> doing the same thing in different files, and you decide that it's
> not worth writing a changelog for them, they should probably
> go into a single patch.
>

Mmm... maybe you're right and such splitting is excessive.
It seemed tidier this way, enabling each board on a different patch.

I can squash them on a v2, if you want me to.

Thanks for reviewing!

-- 
Ezequiel García, Free Electrons
Embedded Linux, Kernel and Android Engineering
http://free-electrons.com


More information about the devicetree-discuss mailing list