[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/6] i.MX: simplify CCM to only handle clock req
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH 2/6] i.MX: simplify CCM to only handle clock required by timers. |
Date: |
Wed, 3 Feb 2016 10:44:33 +0000 |
On 2 February 2016 at 22:22, Jean-Christophe DUBOIS <address@hidden> wrote:
> Peter Maydell wrote:
>> These are just renaming NOCLK to CLK_NONE and fixing formatting?
>> Again, please don't put that in the same patch as substantive
>> code changes.
>
>
> I just wanted to make things more coherent at the naming convention level.
>
> But if you prefer NOCLK, I'll put it back.
I have no preference either way about the name. It's just hard
to review patches if they mix lots of cleanups in at once,
and especially if they mix stylistic changes in with
behaviour changes. It's probably sufficient just to split this
patch up into logically distinct changes with suitable commit
messages.
thanks
-- PMM