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日 16:00
> To: linux-media@xxxxxxxxxxxxxxx; dl-linux-imx <linux-imx@xxxxxxx>;
> devicetree@xxxxxxxxxxxxxxx; G.N. Zhou (OSS) <guoniu.zhou@xxxxxxxxxxx>;
> 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,
> 
> Am Donnerstag, 29. Juni 2023, 09:07:23 CEST schrieb G.N. Zhou (OSS):
> > 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.
> 
> Thanks for confirmation.
> 
> > The input is MIPI CSI-2 by default.(We use default value in the
> > serials)
> 
> Thanks. Is there already a (preliminary) driver for the MIPI CSI-2 peripheral?

Yes, I'm preparing to send out it these days.

> 
> Thanks and best regards,
> Alexander
> 
> > >
> > > 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/
> > >
> >
> >
> 
> 
> --
> 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