On Tue, Feb 25, 2020 at 04:36:32PM +0000, Ioana Ciornei wrote:

quoted

Subject: [PATCH net-next 5/8] net: dpaa2-mac: use resolved link config in
mac_link_up()

Convert the DPAA2 ethernet driver to use the finalised link parameters in
mac_link_up() rather than the parameters in mac_config(), which are more
suited to the needs of the DPAA2 MC firmware than those available via
mac_config().

Signed-off-by: Russell King <redacted>

Tested-by: Ioana Ciornei <ioana.ciornei@nxp.com>

Thanks.

quoted

+
+		/* This is lossy; the firmware really should take the pause
+		 * enablement status rather than pause/asym pause status.
+		 */

In what sense it's lossy? I cannot see how information can be lost by translating the rx/tx pause state to pause/asym.
If it's just about the unnecessary double translation, then I agree.. this could have been done in an easier manner.

If you're just translating rx/tx to pause/asym and then doing the
reverse, then it isn't lossy, but if the firmware is resolving
pause/asym according to the table in IEEE 802.3, then it will be
lossy.

If the firmware doesn't interpret the bits, then why not do the
sensible thing and just pass the enablement status rather than
trying to confusingly encode it back to pause/asym?

-- 
RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
FTTC broadband for 0.8mile line in suburbia: sync at 12.1Mbps down 622kbps up
According to speedtest.net: 11.9Mbps down 500kbps up

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel