Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1232212

Re: [PATCH 3/5] drm: omapdrm: tiler: Fix module autoload for OF platform driver

From Tomi Valkeinen <tomi.valkeinen@ti.com>
Newsgroups linux.kernel
Subject Re: [PATCH 3/5] drm: omapdrm: tiler: Fix module autoload for OF platform driver
Date 2015-09-24 17:50 +0200
Message-ID <qcgWv-7mP-29@gated-at.bofh.it> (permalink)
References <q9Ime-6cb-29@gated-at.bofh.it> <qccg9-Bw-1@gated-at.bofh.it> <qcgMP-7bv-45@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


[Multipart message — attachments visible in raw view] - view raw

On 24/09/15 18:36, Luis de Bethencourt wrote:

> I am a bit confused.

Yes, it's an interesting mess due to legacy reasons. Maybe we manage to
fix it some day...

> So how the OMAP DRM auto loading is supposed to work when using Device Trees?

omapdrm isn't a real HW device driver at the moment. There's another
driver, omapdss, which is the HW driver, and omapdrm uses omapdss.
omapdrm platform device is created by omap platform code at boot time,
the same way for both DT and non-DT boots.

> As far as I can tell, the main omap drm driver does not have a OF device ID
> table and a .of_match it only has a MODULE_ALIAS("platform:" DRIVER_NAME),
> but the tiler driver (that is built-in the omap drm driver) does have a a
> OF device ID table and I see in DTS that are device nodes using those compatible
> strings
> 
> $ git grep omap4-dmm arch/arm/boot/dts/omap*
> arch/arm/boot/dts/omap4.dtsi:                   compatible = "ti,omap4-dmm";
> 
> Does that mean there is no need for MODULE_ALIAS("platform:" DMM_DRIVER_NAME)?

No, I think that's the thing that makes omapdrm load.

 Tomi

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

Re: [PATCH 3/5] drm: omapdrm: tiler: Fix module autoload for OF  platform driver Tomi Valkeinen <tomi.valkeinen@ti.com> - 2015-09-24 12:50 +0200
  Re: [PATCH 3/5] drm: omapdrm: tiler: Fix module autoload for OF  platform driver Luis de Bethencourt <luis@debethencourt.com> - 2015-09-24 17:40 +0200
    Re: [PATCH 3/5] drm: omapdrm: tiler: Fix module autoload for OF  platform driver Tomi Valkeinen <tomi.valkeinen@ti.com> - 2015-09-24 17:50 +0200
      Re: [PATCH 3/5] drm: omapdrm: tiler: Fix module autoload for OF  platform driver Luis de Bethencourt <luis@debethencourt.com> - 2015-09-24 18:10 +0200
        Re: [PATCH 3/5] drm: omapdrm: tiler: Fix module autoload for OF  platform driver Tomi Valkeinen <tomi.valkeinen@ti.com> - 2015-09-24 18:20 +0200

csiph-web