Multiple issues with omapdrm

[Tomi Valkeinen]

On 05/06/13 13:57, Rob Clark wrote:

> 1) drmOpen("omap") will try to modprobe "omap", not "omapdrm" so we
> need to rename the .ko

Has something been changed that broke that? Or was "omapdrm" just a
badly chosen name from the start? If drmOpen("omapdrm") works now,
doesn't changing the module name break userspace compatibility?

I had a quick look at libdrm. It calls server_info->load_module() but I
couldn't figure out where that call actually goes...

> 2) sorting out the modprobe of panel drivers..  although with the
> current structure of omapdrm+omapdss I can't think of any clean way to
> handle this.  I suppose we could do a hack with a bunch of
> request_module()s

If omapdrm and omapdss were merged, what would be the clean way be? Or
did you mean some other structure?

I'm no expert on this, but my understanding is that udev (or such)
should load the modules for the devices that the board has. If it's a
requirement that the drm drivers are loaded only by a call to drmOpen()
(why is that?), then maybe udev could load only the panel drivers.

 Tomi


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 901 bytes
Desc: OpenPGP digital signature
URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20130605/05d841c9/attachment.pgp>