2022-08-30 09:35:31

by Sudip Mukherjee

[permalink] [raw]
Subject: build failure of next-20220830 due to 5f8cdece42ff ("drm/msm/dsi: switch to DRM_PANEL_BRIDGE")

Hi All,

The builds of arm64 allmodconfig with clang have failed to build
next-20220830 with the error:

drivers/gpu/drm/msm/dsi/dsi_host.c:1903:14: error: variable 'device_node' is uninitialized when used here [-Werror,-Wuninitialized]
of_node_put(device_node);
^~~~~~~~~~~
drivers/gpu/drm/msm/dsi/dsi_host.c:1870:44: note: initialize the variable 'device_node' to silence this warning
struct device_node *endpoint, *device_node;
^
= NULL

git bisect pointed to 5f8cdece42ff ("drm/msm/dsi: switch to DRM_PANEL_BRIDGE")

I will be happy to test any patch or provide any extra log if needed.


--
Regards
Sudip


2022-08-30 12:44:25

by Nathan Chancellor

[permalink] [raw]
Subject: Re: build failure of next-20220830 due to 5f8cdece42ff ("drm/msm/dsi: switch to DRM_PANEL_BRIDGE")

Hi Sudip,

On Tue, Aug 30, 2022 at 10:18:43AM +0100, Sudip Mukherjee (Codethink) wrote:
> Hi All,
>
> The builds of arm64 allmodconfig with clang have failed to build
> next-20220830 with the error:
>
> drivers/gpu/drm/msm/dsi/dsi_host.c:1903:14: error: variable 'device_node' is uninitialized when used here [-Werror,-Wuninitialized]
> of_node_put(device_node);
> ^~~~~~~~~~~
> drivers/gpu/drm/msm/dsi/dsi_host.c:1870:44: note: initialize the variable 'device_node' to silence this warning
> struct device_node *endpoint, *device_node;
> ^
> = NULL
>
> git bisect pointed to 5f8cdece42ff ("drm/msm/dsi: switch to DRM_PANEL_BRIDGE")
>
> I will be happy to test any patch or provide any extra log if needed.

Thanks for the report. I noticed this yesterday and sent a patch:

https://lore.kernel.org/[email protected]/
https://github.com/ClangBuiltLinux/linux/issues/1700

Cheers,
Nathan

2022-08-30 21:08:59

by Dmitry Baryshkov

[permalink] [raw]
Subject: Re: build failure of next-20220830 due to 5f8cdece42ff ("drm/msm/dsi: switch to DRM_PANEL_BRIDGE")

Hi,

On 30/08/2022 15:40, Nathan Chancellor wrote:
> Hi Sudip,
>
> On Tue, Aug 30, 2022 at 10:18:43AM +0100, Sudip Mukherjee (Codethink) wrote:
>> Hi All,
>>
>> The builds of arm64 allmodconfig with clang have failed to build
>> next-20220830 with the error:
>>
>> drivers/gpu/drm/msm/dsi/dsi_host.c:1903:14: error: variable 'device_node' is uninitialized when used here [-Werror,-Wuninitialized]
>> of_node_put(device_node);
>> ^~~~~~~~~~~
>> drivers/gpu/drm/msm/dsi/dsi_host.c:1870:44: note: initialize the variable 'device_node' to silence this warning
>> struct device_node *endpoint, *device_node;
>> ^
>> = NULL
>>
>> git bisect pointed to 5f8cdece42ff ("drm/msm/dsi: switch to DRM_PANEL_BRIDGE")
>>
>> I will be happy to test any patch or provide any extra log if needed.
>
> Thanks for the report. I noticed this yesterday and sent a patch:
>
> https://lore.kernel.org/[email protected]/
> https://github.com/ClangBuiltLinux/linux/issues/1700

Updated and pushed the branch to unbreak the linux-next. Thank you!

>
> Cheers,
> Nathan

--
With best wishes
Dmitry