> +static const struct __initdata dispc_features omap2_dispc_features = { > + .hp_max = 256, > + .vp_max = 255, > + .sw_max = 64, > + .sw_start = 5, > + .fp_start = 15, > + .bp_start = 27, > + .calc_scaling = dispc_ovl_calc_scaling_24xx, > + .calc_core_clk = calc_core_clk_24xx, > +}; > + > +static const struct __initdata dispc_features omap3_2_1_dispc_features = { > + .hp_max = 256, > + .vp_max = 255, > + .sw_max = 64, > + .sw_start = 5, > + .fp_start = 15, > + .bp_start = 27, > + .calc_scaling = dispc_ovl_calc_scaling_34xx, > + .calc_core_clk = calc_core_clk_34xx, > +}; > + > +static const struct __initdata dispc_features omap3_3_0_dispc_features = { > + .hp_max = 4096, > + .vp_max = 4095, > + .sw_max = 256, > + .sw_start = 7, > + .fp_start = 19, > + .bp_start = 31, > + .calc_scaling = dispc_ovl_calc_scaling_34xx, > + .calc_core_clk = calc_core_clk_34xx, > +}; > + > +static const struct __initdata dispc_features omap4_dispc_features = { > + .hp_max = 4096, > + .vp_max = 4095, > + .sw_max = 256, > + .sw_start = 7, > + .fp_start = 19, > + .bp_start = 31, > + .calc_scaling = dispc_ovl_calc_scaling_44xx, > + .calc_core_clk = calc_core_clk_44xx, > +}; > + Here the dispc_features not only mention the omap name but also the revision like omap3_3_0_dispc_features which initializes data for OMAP3430_REV_ES3_0 and higher. May be omap34xx_rev3_0_dispc_features is a better name for this. For others omap44xx_dispc_features kind of name should be ok without revision number being mentioned. What d you say? > +static int __init dispc_init_features(struct device *dev) > +{ > + dispc.feat = devm_kzalloc(dev, sizeof(*dispc.feat), GFP_KERNEL); > + if (!dispc.feat) { > + dev_err(dev, "Failed to allocate DISPC Features\n"); > + return -ENOMEM; > + } > + > + if (cpu_is_omap24xx()) { > + dispc.feat = &omap2_dispc_features; > + } else if (cpu_is_omap34xx()) { > + if (omap_rev() < OMAP3430_REV_ES3_0) > + dispc.feat = &omap3_2_1_dispc_features; > + else > + dispc.feat = &omap3_3_0_dispc_features; > + } else { > + dispc.feat = &omap4_dispc_features; > + } > + > + return 0; > +} > + -- Chandrabhanu Mahapatra Texas Instruments India Pvt. Ltd. -- 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