> On 28 Jul 2017, at 05:39, Simon Glass <s...@chromium.org> wrote:
>
> Hi Philipp,
>
> On 26 July 2017 at 04:40, Philipp Tomsich
> <philipp.toms...@theobroma-systems.com> wrote:
>> The GMAC in the RK3368 once again is identical to the incarnation in
>> the RK3288 and the RK3399, except for where some of the configuration
>> and control registers are located in the GRF.
>>
>> This adds the RK3368-specific logic necessary to reuse this driver.
>>
>> Signed-off-by: Philipp Tomsich <philipp.toms...@theobroma-systems.com>
>>
>> ---
>>
>> Changes in v2:
>> - moves the bit-definitions (for GRF registers) specific to the GMAC
>> into the GMAC driver: these include interface speed selection and
>> configuring the RGMII output mode
>>
>> arch/arm/include/asm/arch-rockchip/grf_rk3368.h | 1 +
>> drivers/net/gmac_rockchip.c | 78
>> +++++++++++++++++++++++++
>> 2 files changed, 79 insertions(+)
>>
>> diff --git a/arch/arm/include/asm/arch-rockchip/grf_rk3368.h
>> b/arch/arm/include/asm/arch-rockchip/grf_rk3368.h
>> index ec42d7a..1f84ff9 100644
>> --- a/arch/arm/include/asm/arch-rockchip/grf_rk3368.h
>> +++ b/arch/arm/include/asm/arch-rockchip/grf_rk3368.h
>> @@ -129,4 +129,5 @@ enum {
>> MCU_CODE_BASE_BIT31_BIT28_SHIFT = 0,
>> MCU_CODE_BASE_BIT31_BIT28_MASK = GENMASK(3, 0),
>> };
>> +
>> #endif
>> diff --git a/drivers/net/gmac_rockchip.c b/drivers/net/gmac_rockchip.c
>> index c9f9e83..586ccbf 100644
>> --- a/drivers/net/gmac_rockchip.c
>> +++ b/drivers/net/gmac_rockchip.c
>> @@ -16,6 +16,7 @@
>> #include <asm/arch/clock.h>
>> #include <asm/arch/hardware.h>
>> #include <asm/arch/grf_rk3288.h>
>> +#include <asm/arch/grf_rk3368.h>
>> #include <asm/arch/grf_rk3399.h>
>> #include <dm/pinctrl.h>
>> #include <dt-bindings/clock/rk3288-cru.h>
>> @@ -83,6 +84,38 @@ static int rk3288_gmac_fix_mac_speed(struct dw_eth_dev
>> *priv)
>> return 0;
>> }
>>
>> +static int rk3368_gmac_fix_mac_speed(struct dw_eth_dev *priv)
>
> What is the plan here? Can we use a ioctl() interface to the grf
> syscon perhaps, to remove this SoC-specific code from the driver?
Indeed. This is the longer-term plan.
Regards,
Philipp.
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot
