From: Maxime Ripard <maxime.ripard@bootlin.com>
To: Jan Kotas <jank@cadence.com>
Cc: mchehab@kernel.org, robh+dt@kernel.org, mark.rutland@arm.com,
linux-media@vger.kernel.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH 3/3] media: Add support for Cadence CSI2TX 2.1
Date: Sat, 20 Jul 2019 10:25:59 +0200 [thread overview]
Message-ID: <20190720082559.pk3odkf5hztgvpif@flea> (raw)
In-Reply-To: <20190718111509.29924-4-jank@cadence.com>
[-- Attachment #1: Type: text/plain, Size: 4320 bytes --]
Hi,
On Thu, Jul 18, 2019 at 12:15:09PM +0100, Jan Kotas wrote:
> This patch adds support for CSI2TX v2.1 version of the controller.
>
> Signed-off-by: Jan Kotas <jank@cadence.com>
> ---
> drivers/media/platform/cadence/cdns-csi2tx.c | 142 +++++++++++++++++++++------
> 1 file changed, 112 insertions(+), 30 deletions(-)
>
> diff --git a/drivers/media/platform/cadence/cdns-csi2tx.c b/drivers/media/platform/cadence/cdns-csi2tx.c
> index 232259c71..e4d08acfb 100644
> --- a/drivers/media/platform/cadence/cdns-csi2tx.c
> +++ b/drivers/media/platform/cadence/cdns-csi2tx.c
> @@ -52,6 +52,17 @@
> #define CSI2TX_STREAM_IF_CFG_REG(n) (0x100 + (n) * 4)
> #define CSI2TX_STREAM_IF_CFG_FILL_LEVEL(n) ((n) & 0x1f)
>
> +/* CSI2TX V2 Registers */
> +#define CSI2TX_V2_DPHY_CFG_REG 0x28
> +#define CSI2TX_V2_DPHY_CFG_RESET BIT(16)
> +#define CSI2TX_V2_DPHY_CFG_CLOCK_MODE BIT(10)
> +#define CSI2TX_V2_DPHY_CFG_MODE_MASK GENMASK(9, 8)
> +#define CSI2TX_V2_DPHY_CFG_MODE_LPDT (2 << 8)
> +#define CSI2TX_V2_DPHY_CFG_MODE_HS (1 << 8)
> +#define CSI2TX_V2_DPHY_CFG_MODE_ULPS (0 << 8)
> +#define CSI2TX_V2_DPHY_CFG_CLK_ENABLE BIT(4)
> +#define CSI2TX_V2_DPHY_CFG_LANE_ENABLE(n) BIT(n)
> +
> #define CSI2TX_LANES_MAX 4
> #define CSI2TX_STREAMS_MAX 4
>
> @@ -70,6 +81,13 @@ struct csi2tx_fmt {
> u32 bpp;
> };
>
> +struct csi2tx_priv;
> +
> +/* CSI2TX Variant Operations */
> +struct csi2tx_vops {
> + void (*dphy_setup)(struct csi2tx_priv *csi2tx);
> +};
> +
> struct csi2tx_priv {
> struct device *dev;
> unsigned int count;
> @@ -82,6 +100,8 @@ struct csi2tx_priv {
>
> void __iomem *base;
>
> + struct csi2tx_vops *vops;
> +
> struct clk *esc_clk;
> struct clk *p_clk;
> struct clk *pixel_clk[CSI2TX_STREAMS_MAX];
> @@ -209,53 +229,92 @@ static const struct v4l2_subdev_pad_ops csi2tx_pad_ops = {
> .set_fmt = csi2tx_set_pad_format,
> };
>
> -static void csi2tx_reset(struct csi2tx_priv *csi2tx)
> +/* Set Wake Up value in the D-PHY */
> +static void csi2tx_dphy_set_wakeup(struct csi2tx_priv *csi2tx)
> {
> - writel(CSI2TX_CONFIG_SRST_REQ, csi2tx->base + CSI2TX_CONFIG_REG);
> -
> - udelay(10);
> + writel(CSI2TX_DPHY_CLK_WAKEUP_ULPS_CYCLES(32),
> + csi2tx->base + CSI2TX_DPHY_CLK_WAKEUP_REG);
> }
>
> -static int csi2tx_start(struct csi2tx_priv *csi2tx)
> +/*
> + * Finishes the D-PHY initialization
> + * reg dphy cfg value to be used
> + */
> +static void csi2tx_dphy_init_finish(struct csi2tx_priv *csi2tx, u32 reg)
> {
> - struct media_entity *entity = &csi2tx->subdev.entity;
> - struct media_link *link;
> unsigned int i;
> - u32 reg;
>
> - csi2tx_reset(csi2tx);
> + udelay(10);
>
> - writel(CSI2TX_CONFIG_CFG_REQ, csi2tx->base + CSI2TX_CONFIG_REG);
> + /* Enable our (clock and data) lanes */
> + reg |= CSI2TX_DPHY_CFG_CLK_ENABLE;
> + for (i = 0; i < csi2tx->num_lanes; i++)
> + reg |= CSI2TX_DPHY_CFG_LANE_ENABLE(csi2tx->lanes[i] - 1);
> + writel(reg, csi2tx->base + CSI2TX_DPHY_CFG_REG);
>
> udelay(10);
>
> - /* Configure our PPI interface with the D-PHY */
> - writel(CSI2TX_DPHY_CLK_WAKEUP_ULPS_CYCLES(32),
> - csi2tx->base + CSI2TX_DPHY_CLK_WAKEUP_REG);
> + /* Switch to HS mode */
> + reg &= ~CSI2TX_DPHY_CFG_MODE_MASK;
> + writel(reg | CSI2TX_DPHY_CFG_MODE_HS,
> + csi2tx->base + CSI2TX_DPHY_CFG_REG);
> +}
> +
> +/* Configures D-PHY in CSIv1.3 */
> +static void csi2tx_dphy_setup(struct csi2tx_priv *csi2tx)
> +{
> + u32 reg;
> + unsigned int i;
> +
> + csi2tx_dphy_set_wakeup(csi2tx);
>
> /* Put our lanes (clock and data) out of reset */
> reg = CSI2TX_DPHY_CFG_CLK_RESET | CSI2TX_DPHY_CFG_MODE_LPDT;
> for (i = 0; i < csi2tx->num_lanes; i++)
> - reg |= CSI2TX_DPHY_CFG_LANE_RESET(csi2tx->lanes[i]);
> + reg |= CSI2TX_DPHY_CFG_LANE_RESET(csi2tx->lanes[i] - 1);
This looks like a separate change?
> writel(reg, csi2tx->base + CSI2TX_DPHY_CFG_REG);
>
> - udelay(10);
> + csi2tx_dphy_init_finish(csi2tx, reg);
> +}
>
> - /* Enable our (clock and data) lanes */
> - reg |= CSI2TX_DPHY_CFG_CLK_ENABLE;
> - for (i = 0; i < csi2tx->num_lanes; i++)
> - reg |= CSI2TX_DPHY_CFG_LANE_ENABLE(csi2tx->lanes[i]);
And you have a similar change here that should be in a separate patch,
with a proper explanation.
Otherwise, the rest looks good.
Maxime
--
Maxime Ripard, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
next prev parent reply other threads:[~2019-07-20 8:26 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-07-18 11:15 [PATCH 0/3] media: Add support for Cadence CSI2TX version 2.1 Jan Kotas
2019-07-18 11:15 ` [PATCH 1/3] media: dt-bindings: Update bindings " Jan Kotas
2019-07-20 8:26 ` Maxime Ripard
2019-07-18 11:15 ` [PATCH 2/3] media: Add lane checks for Cadence CSI2TX Jan Kotas
2019-07-18 11:52 ` Marc Gonzalez
2019-07-18 12:07 ` Jan Kotas
2019-07-20 8:26 ` Maxime Ripard
2019-07-18 11:15 ` [PATCH 3/3] media: Add support for Cadence CSI2TX 2.1 Jan Kotas
2019-07-20 8:25 ` Maxime Ripard [this message]
2019-07-22 7:07 ` Jan Kotas
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20190720082559.pk3odkf5hztgvpif@flea \
--to=maxime.ripard@bootlin.com \
--cc=devicetree@vger.kernel.org \
--cc=jank@cadence.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=mark.rutland@arm.com \
--cc=mchehab@kernel.org \
--cc=robh+dt@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).