[U-Boot] [PATCH v3 2/6] arm: atmel: sama5d3: correct the error define of DIV

Bo Shen voice.shen at atmel.com
Thu Nov 14 07:31:24 CET 2013


Hi Andreas,

On 11/13/2013 08:20 PM, Andreas Bießmann wrote:
> Hi Bo,
>
> On 11/06/2013 06:29 AM, Bo Shen wrote:
>> Correct the error define of DIV.
>>
>> Signed-off-by: Bo Shen <voice.shen at atmel.com>
>>
>> ---
>> Changes in v3:
>>    - None
>>
>> Changes in v2:
>>    - None
>>
>>   arch/arm/include/asm/arch-at91/at91_pmc.h |    4 ++--
>>   1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/arch/arm/include/asm/arch-at91/at91_pmc.h b/arch/arm/include/asm/arch-at91/at91_pmc.h
>> index 003920c..ed5462c 100644
>> --- a/arch/arm/include/asm/arch-at91/at91_pmc.h
>> +++ b/arch/arm/include/asm/arch-at91/at91_pmc.h
>> @@ -124,8 +124,8 @@ typedef struct at91_pmc {
>>   #define AT91_PMC_MCKR_MDIV_MASK		0x00000300
>>   #endif
>>
>> -#define AT91_PMC_MCKR_PLLADIV_1		0x00001000
>> -#define AT91_PMC_MCKR_PLLADIV_2		0x00002000
>> +#define AT91_PMC_MCKR_PLLADIV_1		0x00000000
>> +#define AT91_PMC_MCKR_PLLADIV_2		0x00001000
>
> this change will touch pm9261 board. I dunno if your change is correct
> for that board. I also wonder why one should set bit position 13 in MCKR
> ... I can't find any source what that bit could be. However mature
> documentation for sam9261 (which is the SoC on pm9261) doesn't name bit
> position 12 either.
>
> So I'm fine with your change, but please fix the pm9261 board also (use
> the PLLADIV_2 define).

Ok, thanks.

>>
>>   #define AT91_PMC_IXR_MOSCS		0x00000001
>>   #define AT91_PMC_IXR_LOCKA		0x00000002
>>
>
> Best regards
>
> Andreas Bießmann
>

Best Regards,
Bo Shen



More information about the U-Boot mailing list