[U-Boot] [PATCH v2 01/20] arm: socfpga: Restructure clock manager driver

Ley Foon Tan lftan.linux at gmail.com
Tue Mar 21 07:26:29 UTC 2017


On Fri, Mar 10, 2017 at 12:29 AM, Dinh Nguyen <dinguyen at kernel.org> wrote:
>
>
> On 03/08/2017 06:26 PM, Ley Foon Tan wrote:
>> Restructure clock manager driver in the preparation to support A10.
>> Move the Gen5 specific code to _gen5 files. No functional change.
>>
>> Change all uint32_t to u32 and change to use macro BIT(n) for bit shift.
>>
>> Signed-off-by: Ley Foon Tan <ley.foon.tan at intel.com>
>> ---
>>  arch/arm/mach-socfpga/Makefile                     |   3 +-
>>  arch/arm/mach-socfpga/clock_manager.c              | 510 +--------------------
>>  arch/arm/mach-socfpga/clock_manager_gen5.c         | 510 +++++++++++++++++++++
>>  arch/arm/mach-socfpga/include/mach/clock_manager.h | 312 +------------
>>  .../mach-socfpga/include/mach/clock_manager_gen5.h | 322 +++++++++++++
>>  5 files changed, 846 insertions(+), 811 deletions(-)
>>  create mode 100644 arch/arm/mach-socfpga/clock_manager_gen5.c
>>  create mode 100644 arch/arm/mach-socfpga/include/mach/clock_manager_gen5.h
>>
>
> Please add a -C when you run format-patch, you'll get something like this:
>
>  5 files changed, 104 insertions(+), 947 deletions(-)
>  copy arch/arm/mach-socfpga/{clock_manager.c => clock_manager_gen5.c} (88%)
>  copy arch/arm/mach-socfpga/include/mach/{clock_manager.h =>
> clock_manager_gen5.h} (87%)
>
> It it make things a bit smaller to review for simple file copies.
>
Okay, will do it next revision.
Thanks.

Regards
Ley Foon


More information about the U-Boot mailing list