On 16/09/13 10:18, Archit Taneja wrote:
> Currently, omapdrm, omapfb and omap_vout platform devices are created and
> registered through omap_arch_initcalls. In a multiplatform config. It's
> possible that all the corresponding configs for the above drivers along with
> omapdss config are selected even if the hardware doesn't have a DSS IP.
> 
> If the image is booted on a AM33xx platform, the above drm, fb and v4l devices
> would be registered throuth the omap_arch_initcalls even if omapdss itself 
> isn't
> registered. These platform devices don't cause any harm, but are unnecessary.
> 
> Move the registration of these devices into omap_display_init(), which 
> registers
> omapdss devices and is called only if the platform has DSS hardware.
> 
> Also, the first patch prevents creation of a DMM device when omapdrm device is
> registered. With the removal of address and irq data from the omap4 hwmods, 
> the
> probe of DMM driver fails and omapdrm isn't able to utilize the DMM hardware.
> This will be fixed when DMM DT nodes are added for omap4, omap5 and dra7x.
> 
> Changes in v2:
> - Move device creation for omapfb and omap_vout to omap_display_init too.
> - Keep the DMM DT conversion as a separate patch series since this series does
>   something different now.
> 
> Archit Taneja (4):
>   arm: omap: drm: Don't build device for DMM
>   arm: omap: display: Create omapdrm device inside omap_display_init
>   arm: omap: display: Create omapvrfb and omapfb devices inside
>     omap_display_init
>   arm: omap: display: Create omap_vout device inside omap_display_init
> 
>  arch/arm/mach-omap2/Makefile  |  6 +-----
>  arch/arm/mach-omap2/devices.c | 10 +++++-----
>  arch/arm/mach-omap2/display.c | 28 ++++++++++++++++++++++++++++
>  arch/arm/mach-omap2/display.h |  4 ++++
>  arch/arm/mach-omap2/drm.c     | 24 +++++-------------------
>  arch/arm/mach-omap2/fb.c      | 14 +++++++-------
>  6 files changed, 50 insertions(+), 36 deletions(-)

Acked-by: Tomi Valkeinen <[email protected]>

Tony, since there are no driver changes here, and it's unlikely that
we'd have a conflict between the dss driver changes and these changes, I
think it's best if this goes through linux-omap.

 Tomi


Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to