Re: [PATCH v2 2/3] arch: Remove struct fb_info from video helpers

From: Helge Deller
Date: Thu Mar 28 2024 - 11:01:07 EST


On 3/28/24 14:33, Thomas Zimmermann wrote:
Am 28.03.24 um 12:04 schrieb Helge Deller:
On 3/27/24 21:41, Thomas Zimmermann wrote:
The per-architecture video helpers do not depend on struct fb_info
or anything else from fbdev. Remove it from the interface and replace
fb_is_primary_device() with video_is_primary_device(). The new helper

Since you rename this function, wouldn't something similar to

device_is_primary_display()
    or
device_is_primary_console()
    or
is_primary_graphics_device()
    or
is_primary_display_device()

be a better name?

The video_ prefix is there to signal that the code is part of the video subsystem.

But there's too much code that tried to figure out a default video
device. So I actually have different plans for this function. I'd
like to replace it with a helper that returns the default device
instead of just testing for it. Sample code for x86 is already in
vgaarb.c. [1] The function's name would then be
video_default_device() and return the appropriate struct device*.
video_is_primary device() will be removed. This rename here is the
easiest step towards the new helper. Ok?
Sounds ok.

Helge