[PATCH 2/2] fbdev: Add support for the nomodeset kernel parameter

[Thomas Zimmermann]

Hi

Am 11.11.22 um 11:49 schrieb Helge Deller:
> On 11/11/22 10:49, Javier Martinez Canillas wrote:
>> On 11/8/22 09:16, Thomas Zimmermann wrote:
>>> Hi
>>
>> [...]
>>
>>>
>>> My proposal would be to add a little helper to fbdev that includes your
>>> suggestions:
>>>
>>>     bool fb_modesetting_disabled(const char *drvname)
>>>     {
>>>        fwonly = video_firmware_drivers_only()
>>>        if (fbonly && drvname)
>>>     pr_warn("")
>>>        return fbonly;
>>>     }
> 
> I'm still wondering why you can't simply merge it with what is printed in
>   [1] 
> https://elixir.bootlin.com/linux/latest/source/drivers/gpu/drm/drm_nomodeset.c#L18

I don't understand. That message is still there after moving the code 
around. It is always printed if the nomodeset parameter is given.

In addition to that, you get a per-driver warning for fbdev, so that 
it's clear which drivers are affected.

Best regards
Thomas

> 
> Other than that, your the proposal is okay, if you adjust your patches to
> call this new function "fb_modesetting_disabled()" instead of
> calling video_firmware_drivers_only() like this:
> 
>> +    if (video_firmware_drivers_only())
>> +        return -ENODEV;
> 
> Helge

-- 
Thomas Zimmermann
Graphics Driver Developer
SUSE Software Solutions Germany GmbH
Maxfeldstr. 5, 90409 Nürnberg, Germany
(HRB 36809, AG Nürnberg)
Geschäftsführer: Ivo Totev
-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_signature
Type: application/pgp-signature
Size: 840 bytes
Desc: OpenPGP digital signature
URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20221111/efd4119e/attachment.sig>