Re: [PATCH v4 14/17] media: em28xx: Add include for struct usb_device_id

Hello,

On Mon, Jul 27, 2026 at 12:14:51PM +0200, Hans Verkuil wrote:
> On 30/06/2026 11:24, Uwe Kleine-König (The Capable Hub) wrote:
> > Traditionally <linux/mod_devicetable.h> was a header defining a plethora
> > of structs, among them struct usb_device_id. This was split now with the
> > objective that only the relevant bits are included.
> > 
> > Currently <linux/mod_devicetable.h> is transitively included in
> > drivers/media/usb/em28xx/em28xx.h via:
> > 
> > 	drivers/media/usb/em28xx/em28xx.h ->
> > 	<linux/i2c.h> ->
> > 	<linux/acpi.h> ->
> > 	<linux/device.h> ->
> > 	<linux/device/driver.h> ->
> > 	<linux/mod_devicetable.h
> > 
> > To keep struct usb_device_id available once <linux/device/driver.h>
> > stops including <linux/mod_devicetable.h>, include it the header
> > providing that struct explictly.
> > 
> > Acked-by: Danilo Krummrich <[email protected]>
> > Acked-by: Takashi Sakamoto <[email protected]>
> > Link: https://patch.msgid.link/b859817d9f6ec7450302a231e093a5ad5852cb91.1782682124.git.ukleinek@kernel.org
> > Signed-off-by: Uwe Kleine-König (The Capable Hub) <[email protected]>
> 
> Acked-by: Hans Verkuil <[email protected]>

Note you're late to the party. This patch already became commit
c19f08f796cb ("media: em28xx: Add include for struct usb_device_id") and
is part of v7.2-rc2.

Best regards
Uwe

-----BEGIN PGP SIGNATURE-----

iQEzBAABCgAdFiEEP4GsaTp6HlmJrf7Tj4D7WH0S/k4FAmpp8YAACgkQj4D7WH0S
/k4f9wf/R+9fqRYwzLJxLAcYWyOPLtgppccbE0SAa6G3ubLnlhF//Kc939iAa+Hk
3mrxyhJev37lJ5YCMmxs1pWazLG2jhGK624NC0VRdKN4NE3M72SEFzhiq3EXQfav
Qafv0Y7ne57ehIPokORapgv+CCQ10LIotIwqjKzfMHoAUaPxagsuDW8WF1kmgqf5
73ravfx6oL/TXx+sSTNjUfx5MKygafje/mkO4sDHmnnAbcJHme/blsxZvZXS8mJ6
Gh5F2r7JaxEXioRl6BbrCMzRZ/XvsMc2EkDRuZtJlhP3EcBjPx8gAXvmOUVS4f5b
OnWfol8NMgCjseFVER/mdwmU2jYcfA==
=PUla
-----END PGP SIGNATURE-----