From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Warren Date: Thu, 30 Apr 2020 16:52:28 -0600 Subject: [PATCH 8/8] net: gmac_rockchip: Add dwc_eth_qos support In-Reply-To: <20200430104527.31248-1-david.wu@rock-chips.com> References: <20200430103656.29728-1-david.wu@rock-chips.com> <20200430104527.31248-1-david.wu@rock-chips.com> Message-ID: <017cf400-4866-dfbd-023c-73451a7baafb@wwwdotorg.org> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 4/30/20 4:45 AM, David Wu wrote: > Change the original data structure so that Rockchip's Soc > gmac controller can support the designware.c and dwc_eth_qos.c > drivers, a Soc can only support one. I'm really confused; with a filename like gmac_rockchip.c that sounds like it's driver for a MAC device. DWC EQoS is also a MAC device. The two shouldn't be related or coupled in any way. I think what you need is to completely drop this patch (and the patch which creates dwc_eth_qos.h), and instead make the DWC EQoS driver itself directly support the Rockchip SoC by adding RK's compatible value to the list of compatible values that the EQoS driver supports, along with new probe functions etc. Maybe this requires splitting some PHY code out of gmac_rockchip into a common/separate PHY driver? I haven't looked at the code to know if that's required.