From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marek Vasut Date: Sun, 24 Jan 2021 15:52:45 +0100 Subject: [PATCH] net: fec: Get rid of FEC_ENET_ENABLE_[TR]XC_DELAY In-Reply-To: References: <57e65490-aaa1-5886-ceba-c3c5ac63fab2@denx.de> <0932936a-9870-d4da-e946-b5d379611abc@denx.de> Message-ID: <208a762f-3d41-a7f2-f21f-01589ebd27d8@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 1/24/21 3:35 PM, Oleksandr Suvorov wrote: > On Sun, Jan 24, 2021 at 12:04 AM Stefano Babic wrote: >> >> Hi Marek, >> >> On 23.01.21 20:57, Marek Vasut wrote: >>> On 1/23/21 8:54 PM, sbabic at denx.de wrote: >>>>> The FEC_ENET_ENABLE_.XC_DELAY setting is only used by one system which >>>>> supports OF control, correctly set the rgmii-id PHY mode in the DT and >>>>> then determine whether or not to enable the FEC RXC/TXC internal delay >>>>> based on the PHY mode. >>> >>> [...] >>> >>>> Applied to u-boot-imx, master, thanks ! >>> >>> Didn't this fail to build on mx25 ? >> >> Yes, sorry, this is a false positive - patch has not been applied. > > Anyway, as there will be another solution that won't break mx25/mx35, > I should retest the new patch version on apalis, so I'm calling off my > review so far. The only change required to that patch is an extra ifdef DM_ETH, so for mx8 apalis nothing changed, since you have DM_ETH enabled.