lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Message-ID: <e7a7c38c-ebe7-1cea-4099-3cf3a4483ac7@gmail.com> Date: Wed, 22 May 2019 22:18:32 +0200 From: Heiner Kallweit <hkallweit1@...il.com> To: Florian Fainelli <f.fainelli@...il.com>, Madalin-cristian Bucur <madalin.bucur@....com>, Shawn Guo <shawnguo@...nel.org>, Li Yang <leoyang.li@....com>, Rob Herring <robh+dt@...nel.org>, Mark Rutland <mark.rutland@....com>, Andrew Lunn <andrew@...n.ch>, David Miller <davem@...emloft.net> Cc: "netdev@...r.kernel.org" <netdev@...r.kernel.org>, "linux-arm-kernel@...ts.infradead.org" <linux-arm-kernel@...ts.infradead.org>, "devicetree@...r.kernel.org" <devicetree@...r.kernel.org> Subject: Re: [PATCH net-next 2/2] net: phy: aquantia: add USXGMII support On 22.05.2019 22:07, Florian Fainelli wrote: > On 5/22/19 12:58 PM, Heiner Kallweit wrote: >> So far we didn't support mode USXGMII, and in order to not break the >> two Freescale boards mode XGMII was accepted for the AQR107 family >> even though it doesn't support XGMII. Add USXGMII support to the >> Aquantia PHY driver and change the phy connection type for the two >> boards. >> >> As an additional note: Even though the handle is named aqr106 >> there seem to be LS1046A boards with an AQR107. >> >> Signed-off-by: Heiner Kallweit <hkallweit1@...il.com> > > You can probably split the DTS changes and the PHY driver changes into a > separate commits and just have the DTS changes come last? With that: > To split the patches I would have to do: 1. Add USXGMII support to Aquantia PHY driver 2. DTS changes 3. Don't accept XGMII any longer in Aquantia PHY driver This seemed to me to be too much overhead considering the very small change. Just making the DTS changes a separate patch would break bisecting. > Reviewed-by: Florian Fainelli <f.fainelli@...il.com> >
Powered by blists - more mailing lists