2024-04-26 19:28:44

by Anatoliy Klymenko

[permalink] [raw]
Subject: [PATCH v3 0/2] Fix Kernel CI issues

Fix number of CI reported W=1 build issues.

Patch 1/2: Fix function arguments description.
Closes: https://lore.kernel.org/oe-kbuild-all/[email protected]/

Patch 2/2: Fix clang compilation error.
Closes: https://lore.kernel.org/oe-kbuild-all/[email protected]/

Signed-off-by: Anatoliy Klymenko <[email protected]>
---
Changes in v3:
- Add Signed-off-by tag.

- Link to v2: https://lore.kernel.org/r/[email protected]

Changes in v2:
- Compilation error fix added.

- Link to v1: https://lore.kernel.org/r/[email protected]

---
Anatoliy Klymenko (2):
drm: xlnx: zynqmp_dpsub: Fix few function comments
drm: xlnx: zynqmp_dpsub: Fix compilation error

drivers/gpu/drm/xlnx/zynqmp_disp.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
---
base-commit: 2bdb481bf7a93c22b9fea8daefa2834aab23a70f
change-id: 20240425-dp-live-fmt-fix-a10bf7973596

Best regards,
--
Anatoliy Klymenko <[email protected]>



2024-04-27 07:48:29

by Tomi Valkeinen

[permalink] [raw]
Subject: Re: [PATCH v3 0/2] Fix Kernel CI issues

On 26/04/2024 22:27, Anatoliy Klymenko wrote:
> Fix number of CI reported W=1 build issues.
>
> Patch 1/2: Fix function arguments description.
> Closes: https://lore.kernel.org/oe-kbuild-all/[email protected]/
>
> Patch 2/2: Fix clang compilation error.
> Closes: https://lore.kernel.org/oe-kbuild-all/[email protected]/
>
> Signed-off-by: Anatoliy Klymenko <[email protected]>
> ---
> Changes in v3:
> - Add Signed-off-by tag.
>
> - Link to v2: https://lore.kernel.org/r/[email protected]
>
> Changes in v2:
> - Compilation error fix added.
>
> - Link to v1: https://lore.kernel.org/r/[email protected]
>
> ---
> Anatoliy Klymenko (2):
> drm: xlnx: zynqmp_dpsub: Fix few function comments
> drm: xlnx: zynqmp_dpsub: Fix compilation error
>
> drivers/gpu/drm/xlnx/zynqmp_disp.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
> ---
> base-commit: 2bdb481bf7a93c22b9fea8daefa2834aab23a70f
> change-id: 20240425-dp-live-fmt-fix-a10bf7973596
>
> Best regards,

Thanks, pushed to drm-misc-next.

Tomi


2024-05-03 16:27:43

by Nathan Chancellor

[permalink] [raw]
Subject: Re: [PATCH v3 0/2] Fix Kernel CI issues

Hi Tomi,

On Sat, Apr 27, 2024 at 10:48:16AM +0300, Tomi Valkeinen wrote:
> On 26/04/2024 22:27, Anatoliy Klymenko wrote:
> > Fix number of CI reported W=1 build issues.
> >
> > Patch 1/2: Fix function arguments description.
> > Closes: https://lore.kernel.org/oe-kbuild-all/[email protected]/
> >
> > Patch 2/2: Fix clang compilation error.
> > Closes: https://lore.kernel.org/oe-kbuild-all/[email protected]/
> >
> > Signed-off-by: Anatoliy Klymenko <[email protected]>
> > ---
> > Changes in v3:
> > - Add Signed-off-by tag.
> >
> > - Link to v2: https://lore.kernel.org/r/[email protected]
> >
> > Changes in v2:
> > - Compilation error fix added.
> >
> > - Link to v1: https://lore.kernel.org/r/[email protected]
> >
> > ---
> > Anatoliy Klymenko (2):
> > drm: xlnx: zynqmp_dpsub: Fix few function comments
> > drm: xlnx: zynqmp_dpsub: Fix compilation error
> >
> > drivers/gpu/drm/xlnx/zynqmp_disp.c | 6 +++---
> > 1 file changed, 3 insertions(+), 3 deletions(-)
> > ---
> > base-commit: 2bdb481bf7a93c22b9fea8daefa2834aab23a70f
> > change-id: 20240425-dp-live-fmt-fix-a10bf7973596
> >
> > Best regards,
>
> Thanks, pushed to drm-misc-next.

I think the second patch also needs to go to drm-misc-next-fixes? The
clang warning fixed by it has returned in next-20240503 because it
appears that for-linux-next was switch from drm-misc-next to
drm-misc-next-fixes, as I see for-linux-next was pointing to commit
235e60653f8d ("drm/debugfs: Drop conditionals around of_node pointers")
on drm-misc-next in next-20240502 but it is now pointing to commit
be3f3042391d ("drm: zynqmp_dpsub: Always register bridge") on
drm-misc-next-fixes in next-20240503.

Cheers,
Nathan

2024-05-03 17:28:54

by Tomi Valkeinen

[permalink] [raw]
Subject: Re: [PATCH v3 0/2] Fix Kernel CI issues

On 03/05/2024 19:27, Nathan Chancellor wrote:
> Hi Tomi,
>
> On Sat, Apr 27, 2024 at 10:48:16AM +0300, Tomi Valkeinen wrote:
>> On 26/04/2024 22:27, Anatoliy Klymenko wrote:
>>> Fix number of CI reported W=1 build issues.
>>>
>>> Patch 1/2: Fix function arguments description.
>>> Closes: https://lore.kernel.org/oe-kbuild-all/[email protected]/
>>>
>>> Patch 2/2: Fix clang compilation error.
>>> Closes: https://lore.kernel.org/oe-kbuild-all/[email protected]/
>>>
>>> Signed-off-by: Anatoliy Klymenko <[email protected]>
>>> ---
>>> Changes in v3:
>>> - Add Signed-off-by tag.
>>>
>>> - Link to v2: https://lore.kernel.org/r/[email protected]
>>>
>>> Changes in v2:
>>> - Compilation error fix added.
>>>
>>> - Link to v1: https://lore.kernel.org/r/[email protected]
>>>
>>> ---
>>> Anatoliy Klymenko (2):
>>> drm: xlnx: zynqmp_dpsub: Fix few function comments
>>> drm: xlnx: zynqmp_dpsub: Fix compilation error
>>>
>>> drivers/gpu/drm/xlnx/zynqmp_disp.c | 6 +++---
>>> 1 file changed, 3 insertions(+), 3 deletions(-)
>>> ---
>>> base-commit: 2bdb481bf7a93c22b9fea8daefa2834aab23a70f
>>> change-id: 20240425-dp-live-fmt-fix-a10bf7973596
>>>
>>> Best regards,
>>
>> Thanks, pushed to drm-misc-next.
>
> I think the second patch also needs to go to drm-misc-next-fixes? The
> clang warning fixed by it has returned in next-20240503 because it
> appears that for-linux-next was switch from drm-misc-next to
> drm-misc-next-fixes, as I see for-linux-next was pointing to commit
> 235e60653f8d ("drm/debugfs: Drop conditionals around of_node pointers")
> on drm-misc-next in next-20240502 but it is now pointing to commit
> be3f3042391d ("drm: zynqmp_dpsub: Always register bridge") on
> drm-misc-next-fixes in next-20240503.

Oh. Hmm, did I just hit the feature-freeze point with the fixes...

Now I'm unsure where I should push these (if anywhere), as they already
are in drm-misc-next.

DRM Misc maintainers, can you give me a hint? =)

Tomi


2024-05-17 20:27:07

by Jakub Kicinski

[permalink] [raw]
Subject: Re: [PATCH v3 0/2] Fix Kernel CI issues

On Fri, 3 May 2024 20:25:49 +0300 Tomi Valkeinen wrote:
> > I think the second patch also needs to go to drm-misc-next-fixes? The
> > clang warning fixed by it has returned in next-20240503 because it
> > appears that for-linux-next was switch from drm-misc-next to
> > drm-misc-next-fixes, as I see for-linux-next was pointing to commit
> > 235e60653f8d ("drm/debugfs: Drop conditionals around of_node pointers")
> > on drm-misc-next in next-20240502 but it is now pointing to commit
> > be3f3042391d ("drm: zynqmp_dpsub: Always register bridge") on
> > drm-misc-next-fixes in next-20240503.
>
> Oh. Hmm, did I just hit the feature-freeze point with the fixes...
>
> Now I'm unsure where I should push these (if anywhere), as they already
> are in drm-misc-next.
>
> DRM Misc maintainers, can you give me a hint? =)

This is now breaking allmodconfig build of Linus's tree.
Could you please get it fixed ASAP?