[U-Boot] [PATCH v3 3/9] ARM: stm32: cleanup stm32f7 files

[Joe Hershberger]

On Thu, Dec 1, 2016 at 12:18 PM, vikas <vikas.manocha at st.com> wrote:
> Hi Michael,
>
> On 11/24/2016 11:10 AM, Michael Kurz wrote:
>> Cleanup stm32f7 files:
>> - use BIT macro
>> - use GENMASK macro
>
> good.
>
>> - use rcc struct instead of macro additions
>
> Macro definitions are better than struct to make rcc compatible throughout the stm32f7 family in case of additional registers and also to reuse it for stm32f4. At present we cant use same rcc struct for stm32f4 and stm32f7 because of two additional registers in stm32f7.
> So keep the macros for rcc, we would move it for both stm32f7 and stm32f4.

Just 2 extra regs, or they change the position of the existing regs?

> rcc struct shouldn't be in for stm32f7 in first place, the last patch which added it went unnoticed.
>
>>
>> Signed-off-by: Michael Kurz <michi.kurz at gmail.com>
>>
>> ---
>>
>> Changes in v3:
>> - Removed 'prefix all constants with STM32_'
>> - Reverted move of header into source file (rcc.h -> clock.c)
>>
>> Changes in v2:
>> - Add cleanup patch
>>
>>  arch/arm/include/asm/arch-stm32f7/fmc.h    |   6 +-
>>  arch/arm/include/asm/arch-stm32f7/gpt.h    |   6 +-
>>  arch/arm/include/asm/arch-stm32f7/rcc.h    |  50 ++++++----
>>  arch/arm/include/asm/arch-stm32f7/stm32.h  |   8 +-
>>  arch/arm/mach-stm32/stm32f7/clock.c        | 154 ++++++++++++-----------------
>>  board/st/stm32f746-disco/stm32f746-disco.c |   7 +-
>>  6 files changed, 107 insertions(+), 124 deletions(-)
>>
>> diff --git a/arch/arm/include/asm/arch-stm32f7/fmc.h b/arch/arm/include/asm/arch-stm32f7/fmc.h
>> index 7dd5077..d61a86f 100644
>> --- a/arch/arm/include/asm/arch-stm32f7/fmc.h
>> +++ b/arch/arm/include/asm/arch-stm32f7/fmc.h
>> @@ -58,12 +58,12 @@ struct stm32_fmc_regs {
>>  #define FMC_SDCMR_MODE_SELFREFRESH     5
>>  #define FMC_SDCMR_MODE_POWERDOWN       6
>>
>> -#define FMC_SDCMR_BANK_1               (1 << 4)
>> -#define FMC_SDCMR_BANK_2               (1 << 3)
>> +#define FMC_SDCMR_BANK_1               BIT(4)
>> +#define FMC_SDCMR_BANK_2               BIT(3)
>>
>
> [...]