Re: [PATCH v3 02/16] media: ti: vpe: #include <linux/platform_device.h> explicitly

[Yemike Abhilash Chandra]

Hi,
Thanks for the patch.

On 29/06/26 03:28, Uwe Kleine-König (The Capable Hub) wrote:
> The driver uses several symbols and structs defined in that header. The
> header is currently included transitively via
>
> 	"vip.h" ->
> 	<media/v4l2-ctrls.h> ->
> 	<media/media-request.h> ->
> 	<media/media-device.h> ->
> 	<linux/platform_device.h>
>
> which seems to be on the lower end of the scale between random and
> reliable.
>
> Acked-by: Danilo Krummrich <dakr@xxxxxxxxxx>
> Signed-off-by: Uwe Kleine-König (The Capable Hub) <u.kleine-koenig@xxxxxxxxxxxx>

Reviewed-by: Yemike Abhilash Chandra <y-abhilashchandra@xxxxxx>

Thanks and Regards,
Yemike Abhilash Chandra

> ---
>   drivers/media/platform/ti/vpe/vip.c | 1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/drivers/media/platform/ti/vpe/vip.c b/drivers/media/platform/ti/vpe/vip.c
> index cb0a5a07a3d4..e56a95f53ea9 100644
> --- a/drivers/media/platform/ti/vpe/vip.c
> +++ b/drivers/media/platform/ti/vpe/vip.c
> @@ -16,6 +16,7 @@
>   #include <linux/interrupt.h>
>   #include <linux/module.h>
>   #include <linux/workqueue.h>
> +#include <linux/platform_device.h>
>   #include <linux/pm_runtime.h>
>   #include <linux/sched.h>
>   #include <linux/mfd/syscon.h>