# Re: \[PATCH 2/4\] vgacon: rework screen\_info \#ifdef checks

- From: Javier Martinez Canillas \<javierm@redhat.com\>
- Date: 2023-07-07 15:28:10 
- List: linux-fbdev
- Page: https://ratatoskr.run/linux-fbdev/2023/07/2257392
- Thread: https://ratatoskr.run/linux-fbdev/2023/07/2257062/t.md

```text
"Arnd Bergmann" [ref] writes:

> On Fri, Jul 7, 2023, at 15:40, Javier Martinez Canillas wrote:

[...]

>> And this is only used by mdacon (not supported by ia64), vgacon and
>> vga16fb (not supported by ia64 either).
>>
>> So this could just be guarded just by CONFIG_VGA_CONSOLE for ia64 ?
>
> Right, I though about doing this more accurately, but in the end
> went for the simplest change rather than spending much more time
> trying to clean up the unused variables etc.
>
> Let me know if you'd prefer me to respin this part, otherwise
> I'd call the ia64 bit good enough for the purpose of the series.
>

No need to re-spin, agreed that makes sense to keep it simpler.

-- 
Best regards,

Javier Martinez Canillas
Core Platforms
Red Hat

```