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

- From: Thomas Zimmermann \<tzimmermann@suse.de\>
- Date: 2022-11-11 11:42:07 
- List: linux-fbdev
- Page: https://ratatoskr.run/linux-fbdev/2022/11/9227322
- Thread: https://ratatoskr.run/linux-fbdev/2022/11/9227040/t.md

```text
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
```

## Attachments

- [OpenPGP\_signature](https://ratatoskr.run/linux-fbdev/2022/11/9227322/attachment/0) (application/pgp-signature, 840 bytes)