Am Samstag, den 07.06.2014, 14:56 -0700 schrieb Steve Longerbeam: > Signed-off-by: Jiada Wang <jiada_wang@xxxxxxxxxx> > --- > drivers/staging/imx-drm/ipu-v3/ipu-common.c | 18 ++++++++++++++++++ > include/linux/platform_data/imx-ipu-v3.h | 1 + > 2 files changed, 19 insertions(+) > > diff --git a/drivers/staging/imx-drm/ipu-v3/ipu-common.c b/drivers/staging/imx-drm/ipu-v3/ipu-common.c > index f8e8c56..2d95a7c 100644 > --- a/drivers/staging/imx-drm/ipu-v3/ipu-common.c > +++ b/drivers/staging/imx-drm/ipu-v3/ipu-common.c > @@ -829,10 +829,28 @@ static struct ipu_devtype ipu_type_imx6q = { > .type = IPUV3H, > }; > > +static struct ipu_devtype ipu_type_imx6dl = { > + .name = "IPUv3HDL", > + .cm_ofs = 0x00200000, > + .cpmem_ofs = 0x00300000, > + .srm_ofs = 0x00340000, > + .tpm_ofs = 0x00360000, > + .csi0_ofs = 0x00230000, > + .csi1_ofs = 0x00238000, > + .disp0_ofs = 0x00240000, > + .disp1_ofs = 0x00248000, > + .smfc_ofs = 0x00250000, > + .ic_ofs = 0x00220000, > + .vdi_ofs = 0x00268000, > + .dc_tmpl_ofs = 0x00380000, > + .type = IPUV3HDL, > +}; This just duplicates ipu_type_imx6. Do I understand correctly that this new type was added just to account for the different input multiplexer setup between i.MX6Q and i.MX6DL outside of the IPU? This would not be necessary if we describe the multiplexers as separate v4l2_subdev entities. The same applies to the following patch 05/43. regards Philipp -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html