On 2012-11-28 12:41, Chandrabhanu Mahapatra wrote: > The DISPC specific dss_param_range are moved from struct omap_dss_features to > corresponding DSS version specific dispc_param_range struct, initialized in > dispc_features thereby enabling local access. The mgr_width_max and > mgr_height_max, members of dispc_features, are also moved to dispc_param_range. > > Signed-off-by: Chandrabhanu Mahapatra <cmahapatra@xxxxxx> > --- > drivers/video/omap2/dss/dispc.c | 80 +++++++++++++++++++++++--------- > drivers/video/omap2/dss/dss.h | 4 ++ > drivers/video/omap2/dss/dss_features.c | 16 ------- > drivers/video/omap2/dss/dss_features.h | 3 -- > 4 files changed, 63 insertions(+), 40 deletions(-) I think the exact same comments, as to the previous patch, apply here. Instead of creating param_range, use the dss_param_range. And see if dispc.feat->params[FEAT_PARAM_PCD].min style lines can be simplified with helper func/pointer. Tomi
Attachment:
signature.asc
Description: OpenPGP digital signature
