Hi Tony, On 07/07/2014 02:46 PM, Tony Lindgren wrote: > * Jyri Sarha <jsarha@xxxxxx> [140626 12:26]: >> This code is not working currently and it can be removed. There is a >> conflict in sharing resources with the actual HDMI driver and with >> the ASoC HDMI audio DAI driver. > > Is this OK to queue for v3.17 or do we need this for v3.16-rc series? Jyri is away currently, but I think it would be great if this patch goes in for 3.16. The patch does not introduce regression since the OMAP4/5 HDMI is in non working state in the kernel so removing the platform device right now will ease up the merger of the new and working code. Regards, Péter > > Regards, > > Tony > >> Signed-off-by: Jyri Sarha <jsarha@xxxxxx> >> --- >> arch/arm/mach-omap2/devices.c | 28 ---------------------------- >> 1 file changed, 28 deletions(-) >> >> diff --git a/arch/arm/mach-omap2/devices.c b/arch/arm/mach-omap2/devices.c >> index 592ba0a..b6f8f34 100644 >> --- a/arch/arm/mach-omap2/devices.c >> +++ b/arch/arm/mach-omap2/devices.c >> @@ -297,33 +297,6 @@ static void omap_init_audio(void) >> static inline void omap_init_audio(void) {} >> #endif >> >> -#if defined(CONFIG_SND_OMAP_SOC_OMAP_HDMI) || \ >> - defined(CONFIG_SND_OMAP_SOC_OMAP_HDMI_MODULE) >> - >> -static struct platform_device omap_hdmi_audio = { >> - .name = "omap-hdmi-audio", >> - .id = -1, >> -}; >> - >> -static void __init omap_init_hdmi_audio(void) >> -{ >> - struct omap_hwmod *oh; >> - struct platform_device *pdev; >> - >> - oh = omap_hwmod_lookup("dss_hdmi"); >> - if (!oh) >> - return; >> - >> - pdev = omap_device_build("omap-hdmi-audio-dai", -1, oh, NULL, 0); >> - WARN(IS_ERR(pdev), >> - "Can't build omap_device for omap-hdmi-audio-dai.\n"); >> - >> - platform_device_register(&omap_hdmi_audio); >> -} >> -#else >> -static inline void omap_init_hdmi_audio(void) {} >> -#endif >> - >> #if defined(CONFIG_SPI_OMAP24XX) || defined(CONFIG_SPI_OMAP24XX_MODULE) >> >> #include <linux/platform_data/spi-omap2-mcspi.h> >> @@ -459,7 +432,6 @@ static int __init omap2_init_devices(void) >> */ >> omap_init_audio(); >> omap_init_camera(); >> - omap_init_hdmi_audio(); >> omap_init_mbox(); >> /* If dtb is there, the devices will be created dynamically */ >> if (!of_have_populated_dt()) { >> -- >> 1.7.9.5 >> >> -- >> To unsubscribe from this list: send the line "unsubscribe linux-omap" in >> the body of a message to majordomo@xxxxxxxxxxxxxxx >> More majordomo info at http://vger.kernel.org/majordomo-info.html >> -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html