[PATCH v2 11/12] net: dwc_eth_qos_rockchip: Add support for RK3506

[Kever Yang]

On 2026/2/1 07:38, Jonas Karlman wrote:
> Rockchip RK3506 has two Ethernet controllers based on Synopsys DWC
> Ethernet QoS IP.
>
> Add initial support for the RK3506 GMAC variant.
>
> Signed-off-by: Jonas Karlman <jonas at kwiboo.se>
Reviewed-by: Kever Yang <kever.yang at rock-chips.com>

Thanks,
- Kever
> ---
> v2: No change
> ---
>   drivers/net/dwc_eth_qos.c          |  4 ++
>   drivers/net/dwc_eth_qos_rockchip.c | 86 ++++++++++++++++++++++++++++++
>   2 files changed, 90 insertions(+)
>
> diff --git a/drivers/net/dwc_eth_qos.c b/drivers/net/dwc_eth_qos.c
> index 8a396d0b29ea..0f31d6468456 100644
> --- a/drivers/net/dwc_eth_qos.c
> +++ b/drivers/net/dwc_eth_qos.c
> @@ -1620,6 +1620,10 @@ static const struct udevice_id eqos_ids[] = {
>   	},
>   #endif
>   #if IS_ENABLED(CONFIG_DWC_ETH_QOS_ROCKCHIP)
> +	{
> +		.compatible = "rockchip,rk3506-gmac",
> +		.data = (ulong)&eqos_rockchip_config
> +	},
>   	{
>   		.compatible = "rockchip,rk3528-gmac",
>   		.data = (ulong)&eqos_rockchip_config
> diff --git a/drivers/net/dwc_eth_qos_rockchip.c b/drivers/net/dwc_eth_qos_rockchip.c
> index d646d3ebac87..4ccefda0ef5e 100644
> --- a/drivers/net/dwc_eth_qos_rockchip.c
> +++ b/drivers/net/dwc_eth_qos_rockchip.c
> @@ -50,6 +50,80 @@ struct rockchip_platform_data {
>   	(((tx) ? soc##_GMAC_TXCLK_DLY_ENABLE : soc##_GMAC_TXCLK_DLY_DISABLE) | \
>   	 ((rx) ? soc##_GMAC_RXCLK_DLY_ENABLE : soc##_GMAC_RXCLK_DLY_DISABLE))
>   
> +#define RK3506_GRF_SOC_CON8		0x0020
> +#define RK3506_GRF_SOC_CON11		0x002c
> +
> +#define RK3506_GMAC_RMII_MODE		GRF_BIT(1)
> +
> +#define RK3506_GMAC_CLK_RMII_DIV2	GRF_BIT(3)
> +#define RK3506_GMAC_CLK_RMII_DIV20	GRF_CLR_BIT(3)
> +
> +#define RK3506_GMAC_CLK_SELECT_CRU	GRF_CLR_BIT(5)
> +#define RK3506_GMAC_CLK_SELECT_IO	GRF_BIT(5)
> +
> +#define RK3506_GMAC_CLK_RMII_GATE	GRF_BIT(2)
> +#define RK3506_GMAC_CLK_RMII_NOGATE	GRF_CLR_BIT(2)
> +
> +static int rk3506_set_to_rgmii(struct udevice *dev,
> +			       int tx_delay, int rx_delay)
> +{
> +	return -EINVAL;
> +}
> +
> +static int rk3506_set_to_rmii(struct udevice *dev)
> +{
> +	struct eth_pdata *pdata = dev_get_plat(dev);
> +	struct rockchip_platform_data *data = pdata->priv_pdata;
> +	u32 reg;
> +
> +	reg = data->id == 1 ? RK3506_GRF_SOC_CON11 :
> +			      RK3506_GRF_SOC_CON8;
> +	regmap_write(data->grf, reg, RK3506_GMAC_RMII_MODE);
> +
> +	return 0;
> +}
> +
> +static int rk3506_set_gmac_speed(struct udevice *dev)
> +{
> +	struct eqos_priv *eqos = dev_get_priv(dev);
> +	struct eth_pdata *pdata = dev_get_plat(dev);
> +	struct rockchip_platform_data *data = pdata->priv_pdata;
> +	u32 val, reg;
> +
> +	switch (eqos->phy->speed) {
> +	case SPEED_10:
> +		val = RK3506_GMAC_CLK_RMII_DIV20;
> +		break;
> +	case SPEED_100:
> +		val = RK3506_GMAC_CLK_RMII_DIV2;
> +		break;
> +	default:
> +		return -EINVAL;
> +	}
> +
> +	reg = data->id == 1 ? RK3506_GRF_SOC_CON11 :
> +			      RK3506_GRF_SOC_CON8;
> +	regmap_write(data->grf, reg, val);
> +
> +	return 0;
> +}
> +
> +static void rk3506_set_clock_selection(struct udevice *dev, bool enable)
> +{
> +	struct eth_pdata *pdata = dev_get_plat(dev);
> +	struct rockchip_platform_data *data = pdata->priv_pdata;
> +	u32 val, reg;
> +
> +	val = data->clock_input ? RK3506_GMAC_CLK_SELECT_IO :
> +				  RK3506_GMAC_CLK_SELECT_CRU;
> +	val |= enable ? RK3506_GMAC_CLK_RMII_NOGATE :
> +			RK3506_GMAC_CLK_RMII_GATE;
> +
> +	reg = data->id == 1 ? RK3506_GRF_SOC_CON11 :
> +			      RK3506_GRF_SOC_CON8;
> +	regmap_write(data->grf, reg, val);
> +}
> +
>   #define RK3528_VO_GRF_GMAC_CON		0x0018
>   #define RK3528_VPU_GRF_GMAC_CON5	0x0018
>   #define RK3528_VPU_GRF_GMAC_CON6	0x001c
> @@ -534,6 +608,18 @@ static void rk3588_set_clock_selection(struct udevice *dev, bool enable)
>   }
>   
>   static const struct rk_gmac_ops rk_gmac_ops[] = {
> +	{
> +		.compatible = "rockchip,rk3506-gmac",
> +		.set_to_rgmii = rk3506_set_to_rgmii,
> +		.set_to_rmii = rk3506_set_to_rmii,
> +		.set_gmac_speed = rk3506_set_gmac_speed,
> +		.set_clock_selection = rk3506_set_clock_selection,
> +		.regs = {
> +			0xff4c8000, /* gmac0 */
> +			0xff4d0000, /* gmac1 */
> +			0x0, /* sentinel */
> +		},
> +	},
>   	{
>   		.compatible = "rockchip,rk3528-gmac",
>   		.set_to_rgmii = rk3528_set_to_rgmii,