Re: next build: 9 warnings 0 failures (next/next-20170215)

From: Hans Verkuil
Date: Wed Feb 15 2017 - 03:42:31 EST


On 02/15/2017 09:39 AM, Arnd Bergmann wrote:
> On Wed, Feb 15, 2017 at 9:30 AM, Hans Verkuil <hverkuil@xxxxxxxxx> wrote:
>> On 02/15/2017 09:24 AM, Arnd Bergmann wrote:
>>> On Wed, Feb 15, 2017 at 9:02 AM, Olof's autobuilder <build@xxxxxxxxx> wrote:
>
>>>> 1 drivers/media/platform/coda/imx-vdoa.c:333:571: warning: passing argument 1 of 'platform_driver_unregister' discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
>>>> 1 drivers/media/platform/coda/imx-vdoa.c:333:625: warning: passing argument 1 of 'platform_driver_unregister' discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
>>>> 1 drivers/usb/gadget/udc/atmel_usba_udc.c:636:38: warning: 'ept_cfg' may be used uninitialized in this function [-Wmaybe-uninitialized]
>>>> 2 drivers/media/platform/coda/imx-vdoa.c:333:181: warning: passing argument 1 of '__platform_driver_register' discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
>>>
>>> This was "[media] coda/imx-vdoa: platform_driver should not be const",
>>> https://patchwork.linuxtv.org/patch/39288/ Hans already marked this as
>>> 'merged', so I assume it's on its way in, but just hasn't appeared in
>>> linux-next as of today.
>>
>> It's part of a pull request for 4.12 for Mauro to pick up. If this should
>> go into 4.10 or 4.11, then Greg should take this particular patch. Mauro is moving
>> to a new house this week and won't have time (or quite possibly even network
>> connectivity).
>
> Ok, good to know, thanks for your quick reply. I checked the commit
> that caused the
> warning, d2fe28feaebb ("[media] coda/imx-vdoa: constify structs"), and
> it is currently
> part of git://linuxtv.org/media_tree.git#master . My fix should go on
> top of this branch
> and picked up by whoever is going to send it during the merge window.
> As I'm reverting
> half of Mauro's patch, we can't merge it through any other tree at the moment.

OK, we'll take care of this in that case.

I hope I'll remember this :-)

Regards,

Hans