[PATCH] bochs: convert to drm_fb_helper_fbdev_setup/teardown
Gerd Hoffmann
kraxel at redhat.com
Thu Sep 6 07:23:54 UTC 2018
Hi,
> > You can probably get rid of this one if you're refactoring even more. The
> > generic fb_probe implementation (already merged) plus gem-shmem support
> > for it (still in flight) from Noralf should be able to pull that off. That
> > gives you the fb_mmap implementation, but with 100% generic code instead
> > of a driver specific hack like Max did.
>
> Aside from the warning, I have not observed actual issues. This patch
> was prepared on top of v4.18.1 but the new drm_fb_helper_generic_probe
> helper is in master (future 4.19). I suppose that it can be done as a
> future cleanup. Nice work Noralf on reducing duplication!
FYI: qemu kms driver patches go through drm-misc-next, so you can also
work against that branch.
> > I'll leave merging to Gerd.
>
> Thanks, I somehow missed a patch. This one does not compile due to
> "fb.initialized" still being used in bochs_drv.c. Removal is trivial,
> I'll wait for some more feedback and then send a v2 with another patch
> prepended.
Patch looks good to me (except for the build failure which you've
noticed already).
cheers,
Gerd
More information about the dri-devel
mailing list