> -----Original Message----- > From: arun c [mailto:arun.edarath@xxxxxxxxx] > Sent: Tuesday, July 29, 2008 5:11 PM > To: Syed Mohammed, Khasim > Cc: linux-omap@xxxxxxxxxxxxxxx > Subject: Re: Does DSS support 24 bits_per_pixel? > > On Tue, Jul 29, 2008 at 4:37 PM, Syed Mohammed, Khasim <khasim@xxxxxx> wrote: > > > >> Hi all, > >> > >> Does omap (2&3) DSS support 24 bit output? > >> > >> I was trying to enable DVI support for omap2evm(data lines=24, bits > >> per pixel =24), I couldn't find > >> 24 bit support in omapfb driver(git) and omap24xxfb(linux-2.6.14-omap2-v5). > >> > >> In TRM its given that it supports 24 BPP. Then why its absent in both > >> the drivers?. Does the hardware > >> really support 24 bit format? > >> > > Hardware does support RGB 24 packed and unpacked modes. > > > > In TI code it is implemented in V4L based Video driver and also in FB driver, > omap2_disp_config_gfxlayer selects the bpp and configures DISPC_GFX_ATTRIBUTES register accordingly. > > > > But in the TI code, > > omap24xxfb_check_mode(const struct omap24xxfb_info *oinfo, > struct fb_var_screeninfo *var) > { > u32 pixclock, clkdiv; > u32 display_xres, display_yres; > int output_dev = omap2_disp_get_output_dev(OMAP2_GRAPHICS); > > omap2_disp_get_panel_size(output_dev, &display_xres, &display_yres); > if (oinfo->rotation_support) { > > if (var->rotate % 90 != 0) > return -EINVAL; > > if (!((var->bits_per_pixel == 8) || > (var->bits_per_pixel == 16))) > return -EINVAL; > As this was the last update for OMAP2 platforms, I expect it to have support for 24 as well. I am downloading the code base to check for the same. In the latest code for 3530 I have if (!((var->bits_per_pixel == 8) || (var->bits_per_pixel == 16)|| (var->bits_per_pixel == 32))) { Actually you can even refer to omap3530 code base and port this functionality to OMAP2. Regards, Khasim http://khasim.blogspot.com -- 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
