RE: [PATCH v5 3/3] media: nxp: imx8-isi: add ISI support for i.MX93

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Alexander,

> -----Original Message-----
> From: Alexander Stein <alexander.stein@xxxxxxxxxxxxxxx>
> Sent: 2023年6月29日 14:43
> To: linux-media@xxxxxxxxxxxxxxx; dl-linux-imx <linux-imx@xxxxxxx>;
> devicetree@xxxxxxxxxxxxxxx; G.N. Zhou (OSS) <guoniu.zhou@xxxxxxxxxxx>
> Cc: laurent.pinchart@xxxxxxxxxxxxxxxx; mchehab@xxxxxxxxxx;
> robh+dt@xxxxxxxxxx; krzysztof.kozlowski+dt@xxxxxxxxxx; conor+dt@xxxxxxxxxx;
> Xavier Roumegue (OSS) <xavier.roumegue@xxxxxxxxxxx>;
> kernel@xxxxxxxxxxxxxx; jacopo.mondi@xxxxxxxxxxxxxxxx;
> sakari.ailus@xxxxxxxxxxxxxxx
> Subject: Re: [PATCH v5 3/3] media: nxp: imx8-isi: add ISI support for i.MX93
> 
> Caution: This is an external email. Please take care when clicking links or opening
> attachments. When in doubt, report the message using the 'Report this email'
> button
> 
> 
> Hi Guoniu,
> 
> thanks for the patch series.
> 
> Am Donnerstag, 29. Juni 2023, 03:36:21 CEST schrieb
> guoniu.zhou@xxxxxxxxxxx:
> > ********************
> > Achtung externe E-Mail: Öffnen Sie Anhänge und Links nur, wenn Sie
> > wissen, dass diese aus einer sicheren Quelle stammen und sicher sind.
> > Leiten Sie die E-Mail im Zweifelsfall zur Prüfung an den IT-Helpdesk weiter.
> > Attention external email: Open attachments and links only if you know
> > that they are from a secure source and are safe. In doubt forward the
> > email to the IT-Helpdesk to check it. ********************
> >
> > From: "Guoniu.zhou" <guoniu.zhou@xxxxxxx>
> >
> > i.MX93 use a different gasket which has different register definition
> > compared with i.MX8. Hence implement the gasket callbacks in order to
> > add ISI support for i.MX93.
> >
> > Signed-off-by: Guoniu.zhou <guoniu.zhou@xxxxxxx>
> > Reviewed-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx>
> > ---
> >  .../platform/nxp/imx8-isi/imx8-isi-core.c     | 15 ++++++++++
> >  .../platform/nxp/imx8-isi/imx8-isi-core.h     |  2 ++
> >  .../platform/nxp/imx8-isi/imx8-isi-gasket.c   | 30 +++++++++++++++++++
> >  3 files changed, 47 insertions(+)
> >
> > diff --git a/drivers/media/platform/nxp/imx8-isi/imx8-isi-core.c
> > b/drivers/media/platform/nxp/imx8-isi/imx8-isi-core.c index
> > 5165f8960c2c..27bd18b7ee65 100644
> > --- a/drivers/media/platform/nxp/imx8-isi/imx8-isi-core.c
> > +++ b/drivers/media/platform/nxp/imx8-isi/imx8-isi-core.c
> > @@ -307,6 +307,20 @@ static const struct mxc_isi_plat_data
> mxc_imx8mp_data =
> > { .has_36bit_dma              = true,
> >  };
> >
> > +static const struct mxc_isi_plat_data mxc_imx93_data = {
> > +     .model                  = MXC_ISI_IMX93,
> > +     .num_ports              = 1,
> > +     .num_channels           = 1,
> > +     .reg_offset             = 0,
> > +     .ier_reg                = &mxc_imx8_isi_ier_v2,
> > +     .set_thd                = &mxc_imx8_isi_thd_v1,
> > +     .clks                   = mxc_imx8mn_clks,
> > +     .num_clks               = ARRAY_SIZE(mxc_imx8mn_clks),
> > +     .buf_active_reverse     = true,
> > +     .gasket_ops             = &mxc_imx93_gasket_ops,
> > +     .has_36bit_dma          = false,
> > +};
> > +
> >  /*
> > ----------------------------------------------------------------------
> > -----
> > -- * Power management
> >   */
> > @@ -518,6 +532,7 @@ static int mxc_isi_remove(struct platform_device
> > *pdev)  static const struct of_device_id mxc_isi_of_match[] = {
> >       { .compatible = "fsl,imx8mn-isi", .data = &mxc_imx8mn_data },
> >       { .compatible = "fsl,imx8mp-isi", .data = &mxc_imx8mp_data },
> > +     { .compatible = "fsl,imx93-isi", .data = &mxc_imx93_data },
> >       { /* sentinel */ },
> >  };
> >  MODULE_DEVICE_TABLE(of, mxc_isi_of_match); diff --git
> > a/drivers/media/platform/nxp/imx8-isi/imx8-isi-core.h
> > b/drivers/media/platform/nxp/imx8-isi/imx8-isi-core.h index
> > 78ca047d93d1..2810ebe9b5f7 100644
> > --- a/drivers/media/platform/nxp/imx8-isi/imx8-isi-core.h
> > +++ b/drivers/media/platform/nxp/imx8-isi/imx8-isi-core.h
> > @@ -158,6 +158,7 @@ struct mxc_gasket_ops {  enum model {
> >       MXC_ISI_IMX8MN,
> >       MXC_ISI_IMX8MP,
> > +     MXC_ISI_IMX93,
> >  };
> >
> >  struct mxc_isi_plat_data {
> > @@ -295,6 +296,7 @@ struct mxc_isi_dev {  };
> >
> >  extern const struct mxc_gasket_ops mxc_imx8_gasket_ops;
> > +extern const struct mxc_gasket_ops mxc_imx93_gasket_ops;
> >
> >  int mxc_isi_crossbar_init(struct mxc_isi_dev *isi);  void
> > mxc_isi_crossbar_cleanup(struct mxc_isi_crossbar *xbar); diff --git
> > a/drivers/media/platform/nxp/imx8-isi/imx8-isi-gasket.c
> > b/drivers/media/platform/nxp/imx8-isi/imx8-isi-gasket.c index
> > 1d632dc60699..50ac1d3a2b6f 100644
> > --- a/drivers/media/platform/nxp/imx8-isi/imx8-isi-gasket.c
> > +++ b/drivers/media/platform/nxp/imx8-isi/imx8-isi-gasket.c
> > @@ -53,3 +53,33 @@ const struct mxc_gasket_ops mxc_imx8_gasket_ops = {
> >       .enable = mxc_imx8_gasket_enable,
> >       .disable = mxc_imx8_gasket_disable,  };
> > +
> > +/*
> > ----------------------------------------------------------------------
> > -----
> > -- + * i.MX93 gasket
> > + **/
> > +
> > +#define DISP_MIX_CAMERA_MUX                     0x30
> 
> Which peripheral does this refer to? I would assume it is the Media Mix Domain
> Block Control, but there is no register at 0x30 mentioned in the reference manual.
> You have some additional information?
> 
> Also which type of input did you use? MIPI-CSI2 or parallel interface?

Yes, it's Media Mix Domain Block Control(I name it Disp_Mix due to history reason).

I check RM in NXP Website and don't found 0x30 as you said since it's REV2, not latest one which
still under review.

The input is MIPI CSI-2 by default.(We use default value in the serials)

> 
> Thanks and best regards,
> Alexander
> 
> > +#define DISP_MIX_CAMERA_MUX_DATA_TYPE(x)        (((x) & 0x3f) << 3)
> > +#define DISP_MIX_CAMERA_MUX_GASKET_ENABLE       BIT(16)
> > +static void mxc_imx93_gasket_enable(struct mxc_isi_dev *isi,
> > +                                 const struct v4l2_mbus_frame_desc
> *fd,
> > +                                 const struct v4l2_mbus_framefmt
> *fmt,
> > +                                 const unsigned int port) {
> > +     u32 val;
> > +
> > +     val = DISP_MIX_CAMERA_MUX_DATA_TYPE(fd->entry[0].bus.csi2.dt);
> > +     val |= DISP_MIX_CAMERA_MUX_GASKET_ENABLE;
> > +     regmap_write(isi->gasket, DISP_MIX_CAMERA_MUX, val); }
> > +
> > +static void mxc_imx93_gasket_disable(struct mxc_isi_dev *isi,
> > +                                  unsigned int port) {
> > +     regmap_write(isi->gasket, DISP_MIX_CAMERA_MUX, 0); }
> > +
> > +const struct mxc_gasket_ops mxc_imx93_gasket_ops = {
> > +     .enable = mxc_imx93_gasket_enable,
> > +     .disable = mxc_imx93_gasket_disable, };
> 
> 
> --
> TQ-Systems GmbH | Mühlstraße 2, Gut Delling | 82229 Seefeld, Germany
> Amtsgericht München, HRB 105018
> Geschäftsführer: Detlef Schneider, Rüdiger Stahl, Stefan Schneider
> http://www.tq-group.com/
> 





[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux