From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f46.google.com (mail-wm1-f46.google.com [209.85.128.46]) (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 2467D363C46 for ; Fri, 12 Jun 2026 08:22:03 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.46 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781252528; cv=none; b=fEcVg1X5jE+XKPSRYZ8oeeC+8WCbdt6+1j9HAphPp1ldNMdFxyvggbf7HqtXmQbqz6I1VeiVc8ljM9kUC8OfqxEmohJL3foGJyb79voznYNIeVhiO2AcZZk1Me0dB+u1HsBdZT3szlS3sM3sxD8opiUBQRL5OMX5tOc4/+++tJ8= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781252528; c=relaxed/simple; bh=RCPp9Uf3HlobMkUHzc+LbiqpDKpB8kHX9XRhX5NJ9jQ=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=kpyue/xg7Fd6pI4ygFJ5lecEO6PhkvQlKlFBP8wok2gUg3d35T7UsRj1xl2gtPaaexryXlXqKatabhTkLP5L6YediJGVFSLceKgQfl3ij6t8fKTUZkMEObSgg+PlPPcMwtQsZsQ7mOalcTPpkAivJtYjCXVG1Zle9Q+AczZ1i70= 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=kbxRFeDD; arc=none smtp.client-ip=209.85.128.46 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="kbxRFeDD" Received: by mail-wm1-f46.google.com with SMTP id 5b1f17b1804b1-490e1904089so5700355e9.0 for ; Fri, 12 Jun 2026 01:22:03 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre.com; s=google; t=1781252522; x=1781857322; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=ZjYxZMHle7kM1Z7TryoA92PGleAxWUO0wF2e7W9mOj0=; b=kbxRFeDDDLJ+G4t1KNyf4WHKjIKfJgPRvzOKdqxN5UGsYJ8rxTdKvQETSS9ZQE6Dt3 a9xPPx8Z6QIdnlTpAjJBTn0Nx+c5pvgIJw40hgmwPKmH+H8LHuDeD1/bmee4+KlQu+jY VnM/TWKT+NrdawohiX4T/Et87ToJCc222vE+UcvYgV2m/2LiKCqUgHdnx2XWIgGQKTzW 17NZfvFJNEJL5T7GFORAj1pS5DfjayffuBW3P/KEnmTzrn3WVPwOY8FWkFgSq22RAiP0 klATAsu5KLAC3GGqZS/C0H85MPBd53aAZWoVvT2giZ59nqJnBHgpDIOh0uG7fRbpO0PA X0Cg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1781252522; x=1781857322; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to; bh=ZjYxZMHle7kM1Z7TryoA92PGleAxWUO0wF2e7W9mOj0=; b=pn0zhvXDSQBQHyBCtd5CJepgEH+wMeFc1U835lXtmM+iMKmb9H1ISE6pZP+KkqiAd5 YYt5J5WswWxlcGvLSopy3aRshz5d1eEnW7Z5Ibi3rJ0voNzkt6c3eUCPzY5fZ1fbBQNd igR0Zq6WwcBj9poWzhunckjpp0OwI7eDpH+QFEN4N0yNXi+s5f0Hr2+oxxtlqe5xlnnJ fc4/av24zapsCWwRYduNXpTVm+Xu3BOWmOo0ue8MldkbISiPVY5Hug2Ugactu5lEb1Q4 BJX8LGRFMKzVsoWounVmtKrygVVzyWYP9taHArayLpV/WfpoRVq8Pc0+3Iff/e5NB1Gy JMYA== X-Forwarded-Encrypted: i=1; AFNElJ8MoREwpMzQ9HgJ26/4Sn7CuheFqOCU9kLg0A3lL/nd2VSxLKbTMO7tzlsyAcyRHNXMtitLxLhbMUKTBZI=@vger.kernel.org X-Gm-Message-State: AOJu0YxoDcjesuN0Jy1KzA1zmO0f+AlB9YukrNl5fIz0ZoI3SuBnf12f NvmxkUSwvNcJ/lrjVoRwnZa2cKRpeh7pTKXQNOFAhcSjdyblJzcYr3qG7kzYt0e6FV+Lf9N/735 DJXUv X-Gm-Gg: Acq92OFynqfDDYE/q3Na+dAFJ3Ge3ik7KHARnwemHwlOQ4qjns6Pi42tYPfLGUlzVLZ fK3JLoAXACvVm8IWwxNuu5f6WJc4IziPKZ1iPTuJ7q0rxpdRtWqV7Un17p9ykUpsy5CnnY4oNx8 kqseRHkicU7OWuvlXwwqop9DC5dsVyxIIjmQUdyQPMc1/aAd7SawASHy5Q+BhVrqfpuvufXILwO 6wfIoXRmFfJP7wlyXXkj1bM0ZTHGkayoZLHp/OxdKAf5JJEsEeZchieQkls/huwcU20jyaWOMmu i8z58sSsUed7AagktsDAGR5uCHlROQL6O0l9wq7To2g89f58yG9MxD5b6GTYIuYTvif25nBPyUw 9mzeKl1JoA5kdWUYhZGRAj660Ni8WB2J2Qb6MrGQ004qDphvBkqkYa5dqjsgwuxVot8szvHyWS+ pl8eS2mt7qy5vy71vyfvo0u8NBOqpqkrKS7d8UDU+PXe4yYKd4dZ3pBztby8M+nrCjt9+YCrrFT tIx45MZWVNbYA== X-Received: by 2002:a05:600c:4685:b0:490:bad8:329c with SMTP id 5b1f17b1804b1-490ec4d4f64mr22763115e9.12.1781252522336; Fri, 12 Jun 2026 01:22:02 -0700 (PDT) Received: from localhost (p200300f65f47db040e4e5c1e98e7f0ff.dip0.t-ipconnect.de. [2003:f6:5f47:db04:e4e:5c1e:98e7:f0ff]) by smtp.gmail.com with UTF8SMTPSA id ffacd0b85a97d-4606f2e592csm3583369f8f.36.2026.06.12.01.22.01 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 12 Jun 2026 01:22:01 -0700 (PDT) From: =?UTF-8?q?Uwe=20Kleine-K=C3=B6nig=20=28The=20Capable=20Hub=29?= To: Damien Le Moal , Niklas Cassel Cc: linux-ide@vger.kernel.org, linux-kernel@vger.kernel.org, Damien Le Moal Subject: [PATCH v3 1/2] ata: Drop unused assignments of pci_device_id driver data Date: Fri, 12 Jun 2026 10:21:47 +0200 Message-ID: <75fcd02edb42a71cbd83f2de24ebeca70f978003.1781252168.git.ukleinek@kernel.org> X-Mailer: git-send-email 2.47.3 In-Reply-To: References: Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 X-Developer-Signature: v=1; a=openpgp-sha256; l=5907; i=u.kleine-koenig@baylibre.com; h=from:subject:message-id; bh=RCPp9Uf3HlobMkUHzc+LbiqpDKpB8kHX9XRhX5NJ9jQ=; b=owEBbQGS/pANAwAKAY+A+1h9Ev5OAcsmYgBqK8GdXZd3jvx/+q8DsMK5V1toj1P01bYE9YDGy 5XqMOnLg/GJATMEAAEKAB0WIQQ/gaxpOnoeWYmt/tOPgPtYfRL+TgUCaivBnQAKCRCPgPtYfRL+ Tg7nB/9t2HLoTjpN39bPVuCTAA1yePIs7SNPTaijsrhrpe154a5GhIHCbgzJHFnx93tz3OsGMJr GLM3BL/dAH1rdUZSQC9iN2TM4CpHY1T766FxtFKfugRl62+nt8DOERfFG9xmCccj14x2gnqs/N+ MGVM5kldE+nuaS59y2n8v62fnQOAQghmBEVpvTN1xm5qrdeGb/TL6TtxDJTg0NHSy8N0Mjtflx5 PUHwd6zBjUskdnP/Q1vyMbvd1HYnpIgcbYzWawCM5qAAbntnNMMHAaiMKYb11va8/gsVqjqYBou um66JY7ZZiDnzp+ksj3m4a60Mlow2ht4cFbeD3P3Qq+QF6T3 X-Developer-Key: i=u.kleine-koenig@baylibre.com; a=openpgp; fpr=0D2511F322BFAB1C1580266BE2DCDD9132669BD6 Content-Transfer-Encoding: 8bit The drivers explicitly set the .driver_data member of struct pci_device_id to zero without relying on that value. Drop these unused assignments. While touching these arrays, convert the one driver not using PCI_DEVICE to use that macro and align the array's coding style to what is used most for these. (i.e. break very long lines, a single space in the list terminator and no trailing comma.) This patch doesn't modify the compiled array, only its representation in source form benefits. The former was confirmed with builds on x86 and arm64. Reviewed-by: Damien Le Moal Signed-off-by: Uwe Kleine-König (The Capable Hub) --- drivers/ata/pata_atp867x.c | 6 +++--- drivers/ata/pata_cmd640.c | 4 ++-- drivers/ata/pata_jmicron.c | 7 +++++-- drivers/ata/pata_macio.c | 12 ++++++------ drivers/ata/pata_ninja32.c | 14 +++++++------- drivers/ata/pata_sch.c | 2 +- drivers/ata/sata_vsc.c | 14 +++++++++----- 7 files changed, 33 insertions(+), 26 deletions(-) diff --git a/drivers/ata/pata_atp867x.c b/drivers/ata/pata_atp867x.c index 308f86f9e2f0..6154a2bf19be 100644 --- a/drivers/ata/pata_atp867x.c +++ b/drivers/ata/pata_atp867x.c @@ -526,9 +526,9 @@ static int atp867x_reinit_one(struct pci_dev *pdev) #endif static const struct pci_device_id atp867x_pci_tbl[] = { - { PCI_VDEVICE(ARTOP, PCI_DEVICE_ID_ARTOP_ATP867A), 0 }, - { PCI_VDEVICE(ARTOP, PCI_DEVICE_ID_ARTOP_ATP867B), 0 }, - { }, + { PCI_VDEVICE(ARTOP, PCI_DEVICE_ID_ARTOP_ATP867A) }, + { PCI_VDEVICE(ARTOP, PCI_DEVICE_ID_ARTOP_ATP867B) }, + { } }; static struct pci_driver atp867x_driver = { diff --git a/drivers/ata/pata_cmd640.c b/drivers/ata/pata_cmd640.c index 45a7217b136e..84eb36d02d7d 100644 --- a/drivers/ata/pata_cmd640.c +++ b/drivers/ata/pata_cmd640.c @@ -248,8 +248,8 @@ static int cmd640_reinit_one(struct pci_dev *pdev) #endif static const struct pci_device_id cmd640[] = { - { PCI_VDEVICE(CMD, 0x640), 0 }, - { }, + { PCI_VDEVICE(CMD, 0x0640) }, + { } }; static struct pci_driver cmd640_pci_driver = { diff --git a/drivers/ata/pata_jmicron.c b/drivers/ata/pata_jmicron.c index b885f33e8980..d18286cbca1b 100644 --- a/drivers/ata/pata_jmicron.c +++ b/drivers/ata/pata_jmicron.c @@ -148,8 +148,11 @@ static int jmicron_init_one (struct pci_dev *pdev, const struct pci_device_id *i } static const struct pci_device_id jmicron_pci_tbl[] = { - { PCI_VENDOR_ID_JMICRON, PCI_ANY_ID, PCI_ANY_ID, PCI_ANY_ID, - PCI_CLASS_STORAGE_IDE << 8, 0xffff00, 0 }, + { + PCI_DEVICE(PCI_VENDOR_ID_JMICRON, PCI_ANY_ID), + .class = PCI_CLASS_STORAGE_IDE << 8, + .class_mask = 0xffff00 + }, { } /* terminate list */ }; diff --git a/drivers/ata/pata_macio.c b/drivers/ata/pata_macio.c index 9eefdc5df5df..dd93f613f959 100644 --- a/drivers/ata/pata_macio.c +++ b/drivers/ata/pata_macio.c @@ -1368,12 +1368,12 @@ static struct macio_driver pata_macio_driver = }; static const struct pci_device_id pata_macio_pci_match[] = { - { PCI_VDEVICE(APPLE, PCI_DEVICE_ID_APPLE_UNI_N_ATA), 0 }, - { PCI_VDEVICE(APPLE, PCI_DEVICE_ID_APPLE_IPID_ATA100), 0 }, - { PCI_VDEVICE(APPLE, PCI_DEVICE_ID_APPLE_K2_ATA100), 0 }, - { PCI_VDEVICE(APPLE, PCI_DEVICE_ID_APPLE_SH_ATA), 0 }, - { PCI_VDEVICE(APPLE, PCI_DEVICE_ID_APPLE_IPID2_ATA), 0 }, - {}, + { PCI_VDEVICE(APPLE, PCI_DEVICE_ID_APPLE_UNI_N_ATA) }, + { PCI_VDEVICE(APPLE, PCI_DEVICE_ID_APPLE_IPID_ATA100) }, + { PCI_VDEVICE(APPLE, PCI_DEVICE_ID_APPLE_K2_ATA100) }, + { PCI_VDEVICE(APPLE, PCI_DEVICE_ID_APPLE_SH_ATA) }, + { PCI_VDEVICE(APPLE, PCI_DEVICE_ID_APPLE_IPID2_ATA) }, + { } }; static struct pci_driver pata_macio_pci_driver = { diff --git a/drivers/ata/pata_ninja32.c b/drivers/ata/pata_ninja32.c index 76a91013d27d..88db9bd271e8 100644 --- a/drivers/ata/pata_ninja32.c +++ b/drivers/ata/pata_ninja32.c @@ -166,13 +166,13 @@ static int ninja32_reinit_one(struct pci_dev *pdev) #endif static const struct pci_device_id ninja32[] = { - { 0x10FC, 0x0003, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0 }, - { 0x1145, 0x8008, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0 }, - { 0x1145, 0xf008, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0 }, - { 0x1145, 0xf021, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0 }, - { 0x1145, 0xf024, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0 }, - { 0x1145, 0xf02C, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0 }, - { }, + { PCI_DEVICE(0x10FC, 0x0003) }, + { PCI_DEVICE(0x1145, 0x8008) }, + { PCI_DEVICE(0x1145, 0xf008) }, + { PCI_DEVICE(0x1145, 0xf021) }, + { PCI_DEVICE(0x1145, 0xf024) }, + { PCI_DEVICE(0x1145, 0xf02C) }, + { } }; static struct pci_driver ninja32_pci_driver = { diff --git a/drivers/ata/pata_sch.c b/drivers/ata/pata_sch.c index 8356f1f2a025..fadb5377371b 100644 --- a/drivers/ata/pata_sch.c +++ b/drivers/ata/pata_sch.c @@ -42,7 +42,7 @@ static void sch_set_dmamode(struct ata_port *ap, struct ata_device *adev); static const struct pci_device_id sch_pci_tbl[] = { /* Intel SCH PATA Controller */ - { PCI_VDEVICE(INTEL, PCI_DEVICE_ID_INTEL_SCH_IDE), 0 }, + { PCI_VDEVICE(INTEL, PCI_DEVICE_ID_INTEL_SCH_IDE) }, { } /* terminate list */ }; diff --git a/drivers/ata/sata_vsc.c b/drivers/ata/sata_vsc.c index a53a2dfc1e17..606284a6d387 100644 --- a/drivers/ata/sata_vsc.c +++ b/drivers/ata/sata_vsc.c @@ -400,11 +400,15 @@ static int vsc_sata_init_one(struct pci_dev *pdev, } static const struct pci_device_id vsc_sata_pci_tbl[] = { - { PCI_VENDOR_ID_VITESSE, 0x7174, - PCI_ANY_ID, PCI_ANY_ID, 0x10600, 0xFFFFFF, 0 }, - { PCI_VENDOR_ID_INTEL, 0x3200, - PCI_ANY_ID, PCI_ANY_ID, 0x10600, 0xFFFFFF, 0 }, - + { + PCI_DEVICE(PCI_VENDOR_ID_VITESSE, 0x7174), + .class = 0x10600, + .class_mask = 0xFFFFFF, + }, { + PCI_DEVICE(PCI_VENDOR_ID_INTEL, 0x3200), + .class = 0x10600, + .class_mask = 0xFFFFFF, + }, { } /* terminate list */ }; -- 2.47.3