Hi, On Mon, 2010-11-22 at 12:53 +0530, ext Archit Taneja wrote: > From: Sumit Semwal <sumit.semwal@xxxxxx> > > A new member 'channel' is introduced in omap_dss_device structure to determine > which channel the panel uses. The dss_recheck_connections() called in dss_driver_probe() > to set the correct manager to the corresponding omap_dss_device. The interface drivers > (dsi.c, sdi.c etc) now call dispc functions with dssdev->manager->id as a parameter to > specify the DISPC channel. > > The following dispc functions are changed to incorporate channel as an argument: > -dispc_enable_fifohandcheck() > -dispc_set_lcd_size() > -dispc_set_parallel_interface_mode() > -dispc_set_tft_data_lines() > -dispc_set_lcd_display_type() > -dispc_set_lcd_timings() This patch combines two separate things: 1) the new channel-field + related changes (dss_recheck_connections), and 2) converting dispc functions to accept channel as a parameter. Generally about the whole patch set, I think this is starting to look ok. But two things, which are cosmetical: - I wouldn't mind a bit more verbose commit descriptions. Of course it's easy to say "write better descriptions", and I don't have any direct advice for this. However, remember that the 0000-patch won't be in the git log, so all important information should be available also from the patch descriptions. - The files are getting quite crowded with code that checks for the channel and then do the work with bits/irqs depending on the channel. This makes the code a bit difficult to read. I don't have any clear ideas right now how to make it clearer, but some methods to generalize these kinds of functions would be nice. But this is not so important for the time being, and we can improve it later. Tomi -- 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
