[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