Re: v4.7-rc2 build: 0 failures 5 warnings (v4.7-rc2)

Re: v4.7-rc2 build: 0 failures 5 warnings (v4.7-rc2)

[Arnd Bergmann]

On Monday, June 6, 2016 2:09:07 AM CEST Build bot for Mark Brown wrote:
> arm-allmodconfig : PASS, 0 errors, 2 warnings, 0 section mismatches
> 
> Warnings:
>         ../drivers/gpu/drm/omapdrm/dss/dsi.c:1170:6: warning: unused variable 'r' [-Wunused-variable]
>         ../drivers/gpu/drm/omapdrm/dss/hdmi5.c:123:6: warning: unused variable 'r' [-Wunused-variable]

These got introduced by Fixes: 973999aa0140 ("drm/omap: Remove
regulator API abuse") in 4.7-rc2, and are trivial to fix, I can provide a
patch if needed.

I also have a couple of other regression fixes in my local tree 
that I hope to see in 4.7, but with DRM I never know when to resend
them, as they sometimes do not show up in linux-next before they get
merged into mainline.

This is my current backlog:

d703eb3c7606 [IN NEXT] drm: sun4i: fix probe error handling
ee0d3dafedb7 [IN NEXT] drm: sun4i: print DMA address correctly
4b51817edcd0 [IN NEXT] drm/sun4i: add COMMON_CLK dependency
04197e0d8201 [SUBMITTED 20160516] drm: exynos: mark pm functions as __maybe_unused
1aa3f513d264 [SUBMITTED 20160516] drm: mediatek: add CONFIG_OF dependency
dee4d1c31214 [SUBMITTED 20160511] drm: mediatek: remove IOMMU_DMA select
845cd86005ea [SUBMITTED 20160511] drm/omap: include linux/of.h where needed
d657e5dd56c5 [SUBMITTED 20160511] drm: mediatek: add COMMON_CLK dependency
539c2a57be6f [SUBMITTED x2 20160225] drm: avoid uninitialized timestamp use in wait_vblank
921b0acae44c [SUBMITTED 2015012drm: armada: use dma_addr_t properly

Am I right to assume that as the first eight that are all regressions against
4.6 will show up in 4.7 without me doing anything further, but that I should
resend the last two so they can make it into 4.8, and get backported if
necessary?

	Arnd
_______________________________________________
Kernel-build-reports mailing list
Kernel-build-reports@lists.linaro.org
https://lists.linaro.org/mailman/listinfo/kernel-build-reports

Re: v4.7-rc2 build: 0 failures 5 warnings (v4.7-rc2)

[Peter Ujfalusi]

On 06/06/16 12:44, Arnd Bergmann wrote:
> On Monday, June 6, 2016 2:09:07 AM CEST Build bot for Mark Brown wrote:
>> arm-allmodconfig : PASS, 0 errors, 2 warnings, 0 section mismatches
>>
>> Warnings:
>>         ../drivers/gpu/drm/omapdrm/dss/dsi.c:1170:6: warning: unused variable 'r' [-Wunused-variable]
>>         ../drivers/gpu/drm/omapdrm/dss/hdmi5.c:123:6: warning: unused variable 'r' [-Wunused-variable]
> 
> These got introduced by Fixes: 973999aa0140 ("drm/omap: Remove
> regulator API abuse") in 4.7-rc2, and are trivial to fix, I can provide a
> patch if needed.

Hrm, I don't remember sending that patch, I have sent:
https://lkml.org/lkml/2016/4/28/361

The patch I have sent did not have 'r' unused.

I can send follow-up patches to fix the unused variable.

> 
> I also have a couple of other regression fixes in my local tree 
> that I hope to see in 4.7, but with DRM I never know when to resend
> them, as they sometimes do not show up in linux-next before they get
> merged into mainline.
> 
> This is my current backlog:
> 
> d703eb3c7606 [IN NEXT] drm: sun4i: fix probe error handling
> ee0d3dafedb7 [IN NEXT] drm: sun4i: print DMA address correctly
> 4b51817edcd0 [IN NEXT] drm/sun4i: add COMMON_CLK dependency
> 04197e0d8201 [SUBMITTED 20160516] drm: exynos: mark pm functions as __maybe_unused
> 1aa3f513d264 [SUBMITTED 20160516] drm: mediatek: add CONFIG_OF dependency
> dee4d1c31214 [SUBMITTED 20160511] drm: mediatek: remove IOMMU_DMA select
> 845cd86005ea [SUBMITTED 20160511] drm/omap: include linux/of.h where needed
> d657e5dd56c5 [SUBMITTED 20160511] drm: mediatek: add COMMON_CLK dependency
> 539c2a57be6f [SUBMITTED x2 20160225] drm: avoid uninitialized timestamp use in wait_vblank
> 921b0acae44c [SUBMITTED 2015012drm: armada: use dma_addr_t properly
> 
> Am I right to assume that as the first eight that are all regressions against
> 4.6 will show up in 4.7 without me doing anything further, but that I should
> resend the last two so they can make it into 4.8, and get backported if
> necessary?
> 
> 	Arnd
> _______________________________________________
> Kernel-build-reports mailing list
> Kernel-build-reports@lists.linaro.org
> https://lists.linaro.org/mailman/listinfo/kernel-build-reports
> 


-- 
Péter

Re: v4.7-rc2 build: 0 failures 5 warnings (v4.7-rc2)

[Tomi Valkeinen]

[-- Attachment #1.1: Type: text/plain, Size: 1357 bytes --]



On 06/06/16 13:08, Peter Ujfalusi wrote:
> On 06/06/16 12:44, Arnd Bergmann wrote:
>> On Monday, June 6, 2016 2:09:07 AM CEST Build bot for Mark Brown wrote:
>>> arm-allmodconfig : PASS, 0 errors, 2 warnings, 0 section mismatches
>>>
>>> Warnings:
>>>         ../drivers/gpu/drm/omapdrm/dss/dsi.c:1170:6: warning: unused variable 'r' [-Wunused-variable]
>>>         ../drivers/gpu/drm/omapdrm/dss/hdmi5.c:123:6: warning: unused variable 'r' [-Wunused-variable]
>>
>> These got introduced by Fixes: 973999aa0140 ("drm/omap: Remove
>> regulator API abuse") in 4.7-rc2, and are trivial to fix, I can provide a
>> patch if needed.
> 
> Hrm, I don't remember sending that patch, I have sent:
> https://lkml.org/lkml/2016/4/28/361
> 
> The patch I have sent did not have 'r' unused.
> 
> I can send follow-up patches to fix the unused variable.

Oops.

There was a partial cleanup going for 4.7 merge window, but it missed
that. But arch/arm side cleanups went in, so I was able to do proper
cleanup. But in the end I somehow managed to mess it up, and the 'int
r's were left. Sorry about that.

I have sent a fix patch for those last week.

>> 845cd86005ea [SUBMITTED 20160511] drm/omap: include linux/of.h where needed

This one shouldn't be necessary, as there's d9e32ecda41b ("drm/omap: Fix
missing includes").

 Tomi


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

Re: v4.7-rc2 build: 0 failures 5 warnings (v4.7-rc2)

[Arnd Bergmann]

On Monday, June 6, 2016 1:13:03 PM CEST Tomi Valkeinen wrote:
> 
> >> 845cd86005ea [SUBMITTED 20160511] drm/omap: include linux/of.h where needed
> 
> This one shouldn't be necessary, as there's d9e32ecda41b ("drm/omap: Fix
> missing includes").
> 

Ok, I've dropped mine now, thanks for having a look.

	Arnd