All of lore.kernel.org
 help / color / mirror / Atom feed
From: Samuel Holland <samuel@sholland.org>
To: Andre Przywara <andre.przywara@arm.com>,
	Jagan Teki <jagan@amarulasolutions.com>
Cc: u-boot@lists.denx.de, Simon Glass <sjg@chromium.org>,
	Tom Rini <trini@konsulko.com>,
	Jernej Skrabec <jernej.skrabec@gmail.com>
Subject: Re: [PATCH 2/2] sunxi: dts: Update RGMII phy-mode properties
Date: Fri, 18 Mar 2022 00:31:15 -0500	[thread overview]
Message-ID: <8a0fb350-7536-38df-814e-be6eb90f29aa@sholland.org> (raw)
In-Reply-To: <20220316005443.16260-3-andre.przywara@arm.com>

On 3/15/22 7:54 PM, Andre Przywara wrote:
> Commit f11513d99787 ("net: phy: realtek: Add tx/rx delay config for
> 8211e") made the Realtek PHY driver honour the phy-mode DT property,
> to set up the proper delay scheme for the RX and TX lines. A similar
> change in the kernel revealed that those properties were mostly wrong.
> The kernel DTs got updated over the last few months, but we were missing
> out on the U-Boot version.
> 
> Just sync in the phy-mode properties from the mainline kernel,
> v5.17-rc7, to avoid the breaking DT sync that late in the cycle.
> 
> This fixes Ethernet operation on the affected boards.
> 
> Signed-off-by: Andre Przywara <andre.przywara@arm.com>

Reviewed-by: Samuel Holland <samuel@sholland.org>

Verified against the updated files from Linux. I suppose I broke this by not
sending a DT sync series during the same release cycle. Hopefully we can get DT
syncing unblocked soon and merged for the next release.

Regards,
Samuel

      reply	other threads:[~2022-03-18  5:31 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-03-16  0:54 [PATCH 0/2] sunxi: Fix Ethernet on mostly A20 boards Andre Przywara
2022-03-16  0:54 ` [PATCH 1/2] sunxi: Fix old GMAC pinmux setup Andre Przywara
2022-03-16 16:55   ` Jernej Škrabec
2022-03-16 17:04     ` Andre Przywara
2022-03-21 14:17   ` Petr Štetiar
2022-03-16  0:54 ` [PATCH 2/2] sunxi: dts: Update RGMII phy-mode properties Andre Przywara
2022-03-18  5:31   ` Samuel Holland [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=8a0fb350-7536-38df-814e-be6eb90f29aa@sholland.org \
    --to=samuel@sholland.org \
    --cc=andre.przywara@arm.com \
    --cc=jagan@amarulasolutions.com \
    --cc=jernej.skrabec@gmail.com \
    --cc=sjg@chromium.org \
    --cc=trini@konsulko.com \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.