From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wr1-f42.google.com (mail-wr1-f42.google.com [209.85.221.42]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id E48DD320CAD for ; Wed, 10 Jun 2026 11:58:22 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.42 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781092705; cv=none; b=cJjIQ9UaknujgDRhlMxPV+Hmobk0sCSnK2XromITbjhb+L3+6l2umLqLgTR0UL/zRTFqPwC+k8D8w/sGrBaU0QVKMRFTpbFqXN47T+RgbpmWq1gRLFerQtDH7P2zmdLG9GqQdyw6MiWEgDBpZhJxg5Tvl7sj6LBNIdsrENv1D2k= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781092705; c=relaxed/simple; bh=RWK5XNzHCxbN0CIyBzmzynD3G+sG13mD/TIh+ITpqIo=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=XKZivGQg3wqZgJEShF0luO5X/GXkymzUDsN6vujw8MXEYcbom3vx667Gq+t+Yrx/y1g7q+Ecubp0vy5FBF0JUfsWjNvsFK/xWwYYthGpOQBzYW/8baHqlL3d18j7uBSfojBMFuLfkayybQf2EGqaAnRVi2ehJqQNzYTAN4y21B4= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=baylibre.com; spf=pass smtp.mailfrom=baylibre.com; dkim=pass (2048-bit key) header.d=baylibre.com header.i=@baylibre.com header.b=Q/wzw8wm; arc=none smtp.client-ip=209.85.221.42 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=baylibre.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=baylibre.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=baylibre.com header.i=@baylibre.com header.b="Q/wzw8wm" Received: by mail-wr1-f42.google.com with SMTP id ffacd0b85a97d-45ef5146b56so4896437f8f.0 for ; Wed, 10 Jun 2026 04:58:22 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre.com; s=google; t=1781092701; x=1781697501; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=eHPNJDli0SL72ZV7h41ZBnThr8ba+bZcbFOMnm72cuk=; b=Q/wzw8wmF23lyQPnMPzo/UIa7FzrPgM1/pCGDbodUMlNc0q4zfckel+5/xtk58rSQw oyTeNDEDUw8diIyDIKU9dBlWBfGJJ2gkHo2R6LaY28uiim29TqyJNsoXDlFxsHVft4+B vOlCvU621eLV2XDmGcHQS3qtHFumzCtGVJ/FQNxoku+CIfdctOqRm0ObstA0MmHC3B4k uu+ao+jyfaIRRg36VuQqEc1E5ZmOJwZKli2ychgu857Ds/mF6sKzkm6/QzdsQRT8EPeX Ac2RySG+NNIhdgYNbO4AGGEKgC0i+DWNms5Xt2rCESZ3dS2XsrG9vnBj/g0gyFMc3Xx6 dE3Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1781092701; x=1781697501; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-gg:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=eHPNJDli0SL72ZV7h41ZBnThr8ba+bZcbFOMnm72cuk=; b=THVGC6SOh1bMRzNA3fobVMt6PcgK81zHR6SDKIp9cQ2N/8pWjb+u8VsXcvija4d/YX lUbsEW4UYbJYAOKkQxTVresm2UtPfizN5BYrbEwQk5YWwb2oFQV93C4GyP73jqjnsTXE z3hTUf1VG81+qyAWx2BVeJBrQO71z31Z5pLCd+1kHRbhrPm4oKT83+cXzHXBcMLJTZOg g2oNrCXNK6qJpTZk2FJkPuIOyeGMGKBBckm6fX+7S5jYs/C+kCTg0Nb2cyIwX6pEU+8P 3c5pP0i01ZRJwRN1cq/4AnQn9hsom9tYAyWl2t7exuCgK35wuKLXp7yC9DAkfHWo0ELN u/yA== X-Forwarded-Encrypted: i=1; AFNElJ+ii5Jg6kYMFe7qU2wAlhTWIIhVAVcsHDcBsf2ze/ZJSbg1qD6gD0FxJ2hbLBBvQOQNpsXQTmUvmn//9K8=@vger.kernel.org X-Gm-Message-State: AOJu0Yyzr23HXg63GC5mRfeHlDML6zziVSlNiedJ/iK37pcBlCRNzCja gkXmWbZHLXOjVvviDSFfsuPraQzenSR8PGSpGYEx8sU9X+xPnsKMm61uE0AKxDl/2Tk= X-Gm-Gg: Acq92OEfZ89hKqxlJl5mUQUq7yBQHC6skK0lNl0L4zXMlGvajq7FwOyqhQW20bS12GU 2bcYe74mmmLSH21mzLzXvaMrZUxGa2uScSXUxFMywFTjcwdBVyGNU7g2nq1Ov0SglEXFV7peLz8 Bhvm2dv/KTW3qRvEdtfOAuhp5o/4SbRrxPhH2pvb8dtcQzP2nt9Ms7UjR7EFCwuqh+h7+HT8MRH 1/0u1qjfXJN+kBSxKej6s5E3wgdeBzZlmJHl1Z44XcYWtst/hUCNPOd0ySxDbG3LGq3JUHsg6D3 /rr36+ys5w09C+nLMYnI2kaWbOlTufaRkuchNDwb3Q7w67FAfDL7qROuzVxj0PPi0X4CW7EEkK6 YrA0mXd1/U6WntFbpFpgAgubBtMkzpyZ0vSBrgoJ42zTnzDTUW2ktDaI3Txqw7uuUTN0z8DZP/8 R8Qalru3NeswKuKv6GH2YSk3VmNy8co/6LhJ6F4to= X-Received: by 2002:a05:6000:4b07:b0:460:21e7:330e with SMTP id ffacd0b85a97d-46032b8164fmr31650683f8f.10.1781092701398; Wed, 10 Jun 2026 04:58:21 -0700 (PDT) Received: from localhost ([2a02:8071:56d1:2de0:559d:eec2:887f:c200]) by smtp.gmail.com with UTF8SMTPSA id ffacd0b85a97d-4601f2ed944sm73905661f8f.13.2026.06.10.04.58.18 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 10 Jun 2026 04:58:19 -0700 (PDT) Date: Wed, 10 Jun 2026 13:58:17 +0200 From: Uwe =?utf-8?Q?Kleine-K=C3=B6nig_=28The_Capable_Hub=29?= To: Dmitry Torokhov Cc: Kees Cook , Werner Sembach , Christoffer Sandberg , feng , gongqi <550230171hxy@gmail.com>, linux-input@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v1] Input: Drop unused assignments from pnp_device_id arrays Message-ID: References: Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="impemnflpxbxj5mb" Content-Disposition: inline In-Reply-To: --impemnflpxbxj5mb Content-Type: text/plain; protected-headers=v1; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Subject: Re: [PATCH v1] Input: Drop unused assignments from pnp_device_id arrays MIME-Version: 1.0 Hello Dmitry, On Tue, Jun 09, 2026 at 11:12:49PM -0700, Dmitry Torokhov wrote: > On Tue, Jun 09, 2026 at 04:53:25PM +0200, Uwe Kleine-K=F6nig (The Capable= Hub) wrote: > > Explicitly assigning .driver_data in drivers that don't use this member > > is silly and a bit irritating. Drop these. Also simplify the list > > terminator entry to be just empty to match what most other device_id > > tables do. > >=20 > > There is no changed semantic, not even a change in the compiled result. > >=20 > > Signed-off-by: Uwe Kleine-K=F6nig (The Capable Hub) > > --- > > drivers/input/gameport/ns558.c | 46 +++++++++++----------- > > drivers/input/serio/i8042-acpipnpio.h | 56 +++++++++++++-------------- > > 2 files changed, 51 insertions(+), 51 deletions(-) > >=20 > > diff --git a/drivers/input/gameport/ns558.c b/drivers/input/gameport/ns= 558.c > > index fdece6ec1df3..f70a96c4f1fd 100644 > > --- a/drivers/input/gameport/ns558.c > > +++ b/drivers/input/gameport/ns558.c > > @@ -148,29 +148,29 @@ static int ns558_isa_probe(int io) > > #ifdef CONFIG_PNP > > =20 > > static const struct pnp_device_id pnp_devids[] =3D { > > - [...] > > - { .id =3D "", }, > > + [...] > > + { } >=20 > This goes BOOOM! You have to keep empty .id string as terminator. How so? Given that my patch doesn't modify the resulting ns558.o I doubt that. If .id was a char *, I'd agree, but it's a char[], so there should be no difference (and the compiler agrees). Best regards Uwe --impemnflpxbxj5mb Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQEzBAABCgAdFiEEP4GsaTp6HlmJrf7Tj4D7WH0S/k4FAmopUVYACgkQj4D7WH0S /k7g2Af/Yc1rUQEuTflkJFLILBh2UuhAWIVZreH7tyA24XPf0Xfu0XJ/F0qc3fr3 CeRwHk3LXILcCIeL1kVKRnsVjNdU4JwNrNJbWyQ4DU1VOmU4YESWg7+mxhotKO/y IYTsw3/csZ1ireDEKZ1CcPMHPauzTWNbjp8TJ6vYb53UHNeLXssJLV89rjtXGyhL mUCFl7X5Q5crYp8A+ocJLpYvg2EUVEdGB4GkfthgOcsKaFSZ2PWusiAv1RLwUEGS SOUt/EOIakNZC94XKdegRH7hVFFi0mhrMGplZnyTUp3urDPsl7jkvyUBX6VLlUzE fE92yV5oDnCQTZ1Hf2hG3dI4jjg6yQ== =GHA/ -----END PGP SIGNATURE----- --impemnflpxbxj5mb--