On Mon, 2011-03-14 at 23:28 -0500, Taneja, Archit wrote: > The structures dss_reg_field and dss_clk_source_name have enum members which > specify the register field and the clock source respectively. These members are > not used to choose the correct result in the corresponding feature functions. > Remove these members and change the features array declaration to incorporate > these enums. > > The structure dss_clk_source_name without the enum member is just a pointer to > an string. Remove the structure and use a character pointer directly. > > Signed-off-by: Archit Taneja <archit@xxxxxx> > --- > drivers/video/omap2/dss/dss.c | 10 ++-- > drivers/video/omap2/dss/dss.h | 6 --- > drivers/video/omap2/dss/dss_features.c | 77 ++++++++++++++++---------------- > 3 files changed, 43 insertions(+), 50 deletions(-) Thanks, looks good to me. 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