Re: [PATCH] [media] vcodev: mediatek: add missing include in JPEG decoder driver

2017-03-13 Thread Hans Verkuil
On 03/12/2017 09:13 PM, Jérémy Lefaure wrote: > The driver uses kzalloc and kfree functions. So it should include > linux/slab.h. This header file is implicitly included by v4l2-common.h > if CONFIG_SPI is enabled. But when it is disabled, slab.h is not > included. In this case, the driver does not

Re: [PATCH] [media] vcodev: mediatek: add missing include in JPEG decoder driver

2017-03-12 Thread Rick Chang
On Sun, 2017-03-12 at 16:13 -0400, Jérémy Lefaure wrote: > The driver uses kzalloc and kfree functions. So it should include > linux/slab.h. This header file is implicitly included by v4l2-common.h > if CONFIG_SPI is enabled. But when it is disabled, slab.h is not > included. In this case, the driv

[PATCH] [media] vcodev: mediatek: add missing include in JPEG decoder driver

2017-03-12 Thread Jérémy Lefaure
The driver uses kzalloc and kfree functions. So it should include linux/slab.h. This header file is implicitly included by v4l2-common.h if CONFIG_SPI is enabled. But when it is disabled, slab.h is not included. In this case, the driver does not compile: drivers/media/platform/mtk-jpeg/mtk_jpeg_co