* [PATCH] tee: add forward declaration for struct device @ 2017-05-31 11:21 Jerome Forissier 2017-06-14 9:39 ` Jerome Forissier 0 siblings, 1 reply; 6+ messages in thread From: Jerome Forissier @ 2017-05-31 11:21 UTC (permalink / raw) To: linux-arm-kernel tee_drv.h references struct device, but does not include device.h nor platform_device.h. Therefore, if tee_drv.h is included by some file that does not pull device.h nor platform_device.h beforehand, we have a compile warning. Fix this by adding a forward declaration. Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org> --- include/linux/tee_drv.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/linux/tee_drv.h b/include/linux/tee_drv.h index 8614713..07bd226 100644 --- a/include/linux/tee_drv.h +++ b/include/linux/tee_drv.h @@ -29,6 +29,7 @@ #define TEE_SHM_DMA_BUF BIT(1) /* Memory with dma-buf handle */ #define TEE_SHM_EXT_DMA_BUF BIT(2) /* Memory with dma-buf handle */ +struct device; struct tee_device; struct tee_shm; struct tee_shm_pool; -- 2.7.4 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH] tee: add forward declaration for struct device 2017-05-31 11:21 [PATCH] tee: add forward declaration for struct device Jerome Forissier @ 2017-06-14 9:39 ` Jerome Forissier 2017-06-14 9:46 ` Jens Wiklander 0 siblings, 1 reply; 6+ messages in thread From: Jerome Forissier @ 2017-06-14 9:39 UTC (permalink / raw) To: linux-arm-kernel [+Arnd] Ping? Thanks, -- Jerome On 05/31/2017 01:21 PM, Jerome Forissier wrote: > tee_drv.h references struct device, but does not include device.h nor > platform_device.h. Therefore, if tee_drv.h is included by some file > that does not pull device.h nor platform_device.h beforehand, we have a > compile warning. Fix this by adding a forward declaration. > > Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org> > --- > include/linux/tee_drv.h | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/include/linux/tee_drv.h b/include/linux/tee_drv.h > index 8614713..07bd226 100644 > --- a/include/linux/tee_drv.h > +++ b/include/linux/tee_drv.h > @@ -29,6 +29,7 @@ > #define TEE_SHM_DMA_BUF BIT(1) /* Memory with dma-buf handle */ > #define TEE_SHM_EXT_DMA_BUF BIT(2) /* Memory with dma-buf handle */ > > +struct device; > struct tee_device; > struct tee_shm; > struct tee_shm_pool; > ^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH] tee: add forward declaration for struct device 2017-06-14 9:39 ` Jerome Forissier @ 2017-06-14 9:46 ` Jens Wiklander 2017-06-14 11:01 ` Arnd Bergmann 0 siblings, 1 reply; 6+ messages in thread From: Jens Wiklander @ 2017-06-14 9:46 UTC (permalink / raw) To: linux-arm-kernel On Wed, Jun 14, 2017 at 11:39:50AM +0200, Jerome Forissier wrote: > [+Arnd] > > Ping? > > Thanks, > -- > Jerome > > On 05/31/2017 01:21 PM, Jerome Forissier wrote: > > tee_drv.h references struct device, but does not include device.h nor > > platform_device.h. Therefore, if tee_drv.h is included by some file > > that does not pull device.h nor platform_device.h beforehand, we have a > > compile warning. Fix this by adding a forward declaration. > > > > Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org> > > --- > > include/linux/tee_drv.h | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/include/linux/tee_drv.h b/include/linux/tee_drv.h > > index 8614713..07bd226 100644 > > --- a/include/linux/tee_drv.h > > +++ b/include/linux/tee_drv.h > > @@ -29,6 +29,7 @@ > > #define TEE_SHM_DMA_BUF BIT(1) /* Memory with dma-buf handle */ > > #define TEE_SHM_EXT_DMA_BUF BIT(2) /* Memory with dma-buf handle */ > > > > +struct device; > > struct tee_device; > > struct tee_shm; > > struct tee_shm_pool; > > Looks good to me. Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org> -- Thanks, Jens ^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH] tee: add forward declaration for struct device 2017-06-14 9:46 ` Jens Wiklander @ 2017-06-14 11:01 ` Arnd Bergmann 2017-06-14 11:06 ` Jerome Forissier 0 siblings, 1 reply; 6+ messages in thread From: Arnd Bergmann @ 2017-06-14 11:01 UTC (permalink / raw) To: linux-arm-kernel On Wed, Jun 14, 2017 at 11:46 AM, Jens Wiklander <jens.wiklander@linaro.org> wrote: > On Wed, Jun 14, 2017 at 11:39:50AM +0200, Jerome Forissier wrote: >> [+Arnd] >> >> Ping? >> >> Thanks, >> -- >> Jerome >> >> On 05/31/2017 01:21 PM, Jerome Forissier wrote: >> > tee_drv.h references struct device, but does not include device.h nor >> > platform_device.h. Therefore, if tee_drv.h is included by some file >> > that does not pull device.h nor platform_device.h beforehand, we have a >> > compile warning. Fix this by adding a forward declaration. >> > >> > Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org> Acked-by: Arnd Bergmann <arnd@arndb.de> Do we need this to fix a warning in mainline, in linux-next, or only in combination with some other patches? I have not run into this warning in my build testing. >> > include/linux/tee_drv.h | 1 + >> > 1 file changed, 1 insertion(+) >> > >> > diff --git a/include/linux/tee_drv.h b/include/linux/tee_drv.h >> > index 8614713..07bd226 100644 >> > --- a/include/linux/tee_drv.h >> > +++ b/include/linux/tee_drv.h >> > @@ -29,6 +29,7 @@ >> > #define TEE_SHM_DMA_BUF BIT(1) /* Memory with dma-buf handle */ >> > #define TEE_SHM_EXT_DMA_BUF BIT(2) /* Memory with dma-buf handle */ >> > >> > +struct device; >> > struct tee_device; >> > struct tee_shm; >> > struct tee_shm_pool; >> > > > Looks good to me. > > Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org> Can you pick up the patch in your git tree and send a pull request for the appropriate release (4.12-fixes or 4.13)? If you don't expect to send anything else for tee in that release, you can also forward the patch to arm at kernel.org and ask for inclusion. If you just reply with the 'Reviewed-by', I would not expect to have to do anything in the arm-soc tree. Arnd ^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH] tee: add forward declaration for struct device 2017-06-14 11:01 ` Arnd Bergmann @ 2017-06-14 11:06 ` Jerome Forissier 2017-06-14 11:59 ` Arnd Bergmann 0 siblings, 1 reply; 6+ messages in thread From: Jerome Forissier @ 2017-06-14 11:06 UTC (permalink / raw) To: linux-arm-kernel On 06/14/2017 01:01 PM, Arnd Bergmann wrote: > On Wed, Jun 14, 2017 at 11:46 AM, Jens Wiklander > <jens.wiklander@linaro.org> wrote: >> On Wed, Jun 14, 2017 at 11:39:50AM +0200, Jerome Forissier wrote: >>> [+Arnd] >>> >>> Ping? >>> >>> Thanks, >>> -- >>> Jerome >>> >>> On 05/31/2017 01:21 PM, Jerome Forissier wrote: >>>> tee_drv.h references struct device, but does not include device.h nor >>>> platform_device.h. Therefore, if tee_drv.h is included by some file >>>> that does not pull device.h nor platform_device.h beforehand, we have a >>>> compile warning. Fix this by adding a forward declaration. >>>> >>>> Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org> > > Acked-by: Arnd Bergmann <arnd@arndb.de> > > Do we need this to fix a warning in mainline, in linux-next, or only > in combination with some other patches? Only with some other patches. -- Jerome > I have not run into this warning in my build testing. > >>>> include/linux/tee_drv.h | 1 + >>>> 1 file changed, 1 insertion(+) >>>> >>>> diff --git a/include/linux/tee_drv.h b/include/linux/tee_drv.h >>>> index 8614713..07bd226 100644 >>>> --- a/include/linux/tee_drv.h >>>> +++ b/include/linux/tee_drv.h >>>> @@ -29,6 +29,7 @@ >>>> #define TEE_SHM_DMA_BUF BIT(1) /* Memory with dma-buf handle */ >>>> #define TEE_SHM_EXT_DMA_BUF BIT(2) /* Memory with dma-buf handle */ >>>> >>>> +struct device; >>>> struct tee_device; >>>> struct tee_shm; >>>> struct tee_shm_pool; >>>> >> >> Looks good to me. >> >> Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org> > > Can you pick up the patch in your git tree and send a pull request for > the appropriate release (4.12-fixes or 4.13)? > > If you don't expect to send anything else for tee in that release, > you can also forward the patch to arm at kernel.org and ask for > inclusion. If you just reply with the 'Reviewed-by', I would not > expect to have to do anything in the arm-soc tree. > > Arnd > ^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH] tee: add forward declaration for struct device 2017-06-14 11:06 ` Jerome Forissier @ 2017-06-14 11:59 ` Arnd Bergmann 0 siblings, 0 replies; 6+ messages in thread From: Arnd Bergmann @ 2017-06-14 11:59 UTC (permalink / raw) To: linux-arm-kernel On Wed, Jun 14, 2017 at 1:06 PM, Jerome Forissier <jerome.forissier@linaro.org> wrote: > On 06/14/2017 01:01 PM, Arnd Bergmann wrote: >> On Wed, Jun 14, 2017 at 11:46 AM, Jens Wiklander >> <jens.wiklander@linaro.org> wrote: >>> On Wed, Jun 14, 2017 at 11:39:50AM +0200, Jerome Forissier wrote: >>>> [+Arnd] >>>> >>>> Ping? >>>> >>>> Thanks, >>>> -- >>>> Jerome >>>> >>>> On 05/31/2017 01:21 PM, Jerome Forissier wrote: >>>>> tee_drv.h references struct device, but does not include device.h nor >>>>> platform_device.h. Therefore, if tee_drv.h is included by some file >>>>> that does not pull device.h nor platform_device.h beforehand, we have a >>>>> compile warning. Fix this by adding a forward declaration. >>>>> >>>>> Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org> >> >> Acked-by: Arnd Bergmann <arnd@arndb.de> >> >> Do we need this to fix a warning in mainline, in linux-next, or only >> in combination with some other patches? > > Only with some other patches. Ok, in that case, Jens can just queue up the patch in his tree now, and it will be there in time when other patches get applied on top, no need to have this in arm-soc by itself. Thanks, Arnd ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2017-06-14 11:59 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2017-05-31 11:21 [PATCH] tee: add forward declaration for struct device Jerome Forissier 2017-06-14 9:39 ` Jerome Forissier 2017-06-14 9:46 ` Jens Wiklander 2017-06-14 11:01 ` Arnd Bergmann 2017-06-14 11:06 ` Jerome Forissier 2017-06-14 11:59 ` Arnd Bergmann
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox