[U-Boot] [PATCH] net: lpc32xx: correct command register reset value

Albert ARIBAUD albert.aribaud at 3adev.fr
Mon Jun 29 10:25:30 CEST 2015


Bonjour Vladimir,

Le Sun, 28 Jun 2015 06:03:38 +0300, Vladimir Zapolskiy <vz at mleia.com> a
écrit :

> According to LPC32x0 User Manual the following bits in Command
> register 0x3106_0100 are defined:
> 
>   Bit    Symbol
>     2  - Unused
>     3  RegReset
>     4   TxReset
>     5   RxReset
> 
> Fix wrong (1-bit shifted right) COMMAND_RESETS value, which sets
> an unused bit, but neglects RxReset.
> 
> Signed-off-by: Vladimir Zapolskiy <vz at mleia.com>
> ---
>  drivers/net/lpc32xx_eth.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/net/lpc32xx_eth.c b/drivers/net/lpc32xx_eth.c
> index 8dcbb4a..72451a9 100644
> --- a/drivers/net/lpc32xx_eth.c
> +++ b/drivers/net/lpc32xx_eth.c
> @@ -170,7 +170,7 @@ struct lpc32xx_eth_registers {
>  #define COMMAND_PASSRUNTFRAME 0x00000040
>  #define COMMAND_FULL_DUPLEX   0x00000400
>  /* Helper: general reset */
> -#define COMMAND_RESETS        0x0000001C
> +#define COMMAND_RESETS        0x00000038
>  
>  /* STATUS register bitfields/masks and offsets (see Table 283) */
>  #define STATUS_RXSTATUS 0x00000001

Acked-by: Albert ARIBAUD (3ADEV) <albert.aribaud at 3adev.fr>

Thanks for pointing this bug out!

Cordialement,
Albert ARIBAUD
3ADEV


More information about the U-Boot mailing list