[Openpvrsgx-devgroup] [PATCH] staging: pvr: Add a simplified pvr-drv.c as replacement for messy pvr_drm.c

Tony Lindgren tony at atomide.com
Fri Nov 8 22:16:10 CET 2019


* H. Nikolaus Schaller <hns at goldelico.com> [191108 07:31]:
> Hi Tony,
> 
> > Am 07.11.2019 um 17:55 schrieb Tony Lindgren <tony at atomide.com>:
> > 
> > 
> >>> So now we could remove the old files to avoid confusion:
> >>> 
> >>> drivers/staging/pvr/1.14.3699939/eurasia_km/services4/srvkm/env/linux/pvr_drm.c
> >>> drivers/staging/pvr/1.14.3699939/eurasia_km/services4/srvkm/env/linux/module.c
> 
> removing them and removing services4/srvkm/env/linux/module.o from the Makefile seems to lead to build errors for a clean make:
> 
>   Building modules, stage 2.
>   MODPOST 666 modules - due to target is PHONY
> WARNING: modpost: missing MODULE_LICENSE() in drivers/staging/pvr/1.14.3699939/eurasia_km/pvrsrvkm_omap3_sgx530_121.o

Oh OK thanks for checking. It might be worth waiting for
v2 of the $subject patch if you did not apply it yet.
I also noticed a new PM runtime warning that I need to
track down. And I missed adding the postclose call.

This probably is fixed by adding the standard
MODULE_LICENSE() to pvr-drv.c.

> ERROR: "PVRSRVOpen" [drivers/staging/pvr/1.14.3699939/eurasia_km/pvrsrvkm_dra7_sgx544_116.ko] undefined!
> ERROR: "gui32ReleasePID" [drivers/staging/pvr/1.14.3699939/eurasia_km/pvrsrvkm_dra7_sgx544_116.ko] undefined!
> ERROR: "PVRCore_Init" [drivers/staging/pvr/1.14.3699939/eurasia_km/pvrsrvkm_dra7_sgx544_116.ko] undefined!
> ERROR: "PVRSRVDriverShutdown" [drivers/staging/pvr/1.14.3699939/eurasia_km/pvrsrvkm_dra7_sgx544_116.ko] undefined!
> ERROR: "PVRSRVDriverResume" [drivers/staging/pvr/1.14.3699939/eurasia_km/pvrsrvkm_dra7_sgx544_116.ko] undefined!
> ERROR: "PVRCore_Cleanup" [drivers/staging/pvr/1.14.3699939/eurasia_km/pvrsrvkm_dra7_sgx544_116.ko] undefined!
> ERROR: "gPVRSRVLock" [drivers/staging/pvr/1.14.3699939/eurasia_km/pvrsrvkm_dra7_sgx544_116.ko] undefined!
> ERROR: "PVRSRVDriverSuspend" [drivers/staging/pvr/1.14.3699939/eurasia_km/pvrsrvkm_dra7_sgx544_116.ko] undefined!

Hmm OK I'll take a look, those should be all already
added to pvr-drv.h.

> I haven't digged into the code what is missing.
> 
> Maybe we need to export something? Or the Makefile does not properly handle
> 
> $(TARGET) += \
> 	../../pvr-drv.o \
> 
> for modules?
> 
> BTW: another candidate for removal could be:
> 
> drivers/staging/pvr/1.14.3699939//eurasia_km/services4/srvkm/env/linux/pvr_drm.h
> 
> but it is used in more places:

Yeah we should be able to drop that too. And anything
else related to a typical Linux driver.

BTW, I still have had no luck with seeing the demos working..

Trying with am335x or am437x (after patching it) I get
these kind of errors when trying to run the demos after
running pvrsrvctl --start --no-module:

libEGL warning: DRI2: xcb_connect failed
...

Any idea what I might be missing?

Regards,

Tony


More information about the openpvrsgx-devgroup mailing list