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;
From:   "Katakam, Harini" <harini.katakam@amd.com>
To:     Andrew Lunn <andrew@lunn.ch>
CC:     "robh+dt@kernel.org" <robh+dt@kernel.org>,
        "hkallweit1@gmail.com" <hkallweit1@gmail.com>,
        "linux@armlinux.org.uk" <linux@armlinux.org.uk>,
        "davem@davemloft.net" <davem@davemloft.net>,
        "kuba@kernel.org" <kuba@kernel.org>,
        "edumazet@google.com" <edumazet@google.com>,
        "pabeni@redhat.com" <pabeni@redhat.com>,
        "vladimir.oltean@nxp.com" <vladimir.oltean@nxp.com>,
        "wsa+renesas@sang-engineering.com" <wsa+renesas@sang-engineering.com>,
        "krzysztof.kozlowski+dt@linaro.org" 
        <krzysztof.kozlowski+dt@linaro.org>,
        "simon.horman@corigine.com" <simon.horman@corigine.com>,
        "netdev@vger.kernel.org" <netdev@vger.kernel.org>,
        "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "harinikatakamlinux@gmail.com" <harinikatakamlinux@gmail.com>,
        "Simek, Michal" <michal.simek@amd.com>,
        "Pandey, Radhey Shyam" <radhey.shyam.pandey@amd.com>
Subject: RE: [PATCH net-next v2 1/3] phy: mscc: Use PHY_ID_MATCH_VENDOR to
 minimize PHY ID table
Thread-Topic: [PATCH net-next v2 1/3] phy: mscc: Use PHY_ID_MATCH_VENDOR to
 minimize PHY ID table
Thread-Index: AQHZeCv01AQKu6wCIUmTin6SYmuMta89mNcAgAA/MSA=
Date:   Wed, 26 Apr 2023 17:27:51 +0000
Message-ID: <BYAPR12MB47737F393790808D3A90A07D9E659@BYAPR12MB4773.namprd12.prod.outlook.com>
References: <20230426104313.28950-1-harini.katakam@amd.com>
 <20230426104313.28950-2-harini.katakam@amd.com>
 <c120d1bc-4b67-4280-a843-d269fdb27e20@lunn.ch>
In-Reply-To: <c120d1bc-4b67-4280-a843-d269fdb27e20@lunn.ch>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?UP6rrwQdioNd6bnB1DqwmbX3HT7GZYi4jYnoUvS965D/Dj8p8uI1ssBYEnpH?=
 =?us-ascii?Q?8kAsetX6vgB3m0ZW0xkJvrIwKdl3VsY/Oe4cBtgBFAHPQp5aDDzjLBHpTSMy?=
 =?us-ascii?Q?GPcBzjZOzJY37Tc1rH5rgSpIn8vsdf8QNq2oAo25eeD1QdxzfnRrGGNzx/aY?=
 =?us-ascii?Q?R7FDAYHlSAZLL3SJfTc334FYFokGEeKsOWH+O9p5enH7P907k9wocE3/AAve?=
 =?us-ascii?Q?RZOonTSs2TK1xl2TOrzhqgjYRoba4o+bv7H80xUUDiD8lBsYmc7oVWaXVGg3?=
 =?us-ascii?Q?J+2/titr/XQxxQ5VIF82/Xu41CnRS28LOJjse6mkGgbFEGqWCeQ8fl+CZ1Gv?=
 =?us-ascii?Q?JI2lyaL/HcnDo7W6qJxlHF4CpJOATYRnPCbfd1MA3fRCM+oAtw8kjuzxtQVF?=
 =?us-ascii?Q?ilmQFq9cRLQWX6ImkP2AGR4k+buXAVUUsVkG5dHGzJkc1ZO8vtiiEpDkU0pr?=
 =?us-ascii?Q?TQdj8h9Vynb+c9/MDrEba6xlEP0CkSlsoYE1GGPM6Lkz5plFBAAvBTsEchjG?=
 =?us-ascii?Q?xf3R59DtsfXqj3HHYQ4H2TGcC1sxMtr4xClu4Kr6zGfKPP4xH7zcjK4zNBG2?=
 =?us-ascii?Q?iL3bhOdOJf3E9233wo4TEgZNP0toZbDPdRlGVdDUJ2DcALWM7AGcqFrSeXpb?=
 =?us-ascii?Q?Q0VuINbCxzapCgpxMlzhU3ZCy4Y0KrKQziWc8biJf8YfZhAvXJs5dj0W0lSu?=
 =?us-ascii?Q?IAXBBc47A0fATuuFlIKYtgAO+bXiIHCVq4u29NeH5o6xRZTTGZJkguN8z78A?=
 =?us-ascii?Q?9+hYUpHZGp+gfYHliJkg2Z6j2SUdr8ArHne9YuxfMd+WG6yIvb0uG6VN9dVc?=
 =?us-ascii?Q?Vnmg9Kn9zkrpBmxwXzZ0j6QZgjHnUgLldNsBo9u+AhxhVHs1bJKwHGrO35Ie?=
 =?us-ascii?Q?fccYbdfAIbcply0X9g7Q6E3Jp59R4VrWNmo/FNOwwRbBWucji/PhMphUefio?=
 =?us-ascii?Q?A1S7ZHQOREMujWfkj3KTTDU1h/mmzInVKwo/7bjFIaJ0WDEBv6Ctscmp5mTS?=
 =?us-ascii?Q?giMC7vMzBX58cKvVV64FFOP9qJNjLSSpdSHoa9yFryfJpnAz678q8gg3gZlH?=
 =?us-ascii?Q?tY0JU+8C+sPNTx5Zrmf4mqGKIgz2rHo0U2j3bYjnASxHX9y9ebJHGwSJvpTs?=
 =?us-ascii?Q?hUZSuD5AXkKwZ7eSwypWc4pPudGuaCpSa1/U+xqeApQuIQxgjJgNsUZ289G8?=
 =?us-ascii?Q?zb4YFzodUOZulyjxEr+rH8o8R9TD4qNrSJ6xFctjwkj/FOCO4wq3X1fX7JYz?=
 =?us-ascii?Q?mH8N0RYC7CgnCvGN/NxRo4UBo8jqmOp0UPyRK53ZLGeZ2csJ7kS5zc9YSDah?=
 =?us-ascii?Q?fFY3BDWp/GwhCC+pNbrvZW6d8gy/YifFpr/FRr82sqerEXnbHIaptyGh1OU6?=
 =?us-ascii?Q?03rmNDuZJpt85+WE6sLxsr2XFoMAQW2NfMWDQTz+mBwSIaEzweO9bAdXc8uw?=
 =?us-ascii?Q?inoXIa4z1p30LEA5CyNThZw4PSzFUq85an+j7CeRXvNbDT5tdPfiM3nSFlE+?=
 =?us-ascii?Q?65Tznz2lFYizLi8NG7Yrov4HDNdEPh1nMOCv7E/UF8P94RG4Cn0GdzJqM1G5?=
 =?us-ascii?Q?ZNAEVmeLl9Il10YCLfo=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: BYAPR12MB4773.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 436d7364-c3ea-41c2-b237-08db467b8fdb
X-MS-Exchange-CrossTenant-originalarrivaltime: 26 Apr 2023 17:27:51.1276
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 3dd8961f-e488-4e60-8e11-a82d994e183d
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: R7jbl9iZc2ecoJncqNS3Qi7IrGukk4O+KaQOQMW5bivfIJiPgTFeIS3w0uZASesM
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DS0PR12MB8041
X-Spam-Status: No, score=-1.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FORGED_SPF_HELO,
        RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_NONE,
        T_SCC_BODY_TEXT_LINE,URIBL_BLOCKED autolearn=no 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

Hi Andrew,

> -----Original Message-----
> From: Andrew Lunn <andrew@lunn.ch>
> Sent: Wednesday, April 26, 2023 7:10 PM
> To: Katakam, Harini <harini.katakam@amd.com>
> Cc: robh+dt@kernel.org; hkallweit1@gmail.com; linux@armlinux.org.uk;
> davem@davemloft.net; kuba@kernel.org; edumazet@google.com;
> pabeni@redhat.com; vladimir.oltean@nxp.com; wsa+renesas@sang-
> engineering.com; krzysztof.kozlowski+dt@linaro.org;
> simon.horman@corigine.com; netdev@vger.kernel.org;
> devicetree@vger.kernel.org; linux-kernel@vger.kernel.org;
> harinikatakamlinux@gmail.com; Simek, Michal <michal.simek@amd.com>;
> Pandey, Radhey Shyam <radhey.shyam.pandey@amd.com>
> Subject: Re: [PATCH net-next v2 1/3] phy: mscc: Use
> PHY_ID_MATCH_VENDOR to minimize PHY ID table
>=20
> On Wed, Apr 26, 2023 at 04:13:11PM +0530, Harini Katakam wrote:
> > All the PHY devices variants specified have the same mask and hence
> > can be simplified to one vendor look up for 0xfffffff0.
> > Any individual config can be identified by PHY_ID_MATCH_EXACT in the
> > respective structure.
> >
> > Signed-off-by: Harini Katakam <harini.katakam@amd.com>
>=20
<snip>
> > +	{ PHY_ID_MATCH_VENDOR(0xfffffff0) },
>=20
> The vendor ID is 0xfffffff0 ???

Sorry this is supposed to be 0x00070400 (Microsemi OUI).
Will fix in the next version.

Regards,
Harini