Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:6 as permitted sender) client-ip=2620:137:e000::3:6;
Date:   Sun, 3 Dec 2023 17:15:03 +0200
From:   Vladimir Oltean <vladimir.oltean@nxp.com>
To:     Bjorn Helgaas <helgaas@kernel.org>
Cc:     linux-pci@vger.kernel.org, Bjorn Helgaas <bhelgaas@google.com>,
        linux-kernel@vger.kernel.org, Wasim Khan <wasim.khan@nxp.com>,
        Wei Fang <wei.fang@nxp.com>,
        Clark Wang <xiaoning.wang@nxp.com>,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Subject: Re: [PATCH pci] PCI: remove the PCI_VENDOR_ID_NXP alias
Message-ID: <20231203151503.vgyy7kidk7xhlles@skbuf>
References: <20231122154241.1371647-1-vladimir.oltean@nxp.com>
 <20231129233827.GA444332@bhelgaas>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20231129233827.GA444332@bhelgaas>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?yw1QiRbsfUTeQd8QThBnBIMDLPC7bgMJCKp1Fco+FbzCZYn/XNcedoQSM9nZ?=
 =?us-ascii?Q?EQnl5H6fAPPh8lyIRpuI++5vlGIerEtw3hu0jIlmu4/daOkyndMZnsnNrJiH?=
 =?us-ascii?Q?rozOt/agSBaNuDGoDwA44Ypq4pHqmPG93H7lxk5nuE6HwIS1EhrNNBSscA+6?=
 =?us-ascii?Q?qsykGq+/RKToXWFXoBh5Jw26ycPrCN4t8hke0RMOct++bJ8/yKsrNsHHPVW7?=
 =?us-ascii?Q?GviENf24DusO5QX/Hw8umxPVikFlwiqo0Fo4QEJf8twdgQmDGbs97tM2Wa2T?=
 =?us-ascii?Q?saI4CC/tRxKK6dhFC5Hibj03nKkZdwLnnxoE+8lU/L5qZoe5syHd9y93uO1f?=
 =?us-ascii?Q?tH5d174VmP8N7XNKRlGtRzaWq5qyY4RP82TQdETgk8wjlhkkpUzRd48z6HpL?=
 =?us-ascii?Q?M+4fugUbhDEIV1UNBsE4lXwqt//e/UC+Ykf/L/5FAgk5YIaypJ8rb60e28UC?=
 =?us-ascii?Q?Uk2CulG5HAOa7pAzsj2darcHH4V2WyWrR8tZIIsfLCEtSDU5xe9rtOUNeOFM?=
 =?us-ascii?Q?mCjgd4ZwnqHVW1IE9Fmwl9YZIwKBt0r2o/q0ttlP8DLnHvdMT9WTAPk18TD3?=
 =?us-ascii?Q?rJvO/SnzvDijHNLDX3+Zs0QGgfzEoCgca3FlqayGV1sZYuAhmWNJF+td4B86?=
 =?us-ascii?Q?oPC12hk9wCWXB9q0+W1F3imxbcAvexTCjemSOoyBadi69U4lLkED1++GLPYj?=
 =?us-ascii?Q?jRLX2kjl/Z+ZQWluJ46pzpr1bwJZfMKSv8RrWgmUksYCVuK6HEIsExogBPJf?=
 =?us-ascii?Q?1pQM+Ex+Qrh8LmAMLb2aXaNH4ngl6E7HWl2T62dBH8o4k1ahI+nXUB81r/8H?=
 =?us-ascii?Q?M23TpFVunwLZ0VutOU5N5nue7D722DnPaFmS1+RH4RfyxRiOHwcQlhyduyMq?=
 =?us-ascii?Q?miqciMEK1qUSfY0xSXVmj7wWTPKA83xMCyc5PqaQLX0EO2qfhzVuq5CX/AOl?=
 =?us-ascii?Q?rktjkLw61YDioAlNzooecgKbTz764s+XPiLLhJi7yeXnkAr7tCEXuNZ1tUzr?=
 =?us-ascii?Q?RvJ3Z9jg7yJk1ULI7WTJzNWQl5UMPR64yPOU8nvSJm9Y+pZGEJLNBCJRW3Xz?=
 =?us-ascii?Q?aZMlVk8l9PxYlMcSwGJOWaVtKsPNUCqzEw0QavUYI0UIQumvVBgyFkYWU/a8?=
 =?us-ascii?Q?brBV0ktwbfGfbkNzA9rDfb4/zeIODKbtX6PaRXTqi1P2LjjPVzObwTKKV96H?=
 =?us-ascii?Q?audsWcVH59ZKJ5klEEmZPS6BhPiMM4zUVAf7ZlrSJ6KtKsmlc1OjwmF6zLU2?=
 =?us-ascii?Q?UcxH6czgkRzXUDSV48d+/yU6cNWwpdeLwk3ycSl7/KJM/veqMsQ3bE2SPxRR?=
 =?us-ascii?Q?gPZ17SFeKN52ctowRrW070r6yFdUMM9siFq8Oti/61OHWMIAmppQSeiJDWCX?=
 =?us-ascii?Q?/n2uJXP6hX2NwOYxCawgVcYGEUyehYkNk7g/9LmxqoXHJruLR8gvrMkdsw7g?=
 =?us-ascii?Q?upMJoijg6jy3fExEjd4smzbapRGwCLHAeGDHzThW7jVdZ7OKKKf2pdSRB0ah?=
 =?us-ascii?Q?1VXeigWxKcFCSTE+rYyLIlPAaVjoT6Lq1cROTsoosbEJ2esND1C0LOwq6dHJ?=
 =?us-ascii?Q?4qVq7tD7zxao3RgL1FZsiLmLetacE55ESBI7tYmtczYKeqexmazjBIrQNa6/?=
 =?us-ascii?Q?LQ=3D=3D?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: ed4bbf1c-f387-4716-b28a-08dbf412a237
X-MS-Exchange-CrossTenant-AuthSource: AM0PR04MB6452.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 03 Dec 2023 15:15:07.3060
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: fXCJM+EwA/v6HM33DP5Q7xJr+cvXDF/6eCBJPplSiYpxyNC/lS2RGVcm5WmMFlY/VmzJtsy5YC1ZKXJraGsSwg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR04MB9740
X-Spam-Status: No, score=-0.9 required=5.0 tests=DKIM_SIGNED,DKIM_VALID,
	DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,
	SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=unavailable
	autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on pete.vger.email
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (pete.vger.email [0.0.0.0]); Sun, 03 Dec 2023 07:17:23 -0800 (PST)

On Wed, Nov 29, 2023 at 05:38:27PM -0600, Bjorn Helgaas wrote:
> Hmm.  I can't find the 0x1957 Vendor ID here:
> https://pcisig.com/membership/member-companies which is supposed to
> be the authoritative source AFAICS.
> 
> Also, that page lists 0x1131 as "NXP Semiconductors".
> 
> There's a contact email on that page if it needs updates.

Sorry, I've no clue what's with that list of PCI SIG member companies.
Of course a company which no longer has a legal status cannot be a
member of the PCI SIG.

FWIW, the lspci tool uses this database: https://pci-ids.ucw.cz/

> I don't quite understand the goal here.  The company is now called
> "NXP", and this patch removes PCI_VENDOR_ID_NXP (the only instance of
> "NXP" in pci_ids.h) and uses PCI_VENDOR_ID_FREESCALE (which apparently
> does not exist any more)?
> 
> Why would we remove name of the current company and use the name of a
> company that doesn't exist any more?

Because that company doesn't produce new equipment with the old vendor id.

Aliasing the old vendor id to the current company name is polluting the
Linux definitions.

The goal, quite simply, is to call the Freescale PCI ID "Freescale".