* linux-next: build failure after merge of the drm tree
@ 2023-01-19 2:12 Stephen Rothwell
2023-01-19 22:15 ` [Intel-gfx] " Rodrigo Vivi
0 siblings, 1 reply; 2+ messages in thread
From: Stephen Rothwell @ 2023-01-19 2:12 UTC (permalink / raw)
To: Dave Airlie, Daniel Vetter
Cc: Thomas Zimmermann, Intel Graphics, DRI,
Linux Kernel Mailing List, Linux Next Mailing List
[-- Attachment #1: Type: text/plain, Size: 3631 bytes --]
Hi all,
After merging the drm tree, today's linux-next build (x86_64 allmodconfig)
failed like this:
In file included from drivers/gpu/drm/drm_fb_helper.c:33:
drivers/gpu/drm/drm_fb_helper.c: In function 'drm_fb_helper_single_fb_probe':
drivers/gpu/drm/drm_fb_helper.c:1968:24: error: 'dev' undeclared (first use in this function); did you mean 'cdev'?
1968 | if (dev_is_pci(dev->dev))
| ^~~
include/linux/pci.h:1151:25: note: in definition of macro 'dev_is_pci'
1151 | #define dev_is_pci(d) ((d)->bus == &pci_bus_type)
| ^
drivers/gpu/drm/drm_fb_helper.c:1968:24: note: each undeclared identifier is reported only once for each function it appears in
1968 | if (dev_is_pci(dev->dev))
| ^~~
include/linux/pci.h:1151:25: note: in definition of macro 'dev_is_pci'
1151 | #define dev_is_pci(d) ((d)->bus == &pci_bus_type)
| ^
In file included from include/linux/atomic/atomic-instrumented.h:20,
from include/linux/atomic.h:82,
from include/linux/console.h:17,
from drivers/gpu/drm/drm_fb_helper.c:32:
include/linux/compiler_types.h:299:27: error: expression in static assertion is not an integer
299 | #define __same_type(a, b) __builtin_types_compatible_p(typeof(a), typeof(b))
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/build_bug.h:78:56: note: in definition of macro '__static_assert'
78 | #define __static_assert(expr, msg, ...) _Static_assert(expr, msg)
| ^~~~
include/linux/container_of.h:20:9: note: in expansion of macro 'static_assert'
20 | static_assert(__same_type(*(ptr), ((type *)0)->member) || \
| ^~~~~~~~~~~~~
include/linux/container_of.h:20:23: note: in expansion of macro '__same_type'
20 | static_assert(__same_type(*(ptr), ((type *)0)->member) || \
| ^~~~~~~~~~~
include/linux/pci.h:541:23: note: in expansion of macro 'container_of'
541 | #define to_pci_dev(n) container_of(n, struct pci_dev, dev)
| ^~~~~~~~~~~~
drivers/gpu/drm/drm_fb_helper.c:1969:46: note: in expansion of macro 'to_pci_dev'
1969 | vga_switcheroo_client_fb_set(to_pci_dev(dev->dev), fb_helper->info);
| ^~~~~~~~~~
Caused by commit
cff84bac9922 ("drm/fh-helper: Split fbdev single-probe helper")
interacting with commit
d1d5101452ab ("drm/fb-helper: Set framebuffer for vga-switcheroo clients")
from the drm-misc-fixes tree.
I have added the following merge fix patch.
From: Stephen Rothwell <sfr@canb.auug.org.au>
Date: Thu, 19 Jan 2023 12:42:56 +1100
Subject: [PATCH] fix up for "drm/fb-helper: Set framebuffer for vga-switcheroo clients"
Signed-off-by: Stephen Rothwell <sfr@canb.auug.org.au>
---
drivers/gpu/drm/drm_fb_helper.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/gpu/drm/drm_fb_helper.c b/drivers/gpu/drm/drm_fb_helper.c
index 97049a26fca2..5e445c61252d 100644
--- a/drivers/gpu/drm/drm_fb_helper.c
+++ b/drivers/gpu/drm/drm_fb_helper.c
@@ -1942,6 +1942,7 @@ static int drm_fb_helper_single_fb_probe(struct drm_fb_helper *fb_helper,
int preferred_bpp)
{
struct drm_client_dev *client = &fb_helper->client;
+ struct drm_device *dev = fb_helper->dev;
struct drm_fb_helper_surface_size sizes;
int ret;
--
2.35.1
--
Cheers,
Stephen Rothwell
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [Intel-gfx] linux-next: build failure after merge of the drm tree
2023-01-19 2:12 linux-next: build failure after merge of the drm tree Stephen Rothwell
@ 2023-01-19 22:15 ` Rodrigo Vivi
0 siblings, 0 replies; 2+ messages in thread
From: Rodrigo Vivi @ 2023-01-19 22:15 UTC (permalink / raw)
To: Stephen Rothwell
Cc: Dave Airlie, Daniel Vetter, Intel Graphics,
Linux Next Mailing List, DRI, Thomas Zimmermann,
Linux Kernel Mailing List
On Thu, Jan 19, 2023 at 01:12:22PM +1100, Stephen Rothwell wrote:
> Hi all,
>
> After merging the drm tree, today's linux-next build (x86_64 allmodconfig)
> failed like this:
>
> In file included from drivers/gpu/drm/drm_fb_helper.c:33:
> drivers/gpu/drm/drm_fb_helper.c: In function 'drm_fb_helper_single_fb_probe':
> drivers/gpu/drm/drm_fb_helper.c:1968:24: error: 'dev' undeclared (first use in this function); did you mean 'cdev'?
It's a silent conflict of
cff84bac9922 ("drm/fh-helper: Split fbdev single-probe helper")
with
d1d5101452ab ("drm/fb-helper: Set framebuffer for vga-switcheroo clients")
The resolution is to reintroduce the dev declaration there in
drm_fb_helper_single_fb_probe()
+ struct drm_device *dev = fb_helper->dev;
I hope it helps.
Thanks,
Rodrigo.
> 1968 | if (dev_is_pci(dev->dev))
> | ^~~
> include/linux/pci.h:1151:25: note: in definition of macro 'dev_is_pci'
> 1151 | #define dev_is_pci(d) ((d)->bus == &pci_bus_type)
> | ^
> drivers/gpu/drm/drm_fb_helper.c:1968:24: note: each undeclared identifier is reported only once for each function it appears in
> 1968 | if (dev_is_pci(dev->dev))
> | ^~~
> include/linux/pci.h:1151:25: note: in definition of macro 'dev_is_pci'
> 1151 | #define dev_is_pci(d) ((d)->bus == &pci_bus_type)
> | ^
> In file included from include/linux/atomic/atomic-instrumented.h:20,
> from include/linux/atomic.h:82,
> from include/linux/console.h:17,
> from drivers/gpu/drm/drm_fb_helper.c:32:
> include/linux/compiler_types.h:299:27: error: expression in static assertion is not an integer
> 299 | #define __same_type(a, b) __builtin_types_compatible_p(typeof(a), typeof(b))
> | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
> include/linux/build_bug.h:78:56: note: in definition of macro '__static_assert'
> 78 | #define __static_assert(expr, msg, ...) _Static_assert(expr, msg)
> | ^~~~
> include/linux/container_of.h:20:9: note: in expansion of macro 'static_assert'
> 20 | static_assert(__same_type(*(ptr), ((type *)0)->member) || \
> | ^~~~~~~~~~~~~
> include/linux/container_of.h:20:23: note: in expansion of macro '__same_type'
> 20 | static_assert(__same_type(*(ptr), ((type *)0)->member) || \
> | ^~~~~~~~~~~
> include/linux/pci.h:541:23: note: in expansion of macro 'container_of'
> 541 | #define to_pci_dev(n) container_of(n, struct pci_dev, dev)
> | ^~~~~~~~~~~~
> drivers/gpu/drm/drm_fb_helper.c:1969:46: note: in expansion of macro 'to_pci_dev'
> 1969 | vga_switcheroo_client_fb_set(to_pci_dev(dev->dev), fb_helper->info);
> | ^~~~~~~~~~
>
> Caused by commit
>
> cff84bac9922 ("drm/fh-helper: Split fbdev single-probe helper")
>
> interacting with commit
>
> d1d5101452ab ("drm/fb-helper: Set framebuffer for vga-switcheroo clients")
>
> from the drm-misc-fixes tree.
>
> I have added the following merge fix patch.
>
> From: Stephen Rothwell <sfr@canb.auug.org.au>
> Date: Thu, 19 Jan 2023 12:42:56 +1100
> Subject: [PATCH] fix up for "drm/fb-helper: Set framebuffer for vga-switcheroo clients"
>
> Signed-off-by: Stephen Rothwell <sfr@canb.auug.org.au>
> ---
> drivers/gpu/drm/drm_fb_helper.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/gpu/drm/drm_fb_helper.c b/drivers/gpu/drm/drm_fb_helper.c
> index 97049a26fca2..5e445c61252d 100644
> --- a/drivers/gpu/drm/drm_fb_helper.c
> +++ b/drivers/gpu/drm/drm_fb_helper.c
> @@ -1942,6 +1942,7 @@ static int drm_fb_helper_single_fb_probe(struct drm_fb_helper *fb_helper,
> int preferred_bpp)
> {
> struct drm_client_dev *client = &fb_helper->client;
> + struct drm_device *dev = fb_helper->dev;
> struct drm_fb_helper_surface_size sizes;
> int ret;
>
> --
> 2.35.1
>
> --
> Cheers,
> Stephen Rothwell
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2023-01-19 22:33 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-01-19 2:12 linux-next: build failure after merge of the drm tree Stephen Rothwell
2023-01-19 22:15 ` [Intel-gfx] " Rodrigo Vivi
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).