Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Subject: Re: [PATCH net-next v3 12/47] net: phy: aquantia: Add support for
 AQR115
To:     Andrew Lunn <andrew@lunn.ch>
Cc:     "David S . Miller" <davem@davemloft.net>,
        Jakub Kicinski <kuba@kernel.org>,
        Madalin Bucur <madalin.bucur@nxp.com>, netdev@vger.kernel.org,
        Paolo Abeni <pabeni@redhat.com>,
        Eric Dumazet <edumazet@google.com>,
        linux-arm-kernel@lists.infradead.org,
        Russell King <linux@armlinux.org.uk>,
        linux-kernel@vger.kernel.org,
        Alexandru Marginean <alexandru.marginean@nxp.com>,
        Heiner Kallweit <hkallweit1@gmail.com>,
        Vladimir Oltean <olteanv@gmail.com>
References: <20220715215954.1449214-1-sean.anderson@seco.com>
 <20220715215954.1449214-13-sean.anderson@seco.com> <YtMAvHXuI76cTLAj@lunn.ch>
From:   Sean Anderson <sean.anderson@seco.com>
Message-ID: <afb79b32-5dfd-d1fa-130e-9d442c4b6100@seco.com>
Date:   Sat, 16 Jul 2022 18:42:27 -0400
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101
 Thunderbird/68.12.0
In-Reply-To: <YtMAvHXuI76cTLAj@lunn.ch>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Language: en-US
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?VldSSnVOc0dFZTM3aE5CNVlGNllhcUpsbkNVVXJxdmJMcndYRDFmZVgxN1g1?=
 =?utf-8?B?L3JZRFk1YWdTbmdUWDNDZkxNZ25NTGJWY3JTai9VaURJU1Q4dVNEMlgwREhT?=
 =?utf-8?B?UzlYOG1LRmpQaktUTFdaQi93RWlQVkhSVjNYQU1VZ0Y2VXFoeG51MGJsOXNT?=
 =?utf-8?B?Wm9aUzB5R1N2T3hobERpc2hNV2gwTnVNV1BvS0lKczNDQnFvZmxZUnVWSzVr?=
 =?utf-8?B?a2x1QWFiQWEvQkM4Sm9vUDlHSTRXcmYzOVp0OXE4UGQrbGFLSW54WnFCWk1P?=
 =?utf-8?B?TUc0NXQxNHlBRE9udmFLQklYY2RUbWE2Q202a3VrU3JCN3p3SFI0V0VpeWM1?=
 =?utf-8?B?YlpWS1B5R3Z3Y3JGZnc1MHo1WkR4bUdBai9wSlNvWWY2U1piUEYzV2hFajRQ?=
 =?utf-8?B?UHJxK3o1b05jRjU1ajdOZFVPbEd5YlJGZm9kWE4yR20xRGtqMk1lbE0yWW4r?=
 =?utf-8?B?K0MyYmlUYlg5NmIwTTBjRHY3dUg4WFJWSXhXb3BIYTlJWExhTVp2L0dBSHhE?=
 =?utf-8?B?VXNBRUtyazJpRURFVnphRnRVYUZGTitsWm4zc3RNR3F1aHR2aThoSUpyQU1v?=
 =?utf-8?B?RFhKWVN5Q2NRd1lZWVNnMUNmeEVSOXE1U1ZIdFNNT1Y4Wm5OdThSNXJCZUZy?=
 =?utf-8?B?ZTNHMTN5ZFdvRElMa2pOUWlTNWZ5MFU0SEg1QnVOaXN1d2todlhHUlRsRCtu?=
 =?utf-8?B?UE5SaEthM1d2M3VMQ2FJVEJyWWZpZUpIcUh1aE9nQVFGWGM2MHN1Z1ZuVGxS?=
 =?utf-8?B?WkNQU0V3SlBFMWtiOHpGWmxUYk5TbjhhK1FqZzNDNXRVTlMxbTVSdklQckJD?=
 =?utf-8?B?NUxZT3dBQ2hTRmk3TnVNOGJZM1FzTXI4QThHWng2K0tzdU1MTlRybldaS1BQ?=
 =?utf-8?B?NGpFNzVDUkJoSVlGdzh0WXNzYThBMlhFRHhubWFJSW1PSEpXVmtaeXZkK0dL?=
 =?utf-8?B?aHAzWHhPbFA5M3RpMkJBRnA1enh3Q1JZK1A3TzRCY0htVm9iWEFtSjVsZ280?=
 =?utf-8?B?UldJYjg0SFo0VCtLTW1rRTV0S0ZoY0luMVg3eDhWVXhVMXZ1WGgrNjQvUUM5?=
 =?utf-8?B?bWJDb3BWeCsxVjA2ZGtjMEZVZFFlb3lPUWlrZjY3ZWhIMkJFYWxjRXhPRVBi?=
 =?utf-8?B?NGpRUmRKcmR4UTVaSXc1U2UyNlNKNGtXNkRURzFPcWxKMnhpbTcwY0x3WDF3?=
 =?utf-8?B?eFhHdXYva0dVdHd6RDVKN3RLSXVGYjRZcjUzb2plSHRLZW0yWmRRaWFQcjNC?=
 =?utf-8?B?T1VXOTVtZ2VhQ2ttMjFnemFFWlZiczFEUno2NnMycC9HVkNhV2pJbXRrWU9z?=
 =?utf-8?B?LzkzSnBTYjQ2Nldrd2FUL3FNckJwK01wZVRIdndzZGs1cTVHaFluTHQ0QXc2?=
 =?utf-8?B?cmI4eDNlU2t3UCtIQUNoUTZQZzBDNHFWQXlxSWNiTEtIV0ltZXIwbytuWmx5?=
 =?utf-8?B?bXdSNE40WSswaFhxbzAzeTczS25BNjk0R0NCR2lrbjArQVloUm4xbEtHVDVK?=
 =?utf-8?B?dkttSEpjcE16eHl3Q3Y4WWo3eE9DZU5oVGNRc0dmRmRqenBxYzZiY1pBUk1I?=
 =?utf-8?B?RTFvdEozTjQ1dUFkNVBQamFpREZjV3lpVlR4dTBPNGU3anVpbWV5bmdLZW1H?=
 =?utf-8?B?MHJEaEJQYUtkT3RPdUFQQUR2QjdINVFpam5rL1ZEeG0zYW9IQXdSSEgreGlW?=
 =?utf-8?B?T2NxTmh0cUJwenoxQWVDN242ODhLMnBWVFJwMm1mdWtnK1ZoTy9nc2lmWkVM?=
 =?utf-8?B?OEdnN3FLNS9SNE9ERWZzMEZySnZnS0VtN3FxNStuaHlzTXFwcytFMVJMbHp2?=
 =?utf-8?B?dUNCaVJ1cFFRa1B0OTlVWmlkc1RXVTArKytNdWxjcVNmbnBpQittSFlFRk5s?=
 =?utf-8?B?UFQrb2E1VHBScWdabmYyVWw3ak5GdTA3QnJ0ZEg1Z0V3ZG5FOUcweTBEWnQ5?=
 =?utf-8?B?L01tbXNqcU5naHhpb2ppU29xTnNqa0dDVXR6cExEUCtlQStEU3l1TUVocHFU?=
 =?utf-8?B?Yi8rYk9jMTI5MXF2ZEs3bWtrdy9zR1lmajlUQWdGU2lWd0VzdEhabHZ3K0U3?=
 =?utf-8?B?LytJNEFXOHlDa3RZNGZrdnJ5Ynd3RU9NNmx6ZytJVGdFTU8yRExxOHYxU015?=
 =?utf-8?Q?R1BLKsM6Jvw8vtKvzFKsIvZnw?=
X-OriginatorOrg: seco.com
X-MS-Exchange-CrossTenant-Network-Message-Id: badb141a-7fc2-4c62-f720-08da677c7771
X-MS-Exchange-CrossTenant-AuthSource: DB7PR03MB4972.eurprd03.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 16 Jul 2022 22:42:30.5315
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: bebe97c3-6438-442e-ade3-ff17aa50e733
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: ngHbHSgu76+pQr3++n4LLngj1NcNWBYJsCsd6Vgw+0SHrYxgp6P9MSIrVM5B2nJ5VH/ADs6g2A7QL5WYxbKP3g==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM6PR03MB5541
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,RCVD_IN_DNSWL_NONE,
        RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS autolearn=ham
        autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 7/16/22 2:17 PM, Andrew Lunn wrote:
> On Fri, Jul 15, 2022 at 05:59:19PM -0400, Sean Anderson wrote:
>> This adds support for the AQR115 (which I have on my LS1046A RDB). I had a
>> quick look over the registers, and it seems to be compatible with the
>> AQR107. I couldn't find this oui anywhere, but that's what I have on my
>> board. It's possible that NXP used a substitute here; I can't confirm
>> the part number since there is a heatsink on top of the phy.
> 
> If i remember correctly, the OUI can be part of the provisioning for
> Aquantia PHYs. And i think there is often per board provisioning,
> specially for the SERDEs configuration. So aQuantia/Marvell probably
> set this OUI, but maybe at NXP request.

Ah, interesting.

> Did you get the part number from the schematic? That should be enough
> to confirm it is a AQR115.

Yes, I got it off the schematic.

>> To avoid breaking <10G ethernet on the LS1046ARDB, we must add this
>> vendor id as an exception to dpaa_phy_init. This will be removed once
>> the DPAA driver is converted to phylink.
> 
> I suggest you split this into two. The PHY changes can be merged right
> away, and is independent of the DPAA. 

The DPAA changes must be merged before the phy changes. At the moment,
sub-10G ethernet still works on the LS1046ARDB. This is because even though
we program an advertisement of only 10G link modes, the phy by default
ignores the programmed advertisement. But by adding a driver for this phy,
the 10G-only advertisement will take effect and no link will be established.
So the DPAA change must come before the phy change.

Since there is no harm, I will split the DPAA change into its own patch and
place it before this one.

> Given the size of this patchset,
> the more you can split it up into parallel submissions the better. So
> please submit the PHY patches independent of the rest.

Yes, that is the strategy outlined in the cover letter.

>> Signed-off-by: Sean Anderson <sean.anderson@seco.com>
> 
> For the aquantia_main.c change only:
> 
> Reviewed-by: Andrew Lunn <andrew@lunn.ch>
> 
>      Andrew
> 

--Sean