Gerd Hoffmann
2018-Sep-06 07:23 UTC
[PATCH] bochs: convert to drm_fb_helper_fbdev_setup/teardown
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