Re: [PATCH] tee: add forward declaration for struct device

From: Arnd Bergmann
Date: Wed Jun 14 2017 - 07:59:49 EST


On Wed, Jun 14, 2017 at 1:06 PM, Jerome Forissier
<jerome.forissier@xxxxxxxxxx> wrote:
> On 06/14/2017 01:01 PM, Arnd Bergmann wrote:
>> On Wed, Jun 14, 2017 at 11:46 AM, Jens Wiklander
>> <jens.wiklander@xxxxxxxxxx> 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@xxxxxxxxxx>
>>
>> Acked-by: Arnd Bergmann <arnd@xxxxxxxx>
>>
>> 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