* Re: [PATCH] arch/sh: Drop CONFIG_FIRMWARE_EDID from defconfig files
2026-04-01 8:32 [PATCH] arch/sh: Drop CONFIG_FIRMWARE_EDID from defconfig files Thomas Zimmermann
@ 2026-04-01 8:46 ` John Paul Adrian Glaubitz
2026-04-01 8:53 ` Thomas Zimmermann
2026-04-01 8:55 ` Geert Uytterhoeven
2026-04-01 13:25 ` John Paul Adrian Glaubitz
2 siblings, 1 reply; 8+ messages in thread
From: John Paul Adrian Glaubitz @ 2026-04-01 8:46 UTC (permalink / raw)
To: Thomas Zimmermann, ysato, dalias, arnd; +Cc: linux-sh, linux-kernel, dri-devel
Hi Thomas,
On Wed, 2026-04-01 at 10:32 +0200, Thomas Zimmermann wrote:
> CONFIG_FIRMWARE_EDID=y depends on X86 or EFI_GENERIC_STUB. Neither is
> true here, so drop the lines from the defconfig files.
>
> Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de>
> ---
> arch/sh/configs/dreamcast_defconfig | 1 -
> arch/sh/configs/hp6xx_defconfig | 1 -
> arch/sh/configs/se7343_defconfig | 1 -
> arch/sh/configs/se7780_defconfig | 1 -
> 4 files changed, 4 deletions(-)
>
> diff --git a/arch/sh/configs/dreamcast_defconfig b/arch/sh/configs/dreamcast_defconfig
> index dd58797e8298..b31bf17fe112 100644
> --- a/arch/sh/configs/dreamcast_defconfig
> +++ b/arch/sh/configs/dreamcast_defconfig
> @@ -50,7 +50,6 @@ CONFIG_HW_RANDOM=y
> CONFIG_WATCHDOG=y
> CONFIG_SH_WDT=y
> CONFIG_FB=y
> -CONFIG_FIRMWARE_EDID=y
> CONFIG_FB_PVR2=y
> CONFIG_FRAMEBUFFER_CONSOLE=y
> CONFIG_FONTS=y
> diff --git a/arch/sh/configs/hp6xx_defconfig b/arch/sh/configs/hp6xx_defconfig
> index 04a9fcb4342a..b6116a203a27 100644
> --- a/arch/sh/configs/hp6xx_defconfig
> +++ b/arch/sh/configs/hp6xx_defconfig
> @@ -35,7 +35,6 @@ CONFIG_SERIAL_SH_SCI_CONSOLE=y
> CONFIG_LEGACY_PTY_COUNT=64
> # CONFIG_HWMON is not set
> CONFIG_FB=y
> -CONFIG_FIRMWARE_EDID=y
> CONFIG_FB_HIT=y
> CONFIG_FB_SH_MOBILE_LCDC=y
> CONFIG_FRAMEBUFFER_CONSOLE=y
> diff --git a/arch/sh/configs/se7343_defconfig b/arch/sh/configs/se7343_defconfig
> index 2d4d1f974f14..b3ce8a502787 100644
> --- a/arch/sh/configs/se7343_defconfig
> +++ b/arch/sh/configs/se7343_defconfig
> @@ -57,7 +57,6 @@ CONFIG_I2C=y
> CONFIG_I2C_SH_MOBILE=y
> # CONFIG_HWMON is not set
> CONFIG_FB=y
> -CONFIG_FIRMWARE_EDID=y
> CONFIG_FB_SH_MOBILE_LCDC=m
> CONFIG_SOUND=y
> CONFIG_SND=y
> diff --git a/arch/sh/configs/se7780_defconfig b/arch/sh/configs/se7780_defconfig
> index 13fa6a59b8f1..9e96b000cb99 100644
> --- a/arch/sh/configs/se7780_defconfig
> +++ b/arch/sh/configs/se7780_defconfig
> @@ -60,7 +60,6 @@ CONFIG_SERIAL_SH_SCI_CONSOLE=y
> # CONFIG_HW_RANDOM is not set
> CONFIG_THERMAL=y
> CONFIG_FB=y
> -CONFIG_FIRMWARE_EDID=y
> CONFIG_FB_SH_MOBILE_LCDC=m
> CONFIG_FRAMEBUFFER_CONSOLE=y
> CONFIG_LOGO=y
Thanks for catching this! I wonder where this configuration entry comes from though.
My SH7785LCR board does have a DVI output as well as a Silicon Motion SM-501 video
chip though, so I think it should be able to perform EDID in general. Or is the
protocol actually tied to the x86 architecture? What about monitors on a PowerPC
workstation, for example? Does EDID not work there at all?
Thanks,
Adrian
--
.''`. John Paul Adrian Glaubitz
: :' : Debian Developer
`. `' Physicist
`- GPG: 62FF 8A75 84E0 2956 9546 0006 7426 3B37 F5B5 F913
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] arch/sh: Drop CONFIG_FIRMWARE_EDID from defconfig files
2026-04-01 8:46 ` John Paul Adrian Glaubitz
@ 2026-04-01 8:53 ` Thomas Zimmermann
2026-04-01 12:45 ` Jani Nikula
2026-04-01 13:24 ` John Paul Adrian Glaubitz
0 siblings, 2 replies; 8+ messages in thread
From: Thomas Zimmermann @ 2026-04-01 8:53 UTC (permalink / raw)
To: John Paul Adrian Glaubitz, ysato, dalias, arnd
Cc: linux-sh, linux-kernel, dri-devel
Hi
Am 01.04.26 um 10:46 schrieb John Paul Adrian Glaubitz:
> Hi Thomas,
>
> On Wed, 2026-04-01 at 10:32 +0200, Thomas Zimmermann wrote:
>> CONFIG_FIRMWARE_EDID=y depends on X86 or EFI_GENERIC_STUB. Neither is
>> true here, so drop the lines from the defconfig files.
>>
>> Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de>
>> ---
>> arch/sh/configs/dreamcast_defconfig | 1 -
>> arch/sh/configs/hp6xx_defconfig | 1 -
>> arch/sh/configs/se7343_defconfig | 1 -
>> arch/sh/configs/se7780_defconfig | 1 -
>> 4 files changed, 4 deletions(-)
>>
>> diff --git a/arch/sh/configs/dreamcast_defconfig b/arch/sh/configs/dreamcast_defconfig
>> index dd58797e8298..b31bf17fe112 100644
>> --- a/arch/sh/configs/dreamcast_defconfig
>> +++ b/arch/sh/configs/dreamcast_defconfig
>> @@ -50,7 +50,6 @@ CONFIG_HW_RANDOM=y
>> CONFIG_WATCHDOG=y
>> CONFIG_SH_WDT=y
>> CONFIG_FB=y
>> -CONFIG_FIRMWARE_EDID=y
>> CONFIG_FB_PVR2=y
>> CONFIG_FRAMEBUFFER_CONSOLE=y
>> CONFIG_FONTS=y
>> diff --git a/arch/sh/configs/hp6xx_defconfig b/arch/sh/configs/hp6xx_defconfig
>> index 04a9fcb4342a..b6116a203a27 100644
>> --- a/arch/sh/configs/hp6xx_defconfig
>> +++ b/arch/sh/configs/hp6xx_defconfig
>> @@ -35,7 +35,6 @@ CONFIG_SERIAL_SH_SCI_CONSOLE=y
>> CONFIG_LEGACY_PTY_COUNT=64
>> # CONFIG_HWMON is not set
>> CONFIG_FB=y
>> -CONFIG_FIRMWARE_EDID=y
>> CONFIG_FB_HIT=y
>> CONFIG_FB_SH_MOBILE_LCDC=y
>> CONFIG_FRAMEBUFFER_CONSOLE=y
>> diff --git a/arch/sh/configs/se7343_defconfig b/arch/sh/configs/se7343_defconfig
>> index 2d4d1f974f14..b3ce8a502787 100644
>> --- a/arch/sh/configs/se7343_defconfig
>> +++ b/arch/sh/configs/se7343_defconfig
>> @@ -57,7 +57,6 @@ CONFIG_I2C=y
>> CONFIG_I2C_SH_MOBILE=y
>> # CONFIG_HWMON is not set
>> CONFIG_FB=y
>> -CONFIG_FIRMWARE_EDID=y
>> CONFIG_FB_SH_MOBILE_LCDC=m
>> CONFIG_SOUND=y
>> CONFIG_SND=y
>> diff --git a/arch/sh/configs/se7780_defconfig b/arch/sh/configs/se7780_defconfig
>> index 13fa6a59b8f1..9e96b000cb99 100644
>> --- a/arch/sh/configs/se7780_defconfig
>> +++ b/arch/sh/configs/se7780_defconfig
>> @@ -60,7 +60,6 @@ CONFIG_SERIAL_SH_SCI_CONSOLE=y
>> # CONFIG_HW_RANDOM is not set
>> CONFIG_THERMAL=y
>> CONFIG_FB=y
>> -CONFIG_FIRMWARE_EDID=y
>> CONFIG_FB_SH_MOBILE_LCDC=m
>> CONFIG_FRAMEBUFFER_CONSOLE=y
>> CONFIG_LOGO=y
> Thanks for catching this! I wonder where this configuration entry comes from though.
>
> My SH7785LCR board does have a DVI output as well as a Silicon Motion SM-501 video
> chip though, so I think it should be able to perform EDID in general. Or is the
> protocol actually tied to the x86 architecture? What about monitors on a PowerPC
> workstation, for example? Does EDID not work there at all?
Yes, your system can fetch the EDID via the Linux driver. But
FIRMWARE_EDID is a special feature where the EDID gets delivered by the
boot loader. This has always only been possible on x86 or with EFI.
Best regards
Thomas
>
> Thanks,
> Adrian
>
--
--
Thomas Zimmermann
Graphics Driver Developer
SUSE Software Solutions Germany GmbH
Frankenstr. 146, 90461 Nürnberg, Germany, www.suse.com
GF: Jochen Jaser, Andrew McDonald, Werner Knoblich, (HRB 36809, AG Nürnberg)
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] arch/sh: Drop CONFIG_FIRMWARE_EDID from defconfig files
2026-04-01 8:53 ` Thomas Zimmermann
@ 2026-04-01 12:45 ` Jani Nikula
2026-04-01 13:24 ` John Paul Adrian Glaubitz
1 sibling, 0 replies; 8+ messages in thread
From: Jani Nikula @ 2026-04-01 12:45 UTC (permalink / raw)
To: Thomas Zimmermann, John Paul Adrian Glaubitz, ysato, dalias, arnd
Cc: linux-sh, linux-kernel, dri-devel
On Wed, 01 Apr 2026, Thomas Zimmermann <tzimmermann@suse.de> wrote:
> Am 01.04.26 um 10:46 schrieb John Paul Adrian Glaubitz:
>> My SH7785LCR board does have a DVI output as well as a Silicon Motion SM-501 video
>> chip though, so I think it should be able to perform EDID in general. Or is the
>> protocol actually tied to the x86 architecture? What about monitors on a PowerPC
>> workstation, for example? Does EDID not work there at all?
>
> Yes, your system can fetch the EDID via the Linux driver. But
> FIRMWARE_EDID is a special feature where the EDID gets delivered by the
> boot loader. This has always only been possible on x86 or with EFI.
And then there's the completely separate CONFIG_DRM_LOAD_EDID_FIRMWARE
config to allow loading the EDID through the kernel firmware loader. Not
confusing or misleading at all. ;D
BR,
Jani.
--
Jani Nikula, Intel
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] arch/sh: Drop CONFIG_FIRMWARE_EDID from defconfig files
2026-04-01 8:53 ` Thomas Zimmermann
2026-04-01 12:45 ` Jani Nikula
@ 2026-04-01 13:24 ` John Paul Adrian Glaubitz
1 sibling, 0 replies; 8+ messages in thread
From: John Paul Adrian Glaubitz @ 2026-04-01 13:24 UTC (permalink / raw)
To: Thomas Zimmermann, ysato, dalias, arnd; +Cc: linux-sh, linux-kernel, dri-devel
Hi Thomas,
On Wed, 2026-04-01 at 10:53 +0200, Thomas Zimmermann wrote:
> Hi
>
> Am 01.04.26 um 10:46 schrieb John Paul Adrian Glaubitz:
> > Hi Thomas,
> >
> > On Wed, 2026-04-01 at 10:32 +0200, Thomas Zimmermann wrote:
> > > CONFIG_FIRMWARE_EDID=y depends on X86 or EFI_GENERIC_STUB. Neither is
> > > true here, so drop the lines from the defconfig files.
> > >
> > > Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de>
> > > ---
> > > arch/sh/configs/dreamcast_defconfig | 1 -
> > > arch/sh/configs/hp6xx_defconfig | 1 -
> > > arch/sh/configs/se7343_defconfig | 1 -
> > > arch/sh/configs/se7780_defconfig | 1 -
> > > 4 files changed, 4 deletions(-)
> > >
> > > diff --git a/arch/sh/configs/dreamcast_defconfig b/arch/sh/configs/dreamcast_defconfig
> > > index dd58797e8298..b31bf17fe112 100644
> > > --- a/arch/sh/configs/dreamcast_defconfig
> > > +++ b/arch/sh/configs/dreamcast_defconfig
> > > @@ -50,7 +50,6 @@ CONFIG_HW_RANDOM=y
> > > CONFIG_WATCHDOG=y
> > > CONFIG_SH_WDT=y
> > > CONFIG_FB=y
> > > -CONFIG_FIRMWARE_EDID=y
> > > CONFIG_FB_PVR2=y
> > > CONFIG_FRAMEBUFFER_CONSOLE=y
> > > CONFIG_FONTS=y
> > > diff --git a/arch/sh/configs/hp6xx_defconfig b/arch/sh/configs/hp6xx_defconfig
> > > index 04a9fcb4342a..b6116a203a27 100644
> > > --- a/arch/sh/configs/hp6xx_defconfig
> > > +++ b/arch/sh/configs/hp6xx_defconfig
> > > @@ -35,7 +35,6 @@ CONFIG_SERIAL_SH_SCI_CONSOLE=y
> > > CONFIG_LEGACY_PTY_COUNT=64
> > > # CONFIG_HWMON is not set
> > > CONFIG_FB=y
> > > -CONFIG_FIRMWARE_EDID=y
> > > CONFIG_FB_HIT=y
> > > CONFIG_FB_SH_MOBILE_LCDC=y
> > > CONFIG_FRAMEBUFFER_CONSOLE=y
> > > diff --git a/arch/sh/configs/se7343_defconfig b/arch/sh/configs/se7343_defconfig
> > > index 2d4d1f974f14..b3ce8a502787 100644
> > > --- a/arch/sh/configs/se7343_defconfig
> > > +++ b/arch/sh/configs/se7343_defconfig
> > > @@ -57,7 +57,6 @@ CONFIG_I2C=y
> > > CONFIG_I2C_SH_MOBILE=y
> > > # CONFIG_HWMON is not set
> > > CONFIG_FB=y
> > > -CONFIG_FIRMWARE_EDID=y
> > > CONFIG_FB_SH_MOBILE_LCDC=m
> > > CONFIG_SOUND=y
> > > CONFIG_SND=y
> > > diff --git a/arch/sh/configs/se7780_defconfig b/arch/sh/configs/se7780_defconfig
> > > index 13fa6a59b8f1..9e96b000cb99 100644
> > > --- a/arch/sh/configs/se7780_defconfig
> > > +++ b/arch/sh/configs/se7780_defconfig
> > > @@ -60,7 +60,6 @@ CONFIG_SERIAL_SH_SCI_CONSOLE=y
> > > # CONFIG_HW_RANDOM is not set
> > > CONFIG_THERMAL=y
> > > CONFIG_FB=y
> > > -CONFIG_FIRMWARE_EDID=y
> > > CONFIG_FB_SH_MOBILE_LCDC=m
> > > CONFIG_FRAMEBUFFER_CONSOLE=y
> > > CONFIG_LOGO=y
> > Thanks for catching this! I wonder where this configuration entry comes from though.
> >
> > My SH7785LCR board does have a DVI output as well as a Silicon Motion SM-501 video
> > chip though, so I think it should be able to perform EDID in general. Or is the
> > protocol actually tied to the x86 architecture? What about monitors on a PowerPC
> > workstation, for example? Does EDID not work there at all?
>
> Yes, your system can fetch the EDID via the Linux driver. But
> FIRMWARE_EDID is a special feature where the EDID gets delivered by the
> boot loader. This has always only been possible on x86 or with EFI.
Ah, thanks. I wasn't aware of that feature. Let me add my RB then.
I'll try to merge some patches this week.
Adrian
--
.''`. John Paul Adrian Glaubitz
: :' : Debian Developer
`. `' Physicist
`- GPG: 62FF 8A75 84E0 2956 9546 0006 7426 3B37 F5B5 F913
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] arch/sh: Drop CONFIG_FIRMWARE_EDID from defconfig files
2026-04-01 8:32 [PATCH] arch/sh: Drop CONFIG_FIRMWARE_EDID from defconfig files Thomas Zimmermann
2026-04-01 8:46 ` John Paul Adrian Glaubitz
@ 2026-04-01 8:55 ` Geert Uytterhoeven
2026-04-01 9:07 ` Thomas Zimmermann
2026-04-01 13:25 ` John Paul Adrian Glaubitz
2 siblings, 1 reply; 8+ messages in thread
From: Geert Uytterhoeven @ 2026-04-01 8:55 UTC (permalink / raw)
To: Thomas Zimmermann
Cc: ysato, dalias, glaubitz, arnd, linux-sh, linux-kernel, dri-devel,
Linux ARM, linuxppc-dev, linux-mips
Hi Thomas,
CC arm/mips/ppc, as you sent similar patches for these arches.
On Wed, 1 Apr 2026 at 10:40, Thomas Zimmermann <tzimmermann@suse.de> wrote:
> CONFIG_FIRMWARE_EDID=y depends on X86 or EFI_GENERIC_STUB. Neither is
> true here, so drop the lines from the defconfig files.
>
> Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de>
Thanks for your patch!
Upon first look, your changes match the (current) dependencies
of FIRMWARE_EDID. The dependency on X86 was added in commit
7e35fc7ab433683f ("video: Make CONFIG_FIRMWARE_EDID generally
available") in v6.17-rc1.
However, CONFIG_FIRMWARE_EDID also protects fb_firmware_edid(),
which seems to extract the EDID from the PCI ROM, and is thus not
x86-specific? That function is only ever called by three fbdev drivers
(i810, nv, savagefb), though.
I assume none of these work on SuperH, so
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [PATCH] arch/sh: Drop CONFIG_FIRMWARE_EDID from defconfig files
2026-04-01 8:55 ` Geert Uytterhoeven
@ 2026-04-01 9:07 ` Thomas Zimmermann
0 siblings, 0 replies; 8+ messages in thread
From: Thomas Zimmermann @ 2026-04-01 9:07 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: ysato, dalias, glaubitz, arnd, linux-sh, linux-kernel, dri-devel,
Linux ARM, linuxppc-dev, linux-mips
Hi Geert
Am 01.04.26 um 10:55 schrieb Geert Uytterhoeven:
> Hi Thomas,
>
> CC arm/mips/ppc, as you sent similar patches for these arches.
>
> On Wed, 1 Apr 2026 at 10:40, Thomas Zimmermann <tzimmermann@suse.de> wrote:
>> CONFIG_FIRMWARE_EDID=y depends on X86 or EFI_GENERIC_STUB. Neither is
>> true here, so drop the lines from the defconfig files.
>>
>> Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de>
> Thanks for your patch!
>
> Upon first look, your changes match the (current) dependencies
> of FIRMWARE_EDID. The dependency on X86 was added in commit
> 7e35fc7ab433683f ("video: Make CONFIG_FIRMWARE_EDID generally
> available") in v6.17-rc1.
> However, CONFIG_FIRMWARE_EDID also protects fb_firmware_edid(),
> which seems to extract the EDID from the PCI ROM, and is thus not
The PCI code in the function tests whether the BIOS ROM has been
shadowed into system memory. That's a common way of testing if the PCI
device is the primary graphics card. The provided EDID is only for the
primary device.
The test shouldn't actually be there any more. We now have
video_is_primary_device() for this.
> x86-specific? That function is only ever called by three fbdev drivers
> (i810, nv, savagefb), though.
I know. But IDK why it hasn't been more prominently featured in fbdev.
Nowadays, we use the firmware EDID with DRM's generic drivers on any
hardware.
>
> I assume none of these work on SuperH, so
> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
Thanks
Best regards
Thomas
>
> Gr{oetje,eeting}s,
>
> Geert
>
--
--
Thomas Zimmermann
Graphics Driver Developer
SUSE Software Solutions Germany GmbH
Frankenstr. 146, 90461 Nürnberg, Germany, www.suse.com
GF: Jochen Jaser, Andrew McDonald, Werner Knoblich, (HRB 36809, AG Nürnberg)
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] arch/sh: Drop CONFIG_FIRMWARE_EDID from defconfig files
2026-04-01 8:32 [PATCH] arch/sh: Drop CONFIG_FIRMWARE_EDID from defconfig files Thomas Zimmermann
2026-04-01 8:46 ` John Paul Adrian Glaubitz
2026-04-01 8:55 ` Geert Uytterhoeven
@ 2026-04-01 13:25 ` John Paul Adrian Glaubitz
2 siblings, 0 replies; 8+ messages in thread
From: John Paul Adrian Glaubitz @ 2026-04-01 13:25 UTC (permalink / raw)
To: Thomas Zimmermann, ysato, dalias, arnd; +Cc: linux-sh, linux-kernel, dri-devel
On Wed, 2026-04-01 at 10:32 +0200, Thomas Zimmermann wrote:
> CONFIG_FIRMWARE_EDID=y depends on X86 or EFI_GENERIC_STUB. Neither is
> true here, so drop the lines from the defconfig files.
>
> Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de>
> ---
> arch/sh/configs/dreamcast_defconfig | 1 -
> arch/sh/configs/hp6xx_defconfig | 1 -
> arch/sh/configs/se7343_defconfig | 1 -
> arch/sh/configs/se7780_defconfig | 1 -
> 4 files changed, 4 deletions(-)
>
> diff --git a/arch/sh/configs/dreamcast_defconfig b/arch/sh/configs/dreamcast_defconfig
> index dd58797e8298..b31bf17fe112 100644
> --- a/arch/sh/configs/dreamcast_defconfig
> +++ b/arch/sh/configs/dreamcast_defconfig
> @@ -50,7 +50,6 @@ CONFIG_HW_RANDOM=y
> CONFIG_WATCHDOG=y
> CONFIG_SH_WDT=y
> CONFIG_FB=y
> -CONFIG_FIRMWARE_EDID=y
> CONFIG_FB_PVR2=y
> CONFIG_FRAMEBUFFER_CONSOLE=y
> CONFIG_FONTS=y
> diff --git a/arch/sh/configs/hp6xx_defconfig b/arch/sh/configs/hp6xx_defconfig
> index 04a9fcb4342a..b6116a203a27 100644
> --- a/arch/sh/configs/hp6xx_defconfig
> +++ b/arch/sh/configs/hp6xx_defconfig
> @@ -35,7 +35,6 @@ CONFIG_SERIAL_SH_SCI_CONSOLE=y
> CONFIG_LEGACY_PTY_COUNT=64
> # CONFIG_HWMON is not set
> CONFIG_FB=y
> -CONFIG_FIRMWARE_EDID=y
> CONFIG_FB_HIT=y
> CONFIG_FB_SH_MOBILE_LCDC=y
> CONFIG_FRAMEBUFFER_CONSOLE=y
> diff --git a/arch/sh/configs/se7343_defconfig b/arch/sh/configs/se7343_defconfig
> index 2d4d1f974f14..b3ce8a502787 100644
> --- a/arch/sh/configs/se7343_defconfig
> +++ b/arch/sh/configs/se7343_defconfig
> @@ -57,7 +57,6 @@ CONFIG_I2C=y
> CONFIG_I2C_SH_MOBILE=y
> # CONFIG_HWMON is not set
> CONFIG_FB=y
> -CONFIG_FIRMWARE_EDID=y
> CONFIG_FB_SH_MOBILE_LCDC=m
> CONFIG_SOUND=y
> CONFIG_SND=y
> diff --git a/arch/sh/configs/se7780_defconfig b/arch/sh/configs/se7780_defconfig
> index 13fa6a59b8f1..9e96b000cb99 100644
> --- a/arch/sh/configs/se7780_defconfig
> +++ b/arch/sh/configs/se7780_defconfig
> @@ -60,7 +60,6 @@ CONFIG_SERIAL_SH_SCI_CONSOLE=y
> # CONFIG_HW_RANDOM is not set
> CONFIG_THERMAL=y
> CONFIG_FB=y
> -CONFIG_FIRMWARE_EDID=y
> CONFIG_FB_SH_MOBILE_LCDC=m
> CONFIG_FRAMEBUFFER_CONSOLE=y
> CONFIG_LOGO=y
Looks good to me.
Reviewed-by: John Paul Adrian Glaubitz <glaubitz@physik.fu-berlin.de>
Adrian
--
.''`. John Paul Adrian Glaubitz
: :' : Debian Developer
`. `' Physicist
`- GPG: 62FF 8A75 84E0 2956 9546 0006 7426 3B37 F5B5 F913
^ permalink raw reply [flat|nested] 8+ messages in thread