Re: linux-next: Tree for Jul 13 (drivers/video/fbdev/ps3fb.c)
Hi, Am 13.07.23 um 18:11 schrieb Randy Dunlap: On 7/12/23 19:37, Stephen Rothwell wrote: Hi all, sorry, I've been AFK for a bit. I'll send a fix soon. Best regards Thomas Changes since 20230712: on ppc64: In file included from ../include/linux/device.h:15, from ../arch/powerpc/include/asm/io.h:22, from ../include/linux/io.h:13, from ../include/linux/irq.h:20, from ../arch/powerpc/include/asm/hardirq.h:6, from ../include/linux/hardirq.h:11, from ../include/linux/interrupt.h:11, from ../drivers/video/fbdev/ps3fb.c:25: ../drivers/video/fbdev/ps3fb.c: In function 'ps3fb_probe': ../drivers/video/fbdev/ps3fb.c:1172:40: error: 'struct fb_info' has no member named 'dev' 1172 | dev_driver_string(info->dev), dev_name(info->dev), |^~ ../include/linux/dev_printk.h:110:37: note: in definition of macro 'dev_printk_index_wrap' 110 | _p_func(dev, fmt, ##__VA_ARGS__); \ | ^~~ ../drivers/video/fbdev/ps3fb.c:1171:9: note: in expansion of macro 'dev_info' 1171 | dev_info(info->device, "%s %s, using %u KiB of video memory\n", | ^~~~ ../drivers/video/fbdev/ps3fb.c:1172:61: error: 'struct fb_info' has no member named 'dev' 1172 | dev_driver_string(info->dev), dev_name(info->dev), | ^~ ../include/linux/dev_printk.h:110:37: note: in definition of macro 'dev_printk_index_wrap' 110 | _p_func(dev, fmt, ##__VA_ARGS__); \ | ^~~ ../drivers/video/fbdev/ps3fb.c:1171:9: note: in expansion of macro 'dev_info' 1171 | dev_info(info->device, "%s %s, using %u KiB of video memory\n", | ^~~~ Full randconfig file is attached. -- Thomas Zimmermann Graphics Driver Developer SUSE Software Solutions Germany GmbH Frankenstrasse 146, 90461 Nuernberg, Germany GF: Ivo Totev, Andrew Myers, Andrew McDonald, Boudien Moerman HRB 36809 (AG Nuernberg) OpenPGP_signature Description: OpenPGP digital signature
Re: linux-next: Tree for Jul 13 (drivers/video/fbdev/ps3fb.c)
On 7/18/23 13:48, Michael Ellerman wrote: Bagas Sanjaya writes: On Thu, Jul 13, 2023 at 09:11:10AM -0700, Randy Dunlap wrote: on ppc64: In file included from ../include/linux/device.h:15, from ../arch/powerpc/include/asm/io.h:22, from ../include/linux/io.h:13, from ../include/linux/irq.h:20, from ../arch/powerpc/include/asm/hardirq.h:6, from ../include/linux/hardirq.h:11, from ../include/linux/interrupt.h:11, from ../drivers/video/fbdev/ps3fb.c:25: ../drivers/video/fbdev/ps3fb.c: In function 'ps3fb_probe': ../drivers/video/fbdev/ps3fb.c:1172:40: error: 'struct fb_info' has no member named 'dev' 1172 | dev_driver_string(info->dev), dev_name(info->dev), |^~ ../include/linux/dev_printk.h:110:37: note: in definition of macro 'dev_printk_index_wrap' 110 | _p_func(dev, fmt, ##__VA_ARGS__); \ | ^~~ ../drivers/video/fbdev/ps3fb.c:1171:9: note: in expansion of macro 'dev_info' 1171 | dev_info(info->device, "%s %s, using %u KiB of video memory\n", | ^~~~ ../drivers/video/fbdev/ps3fb.c:1172:61: error: 'struct fb_info' has no member named 'dev' 1172 | dev_driver_string(info->dev), dev_name(info->dev), | ^~ ../include/linux/dev_printk.h:110:37: note: in definition of macro 'dev_printk_index_wrap' 110 | _p_func(dev, fmt, ##__VA_ARGS__); \ | ^~~ ../drivers/video/fbdev/ps3fb.c:1171:9: note: in expansion of macro 'dev_info' 1171 | dev_info(info->device, "%s %s, using %u KiB of video memory\n", | ^~~~ Hmm, there is no response from Thomas yet. I guess we should go with reverting bdb616479eff419, right? Regardless, I'm adding this build regression to regzbot so that parties involved are aware of it: #regzbot ^introduced: bdb616479eff419 #regzbot title: build regression in PS3 framebuffer Does regzbot track issues in linux-next? They're not really regressions because they're not in a release yet. Anyway I don't see where bdb616479eff419 comes from. The issue was introduced by: 701d2054fa31 fbdev: Make support for userspace interfaces configurable The driver seems to only use info->dev in that one dev_info() line, which seems purely cosmetic, so I think it could just be removed, eg: diff --git a/drivers/video/fbdev/ps3fb.c b/drivers/video/fbdev/ps3fb.c index d4abcf8aff75..a304a39d712b 100644 --- a/drivers/video/fbdev/ps3fb.c +++ b/drivers/video/fbdev/ps3fb.c @@ -1168,8 +1168,7 @@ static int ps3fb_probe(struct ps3_system_bus_device *dev) ps3_system_bus_set_drvdata(dev, info); - dev_info(info->device, "%s %s, using %u KiB of video memory\n", -dev_driver_string(info->dev), dev_name(info->dev), + dev_info(info->device, "using %u KiB of video memory\n", info->fix.smem_len >> 10); task = kthread_run(ps3fbd, info, DEVICE_NAME); Can you please resend this as proper patch to fbdev and/or drm-misc mailing lists? As it is, it never showed up for me in patchwork... Helge
Re: linux-next: Tree for Jul 13 (drivers/video/fbdev/ps3fb.c)
On 18.07.23 18:15, Randy Dunlap wrote: > On 7/18/23 04:48, Michael Ellerman wrote: >> Bagas Sanjaya writes: >>> On Thu, Jul 13, 2023 at 09:11:10AM -0700, Randy Dunlap wrote: on ppc64: In file included from ../include/linux/device.h:15, from ../arch/powerpc/include/asm/io.h:22, from ../include/linux/io.h:13, from ../include/linux/irq.h:20, from ../arch/powerpc/include/asm/hardirq.h:6, from ../include/linux/hardirq.h:11, from ../include/linux/interrupt.h:11, from ../drivers/video/fbdev/ps3fb.c:25: ../drivers/video/fbdev/ps3fb.c: In function 'ps3fb_probe': ../drivers/video/fbdev/ps3fb.c:1172:40: error: 'struct fb_info' has no member named 'dev' > [...] >> >> Does regzbot track issues in linux-next? Seems your patch didn't make any progress, at least I can't see it in -next. Is there a reason why, or did I miss anything? And yes, sure, I'm aware that it's -next and a driver that people might not enable regularly. But I noticed it and thought "quickly bring it up, might be good to fix this rather sooner than later before other people run into it (and who knows, maybe it'll switch a light in some CI system from red to green as well)" Ciao, Thorsten >> The driver seems to only use info->dev in that one dev_info() line, >> which seems purely cosmetic, so I think it could just be removed, eg: >> >> diff --git a/drivers/video/fbdev/ps3fb.c b/drivers/video/fbdev/ps3fb.c >> index d4abcf8aff75..a304a39d712b 100644 >> --- a/drivers/video/fbdev/ps3fb.c >> +++ b/drivers/video/fbdev/ps3fb.c >> @@ -1168,8 +1168,7 @@ static int ps3fb_probe(struct ps3_system_bus_device >> *dev) >> >> ps3_system_bus_set_drvdata(dev, info); >> >> -dev_info(info->device, "%s %s, using %u KiB of video memory\n", >> - dev_driver_string(info->dev), dev_name(info->dev), >> +dev_info(info->device, "using %u KiB of video memory\n", >> info->fix.smem_len >> 10); >> >> task = kthread_run(ps3fbd, info, DEVICE_NAME); > > > Tested-by: Randy Dunlap # build-tested > > Thanks. >
Re: linux-next: Tree for Jul 13 (drivers/video/fbdev/ps3fb.c)
On 19.07.23 14:36, Bagas Sanjaya wrote: > On 7/18/23 17:06, Thorsten Leemhuis wrote: >> I'm missing something here: >> >> * What makes you think this is caused by bdb616479eff419? I didn't see >> anything in the thread that claims this, but I might be missing something >> * related: if I understand Randy right, this is only happening in -next; >> so why is bdb616479eff419 the culprit, which is also in mainline since >> End of June? > > Actually drivers/video/fbdev/ps3bf.c only had two non-merge commits during > previous cycle: 25ec15abb06194 and bdb616479eff419. The former was simply > adding .owner field in ps3fb_ops (hence trivial), so I inferred that the > culprit was likely the latter (due to it was being authored by Thomas). As you can see from Michael's reply this was misguided, as it was an external change that broke the driver. This happens all the time, such inferring thus is not possible at all. Ciao, Thorsten
Re: linux-next: Tree for Jul 13 (drivers/video/fbdev/ps3fb.c)
On 7/18/23 17:06, Thorsten Leemhuis wrote: > I'm missing something here: > > * What makes you think this is caused by bdb616479eff419? I didn't see > anything in the thread that claims this, but I might be missing something > * related: if I understand Randy right, this is only happening in -next; > so why is bdb616479eff419 the culprit, which is also in mainline since > End of June? > Actually drivers/video/fbdev/ps3bf.c only had two non-merge commits during previous cycle: 25ec15abb06194 and bdb616479eff419. The former was simply adding .owner field in ps3fb_ops (hence trivial), so I inferred that the culprit was likely the latter (due to it was being authored by Thomas). Thanks for the question. -- An old man doll... just what I always wanted! - Clara
Re: linux-next: Tree for Jul 13 (drivers/video/fbdev/ps3fb.c)
On 7/18/23 04:48, Michael Ellerman wrote: > Bagas Sanjaya writes: >> On Thu, Jul 13, 2023 at 09:11:10AM -0700, Randy Dunlap wrote: >>> on ppc64: >>> >>> In file included from ../include/linux/device.h:15, >>> from ../arch/powerpc/include/asm/io.h:22, >>> from ../include/linux/io.h:13, >>> from ../include/linux/irq.h:20, >>> from ../arch/powerpc/include/asm/hardirq.h:6, >>> from ../include/linux/hardirq.h:11, >>> from ../include/linux/interrupt.h:11, >>> from ../drivers/video/fbdev/ps3fb.c:25: >>> ../drivers/video/fbdev/ps3fb.c: In function 'ps3fb_probe': >>> ../drivers/video/fbdev/ps3fb.c:1172:40: error: 'struct fb_info' has no >>> member named 'dev' >>> 1172 | dev_driver_string(info->dev), dev_name(info->dev), >>> |^~ >>> ../include/linux/dev_printk.h:110:37: note: in definition of macro >>> 'dev_printk_index_wrap' >>> 110 | _p_func(dev, fmt, ##__VA_ARGS__); >>> \ >>> | ^~~ >>> ../drivers/video/fbdev/ps3fb.c:1171:9: note: in expansion of macro >>> 'dev_info' >>> 1171 | dev_info(info->device, "%s %s, using %u KiB of video >>> memory\n", >>> | ^~~~ >>> ../drivers/video/fbdev/ps3fb.c:1172:61: error: 'struct fb_info' has no >>> member named 'dev' >>> 1172 | dev_driver_string(info->dev), dev_name(info->dev), >>> | ^~ >>> ../include/linux/dev_printk.h:110:37: note: in definition of macro >>> 'dev_printk_index_wrap' >>> 110 | _p_func(dev, fmt, ##__VA_ARGS__); >>> \ >>> | ^~~ >>> ../drivers/video/fbdev/ps3fb.c:1171:9: note: in expansion of macro >>> 'dev_info' >>> 1171 | dev_info(info->device, "%s %s, using %u KiB of video >>> memory\n", >>> | ^~~~ >>> >>> >> >> Hmm, there is no response from Thomas yet. I guess we should go with >> reverting bdb616479eff419, right? Regardless, I'm adding this build >> regression >> to regzbot so that parties involved are aware of it: >> >> #regzbot ^introduced: bdb616479eff419 >> #regzbot title: build regression in PS3 framebuffer > > Does regzbot track issues in linux-next? > > They're not really regressions because they're not in a release yet. > > Anyway I don't see where bdb616479eff419 comes from. > > The issue was introduced by: > > 701d2054fa31 fbdev: Make support for userspace interfaces configurable > > The driver seems to only use info->dev in that one dev_info() line, > which seems purely cosmetic, so I think it could just be removed, eg: > > diff --git a/drivers/video/fbdev/ps3fb.c b/drivers/video/fbdev/ps3fb.c > index d4abcf8aff75..a304a39d712b 100644 > --- a/drivers/video/fbdev/ps3fb.c > +++ b/drivers/video/fbdev/ps3fb.c > @@ -1168,8 +1168,7 @@ static int ps3fb_probe(struct ps3_system_bus_device > *dev) > > ps3_system_bus_set_drvdata(dev, info); > > - dev_info(info->device, "%s %s, using %u KiB of video memory\n", > - dev_driver_string(info->dev), dev_name(info->dev), > + dev_info(info->device, "using %u KiB of video memory\n", >info->fix.smem_len >> 10); > > task = kthread_run(ps3fbd, info, DEVICE_NAME); Tested-by: Randy Dunlap # build-tested Thanks. -- ~Randy
Re: linux-next: Tree for Jul 13 (drivers/video/fbdev/ps3fb.c)
Michael, thx for looking into this! On 18.07.23 13:48, Michael Ellerman wrote: > Bagas Sanjaya writes: >> On Thu, Jul 13, 2023 at 09:11:10AM -0700, Randy Dunlap wrote: >>> on ppc64: >>> >>> In file included from ../include/linux/device.h:15, >>> from ../arch/powerpc/include/asm/io.h:22, >>> from ../include/linux/io.h:13, >>> from ../include/linux/irq.h:20, >>> from ../arch/powerpc/include/asm/hardirq.h:6, >>> from ../include/linux/hardirq.h:11, >>> from ../include/linux/interrupt.h:11, >>> from ../drivers/video/fbdev/ps3fb.c:25: >>> ../drivers/video/fbdev/ps3fb.c: In function 'ps3fb_probe': >>> ../drivers/video/fbdev/ps3fb.c:1172:40: error: 'struct fb_info' has no >>> member named 'dev' >>> 1172 | dev_driver_string(info->dev), dev_name(info->dev), >>> |^~ >>> ../include/linux/dev_printk.h:110:37: note: in definition of macro >>> 'dev_printk_index_wrap' >>> 110 | _p_func(dev, fmt, ##__VA_ARGS__); >>> \ >>> | ^~~ >>> ../drivers/video/fbdev/ps3fb.c:1171:9: note: in expansion of macro >>> 'dev_info' >>> 1171 | dev_info(info->device, "%s %s, using %u KiB of video >>> memory\n", >>> | ^~~~ >>> ../drivers/video/fbdev/ps3fb.c:1172:61: error: 'struct fb_info' has no >>> member named 'dev' >>> 1172 | dev_driver_string(info->dev), dev_name(info->dev), >>> | ^~ >>> ../include/linux/dev_printk.h:110:37: note: in definition of macro >>> 'dev_printk_index_wrap' >>> 110 | _p_func(dev, fmt, ##__VA_ARGS__); >>> \ >>> | ^~~ >>> ../drivers/video/fbdev/ps3fb.c:1171:9: note: in expansion of macro >>> 'dev_info' >>> 1171 | dev_info(info->device, "%s %s, using %u KiB of video >>> memory\n", >>> | ^~~~ >>> >>> >> >> Hmm, there is no response from Thomas yet. I guess we should go with >> reverting bdb616479eff419, right? Regardless, I'm adding this build >> regression >> to regzbot so that parties involved are aware of it: >> >> #regzbot ^introduced: bdb616479eff419 >> #regzbot title: build regression in PS3 framebuffer > > Does regzbot track issues in linux-next? It can, I made sure of that in case somebody want to use this sooner or later (and it wasn't much work), but I don't actively use this functionally right now and do not plan to do so, there are more important issues to spend time on. > They're not really regressions because they're not in a release yet. > > Anyway I don't see where bdb616479eff419 comes from. That makes two of us :-D > The issue was introduced by: > > 701d2054fa31 fbdev: Make support for userspace interfaces configurable Ahh, that makes a lot more sense. While at it, let me tell regzbot: #regzbot introduced: 701d2054fa31 Ciao, Thorsten
Re: linux-next: Tree for Jul 13 (drivers/video/fbdev/ps3fb.c)
Bagas Sanjaya writes: > On Thu, Jul 13, 2023 at 09:11:10AM -0700, Randy Dunlap wrote: >> on ppc64: >> >> In file included from ../include/linux/device.h:15, >> from ../arch/powerpc/include/asm/io.h:22, >> from ../include/linux/io.h:13, >> from ../include/linux/irq.h:20, >> from ../arch/powerpc/include/asm/hardirq.h:6, >> from ../include/linux/hardirq.h:11, >> from ../include/linux/interrupt.h:11, >> from ../drivers/video/fbdev/ps3fb.c:25: >> ../drivers/video/fbdev/ps3fb.c: In function 'ps3fb_probe': >> ../drivers/video/fbdev/ps3fb.c:1172:40: error: 'struct fb_info' has no >> member named 'dev' >> 1172 | dev_driver_string(info->dev), dev_name(info->dev), >> |^~ >> ../include/linux/dev_printk.h:110:37: note: in definition of macro >> 'dev_printk_index_wrap' >> 110 | _p_func(dev, fmt, ##__VA_ARGS__); >>\ >> | ^~~ >> ../drivers/video/fbdev/ps3fb.c:1171:9: note: in expansion of macro 'dev_info' >> 1171 | dev_info(info->device, "%s %s, using %u KiB of video >> memory\n", >> | ^~~~ >> ../drivers/video/fbdev/ps3fb.c:1172:61: error: 'struct fb_info' has no >> member named 'dev' >> 1172 | dev_driver_string(info->dev), dev_name(info->dev), >> | ^~ >> ../include/linux/dev_printk.h:110:37: note: in definition of macro >> 'dev_printk_index_wrap' >> 110 | _p_func(dev, fmt, ##__VA_ARGS__); >>\ >> | ^~~ >> ../drivers/video/fbdev/ps3fb.c:1171:9: note: in expansion of macro 'dev_info' >> 1171 | dev_info(info->device, "%s %s, using %u KiB of video >> memory\n", >> | ^~~~ >> >> > > Hmm, there is no response from Thomas yet. I guess we should go with > reverting bdb616479eff419, right? Regardless, I'm adding this build regression > to regzbot so that parties involved are aware of it: > > #regzbot ^introduced: bdb616479eff419 > #regzbot title: build regression in PS3 framebuffer Does regzbot track issues in linux-next? They're not really regressions because they're not in a release yet. Anyway I don't see where bdb616479eff419 comes from. The issue was introduced by: 701d2054fa31 fbdev: Make support for userspace interfaces configurable The driver seems to only use info->dev in that one dev_info() line, which seems purely cosmetic, so I think it could just be removed, eg: diff --git a/drivers/video/fbdev/ps3fb.c b/drivers/video/fbdev/ps3fb.c index d4abcf8aff75..a304a39d712b 100644 --- a/drivers/video/fbdev/ps3fb.c +++ b/drivers/video/fbdev/ps3fb.c @@ -1168,8 +1168,7 @@ static int ps3fb_probe(struct ps3_system_bus_device *dev) ps3_system_bus_set_drvdata(dev, info); - dev_info(info->device, "%s %s, using %u KiB of video memory\n", -dev_driver_string(info->dev), dev_name(info->dev), + dev_info(info->device, "using %u KiB of video memory\n", info->fix.smem_len >> 10); task = kthread_run(ps3fbd, info, DEVICE_NAME); cheers
Re: linux-next: Tree for Jul 13 (drivers/video/fbdev/ps3fb.c)
On 18.07.23 05:32, Bagas Sanjaya wrote: > On Thu, Jul 13, 2023 at 09:11:10AM -0700, Randy Dunlap wrote: >> On 7/12/23 19:37, Stephen Rothwell wrote: >>> Changes since 20230712: >> >> on ppc64: >> >> In file included from ../include/linux/device.h:15, >> from ../arch/powerpc/include/asm/io.h:22, >> from ../include/linux/io.h:13, >> from ../include/linux/irq.h:20, >> from ../arch/powerpc/include/asm/hardirq.h:6, >> from ../include/linux/hardirq.h:11, >> from ../include/linux/interrupt.h:11, >> from ../drivers/video/fbdev/ps3fb.c:25: >> ../drivers/video/fbdev/ps3fb.c: In function 'ps3fb_probe': >> ../drivers/video/fbdev/ps3fb.c:1172:40: error: 'struct fb_info' has no >> member named 'dev' >> 1172 | dev_driver_string(info->dev), dev_name(info->dev), >> |^~ >> ../include/linux/dev_printk.h:110:37: note: in definition of macro >> 'dev_printk_index_wrap' >> 110 | _p_func(dev, fmt, ##__VA_ARGS__); >>\ >> | ^~~ >> ../drivers/video/fbdev/ps3fb.c:1171:9: note: in expansion of macro 'dev_info' >> 1171 | dev_info(info->device, "%s %s, using %u KiB of video >> memory\n", >> | ^~~~ >> ../drivers/video/fbdev/ps3fb.c:1172:61: error: 'struct fb_info' has no >> member named 'dev' >> 1172 | dev_driver_string(info->dev), dev_name(info->dev), >> | ^~ >> ../include/linux/dev_printk.h:110:37: note: in definition of macro >> 'dev_printk_index_wrap' >> 110 | _p_func(dev, fmt, ##__VA_ARGS__); >>\ >> | ^~~ >> ../drivers/video/fbdev/ps3fb.c:1171:9: note: in expansion of macro 'dev_info' >> 1171 | dev_info(info->device, "%s %s, using %u KiB of video >> memory\n", >> | ^~~~ > > Hmm, there is no response from Thomas yet. I guess we should go with > reverting bdb616479eff419, right? I'm missing something here: * What makes you think this is caused by bdb616479eff419? I didn't see anything in the thread that claims this, but I might be missing something * related: if I understand Randy right, this is only happening in -next; so why is bdb616479eff419 the culprit, which is also in mainline since End of June? And asking for a revert already is a bit jumping the gun; sure, it would be good to get this fixed, but remember: developers have a lot on their plate and thus sometimes are forced to set priorities; they also sometimes go on vacation or are afk for other reasons; and sometimes they just miss a mail or two. These are just a few reasons why there might be good reasons why Thomas didn't look into this yet, hence please first ask really kindly before asking for a revert. Ciao, Thorsten
Re: linux-next: Tree for Jul 13 (drivers/video/fbdev/ps3fb.c)
On Thu, Jul 13, 2023 at 09:11:10AM -0700, Randy Dunlap wrote: > > > On 7/12/23 19:37, Stephen Rothwell wrote: > > Hi all, > > > > Changes since 20230712: > > > > on ppc64: > > In file included from ../include/linux/device.h:15, > from ../arch/powerpc/include/asm/io.h:22, > from ../include/linux/io.h:13, > from ../include/linux/irq.h:20, > from ../arch/powerpc/include/asm/hardirq.h:6, > from ../include/linux/hardirq.h:11, > from ../include/linux/interrupt.h:11, > from ../drivers/video/fbdev/ps3fb.c:25: > ../drivers/video/fbdev/ps3fb.c: In function 'ps3fb_probe': > ../drivers/video/fbdev/ps3fb.c:1172:40: error: 'struct fb_info' has no member > named 'dev' > 1172 | dev_driver_string(info->dev), dev_name(info->dev), > |^~ > ../include/linux/dev_printk.h:110:37: note: in definition of macro > 'dev_printk_index_wrap' > 110 | _p_func(dev, fmt, ##__VA_ARGS__); > \ > | ^~~ > ../drivers/video/fbdev/ps3fb.c:1171:9: note: in expansion of macro 'dev_info' > 1171 | dev_info(info->device, "%s %s, using %u KiB of video > memory\n", > | ^~~~ > ../drivers/video/fbdev/ps3fb.c:1172:61: error: 'struct fb_info' has no member > named 'dev' > 1172 | dev_driver_string(info->dev), dev_name(info->dev), > | ^~ > ../include/linux/dev_printk.h:110:37: note: in definition of macro > 'dev_printk_index_wrap' > 110 | _p_func(dev, fmt, ##__VA_ARGS__); > \ > | ^~~ > ../drivers/video/fbdev/ps3fb.c:1171:9: note: in expansion of macro 'dev_info' > 1171 | dev_info(info->device, "%s %s, using %u KiB of video > memory\n", > | ^~~~ > > Hmm, there is no response from Thomas yet. I guess we should go with reverting bdb616479eff419, right? Regardless, I'm adding this build regression to regzbot so that parties involved are aware of it: #regzbot ^introduced: bdb616479eff419 #regzbot title: build regression in PS3 framebuffer Thanks. -- An old man doll... just what I always wanted! - Clara
Re: linux-next: Tree for Jul 13 (drivers/video/fbdev/ps3fb.c)
Hi Thomas, On 7/14/23 13:46, Randy Dunlap wrote: > Thomas, > > On 7/13/23 09:11, Randy Dunlap wrote: >> >> >> On 7/12/23 19:37, Stephen Rothwell wrote: >>> Hi all, >>> > > I still see this build error on linux-next 20230714. I still see this build error on linux-next 20230717. > >>> Changes since 20230712: >>> >> >> on ppc64: >> >> In file included from ../include/linux/device.h:15, >> from ../arch/powerpc/include/asm/io.h:22, >> from ../include/linux/io.h:13, >> from ../include/linux/irq.h:20, >> from ../arch/powerpc/include/asm/hardirq.h:6, >> from ../include/linux/hardirq.h:11, >> from ../include/linux/interrupt.h:11, >> from ../drivers/video/fbdev/ps3fb.c:25: >> ../drivers/video/fbdev/ps3fb.c: In function 'ps3fb_probe': >> ../drivers/video/fbdev/ps3fb.c:1172:40: error: 'struct fb_info' has no >> member named 'dev' >> 1172 | dev_driver_string(info->dev), dev_name(info->dev), >> | ^~ >> ../include/linux/dev_printk.h:110:37: note: in definition of macro >> 'dev_printk_index_wrap' >> 110 | _p_func(dev, fmt, ##__VA_ARGS__); \ >> | ^~~ >> ../drivers/video/fbdev/ps3fb.c:1171:9: note: in expansion of macro 'dev_info' >> 1171 | dev_info(info->device, "%s %s, using %u KiB of video memory\n", >> | ^~~~ >> ../drivers/video/fbdev/ps3fb.c:1172:61: error: 'struct fb_info' has no >> member named 'dev' >> 1172 | dev_driver_string(info->dev), dev_name(info->dev), >> | ^~ >> ../include/linux/dev_printk.h:110:37: note: in definition of macro >> 'dev_printk_index_wrap' -- ~Randy [using gmail temporarily while infradead is down]
Re: linux-next: Tree for Jul 13 (drivers/video/fbdev/ps3fb.c)
Thomas, On 7/13/23 09:11, Randy Dunlap wrote: > > > On 7/12/23 19:37, Stephen Rothwell wrote: >> Hi all, >> I still see this build error on linux-next 20230714. >> Changes since 20230712: >> > > on ppc64: > > In file included from ../include/linux/device.h:15, > from ../arch/powerpc/include/asm/io.h:22, > from ../include/linux/io.h:13, > from ../include/linux/irq.h:20, > from ../arch/powerpc/include/asm/hardirq.h:6, > from ../include/linux/hardirq.h:11, > from ../include/linux/interrupt.h:11, > from ../drivers/video/fbdev/ps3fb.c:25: > ../drivers/video/fbdev/ps3fb.c: In function 'ps3fb_probe': > ../drivers/video/fbdev/ps3fb.c:1172:40: error: 'struct fb_info' has no member > named 'dev' > 1172 | dev_driver_string(info->dev), dev_name(info->dev), > |^~ > ../include/linux/dev_printk.h:110:37: note: in definition of macro > 'dev_printk_index_wrap' > 110 | _p_func(dev, fmt, ##__VA_ARGS__); > \ > | ^~~ > ../drivers/video/fbdev/ps3fb.c:1171:9: note: in expansion of macro 'dev_info' > 1171 | dev_info(info->device, "%s %s, using %u KiB of video > memory\n", > | ^~~~ > ../drivers/video/fbdev/ps3fb.c:1172:61: error: 'struct fb_info' has no member > named 'dev' > 1172 | dev_driver_string(info->dev), dev_name(info->dev), > | ^~ > ../include/linux/dev_printk.h:110:37: note: in definition of macro > 'dev_printk_index_wrap' > 110 | _p_func(dev, fmt, ##__VA_ARGS__); > \ > | ^~~ > ../drivers/video/fbdev/ps3fb.c:1171:9: note: in expansion of macro 'dev_info' > 1171 | dev_info(info->device, "%s %s, using %u KiB of video > memory\n", > | ^~~~ > > > Full randconfig file is attached. -- ~Randy
Re: linux-next: Tree for Jul 13 (drivers/video/fbdev/ps3fb.c)
On 7/12/23 19:37, Stephen Rothwell wrote: > Hi all, > > Changes since 20230712: > on ppc64: In file included from ../include/linux/device.h:15, from ../arch/powerpc/include/asm/io.h:22, from ../include/linux/io.h:13, from ../include/linux/irq.h:20, from ../arch/powerpc/include/asm/hardirq.h:6, from ../include/linux/hardirq.h:11, from ../include/linux/interrupt.h:11, from ../drivers/video/fbdev/ps3fb.c:25: ../drivers/video/fbdev/ps3fb.c: In function 'ps3fb_probe': ../drivers/video/fbdev/ps3fb.c:1172:40: error: 'struct fb_info' has no member named 'dev' 1172 | dev_driver_string(info->dev), dev_name(info->dev), |^~ ../include/linux/dev_printk.h:110:37: note: in definition of macro 'dev_printk_index_wrap' 110 | _p_func(dev, fmt, ##__VA_ARGS__); \ | ^~~ ../drivers/video/fbdev/ps3fb.c:1171:9: note: in expansion of macro 'dev_info' 1171 | dev_info(info->device, "%s %s, using %u KiB of video memory\n", | ^~~~ ../drivers/video/fbdev/ps3fb.c:1172:61: error: 'struct fb_info' has no member named 'dev' 1172 | dev_driver_string(info->dev), dev_name(info->dev), | ^~ ../include/linux/dev_printk.h:110:37: note: in definition of macro 'dev_printk_index_wrap' 110 | _p_func(dev, fmt, ##__VA_ARGS__); \ | ^~~ ../drivers/video/fbdev/ps3fb.c:1171:9: note: in expansion of macro 'dev_info' 1171 | dev_info(info->device, "%s %s, using %u KiB of video memory\n", | ^~~~ Full randconfig file is attached. -- ~Randy config-r3073.gz Description: application/gzip