[U-Boot] [PATCH] inka4x0: Add hardware diagnosis functions for inka4x0

Heiko Schocher hs at denx.de
Wed Mar 25 13:16:54 CET 2009


Hello Detlev,

Detlev Zundel wrote:
> Hi Heiko,
>
>   
>>> +#define GPIO_BASE  (u_char *)0x30400000
>>>   
>>>       
>> should be
>>
>> #define GPIO_BASE  (u_char *)CONFIG_SYS_CS3_START
>>     
>
> Fixed, thanks.
>
>   
>>> +#define SERIAL_PORT_BASE	(u_char *)0x80000000
>>>   
>>>       
>> this should be
>>
>> #define SERIAL_PORT_BASE	(u_char *)CONFIG_SYS_CS2_START
>>     
>
> Fixed, thanks.
>
>
> [...]
>
>   
>>> +#define UART_MSR_DCD	0x80	/* Data Carrier Detect */
>>> +#define UART_MSR_DSR	0x20	/* Data Set Ready */
>>> +#define UART_MSR_CTS	0x10	/* Clear to Send */
>>>   
>>>       
>> Are this not common 16xxx UART registers? If so, I think, they
>> should go in include/ns16550.h ...
>>     
>
> Ok, done.
>
> I'm glad I posted this series of patches again with my name added.  On
> the previous posting, the code unfortunately did not get this quality
> review ;)
>   

Think that has nothing to do with your name, more with time ;-)

bye
Heiko

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany 



More information about the U-Boot mailing list