Hi, On Thursday 24 February 2011 07:04 PM, Valkeinen, Tomi wrote:
Only OMAP 3430 hardware has SDI support. The availability of SDI HW can be found out by checking if the LCD channel supports SDI displays. This patch checks for SDI HW support before accessing SDI registers, which fixes a crash on OMAP4 when SDI SW support is compiled in. Signed-off-by: Tomi Valkeinen<tomi.valkeinen@xxxxxx> --- drivers/video/omap2/dss/display.c | 10 ++++++++++ drivers/video/omap2/dss/dss.c | 29 ++++++++++++++++++----------- 2 files changed, 28 insertions(+), 11 deletions(-) diff --git a/drivers/video/omap2/dss/display.c b/drivers/video/omap2/dss/display.c index 3f4fa0b..58459f4 100644 --- a/drivers/video/omap2/dss/display.c +++ b/drivers/video/omap2/dss/display.c @@ -30,6 +30,7 @@ #include<plat/display.h> #include "dss.h" +#include "dss_features.h" static LIST_HEAD(display_list); @@ -392,6 +393,15 @@ void dss_init_device(struct platform_device *pdev, struct device_attribute *attr; int i; int r; + enum omap_display_type supported; + + supported = dss_feat_get_supported_displays(dssdev->channel); + + if (!(supported& dssdev->type)) { + DSSERR("Unsupported display interface for display '%s'.\n", + dssdev->name); + return; + }
This would make it necessary to specify the channel in the board file, especially digit. I think this patch should also add the channel parameters for all board files which add a tv display.
<snip> Archit -- 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