Commit b76ecff8 authored by Thomas Zimmermann's avatar Thomas Zimmermann

fbdev: Warn in hot-unplug workaround for framebuffers without device

A workaround makes fbdev hot-unplugging work for framebuffers without
device. The only user for this feature was offb. As each OF framebuffer
now has an associated platform device, the workaround hould no longer
be triggered. Update it with a warning and rewrite the comment. Fbdev
drivers that trigger the hot-unplug workaround really need to be fixed.
Signed-off-by: default avatarThomas Zimmermann <tzimmermann@suse.de>
Reviewed-by: default avatarJavier Martinez Canillas <javierm@redhat.com>
Suggested-by: default avatarJavier Martinez Canillas <javierm@redhat.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20220419100405.12600-3-tzimmermann@suse.de
parent 52b1b46c
...@@ -1577,14 +1577,12 @@ static void do_remove_conflicting_framebuffers(struct apertures_struct *a, ...@@ -1577,14 +1577,12 @@ static void do_remove_conflicting_framebuffers(struct apertures_struct *a,
* allocate the memory range. * allocate the memory range.
* *
* If it's not a platform device, at least print a warning. A * If it's not a platform device, at least print a warning. A
* fix would add code to remove the device from the system. * fix would add code to remove the device from the system. For
* framebuffers without any Linux device, print a warning as
* well.
*/ */
if (!device) { if (!device) {
/* TODO: Represent each OF framebuffer as its own pr_warn("fb%d: no device set\n", i);
* device in the device hierarchy. For now, offb
* doesn't have such a device, so unregister the
* framebuffer as before without warning.
*/
do_unregister_framebuffer(registered_fb[i]); do_unregister_framebuffer(registered_fb[i]);
} else if (dev_is_platform(device)) { } else if (dev_is_platform(device)) {
registered_fb[i]->forced_out = true; registered_fb[i]->forced_out = true;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment