[U-Boot] [PATCH 01/11 v2] drivers/net/vsc9953: Cleanup patch
Codrin Constantin Ciubotariu
codrin.ciubotariu at freescale.com
Thu Jul 2 16:32:01 CEST 2015
Hi Joe,
> > -#define CONFIG_VSC9953_PORT_ENA 0x00003a00
>
> Why is this value changing? Was it just wrong before?
Sorry, I missed this comment. Yes, the correct value that only enables the port is 0x00002000. The rest of the bits are set to 1 because they are reserved and default values. Since this patch changes the way l2qsys_reg->sys.switch_port_mode[i] is set (out_le32() replaced by setbits_le32()), the other bits are no longer touched when enabling the port:
- out_le32(&l2qsys_reg->sys.switch_port_mode[i],
- CONFIG_VSC9953_PORT_ENA);
+ setbits_le32(&l2qsys_reg->sys.switch_port_mode[i],
+ CONFIG_VSC9953_PORT_ENA)
Should I make a different patch for this change?
Best regards,
Codrin
More information about the U-Boot
mailing list