[PATCH v5 4/5] net: ethernet: cpsw: add support for hardware interface mode config
Sergei Shtylyov
sergei.shtylyov at cogentembedded.com
Fri Aug 23 15:18:05 EDT 2013
On 08/23/2013 10:53 PM, Daniel Mack wrote:
> The cpsw currently lacks code to properly set up the hardware interface
> mode on AM33xx. Other platforms might be equally affected.
> Usually, the bootloader will configure the control module register, so
> probably that's why such support wasn't needed in the past. In suspend
> mode though, this register is modified, and so it needs reprogramming
> after resume.
> This patch adds code that makes use of the previously added and optional
> support for passing the control mode register, and configures the
> correct register bits when the slave is opened.
> The AM33xx also has a bit for each slave to configure the RMII reference
> clock direction. Setting it is now supported by a per-slave DT property.
> This code path introducted by this patch is currently exclusive for
> am33xx.
> Signed-off-by: Daniel Mack <zonque at gmail.com>
[...]
> @@ -40,4 +41,11 @@ struct cpsw_platform_data {
> u32 hw_type; /* hardware type as specified in 'compatible' */
> };
>
> +/* SoC specific definitions for the CONTROL port */
> +#define AM33XX_GMII_SEL_MODE_MII (0)
> +#define AM33XX_GMII_SEL_MODE_RMII (1)
> +#define AM33XX_GMII_SEL_MODE_RGMII (2)
Parens around decimal literals are hardly needed even in macros.
WBR, Sergei
More information about the linux-arm-kernel
mailing list