Re: [PATCH] can: m_can: pci: Remove driver_data

From: Uwe Kleine-König

Date: Mon May 11 2026 - 05:25:27 EST


Hello Markus,

On Mon, May 11, 2026 at 10:49:16AM +0200, Markus Schneider-Pargmann (The Capable Hub) wrote:
> @@ -183,8 +183,8 @@ static SIMPLE_DEV_PM_OPS(m_can_pci_pm_ops,
> m_can_pci_suspend, m_can_pci_resume);
>
> static const struct pci_device_id m_can_pci_id_table[] = {
> - { PCI_VDEVICE(INTEL, 0x4bc1), M_CAN_CLOCK_FREQ_EHL, },
> - { PCI_VDEVICE(INTEL, 0x4bc2), M_CAN_CLOCK_FREQ_EHL, },
> + { PCI_VDEVICE(INTEL, 0x4bc1), },
> + { PCI_VDEVICE(INTEL, 0x4bc2), },

nitpick: typically the trailing comma isn't added if the closing curly
bracket is on the same line.

Otherwise looks good to me

Acked-by: Uwe Kleine-König (The Capable Hub) <u.kleine-koenig@xxxxxxxxxxxx>

(even if you keep the patch as is).

Best regards
Uwe

Attachment: signature.asc
Description: PGP signature