From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from perceval.ideasonboard.com (perceval.ideasonboard.com [213.167.242.64]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 5614B70 for ; Wed, 28 Jul 2021 05:51:54 +0000 (UTC) Received: from pendragon.ideasonboard.com (62-78-145-57.bb.dnainternet.fi [62.78.145.57]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id BF873EE; Wed, 28 Jul 2021 07:51:45 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1627451506; bh=ZB+MfWCEyRrx4Ei2O+bpHnYURN9YC+OnavdTKNrfe9Y=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=a+1gQGL7nOQjvNAV73IED0h+StcuezPlSWcTLY1Ej5EZ16beRIu+1ezg9QTdt3KJL jALGHaOtufNvgJQ6lnfe3xbrJQLcnU2+gyYiZdqeWxP7BgRO1E4FVxMuru0YemTwnw n1bX0IKhA66XDq5Z2PNO17UPST7m3eyAPR3Y3wKY= Date: Wed, 28 Jul 2021 08:51:39 +0300 From: Laurent Pinchart To: Dafna Hirschfeld Cc: Martin Kepplinger , shawnguo@kernel.org, devicetree@vger.kernel.org, festevam@gmail.com, kernel@pengutronix.de, kernel@puri.sm, krzk@kernel.org, linux-arm-kernel@lists.infradead.org, linux-imx@nxp.com, linux-kernel@vger.kernel.org, linux-media@vger.kernel.org, linux-staging@lists.linux.dev, m.felsch@pengutronix.de, mchehab@kernel.org, phone-devel@vger.kernel.org, robh@kernel.org, slongerbeam@gmail.com Subject: Re: [PATCH v9 2/3] media: imx: add a driver for i.MX8MQ mipi csi rx phy and controller Message-ID: References: <20210726082117.2423597-1-martin.kepplinger@puri.sm> <20210726082117.2423597-3-martin.kepplinger@puri.sm> <489914ac-230f-2fc0-4b10-ae158ab6a95c@collabora.com> Precedence: bulk X-Mailing-List: linux-staging@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <489914ac-230f-2fc0-4b10-ae158ab6a95c@collabora.com> Hi Dafna, On Wed, Jul 28, 2021 at 07:02:57AM +0200, Dafna Hirschfeld wrote: > On 26.07.21 10:21, Martin Kepplinger wrote: > > Add a driver to support the i.MX8MQ MIPI CSI receiver. The hardware side > > is based on > > https://source.codeaurora.org/external/imx/linux-imx/tree/drivers/media/platform/imx8/mxc-mipi-csi2_yav.c?h=imx_5.4.70_2.3.0 > > > > It's built as part of VIDEO_IMX7_CSI because that's documented to support > > i.MX8M platforms. This driver adds i.MX8MQ support where currently only the > > i.MX8MM platform has been supported. > > > > Signed-off-by: Martin Kepplinger > > Reviewed-by: Laurent Pinchart > > --- > > drivers/staging/media/imx/Makefile | 1 + > > drivers/staging/media/imx/imx8mq-mipi-csi2.c | 976 +++++++++++++++++++ > > 2 files changed, 977 insertions(+) > > create mode 100644 drivers/staging/media/imx/imx8mq-mipi-csi2.c > > > > diff --git a/drivers/staging/media/imx/Makefile b/drivers/staging/media/imx/Makefile > > index 6ac33275cc97..19c2fc54d424 100644 > > --- a/drivers/staging/media/imx/Makefile > > +++ b/drivers/staging/media/imx/Makefile > > @@ -16,3 +16,4 @@ obj-$(CONFIG_VIDEO_IMX_CSI) += imx6-mipi-csi2.o > > > > obj-$(CONFIG_VIDEO_IMX7_CSI) += imx7-media-csi.o > > obj-$(CONFIG_VIDEO_IMX7_CSI) += imx7-mipi-csis.o > > +obj-$(CONFIG_VIDEO_IMX7_CSI) += imx8mq-mipi-csi2.o > > diff --git a/drivers/staging/media/imx/imx8mq-mipi-csi2.c b/drivers/staging/media/imx/imx8mq-mipi-csi2.c > > new file mode 100644 > > index 000000000000..c4f4fe0c533b > > --- /dev/null > > +++ b/drivers/staging/media/imx/imx8mq-mipi-csi2.c > > @@ -0,0 +1,976 @@ > > +// SPDX-License-Identifier: GPL-2.0 > > +/* > > + * NXP i.MX8MQ SoC series MIPI-CSI2 receiver driver > > + * > > + * Copyright (C) 2021 Purism SPC > > + */ > > + > > +#include > > +#include > > +#include > > +#include > > +#include > > +#include > > +#include > > +#include > > +#include > > +#include > > +#include > > +#include > > +#include > > +#include > > +#include > > +#include > > +#include > > +#include > > +#include > > + > > +#include > > +#include > > +#include > > +#include > > +#include > > + > > +#define MIPI_CSI2_DRIVER_NAME "imx8mq-mipi-csi2" > > +#define MIPI_CSI2_SUBDEV_NAME MIPI_CSI2_DRIVER_NAME > > + > > +#define MIPI_CSI2_PAD_SINK 0 > > +#define MIPI_CSI2_PAD_SOURCE 1 > > +#define MIPI_CSI2_PADS_NUM 2 > > + > > +#define MIPI_CSI2_DEF_PIX_WIDTH 640 > > +#define MIPI_CSI2_DEF_PIX_HEIGHT 480 > > + > > +/* Register map definition */ > > + > > +/* i.MX8MQ CSI-2 controller CSR */ > > +#define CSI2RX_CFG_NUM_LANES 0x100 > > +#define CSI2RX_CFG_DISABLE_DATA_LANES 0x104 > > +#define CSI2RX_BIT_ERR 0x108 > > +#define CSI2RX_IRQ_STATUS 0x10c > > +#define CSI2RX_IRQ_MASK 0x110 > > +#define CSI2RX_IRQ_MASK_ALL 0x1ff > > +#define CSI2RX_IRQ_MASK_ULPS_STATUS_CHANGE 0x8 > > +#define CSI2RX_ULPS_STATUS 0x114 > > +#define CSI2RX_PPI_ERRSOT_HS 0x118 > > +#define CSI2RX_PPI_ERRSOTSYNC_HS 0x11c > > +#define CSI2RX_PPI_ERRESC 0x120 > > +#define CSI2RX_PPI_ERRSYNCESC 0x124 > > +#define CSI2RX_PPI_ERRCONTROL 0x128 > > +#define CSI2RX_CFG_DISABLE_PAYLOAD_0 0x12c > > +#define CSI2RX_CFG_VID_P_FIFO_SEND_LEVEL 0x188 > > +#define CSI2RX_CFG_DISABLE_PAYLOAD_1 0x130 > > + > > +enum { > > + ST_POWERED = 1, > > + ST_STREAMING = 2, > > + ST_SUSPENDED = 4, > > +}; > > + > > +enum imx8mq_mipi_csi_clk { > > + CSI2_CLK_CORE, > > + CSI2_CLK_ESC, > > + CSI2_CLK_UI, > > + CSI2_NUM_CLKS, > > +}; > > + > > +static const char * const imx8mq_mipi_csi_clk_id[CSI2_NUM_CLKS] = { > > + [CSI2_CLK_CORE] = "core", > > + [CSI2_CLK_ESC] = "esc", > > + [CSI2_CLK_UI] = "ui", > > +}; > > + > > +#define CSI2_NUM_CLKS ARRAY_SIZE(imx8mq_mipi_csi_clk_id) > > + > > +#define GPR_CSI2_1_RX_ENABLE BIT(13) > > +#define GPR_CSI2_1_VID_INTFC_ENB BIT(12) > > +#define GPR_CSI2_1_HSEL BIT(10) > > +#define GPR_CSI2_1_CONT_CLK_MODE BIT(8) > > +#define GPR_CSI2_1_S_PRG_RXHS_SETTLE(x) (((x) & 0x3f) << 2) > > + > > +/* > > + * The send level configures the number of entries that must accumulate in > > + * the Pixel FIFO before the data will be transferred to the video output. > > + * See https://community.nxp.com/t5/i-MX-Processors/IMX8M-MIPI-CSI-Host-Controller-send-level/m-p/864005/highlight/true#M131704 > > If there is more important information in that link you think is valuable, > better to write it here in case the link get broken. Good idea. That link also has information about registers 0x180 and 0x184, which should be captured in the form of comments in imx8mq_mipi_csi_set_params(). > > + */ > > +#define CSI2RX_SEND_LEVEL 64 > > + > > +struct csi_state { > > + struct device *dev; > > + void __iomem *regs; > > + struct clk_bulk_data clks[CSI2_NUM_CLKS]; > > + struct reset_control *rst; > > + struct regulator *mipi_phy_regulator; > > + > > + struct v4l2_subdev sd; > > + struct media_pad pads[MIPI_CSI2_PADS_NUM]; > > + struct v4l2_async_notifier notifier; > > + struct v4l2_subdev *src_sd; > > + > > + struct v4l2_fwnode_bus_mipi_csi2 bus; > > + > > + struct mutex lock; /* Protect csi2_fmt, format_mbus, state, hs_settle */ > > + const struct csi2_pix_format *csi2_fmt; > > + struct v4l2_mbus_framefmt format_mbus[MIPI_CSI2_PADS_NUM]; > > + u32 state; > > + u32 hs_settle; > > + > > + struct regmap *phy_gpr; > > + u8 phy_gpr_reg; > > + > > + struct icc_path *icc_path; > > + s32 icc_path_bw; > > +}; > > + > > +/* ----------------------------------------------------------------------------- > > + * Format helpers > > + */ > > + > > +struct csi2_pix_format { > > + u32 code; > > + u8 width; > > +}; > > + > > +static const struct csi2_pix_format imx8mq_mipi_csi_formats[] = { > > + /* RAW (Bayer and greyscale) formats. */ > > + { > > + .code = MEDIA_BUS_FMT_SBGGR8_1X8, > > + .width = 8, > > + }, { > > + .code = MEDIA_BUS_FMT_SGBRG8_1X8, > > + .width = 8, > > + }, { > > + .code = MEDIA_BUS_FMT_SGRBG8_1X8, > > + .width = 8, > > + }, { > > + .code = MEDIA_BUS_FMT_SRGGB8_1X8, > > + .width = 8, > > + }, { > > + .code = MEDIA_BUS_FMT_Y8_1X8, > > + .width = 8, > > + }, { > > + .code = MEDIA_BUS_FMT_SBGGR10_1X10, > > + .width = 10, > > + }, { > > + .code = MEDIA_BUS_FMT_SGBRG10_1X10, > > + .width = 10, > > + }, { > > + .code = MEDIA_BUS_FMT_SGRBG10_1X10, > > + .width = 10, > > + }, { > > + .code = MEDIA_BUS_FMT_SRGGB10_1X10, > > + .width = 10, > > + }, { > > + .code = MEDIA_BUS_FMT_Y10_1X10, > > + .width = 10, > > + }, { > > + .code = MEDIA_BUS_FMT_SBGGR12_1X12, > > + .width = 12, > > + }, { > > + .code = MEDIA_BUS_FMT_SGBRG12_1X12, > > + .width = 12, > > + }, { > > + .code = MEDIA_BUS_FMT_SGRBG12_1X12, > > + .width = 12, > > + }, { > > + .code = MEDIA_BUS_FMT_SRGGB12_1X12, > > + .width = 12, > > + }, { > > + .code = MEDIA_BUS_FMT_Y12_1X12, > > + .width = 12, > > + }, { > > + .code = MEDIA_BUS_FMT_SBGGR14_1X14, > > + .width = 14, > > + }, { > > + .code = MEDIA_BUS_FMT_SGBRG14_1X14, > > + .width = 14, > > + }, { > > + .code = MEDIA_BUS_FMT_SGRBG14_1X14, > > + .width = 14, > > + }, { > > + .code = MEDIA_BUS_FMT_SRGGB14_1X14, > > + .width = 14, > > + }, { > > + /* YUV formats */ > > + .code = MEDIA_BUS_FMT_YUYV8_2X8, > > + .width = 16, > > + }, { > > + .code = MEDIA_BUS_FMT_YUYV8_1X16, > > + .width = 16, > > + } > > +}; > > + > > +static const struct csi2_pix_format *find_csi2_format(u32 code) > > +{ > > + unsigned int i; > > + > > + for (i = 0; i < ARRAY_SIZE(imx8mq_mipi_csi_formats); i++) > > + if (code == imx8mq_mipi_csi_formats[i].code) > > + return &imx8mq_mipi_csi_formats[i]; > > + return NULL; > > +} > > + > > +/* ----------------------------------------------------------------------------- > > + * Hardware configuration > > + */ > > + > > +static inline void imx8mq_mipi_csi_write(struct csi_state *state, u32 reg, u32 val) > > +{ > > + writel(val, state->regs + reg); > > +} > > + > > +static int imx8mq_mipi_csi_sw_reset(struct csi_state *state) > > +{ > > + int ret; > > + > > + /* > > + * these are most likely self-clearing reset bits. to make it > > + * more clear, the reset-imx7 driver should implement the > > + * .reset() operation. > > + */ > > + ret = reset_control_assert(state->rst); > > + if (ret < 0) { > > + dev_err(state->dev, "Failed to assert resets: %d\n", ret); > > + return ret; > > + } > > + > > + return 0; > > +} > > + > > +static void imx8mq_mipi_csi_system_enable(struct csi_state *state, int on) > > +{ > > + if (!on) { > > + imx8mq_mipi_csi_write(state, CSI2RX_CFG_DISABLE_DATA_LANES, 0xf); > > + return; > > + } > > + > > + regmap_update_bits(state->phy_gpr, > > + state->phy_gpr_reg, > > + 0x3fff, > > + GPR_CSI2_1_RX_ENABLE | > > + GPR_CSI2_1_VID_INTFC_ENB | > > + GPR_CSI2_1_HSEL | > > + GPR_CSI2_1_CONT_CLK_MODE | > > + GPR_CSI2_1_S_PRG_RXHS_SETTLE(state->hs_settle)); > > +} > > + > > +static void imx8mq_mipi_csi_set_params(struct csi_state *state) > > +{ > > + int lanes = state->bus.num_data_lanes; > > + > > + imx8mq_mipi_csi_write(state, CSI2RX_CFG_NUM_LANES, lanes - 1); > > + imx8mq_mipi_csi_write(state, CSI2RX_CFG_DISABLE_DATA_LANES, > > + (0xf << lanes) & 0xf); > > + imx8mq_mipi_csi_write(state, CSI2RX_IRQ_MASK, CSI2RX_IRQ_MASK_ALL); > > + imx8mq_mipi_csi_write(state, 0x180, 1); > > + /* vid_vc */ > > + imx8mq_mipi_csi_write(state, 0x184, 1); > > + imx8mq_mipi_csi_write(state, 0x188, CSI2RX_SEND_LEVEL); > > I see CSI2RX_CFG_VID_P_FIFO_SEND_LEVEL is 0x188 so better use the macro. Good point, I missed that. > > +} > > + > > +static int imx8mq_mipi_csi_clk_enable(struct csi_state *state) > > +{ > > + return clk_bulk_prepare_enable(CSI2_NUM_CLKS, state->clks); > > +} > > + > > +static void imx8mq_mipi_csi_clk_disable(struct csi_state *state) > > +{ > > + clk_bulk_disable_unprepare(CSI2_NUM_CLKS, state->clks); > > +} > > + > > +static int imx8mq_mipi_csi_clk_get(struct csi_state *state) > > +{ > > + unsigned int i; > > + > > + for (i = 0; i < CSI2_NUM_CLKS; i++) > > + state->clks[i].id = imx8mq_mipi_csi_clk_id[i]; > > + > > + return devm_clk_bulk_get(state->dev, CSI2_NUM_CLKS, state->clks); > > +} > > + > > +static int imx8mq_mipi_csi_calc_hs_settle(struct csi_state *state) > > +{ > > + s64 link_freq; > > + u32 lane_rate; > > + unsigned long esc_clk_rate; > > + u32 min_ths_settle, max_ths_settle, ths_settle_ns, esc_clk_period_ns; > > + > > + /* Calculate the line rate from the pixel rate. */ > > + link_freq = v4l2_get_link_freq(state->src_sd->ctrl_handler, > > + state->csi2_fmt->width, > > + state->bus.num_data_lanes * 2); > > + if (link_freq < 0) { > > + dev_err(state->dev, "Unable to obtain link frequency: %d\n", > > + (int)link_freq); > > + return link_freq; > > + } > > + > > + lane_rate = link_freq * 2; > > + if (lane_rate < 80000000 || lane_rate > 1500000000) { > > + dev_dbg(state->dev, "Out-of-bound lane rate %u\n", lane_rate); > > + return -EINVAL; > > + } > > + > > + /* > > + * The D-PHY specification requires Ths-settle to be in the range > > + * 85ns + 6*UI to 140ns + 10*UI, with the unit interval UI being half > > + * the clock period. > > + * > > + * The Ths-settle value is expressed in the hardware as a multiple of > > + * the Esc clock period: > > + * > > + * Ths-settle = (PRG_RXHS_SETTLE + 1) * Tperiod of RxClkInEsc > > + * > > + * Due to the one cycle inaccuracy introduced by rounding, the > > + * documentation recommends picking a value away from the boundaries. > > + * Let's pick the average. > > + */ > > + esc_clk_rate = clk_get_rate(state->clks[CSI2_CLK_ESC].clk); > > + if (!esc_clk_rate) { > > + dev_err(state->dev, "Could not get esc clock rate.\n"); > > + return -EINVAL; > > + } > > + > > + dev_dbg(state->dev, "esc clk rate: %lu\n", esc_clk_rate); > > + esc_clk_period_ns = 1000000000 / esc_clk_rate; > > + > > + min_ths_settle = 85 + 6 * 1000000 / (lane_rate / 1000); > > + max_ths_settle = 140 + 10 * 1000000 / (lane_rate / 1000); > > + ths_settle_ns = (min_ths_settle + max_ths_settle) / 2; > > + > > + state->hs_settle = ths_settle_ns / esc_clk_period_ns - 1; > > + > > + dev_dbg(state->dev, "lane rate %u Ths_settle %u hs_settle %u\n", > > + lane_rate, ths_settle_ns, state->hs_settle); > > + > > + return 0; > > +} > > + > > +static int imx8mq_mipi_csi_start_stream(struct csi_state *state) > > +{ > > + int ret; > > + > > + ret = imx8mq_mipi_csi_sw_reset(state); > > + if (ret) > > + return ret; > > + > > + imx8mq_mipi_csi_set_params(state); > > + ret = imx8mq_mipi_csi_calc_hs_settle(state); > > + if (ret) > > + return ret; > > + > > + imx8mq_mipi_csi_system_enable(state, true); > > + > > + return 0; > > +} > > + > > +static void imx8mq_mipi_csi_stop_stream(struct csi_state *state) > > +{ > > + imx8mq_mipi_csi_system_enable(state, false); > > +} > > + > > +/* ----------------------------------------------------------------------------- > > + * V4L2 subdev operations > > + */ > > + > > +static struct csi_state *mipi_sd_to_csi2_state(struct v4l2_subdev *sdev) > > +{ > > + return container_of(sdev, struct csi_state, sd); > > +} > > + > > +static int imx8mq_mipi_csi_s_stream(struct v4l2_subdev *sd, int enable) > > +{ > > + struct csi_state *state = mipi_sd_to_csi2_state(sd); > > + int ret = 0; > > + > > + imx8mq_mipi_csi_write(state, CSI2RX_IRQ_MASK, > > + CSI2RX_IRQ_MASK_ULPS_STATUS_CHANGE); > > + > > + if (enable) { > > + ret = pm_runtime_resume_and_get(state->dev); > > + if (ret < 0) > > + return ret; > > + } > > + > > + mutex_lock(&state->lock); > > + > > + if (enable) { > > + if (state->state & ST_SUSPENDED) { > > + ret = -EBUSY; > > + goto unlock; > > + } > > + > > + ret = imx8mq_mipi_csi_start_stream(state); > > + if (ret < 0) > > + goto unlock; > > + > > + ret = v4l2_subdev_call(state->src_sd, video, s_stream, 1); > > can we be sure that src_sd is not null? I think so, as the sink pad has the MUST_CONNECT flag set, so if no subdev is bound, the pipeline won't validate and .s_stream() won't be called. > > + if (ret < 0) > > + goto unlock; > > + > > + state->state |= ST_STREAMING; > > + } else { > > + v4l2_subdev_call(state->src_sd, video, s_stream, 0); > > + imx8mq_mipi_csi_stop_stream(state); > > + state->state &= ~ST_STREAMING; > > + } > > + > > +unlock: > > + mutex_unlock(&state->lock); > > + > > + if (!enable || ret < 0) > > + pm_runtime_put(state->dev); > > + > > + return ret; > > +} > > + > > +static struct v4l2_mbus_framefmt * > > +imx8mq_mipi_csi_get_format(struct csi_state *state, > > + struct v4l2_subdev_state *sd_state, > > + enum v4l2_subdev_format_whence which, > > + unsigned int pad) > > +{ > > + if (which == V4L2_SUBDEV_FORMAT_TRY) > > + return v4l2_subdev_get_try_format(&state->sd, sd_state, pad); > > + > > + return &state->format_mbus[pad]; > > +} > > + > > +static int imx8mq_mipi_csi_init_cfg(struct v4l2_subdev *sd, > > + struct v4l2_subdev_state *sd_state) > > +{ > > + struct csi_state *state = mipi_sd_to_csi2_state(sd); > > + struct v4l2_mbus_framefmt *fmt_sink; > > + struct v4l2_mbus_framefmt *fmt_source; > > + enum v4l2_subdev_format_whence which; > > + > > + which = sd_state ? V4L2_SUBDEV_FORMAT_TRY : V4L2_SUBDEV_FORMAT_ACTIVE; > > + fmt_sink = imx8mq_mipi_csi_get_format(state, sd_state, which, > > + MIPI_CSI2_PAD_SINK); > > + > > + fmt_sink->code = MEDIA_BUS_FMT_SGBRG10_1X10; > > + fmt_sink->width = MIPI_CSI2_DEF_PIX_WIDTH; > > + fmt_sink->height = MIPI_CSI2_DEF_PIX_HEIGHT; > > + fmt_sink->field = V4L2_FIELD_NONE; > > + > > + fmt_sink->colorspace = V4L2_COLORSPACE_RAW; > > + fmt_sink->xfer_func = V4L2_MAP_XFER_FUNC_DEFAULT(fmt_sink->colorspace); > > + fmt_sink->ycbcr_enc = V4L2_MAP_YCBCR_ENC_DEFAULT(fmt_sink->colorspace); > > + fmt_sink->quantization = > > + V4L2_MAP_QUANTIZATION_DEFAULT(false, fmt_sink->colorspace, > > + fmt_sink->ycbcr_enc); > > + > > + fmt_source = imx8mq_mipi_csi_get_format(state, sd_state, which, > > + MIPI_CSI2_PAD_SOURCE); > > + *fmt_source = *fmt_sink; > > + > > + return 0; > > +} > > + > > +static int imx8mq_mipi_csi_get_fmt(struct v4l2_subdev *sd, > > + struct v4l2_subdev_state *sd_state, > > + struct v4l2_subdev_format *sdformat) > > +{ > > + struct csi_state *state = mipi_sd_to_csi2_state(sd); > > + struct v4l2_mbus_framefmt *fmt; > > + > > + fmt = imx8mq_mipi_csi_get_format(state, sd_state, sdformat->which, > > + sdformat->pad); > > + > > + mutex_lock(&state->lock); > > + > > + sdformat->format = *fmt; > > + > > + mutex_unlock(&state->lock); > > + > > + return 0; > > +} > > + > > +static int imx8mq_mipi_csi_enum_mbus_code(struct v4l2_subdev *sd, > > + struct v4l2_subdev_state *sd_state, > > + struct v4l2_subdev_mbus_code_enum *code) > > +{ > > + struct csi_state *state = mipi_sd_to_csi2_state(sd); > > + > > + /* > > + * We can't transcode in any way, the source format is identical > > + * to the sink format. > > + */ > > + if (code->pad == MIPI_CSI2_PAD_SOURCE) { > > + struct v4l2_mbus_framefmt *fmt; > > + > > + if (code->index > 0) > > + return -EINVAL; > > + > > + fmt = imx8mq_mipi_csi_get_format(state, sd_state, code->which, > > + code->pad); > > + code->code = fmt->code; > > + return 0; > > + } > > + > > + if (code->pad != MIPI_CSI2_PAD_SINK) > > + return -EINVAL; > > + > > + if (code->index >= ARRAY_SIZE(imx8mq_mipi_csi_formats)) > > + return -EINVAL; > > + > > + code->code = imx8mq_mipi_csi_formats[code->index].code; > > + > > + return 0; > > +} > > + > > +static int imx8mq_mipi_csi_set_fmt(struct v4l2_subdev *sd, > > + struct v4l2_subdev_state *sd_state, > > + struct v4l2_subdev_format *sdformat) > > +{ > > + struct csi_state *state = mipi_sd_to_csi2_state(sd); > > + struct csi2_pix_format const *csi2_fmt; > > + struct v4l2_mbus_framefmt *fmt; > > + > > + /* > > + * The device can't transcode in any way, the source format can't be > > + * modified. > > + */ > > + if (sdformat->pad == MIPI_CSI2_PAD_SOURCE) > > + return imx8mq_mipi_csi_get_fmt(sd, sd_state, sdformat); > > + > > + if (sdformat->pad != MIPI_CSI2_PAD_SINK) > > + return -EINVAL; > > + > > + csi2_fmt = find_csi2_format(sdformat->format.code); > > + if (!csi2_fmt) > > + csi2_fmt = &imx8mq_mipi_csi_formats[0]; > > + > > + fmt = imx8mq_mipi_csi_get_format(state, sd_state, sdformat->which, > > + sdformat->pad); > > + > > + mutex_lock(&state->lock); > > + > > + fmt->code = csi2_fmt->code; > > + fmt->width = sdformat->format.width; > > + fmt->height = sdformat->format.height; > > + > > + sdformat->format = *fmt; > > + > > + /* Propagate the format from sink to source. */ > > + fmt = imx8mq_mipi_csi_get_format(state, sd_state, sdformat->which, > > + MIPI_CSI2_PAD_SOURCE); > > + *fmt = sdformat->format; > > + > > + /* Store the CSI2 format descriptor for active formats. */ > > + if (sdformat->which == V4L2_SUBDEV_FORMAT_ACTIVE) > > + state->csi2_fmt = csi2_fmt; > > + > > + mutex_unlock(&state->lock); > > + > > + return 0; > > +} > > + > > +static const struct v4l2_subdev_video_ops imx8mq_mipi_csi_video_ops = { > > + .s_stream = imx8mq_mipi_csi_s_stream, > > +}; > > + > > +static const struct v4l2_subdev_pad_ops imx8mq_mipi_csi_pad_ops = { > > + .init_cfg = imx8mq_mipi_csi_init_cfg, > > + .enum_mbus_code = imx8mq_mipi_csi_enum_mbus_code, > > + .get_fmt = imx8mq_mipi_csi_get_fmt, > > + .set_fmt = imx8mq_mipi_csi_set_fmt, > > +}; > > + > > +static const struct v4l2_subdev_ops imx8mq_mipi_csi_subdev_ops = { > > + .video = &imx8mq_mipi_csi_video_ops, > > + .pad = &imx8mq_mipi_csi_pad_ops, > > +}; > > + > > +/* ----------------------------------------------------------------------------- > > + * Media entity operations > > + */ > > + > > +static const struct media_entity_operations imx8mq_mipi_csi_entity_ops = { > > + .link_validate = v4l2_subdev_link_validate, > > + .get_fwnode_pad = v4l2_subdev_get_fwnode_pad_1_to_1, > > +}; > > + > > +/* ----------------------------------------------------------------------------- > > + * Async subdev notifier > > + */ > > + > > +static struct csi_state * > > +mipi_notifier_to_csi2_state(struct v4l2_async_notifier *n) > > +{ > > + return container_of(n, struct csi_state, notifier); > > +} > > + > > +static int imx8mq_mipi_csi_notify_bound(struct v4l2_async_notifier *notifier, > > + struct v4l2_subdev *sd, > > + struct v4l2_async_subdev *asd) > > +{ > > + struct csi_state *state = mipi_notifier_to_csi2_state(notifier); > > + struct media_pad *sink = &state->sd.entity.pads[MIPI_CSI2_PAD_SINK]; > > + > > + state->src_sd = sd; > > + > > + return v4l2_create_fwnode_links_to_pad(sd, sink, MEDIA_LNK_FL_ENABLED | > > + MEDIA_LNK_FL_IMMUTABLE); > > +} > > + > > +static const struct v4l2_async_notifier_operations imx8mq_mipi_csi_notify_ops = { > > + .bound = imx8mq_mipi_csi_notify_bound, > > +}; > > + > > +static int imx8mq_mipi_csi_async_register(struct csi_state *state) > > +{ > > + struct v4l2_fwnode_endpoint vep = { > > + .bus_type = V4L2_MBUS_CSI2_DPHY, > > + }; > > + struct v4l2_async_subdev *asd; > > + struct fwnode_handle *ep; > > + unsigned int i; > > + int ret; > > + > > + v4l2_async_notifier_init(&state->notifier); > > + > > + ep = fwnode_graph_get_endpoint_by_id(dev_fwnode(state->dev), 0, 0, > > + FWNODE_GRAPH_ENDPOINT_NEXT); > > + if (!ep) > > + return -ENOTCONN; > > + > > + ret = v4l2_fwnode_endpoint_parse(ep, &vep); > > + if (ret) > > + goto err_parse; > > + > > + for (i = 0; i < vep.bus.mipi_csi2.num_data_lanes; ++i) { > > + if (vep.bus.mipi_csi2.data_lanes[i] != i + 1) { > > + dev_err(state->dev, > > + "data lanes reordering is not supported"); > > + ret = -EINVAL; > > + goto err_parse; > > + } > > + } > > + > > + state->bus = vep.bus.mipi_csi2; > > + > > + dev_dbg(state->dev, "data lanes: %d flags: 0x%08x\n", > > + state->bus.num_data_lanes, > > + state->bus.flags); > > + > > + asd = v4l2_async_notifier_add_fwnode_remote_subdev(&state->notifier, > > + ep, struct v4l2_async_subdev); > > + if (IS_ERR(asd)) { > > + ret = PTR_ERR(asd); > > + goto err_parse; > > + } > > + > > + fwnode_handle_put(ep); > > + > > + state->notifier.ops = &imx8mq_mipi_csi_notify_ops; > > + > > + ret = v4l2_async_subdev_notifier_register(&state->sd, &state->notifier); > > + if (ret) > > + return ret; > > + > > + return v4l2_async_register_subdev(&state->sd); > > + > > +err_parse: > > + fwnode_handle_put(ep); > > + > > + return ret; > > +} > > + > > +/* ----------------------------------------------------------------------------- > > + * Suspend/resume > > + */ > > + > > +static int imx8mq_mipi_csi_pm_suspend(struct device *dev, bool runtime) > > +{ > > + struct v4l2_subdev *sd = dev_get_drvdata(dev); > > + struct csi_state *state = mipi_sd_to_csi2_state(sd); > > + int ret = 0; > > + > > + mutex_lock(&state->lock); > > + > > + if (state->state & ST_POWERED) { > > + imx8mq_mipi_csi_stop_stream(state); > > + imx8mq_mipi_csi_clk_disable(state); > > + state->state &= ~ST_POWERED; > > + if (!runtime) > > + state->state |= ST_SUSPENDED; > > + } > > + > > + mutex_unlock(&state->lock); > > + > > + ret = icc_set_bw(state->icc_path, 0, 0); > > + if (ret) > > + dev_err(dev, "icc_set_bw failed with %d\n", ret); > > + > > + return ret ? -EAGAIN : 0; > > +} > > + > > +static int imx8mq_mipi_csi_pm_resume(struct device *dev, bool runtime) > > +{ > > + struct v4l2_subdev *sd = dev_get_drvdata(dev); > > + struct csi_state *state = mipi_sd_to_csi2_state(sd); > > + int ret = 0; > > + > > + ret = icc_set_bw(state->icc_path, 0, state->icc_path_bw); > > + if (ret) { > > + dev_err(dev, "icc_set_bw failed with %d\n", ret); > > + return ret; > > + } > > + > > + mutex_lock(&state->lock); > > + > > + if (!runtime && !(state->state & ST_SUSPENDED)) > > + goto unlock; > > + > > + if (!(state->state & ST_POWERED)) { > > + state->state |= ST_POWERED; > > + ret = imx8mq_mipi_csi_clk_enable(state); > > + } > > + if (state->state & ST_STREAMING) { > > + ret = imx8mq_mipi_csi_start_stream(state); > > + if (ret) > > + goto unlock; > > + } > > + > > + state->state &= ~ST_SUSPENDED; > > + > > +unlock: > > + mutex_unlock(&state->lock); > > + > > + return ret ? -EAGAIN : 0; > > +} > > + > > +static int __maybe_unused imx8mq_mipi_csi_suspend(struct device *dev) > > +{ > > + return imx8mq_mipi_csi_pm_suspend(dev, false); > > +} > > + > > +static int __maybe_unused imx8mq_mipi_csi_resume(struct device *dev) > > +{ > > + return imx8mq_mipi_csi_pm_resume(dev, false); > > +} > > + > > +static int __maybe_unused imx8mq_mipi_csi_runtime_suspend(struct device *dev) > > +{ > > + return imx8mq_mipi_csi_pm_suspend(dev, true); > > +} > > + > > +static int __maybe_unused imx8mq_mipi_csi_runtime_resume(struct device *dev) > > +{ > > + return imx8mq_mipi_csi_pm_resume(dev, true); > > +} > > + > > +static const struct dev_pm_ops imx8mq_mipi_csi_pm_ops = { > > + SET_RUNTIME_PM_OPS(imx8mq_mipi_csi_runtime_suspend, > > + imx8mq_mipi_csi_runtime_resume, > > + NULL) > > + SET_SYSTEM_SLEEP_PM_OPS(imx8mq_mipi_csi_suspend, imx8mq_mipi_csi_resume) > > +}; > > + > > +/* ----------------------------------------------------------------------------- > > + * Probe/remove & platform driver > > + */ > > + > > +static int imx8mq_mipi_csi_subdev_init(struct csi_state *state) > > +{ > > + struct v4l2_subdev *sd = &state->sd; > > + > > + v4l2_subdev_init(sd, &imx8mq_mipi_csi_subdev_ops); > > + sd->owner = THIS_MODULE; > > + snprintf(sd->name, sizeof(sd->name), "%s %s", > > + MIPI_CSI2_SUBDEV_NAME, dev_name(state->dev)); > > + > > + sd->flags |= V4L2_SUBDEV_FL_HAS_DEVNODE; > > + > > + sd->entity.function = MEDIA_ENT_F_VID_IF_BRIDGE; > > + sd->entity.ops = &imx8mq_mipi_csi_entity_ops; > > + > > + sd->dev = state->dev; > > + > > + state->csi2_fmt = &imx8mq_mipi_csi_formats[0]; > > + imx8mq_mipi_csi_init_cfg(sd, NULL); > > + > > + state->pads[MIPI_CSI2_PAD_SINK].flags = MEDIA_PAD_FL_SINK > > + | MEDIA_PAD_FL_MUST_CONNECT; > > + state->pads[MIPI_CSI2_PAD_SOURCE].flags = MEDIA_PAD_FL_SOURCE > > + | MEDIA_PAD_FL_MUST_CONNECT; > > + return media_entity_pads_init(&sd->entity, MIPI_CSI2_PADS_NUM, > > + state->pads); > > +} > > + > > +static void imx8mq_mipi_csi_release_icc(struct platform_device *pdev) > > +{ > > + struct v4l2_subdev *sd = dev_get_drvdata(&pdev->dev); > > + struct csi_state *state = mipi_sd_to_csi2_state(sd); > > + > > + icc_put(state->icc_path); > > +} > > + > > +static int imx8mq_mipi_csi_init_icc(struct platform_device *pdev) > > +{ > > + struct v4l2_subdev *sd = dev_get_drvdata(&pdev->dev); > > + struct csi_state *state = mipi_sd_to_csi2_state(sd); > > + > > + /* Optional interconnect request */ > > + state->icc_path = of_icc_get(&pdev->dev, "dram"); > > + if (IS_ERR_OR_NULL(state->icc_path)) > > + return PTR_ERR_OR_ZERO(state->icc_path); > > + > > + state->icc_path_bw = MBps_to_icc(700); > > + > > + return 0; > > +} > > + > > +static int imx8mq_mipi_csi_parse_dt(struct csi_state *state) > > +{ > > + struct device *dev = state->dev; > > + struct device_node *np = state->dev->of_node; > > + struct device_node *node; > > + phandle ph; > > + u32 out_val[2]; > > + int ret = 0; > > + > > + state->rst = devm_reset_control_array_get_exclusive(dev); > > + if (IS_ERR(state->rst)) { > > + dev_err(dev, "Failed to get reset: %pe\n", state->rst); > > + return PTR_ERR(state->rst); > > + } > > + > > + ret = of_property_read_u32_array(np, "fsl,mipi-phy-gpr", out_val, > > + ARRAY_SIZE(out_val)); > > + if (ret) { > > + dev_err(dev, "no fsl,mipi-phy-gpr property found: %d\n", ret); > > + return ret; > > + } > > + > > + ph = *out_val; > > + > > + node = of_find_node_by_phandle(ph); > > + if (!node) { > > + dev_err(dev, "Error finding node by phandle\n"); > > + return -ENODEV; > > + } > > + state->phy_gpr = syscon_node_to_regmap(node); > > + of_node_put(node); > > + if (IS_ERR(state->phy_gpr)) { > > + dev_err(dev, "failed to get gpr regmap: %pe\n", state->phy_gpr); > > + return PTR_ERR(state->phy_gpr); > > + } > > + > > + state->phy_gpr_reg = out_val[1]; > > + dev_dbg(dev, "phy gpr register set to 0x%x\n", state->phy_gpr_reg); > > + > > + return ret; > > +} > > + > > +static int imx8mq_mipi_csi_probe(struct platform_device *pdev) > > +{ > > + struct device *dev = &pdev->dev; > > + struct csi_state *state; > > + int ret; > > + > > + state = devm_kzalloc(dev, sizeof(*state), GFP_KERNEL); > > + if (!state) > > + return -ENOMEM; > > + > > + state->dev = dev; > > + > > + ret = imx8mq_mipi_csi_parse_dt(state); > > + if (ret < 0) { > > + dev_err(dev, "Failed to parse device tree: %d\n", ret); > > + return ret; > > + } > > + > > + /* Acquire resources. */ > > + state->regs = devm_platform_ioremap_resource(pdev, 0); > > + if (IS_ERR(state->regs)) > > + return PTR_ERR(state->regs); > > + > > + ret = imx8mq_mipi_csi_clk_get(state); > > + if (ret < 0) > > + return ret; > > + > > + platform_set_drvdata(pdev, &state->sd); > > + > > + mutex_init(&state->lock); > > + > > + ret = imx8mq_mipi_csi_subdev_init(state); > > + if (ret < 0) > > + goto mutex; > > + > > + ret = imx8mq_mipi_csi_init_icc(pdev); > > + if (ret) > > + goto mutex; > > + > > + /* Enable runtime PM. */ > > + pm_runtime_enable(dev); > > + if (!pm_runtime_enabled(dev)) { > > + ret = imx8mq_mipi_csi_pm_resume(dev, true); > > + if (ret < 0) > > + goto icc; > > + } > > + > > + ret = imx8mq_mipi_csi_async_register(state); > > + if (ret < 0) > > + goto cleanup; > > + > > + return 0; > > + > > +cleanup: > > + pm_runtime_disable(&pdev->dev); > > + imx8mq_mipi_csi_pm_suspend(&pdev->dev, true); > > + > > + media_entity_cleanup(&state->sd.entity); > > + v4l2_async_notifier_unregister(&state->notifier); > > + v4l2_async_notifier_cleanup(&state->notifier); > > + v4l2_async_unregister_subdev(&state->sd); > > +icc: > > + imx8mq_mipi_csi_release_icc(pdev); > > +mutex: > > + mutex_destroy(&state->lock); > > + > > + return ret; > > +} > > + > > +static int imx8mq_mipi_csi_remove(struct platform_device *pdev) > > +{ > > + struct v4l2_subdev *sd = platform_get_drvdata(pdev); > > + struct csi_state *state = mipi_sd_to_csi2_state(sd); > > + > > + v4l2_async_notifier_unregister(&state->notifier); > > + v4l2_async_notifier_cleanup(&state->notifier); > > + v4l2_async_unregister_subdev(&state->sd); > > + > > + pm_runtime_disable(&pdev->dev); > > + imx8mq_mipi_csi_pm_suspend(&pdev->dev, true); > > + media_entity_cleanup(&state->sd.entity); > > + mutex_destroy(&state->lock); > > + pm_runtime_set_suspended(&pdev->dev); > > + imx8mq_mipi_csi_release_icc(pdev); > > + > > + return 0; > > +} > > + > > +static const struct of_device_id imx8mq_mipi_csi_of_match[] = { > > + { .compatible = "fsl,imx8mq-mipi-csi2", }, > > + { /* sentinel */ }, > > +}; > > +MODULE_DEVICE_TABLE(of, imx8mq_mipi_csi_of_match); > > + > > +static struct platform_driver imx8mq_mipi_csi_driver = { > > + .probe = imx8mq_mipi_csi_probe, > > + .remove = imx8mq_mipi_csi_remove, > > + .driver = { > > + .of_match_table = imx8mq_mipi_csi_of_match, > > + .name = MIPI_CSI2_DRIVER_NAME, > > + .pm = &imx8mq_mipi_csi_pm_ops, > > + }, > > +}; > > + > > +module_platform_driver(imx8mq_mipi_csi_driver); > > + > > +MODULE_DESCRIPTION("i.MX8MQ MIPI CSI-2 receiver driver"); > > +MODULE_AUTHOR("Martin Kepplinger "); > > +MODULE_LICENSE("GPL v2"); > > +MODULE_ALIAS("platform:imx8mq-mipi-csi2"); -- Regards, Laurent Pinchart From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-11.7 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS, UNWANTED_LANGUAGE_BODY,URIBL_BLOCKED autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id DAA84C4338F for ; Wed, 28 Jul 2021 05:54:06 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id A796060F01 for ; Wed, 28 Jul 2021 05:54:06 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org A796060F01 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=ideasonboard.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=GcWW9XKVXZ6xBbxWrk9dHyHq/kRVQ4tcteh9xdxLQT8=; b=fOtSSEoGM+U5Bv S8jJYeoUipUyLWq8KssPd3EqLyU+k4yKva6y8JO/GFEcJbFguTaIq7gezGPG5GfZ76AdKhoNfZDE6 LeL3cnsrh+8Zc9qJg7YR59lUIENM1XzhzEgJzc8/+0pBnMCiCShnN8DMVL/NJm7UcrIaS6hYYOcG2 J6LLRZvKXuaCUVHzDiWt51UkwaL288aW0GzMkLcrVr+/AAnZATa1W7gI/jzGB4aSelUOFlPXWIY7B EGzZsAnR55JbaBMr8yvP0m3vMkXM/JBmk2haZ9ivjHMEvNcALjxudUKQFODSDJpUUZp+IvNpPDhYw kfZ68+JpVHAPOIT+8g5g==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1m8cU1-00HAgO-8s; Wed, 28 Jul 2021 05:51:57 +0000 Received: from perceval.ideasonboard.com ([213.167.242.64]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1m8cTu-00HAez-KJ for linux-arm-kernel@lists.infradead.org; Wed, 28 Jul 2021 05:51:54 +0000 Received: from pendragon.ideasonboard.com (62-78-145-57.bb.dnainternet.fi [62.78.145.57]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id BF873EE; Wed, 28 Jul 2021 07:51:45 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1627451506; bh=ZB+MfWCEyRrx4Ei2O+bpHnYURN9YC+OnavdTKNrfe9Y=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=a+1gQGL7nOQjvNAV73IED0h+StcuezPlSWcTLY1Ej5EZ16beRIu+1ezg9QTdt3KJL jALGHaOtufNvgJQ6lnfe3xbrJQLcnU2+gyYiZdqeWxP7BgRO1E4FVxMuru0YemTwnw n1bX0IKhA66XDq5Z2PNO17UPST7m3eyAPR3Y3wKY= Date: Wed, 28 Jul 2021 08:51:39 +0300 From: Laurent Pinchart To: Dafna Hirschfeld Cc: Martin Kepplinger , shawnguo@kernel.org, devicetree@vger.kernel.org, festevam@gmail.com, kernel@pengutronix.de, kernel@puri.sm, krzk@kernel.org, linux-arm-kernel@lists.infradead.org, linux-imx@nxp.com, linux-kernel@vger.kernel.org, linux-media@vger.kernel.org, linux-staging@lists.linux.dev, m.felsch@pengutronix.de, mchehab@kernel.org, phone-devel@vger.kernel.org, robh@kernel.org, slongerbeam@gmail.com Subject: Re: [PATCH v9 2/3] media: imx: add a driver for i.MX8MQ mipi csi rx phy and controller Message-ID: References: <20210726082117.2423597-1-martin.kepplinger@puri.sm> <20210726082117.2423597-3-martin.kepplinger@puri.sm> <489914ac-230f-2fc0-4b10-ae158ab6a95c@collabora.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <489914ac-230f-2fc0-4b10-ae158ab6a95c@collabora.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210727_225151_046821_B66F1A30 X-CRM114-Status: GOOD ( 39.43 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi Dafna, On Wed, Jul 28, 2021 at 07:02:57AM +0200, Dafna Hirschfeld wrote: > On 26.07.21 10:21, Martin Kepplinger wrote: > > Add a driver to support the i.MX8MQ MIPI CSI receiver. The hardware side > > is based on > > https://source.codeaurora.org/external/imx/linux-imx/tree/drivers/media/platform/imx8/mxc-mipi-csi2_yav.c?h=imx_5.4.70_2.3.0 > > > > It's built as part of VIDEO_IMX7_CSI because that's documented to support > > i.MX8M platforms. This driver adds i.MX8MQ support where currently only the > > i.MX8MM platform has been supported. > > > > Signed-off-by: Martin Kepplinger > > Reviewed-by: Laurent Pinchart > > --- > > drivers/staging/media/imx/Makefile | 1 + > > drivers/staging/media/imx/imx8mq-mipi-csi2.c | 976 +++++++++++++++++++ > > 2 files changed, 977 insertions(+) > > create mode 100644 drivers/staging/media/imx/imx8mq-mipi-csi2.c > > > > diff --git a/drivers/staging/media/imx/Makefile b/drivers/staging/media/imx/Makefile > > index 6ac33275cc97..19c2fc54d424 100644 > > --- a/drivers/staging/media/imx/Makefile > > +++ b/drivers/staging/media/imx/Makefile > > @@ -16,3 +16,4 @@ obj-$(CONFIG_VIDEO_IMX_CSI) += imx6-mipi-csi2.o > > > > obj-$(CONFIG_VIDEO_IMX7_CSI) += imx7-media-csi.o > > obj-$(CONFIG_VIDEO_IMX7_CSI) += imx7-mipi-csis.o > > +obj-$(CONFIG_VIDEO_IMX7_CSI) += imx8mq-mipi-csi2.o > > diff --git a/drivers/staging/media/imx/imx8mq-mipi-csi2.c b/drivers/staging/media/imx/imx8mq-mipi-csi2.c > > new file mode 100644 > > index 000000000000..c4f4fe0c533b > > --- /dev/null > > +++ b/drivers/staging/media/imx/imx8mq-mipi-csi2.c > > @@ -0,0 +1,976 @@ > > +// SPDX-License-Identifier: GPL-2.0 > > +/* > > + * NXP i.MX8MQ SoC series MIPI-CSI2 receiver driver > > + * > > + * Copyright (C) 2021 Purism SPC > > + */ > > + > > +#include > > +#include > > +#include > > +#include > > +#include > > +#include > > +#include > > +#include > > +#include > > +#include > > +#include > > +#include > > +#include > > +#include > > +#include > > +#include > > +#include > > +#include > > +#include > > + > > +#include > > +#include > > +#include > > +#include > > +#include > > + > > +#define MIPI_CSI2_DRIVER_NAME "imx8mq-mipi-csi2" > > +#define MIPI_CSI2_SUBDEV_NAME MIPI_CSI2_DRIVER_NAME > > + > > +#define MIPI_CSI2_PAD_SINK 0 > > +#define MIPI_CSI2_PAD_SOURCE 1 > > +#define MIPI_CSI2_PADS_NUM 2 > > + > > +#define MIPI_CSI2_DEF_PIX_WIDTH 640 > > +#define MIPI_CSI2_DEF_PIX_HEIGHT 480 > > + > > +/* Register map definition */ > > + > > +/* i.MX8MQ CSI-2 controller CSR */ > > +#define CSI2RX_CFG_NUM_LANES 0x100 > > +#define CSI2RX_CFG_DISABLE_DATA_LANES 0x104 > > +#define CSI2RX_BIT_ERR 0x108 > > +#define CSI2RX_IRQ_STATUS 0x10c > > +#define CSI2RX_IRQ_MASK 0x110 > > +#define CSI2RX_IRQ_MASK_ALL 0x1ff > > +#define CSI2RX_IRQ_MASK_ULPS_STATUS_CHANGE 0x8 > > +#define CSI2RX_ULPS_STATUS 0x114 > > +#define CSI2RX_PPI_ERRSOT_HS 0x118 > > +#define CSI2RX_PPI_ERRSOTSYNC_HS 0x11c > > +#define CSI2RX_PPI_ERRESC 0x120 > > +#define CSI2RX_PPI_ERRSYNCESC 0x124 > > +#define CSI2RX_PPI_ERRCONTROL 0x128 > > +#define CSI2RX_CFG_DISABLE_PAYLOAD_0 0x12c > > +#define CSI2RX_CFG_VID_P_FIFO_SEND_LEVEL 0x188 > > +#define CSI2RX_CFG_DISABLE_PAYLOAD_1 0x130 > > + > > +enum { > > + ST_POWERED = 1, > > + ST_STREAMING = 2, > > + ST_SUSPENDED = 4, > > +}; > > + > > +enum imx8mq_mipi_csi_clk { > > + CSI2_CLK_CORE, > > + CSI2_CLK_ESC, > > + CSI2_CLK_UI, > > + CSI2_NUM_CLKS, > > +}; > > + > > +static const char * const imx8mq_mipi_csi_clk_id[CSI2_NUM_CLKS] = { > > + [CSI2_CLK_CORE] = "core", > > + [CSI2_CLK_ESC] = "esc", > > + [CSI2_CLK_UI] = "ui", > > +}; > > + > > +#define CSI2_NUM_CLKS ARRAY_SIZE(imx8mq_mipi_csi_clk_id) > > + > > +#define GPR_CSI2_1_RX_ENABLE BIT(13) > > +#define GPR_CSI2_1_VID_INTFC_ENB BIT(12) > > +#define GPR_CSI2_1_HSEL BIT(10) > > +#define GPR_CSI2_1_CONT_CLK_MODE BIT(8) > > +#define GPR_CSI2_1_S_PRG_RXHS_SETTLE(x) (((x) & 0x3f) << 2) > > + > > +/* > > + * The send level configures the number of entries that must accumulate in > > + * the Pixel FIFO before the data will be transferred to the video output. > > + * See https://community.nxp.com/t5/i-MX-Processors/IMX8M-MIPI-CSI-Host-Controller-send-level/m-p/864005/highlight/true#M131704 > > If there is more important information in that link you think is valuable, > better to write it here in case the link get broken. Good idea. That link also has information about registers 0x180 and 0x184, which should be captured in the form of comments in imx8mq_mipi_csi_set_params(). > > + */ > > +#define CSI2RX_SEND_LEVEL 64 > > + > > +struct csi_state { > > + struct device *dev; > > + void __iomem *regs; > > + struct clk_bulk_data clks[CSI2_NUM_CLKS]; > > + struct reset_control *rst; > > + struct regulator *mipi_phy_regulator; > > + > > + struct v4l2_subdev sd; > > + struct media_pad pads[MIPI_CSI2_PADS_NUM]; > > + struct v4l2_async_notifier notifier; > > + struct v4l2_subdev *src_sd; > > + > > + struct v4l2_fwnode_bus_mipi_csi2 bus; > > + > > + struct mutex lock; /* Protect csi2_fmt, format_mbus, state, hs_settle */ > > + const struct csi2_pix_format *csi2_fmt; > > + struct v4l2_mbus_framefmt format_mbus[MIPI_CSI2_PADS_NUM]; > > + u32 state; > > + u32 hs_settle; > > + > > + struct regmap *phy_gpr; > > + u8 phy_gpr_reg; > > + > > + struct icc_path *icc_path; > > + s32 icc_path_bw; > > +}; > > + > > +/* ----------------------------------------------------------------------------- > > + * Format helpers > > + */ > > + > > +struct csi2_pix_format { > > + u32 code; > > + u8 width; > > +}; > > + > > +static const struct csi2_pix_format imx8mq_mipi_csi_formats[] = { > > + /* RAW (Bayer and greyscale) formats. */ > > + { > > + .code = MEDIA_BUS_FMT_SBGGR8_1X8, > > + .width = 8, > > + }, { > > + .code = MEDIA_BUS_FMT_SGBRG8_1X8, > > + .width = 8, > > + }, { > > + .code = MEDIA_BUS_FMT_SGRBG8_1X8, > > + .width = 8, > > + }, { > > + .code = MEDIA_BUS_FMT_SRGGB8_1X8, > > + .width = 8, > > + }, { > > + .code = MEDIA_BUS_FMT_Y8_1X8, > > + .width = 8, > > + }, { > > + .code = MEDIA_BUS_FMT_SBGGR10_1X10, > > + .width = 10, > > + }, { > > + .code = MEDIA_BUS_FMT_SGBRG10_1X10, > > + .width = 10, > > + }, { > > + .code = MEDIA_BUS_FMT_SGRBG10_1X10, > > + .width = 10, > > + }, { > > + .code = MEDIA_BUS_FMT_SRGGB10_1X10, > > + .width = 10, > > + }, { > > + .code = MEDIA_BUS_FMT_Y10_1X10, > > + .width = 10, > > + }, { > > + .code = MEDIA_BUS_FMT_SBGGR12_1X12, > > + .width = 12, > > + }, { > > + .code = MEDIA_BUS_FMT_SGBRG12_1X12, > > + .width = 12, > > + }, { > > + .code = MEDIA_BUS_FMT_SGRBG12_1X12, > > + .width = 12, > > + }, { > > + .code = MEDIA_BUS_FMT_SRGGB12_1X12, > > + .width = 12, > > + }, { > > + .code = MEDIA_BUS_FMT_Y12_1X12, > > + .width = 12, > > + }, { > > + .code = MEDIA_BUS_FMT_SBGGR14_1X14, > > + .width = 14, > > + }, { > > + .code = MEDIA_BUS_FMT_SGBRG14_1X14, > > + .width = 14, > > + }, { > > + .code = MEDIA_BUS_FMT_SGRBG14_1X14, > > + .width = 14, > > + }, { > > + .code = MEDIA_BUS_FMT_SRGGB14_1X14, > > + .width = 14, > > + }, { > > + /* YUV formats */ > > + .code = MEDIA_BUS_FMT_YUYV8_2X8, > > + .width = 16, > > + }, { > > + .code = MEDIA_BUS_FMT_YUYV8_1X16, > > + .width = 16, > > + } > > +}; > > + > > +static const struct csi2_pix_format *find_csi2_format(u32 code) > > +{ > > + unsigned int i; > > + > > + for (i = 0; i < ARRAY_SIZE(imx8mq_mipi_csi_formats); i++) > > + if (code == imx8mq_mipi_csi_formats[i].code) > > + return &imx8mq_mipi_csi_formats[i]; > > + return NULL; > > +} > > + > > +/* ----------------------------------------------------------------------------- > > + * Hardware configuration > > + */ > > + > > +static inline void imx8mq_mipi_csi_write(struct csi_state *state, u32 reg, u32 val) > > +{ > > + writel(val, state->regs + reg); > > +} > > + > > +static int imx8mq_mipi_csi_sw_reset(struct csi_state *state) > > +{ > > + int ret; > > + > > + /* > > + * these are most likely self-clearing reset bits. to make it > > + * more clear, the reset-imx7 driver should implement the > > + * .reset() operation. > > + */ > > + ret = reset_control_assert(state->rst); > > + if (ret < 0) { > > + dev_err(state->dev, "Failed to assert resets: %d\n", ret); > > + return ret; > > + } > > + > > + return 0; > > +} > > + > > +static void imx8mq_mipi_csi_system_enable(struct csi_state *state, int on) > > +{ > > + if (!on) { > > + imx8mq_mipi_csi_write(state, CSI2RX_CFG_DISABLE_DATA_LANES, 0xf); > > + return; > > + } > > + > > + regmap_update_bits(state->phy_gpr, > > + state->phy_gpr_reg, > > + 0x3fff, > > + GPR_CSI2_1_RX_ENABLE | > > + GPR_CSI2_1_VID_INTFC_ENB | > > + GPR_CSI2_1_HSEL | > > + GPR_CSI2_1_CONT_CLK_MODE | > > + GPR_CSI2_1_S_PRG_RXHS_SETTLE(state->hs_settle)); > > +} > > + > > +static void imx8mq_mipi_csi_set_params(struct csi_state *state) > > +{ > > + int lanes = state->bus.num_data_lanes; > > + > > + imx8mq_mipi_csi_write(state, CSI2RX_CFG_NUM_LANES, lanes - 1); > > + imx8mq_mipi_csi_write(state, CSI2RX_CFG_DISABLE_DATA_LANES, > > + (0xf << lanes) & 0xf); > > + imx8mq_mipi_csi_write(state, CSI2RX_IRQ_MASK, CSI2RX_IRQ_MASK_ALL); > > + imx8mq_mipi_csi_write(state, 0x180, 1); > > + /* vid_vc */ > > + imx8mq_mipi_csi_write(state, 0x184, 1); > > + imx8mq_mipi_csi_write(state, 0x188, CSI2RX_SEND_LEVEL); > > I see CSI2RX_CFG_VID_P_FIFO_SEND_LEVEL is 0x188 so better use the macro. Good point, I missed that. > > +} > > + > > +static int imx8mq_mipi_csi_clk_enable(struct csi_state *state) > > +{ > > + return clk_bulk_prepare_enable(CSI2_NUM_CLKS, state->clks); > > +} > > + > > +static void imx8mq_mipi_csi_clk_disable(struct csi_state *state) > > +{ > > + clk_bulk_disable_unprepare(CSI2_NUM_CLKS, state->clks); > > +} > > + > > +static int imx8mq_mipi_csi_clk_get(struct csi_state *state) > > +{ > > + unsigned int i; > > + > > + for (i = 0; i < CSI2_NUM_CLKS; i++) > > + state->clks[i].id = imx8mq_mipi_csi_clk_id[i]; > > + > > + return devm_clk_bulk_get(state->dev, CSI2_NUM_CLKS, state->clks); > > +} > > + > > +static int imx8mq_mipi_csi_calc_hs_settle(struct csi_state *state) > > +{ > > + s64 link_freq; > > + u32 lane_rate; > > + unsigned long esc_clk_rate; > > + u32 min_ths_settle, max_ths_settle, ths_settle_ns, esc_clk_period_ns; > > + > > + /* Calculate the line rate from the pixel rate. */ > > + link_freq = v4l2_get_link_freq(state->src_sd->ctrl_handler, > > + state->csi2_fmt->width, > > + state->bus.num_data_lanes * 2); > > + if (link_freq < 0) { > > + dev_err(state->dev, "Unable to obtain link frequency: %d\n", > > + (int)link_freq); > > + return link_freq; > > + } > > + > > + lane_rate = link_freq * 2; > > + if (lane_rate < 80000000 || lane_rate > 1500000000) { > > + dev_dbg(state->dev, "Out-of-bound lane rate %u\n", lane_rate); > > + return -EINVAL; > > + } > > + > > + /* > > + * The D-PHY specification requires Ths-settle to be in the range > > + * 85ns + 6*UI to 140ns + 10*UI, with the unit interval UI being half > > + * the clock period. > > + * > > + * The Ths-settle value is expressed in the hardware as a multiple of > > + * the Esc clock period: > > + * > > + * Ths-settle = (PRG_RXHS_SETTLE + 1) * Tperiod of RxClkInEsc > > + * > > + * Due to the one cycle inaccuracy introduced by rounding, the > > + * documentation recommends picking a value away from the boundaries. > > + * Let's pick the average. > > + */ > > + esc_clk_rate = clk_get_rate(state->clks[CSI2_CLK_ESC].clk); > > + if (!esc_clk_rate) { > > + dev_err(state->dev, "Could not get esc clock rate.\n"); > > + return -EINVAL; > > + } > > + > > + dev_dbg(state->dev, "esc clk rate: %lu\n", esc_clk_rate); > > + esc_clk_period_ns = 1000000000 / esc_clk_rate; > > + > > + min_ths_settle = 85 + 6 * 1000000 / (lane_rate / 1000); > > + max_ths_settle = 140 + 10 * 1000000 / (lane_rate / 1000); > > + ths_settle_ns = (min_ths_settle + max_ths_settle) / 2; > > + > > + state->hs_settle = ths_settle_ns / esc_clk_period_ns - 1; > > + > > + dev_dbg(state->dev, "lane rate %u Ths_settle %u hs_settle %u\n", > > + lane_rate, ths_settle_ns, state->hs_settle); > > + > > + return 0; > > +} > > + > > +static int imx8mq_mipi_csi_start_stream(struct csi_state *state) > > +{ > > + int ret; > > + > > + ret = imx8mq_mipi_csi_sw_reset(state); > > + if (ret) > > + return ret; > > + > > + imx8mq_mipi_csi_set_params(state); > > + ret = imx8mq_mipi_csi_calc_hs_settle(state); > > + if (ret) > > + return ret; > > + > > + imx8mq_mipi_csi_system_enable(state, true); > > + > > + return 0; > > +} > > + > > +static void imx8mq_mipi_csi_stop_stream(struct csi_state *state) > > +{ > > + imx8mq_mipi_csi_system_enable(state, false); > > +} > > + > > +/* ----------------------------------------------------------------------------- > > + * V4L2 subdev operations > > + */ > > + > > +static struct csi_state *mipi_sd_to_csi2_state(struct v4l2_subdev *sdev) > > +{ > > + return container_of(sdev, struct csi_state, sd); > > +} > > + > > +static int imx8mq_mipi_csi_s_stream(struct v4l2_subdev *sd, int enable) > > +{ > > + struct csi_state *state = mipi_sd_to_csi2_state(sd); > > + int ret = 0; > > + > > + imx8mq_mipi_csi_write(state, CSI2RX_IRQ_MASK, > > + CSI2RX_IRQ_MASK_ULPS_STATUS_CHANGE); > > + > > + if (enable) { > > + ret = pm_runtime_resume_and_get(state->dev); > > + if (ret < 0) > > + return ret; > > + } > > + > > + mutex_lock(&state->lock); > > + > > + if (enable) { > > + if (state->state & ST_SUSPENDED) { > > + ret = -EBUSY; > > + goto unlock; > > + } > > + > > + ret = imx8mq_mipi_csi_start_stream(state); > > + if (ret < 0) > > + goto unlock; > > + > > + ret = v4l2_subdev_call(state->src_sd, video, s_stream, 1); > > can we be sure that src_sd is not null? I think so, as the sink pad has the MUST_CONNECT flag set, so if no subdev is bound, the pipeline won't validate and .s_stream() won't be called. > > + if (ret < 0) > > + goto unlock; > > + > > + state->state |= ST_STREAMING; > > + } else { > > + v4l2_subdev_call(state->src_sd, video, s_stream, 0); > > + imx8mq_mipi_csi_stop_stream(state); > > + state->state &= ~ST_STREAMING; > > + } > > + > > +unlock: > > + mutex_unlock(&state->lock); > > + > > + if (!enable || ret < 0) > > + pm_runtime_put(state->dev); > > + > > + return ret; > > +} > > + > > +static struct v4l2_mbus_framefmt * > > +imx8mq_mipi_csi_get_format(struct csi_state *state, > > + struct v4l2_subdev_state *sd_state, > > + enum v4l2_subdev_format_whence which, > > + unsigned int pad) > > +{ > > + if (which == V4L2_SUBDEV_FORMAT_TRY) > > + return v4l2_subdev_get_try_format(&state->sd, sd_state, pad); > > + > > + return &state->format_mbus[pad]; > > +} > > + > > +static int imx8mq_mipi_csi_init_cfg(struct v4l2_subdev *sd, > > + struct v4l2_subdev_state *sd_state) > > +{ > > + struct csi_state *state = mipi_sd_to_csi2_state(sd); > > + struct v4l2_mbus_framefmt *fmt_sink; > > + struct v4l2_mbus_framefmt *fmt_source; > > + enum v4l2_subdev_format_whence which; > > + > > + which = sd_state ? V4L2_SUBDEV_FORMAT_TRY : V4L2_SUBDEV_FORMAT_ACTIVE; > > + fmt_sink = imx8mq_mipi_csi_get_format(state, sd_state, which, > > + MIPI_CSI2_PAD_SINK); > > + > > + fmt_sink->code = MEDIA_BUS_FMT_SGBRG10_1X10; > > + fmt_sink->width = MIPI_CSI2_DEF_PIX_WIDTH; > > + fmt_sink->height = MIPI_CSI2_DEF_PIX_HEIGHT; > > + fmt_sink->field = V4L2_FIELD_NONE; > > + > > + fmt_sink->colorspace = V4L2_COLORSPACE_RAW; > > + fmt_sink->xfer_func = V4L2_MAP_XFER_FUNC_DEFAULT(fmt_sink->colorspace); > > + fmt_sink->ycbcr_enc = V4L2_MAP_YCBCR_ENC_DEFAULT(fmt_sink->colorspace); > > + fmt_sink->quantization = > > + V4L2_MAP_QUANTIZATION_DEFAULT(false, fmt_sink->colorspace, > > + fmt_sink->ycbcr_enc); > > + > > + fmt_source = imx8mq_mipi_csi_get_format(state, sd_state, which, > > + MIPI_CSI2_PAD_SOURCE); > > + *fmt_source = *fmt_sink; > > + > > + return 0; > > +} > > + > > +static int imx8mq_mipi_csi_get_fmt(struct v4l2_subdev *sd, > > + struct v4l2_subdev_state *sd_state, > > + struct v4l2_subdev_format *sdformat) > > +{ > > + struct csi_state *state = mipi_sd_to_csi2_state(sd); > > + struct v4l2_mbus_framefmt *fmt; > > + > > + fmt = imx8mq_mipi_csi_get_format(state, sd_state, sdformat->which, > > + sdformat->pad); > > + > > + mutex_lock(&state->lock); > > + > > + sdformat->format = *fmt; > > + > > + mutex_unlock(&state->lock); > > + > > + return 0; > > +} > > + > > +static int imx8mq_mipi_csi_enum_mbus_code(struct v4l2_subdev *sd, > > + struct v4l2_subdev_state *sd_state, > > + struct v4l2_subdev_mbus_code_enum *code) > > +{ > > + struct csi_state *state = mipi_sd_to_csi2_state(sd); > > + > > + /* > > + * We can't transcode in any way, the source format is identical > > + * to the sink format. > > + */ > > + if (code->pad == MIPI_CSI2_PAD_SOURCE) { > > + struct v4l2_mbus_framefmt *fmt; > > + > > + if (code->index > 0) > > + return -EINVAL; > > + > > + fmt = imx8mq_mipi_csi_get_format(state, sd_state, code->which, > > + code->pad); > > + code->code = fmt->code; > > + return 0; > > + } > > + > > + if (code->pad != MIPI_CSI2_PAD_SINK) > > + return -EINVAL; > > + > > + if (code->index >= ARRAY_SIZE(imx8mq_mipi_csi_formats)) > > + return -EINVAL; > > + > > + code->code = imx8mq_mipi_csi_formats[code->index].code; > > + > > + return 0; > > +} > > + > > +static int imx8mq_mipi_csi_set_fmt(struct v4l2_subdev *sd, > > + struct v4l2_subdev_state *sd_state, > > + struct v4l2_subdev_format *sdformat) > > +{ > > + struct csi_state *state = mipi_sd_to_csi2_state(sd); > > + struct csi2_pix_format const *csi2_fmt; > > + struct v4l2_mbus_framefmt *fmt; > > + > > + /* > > + * The device can't transcode in any way, the source format can't be > > + * modified. > > + */ > > + if (sdformat->pad == MIPI_CSI2_PAD_SOURCE) > > + return imx8mq_mipi_csi_get_fmt(sd, sd_state, sdformat); > > + > > + if (sdformat->pad != MIPI_CSI2_PAD_SINK) > > + return -EINVAL; > > + > > + csi2_fmt = find_csi2_format(sdformat->format.code); > > + if (!csi2_fmt) > > + csi2_fmt = &imx8mq_mipi_csi_formats[0]; > > + > > + fmt = imx8mq_mipi_csi_get_format(state, sd_state, sdformat->which, > > + sdformat->pad); > > + > > + mutex_lock(&state->lock); > > + > > + fmt->code = csi2_fmt->code; > > + fmt->width = sdformat->format.width; > > + fmt->height = sdformat->format.height; > > + > > + sdformat->format = *fmt; > > + > > + /* Propagate the format from sink to source. */ > > + fmt = imx8mq_mipi_csi_get_format(state, sd_state, sdformat->which, > > + MIPI_CSI2_PAD_SOURCE); > > + *fmt = sdformat->format; > > + > > + /* Store the CSI2 format descriptor for active formats. */ > > + if (sdformat->which == V4L2_SUBDEV_FORMAT_ACTIVE) > > + state->csi2_fmt = csi2_fmt; > > + > > + mutex_unlock(&state->lock); > > + > > + return 0; > > +} > > + > > +static const struct v4l2_subdev_video_ops imx8mq_mipi_csi_video_ops = { > > + .s_stream = imx8mq_mipi_csi_s_stream, > > +}; > > + > > +static const struct v4l2_subdev_pad_ops imx8mq_mipi_csi_pad_ops = { > > + .init_cfg = imx8mq_mipi_csi_init_cfg, > > + .enum_mbus_code = imx8mq_mipi_csi_enum_mbus_code, > > + .get_fmt = imx8mq_mipi_csi_get_fmt, > > + .set_fmt = imx8mq_mipi_csi_set_fmt, > > +}; > > + > > +static const struct v4l2_subdev_ops imx8mq_mipi_csi_subdev_ops = { > > + .video = &imx8mq_mipi_csi_video_ops, > > + .pad = &imx8mq_mipi_csi_pad_ops, > > +}; > > + > > +/* ----------------------------------------------------------------------------- > > + * Media entity operations > > + */ > > + > > +static const struct media_entity_operations imx8mq_mipi_csi_entity_ops = { > > + .link_validate = v4l2_subdev_link_validate, > > + .get_fwnode_pad = v4l2_subdev_get_fwnode_pad_1_to_1, > > +}; > > + > > +/* ----------------------------------------------------------------------------- > > + * Async subdev notifier > > + */ > > + > > +static struct csi_state * > > +mipi_notifier_to_csi2_state(struct v4l2_async_notifier *n) > > +{ > > + return container_of(n, struct csi_state, notifier); > > +} > > + > > +static int imx8mq_mipi_csi_notify_bound(struct v4l2_async_notifier *notifier, > > + struct v4l2_subdev *sd, > > + struct v4l2_async_subdev *asd) > > +{ > > + struct csi_state *state = mipi_notifier_to_csi2_state(notifier); > > + struct media_pad *sink = &state->sd.entity.pads[MIPI_CSI2_PAD_SINK]; > > + > > + state->src_sd = sd; > > + > > + return v4l2_create_fwnode_links_to_pad(sd, sink, MEDIA_LNK_FL_ENABLED | > > + MEDIA_LNK_FL_IMMUTABLE); > > +} > > + > > +static const struct v4l2_async_notifier_operations imx8mq_mipi_csi_notify_ops = { > > + .bound = imx8mq_mipi_csi_notify_bound, > > +}; > > + > > +static int imx8mq_mipi_csi_async_register(struct csi_state *state) > > +{ > > + struct v4l2_fwnode_endpoint vep = { > > + .bus_type = V4L2_MBUS_CSI2_DPHY, > > + }; > > + struct v4l2_async_subdev *asd; > > + struct fwnode_handle *ep; > > + unsigned int i; > > + int ret; > > + > > + v4l2_async_notifier_init(&state->notifier); > > + > > + ep = fwnode_graph_get_endpoint_by_id(dev_fwnode(state->dev), 0, 0, > > + FWNODE_GRAPH_ENDPOINT_NEXT); > > + if (!ep) > > + return -ENOTCONN; > > + > > + ret = v4l2_fwnode_endpoint_parse(ep, &vep); > > + if (ret) > > + goto err_parse; > > + > > + for (i = 0; i < vep.bus.mipi_csi2.num_data_lanes; ++i) { > > + if (vep.bus.mipi_csi2.data_lanes[i] != i + 1) { > > + dev_err(state->dev, > > + "data lanes reordering is not supported"); > > + ret = -EINVAL; > > + goto err_parse; > > + } > > + } > > + > > + state->bus = vep.bus.mipi_csi2; > > + > > + dev_dbg(state->dev, "data lanes: %d flags: 0x%08x\n", > > + state->bus.num_data_lanes, > > + state->bus.flags); > > + > > + asd = v4l2_async_notifier_add_fwnode_remote_subdev(&state->notifier, > > + ep, struct v4l2_async_subdev); > > + if (IS_ERR(asd)) { > > + ret = PTR_ERR(asd); > > + goto err_parse; > > + } > > + > > + fwnode_handle_put(ep); > > + > > + state->notifier.ops = &imx8mq_mipi_csi_notify_ops; > > + > > + ret = v4l2_async_subdev_notifier_register(&state->sd, &state->notifier); > > + if (ret) > > + return ret; > > + > > + return v4l2_async_register_subdev(&state->sd); > > + > > +err_parse: > > + fwnode_handle_put(ep); > > + > > + return ret; > > +} > > + > > +/* ----------------------------------------------------------------------------- > > + * Suspend/resume > > + */ > > + > > +static int imx8mq_mipi_csi_pm_suspend(struct device *dev, bool runtime) > > +{ > > + struct v4l2_subdev *sd = dev_get_drvdata(dev); > > + struct csi_state *state = mipi_sd_to_csi2_state(sd); > > + int ret = 0; > > + > > + mutex_lock(&state->lock); > > + > > + if (state->state & ST_POWERED) { > > + imx8mq_mipi_csi_stop_stream(state); > > + imx8mq_mipi_csi_clk_disable(state); > > + state->state &= ~ST_POWERED; > > + if (!runtime) > > + state->state |= ST_SUSPENDED; > > + } > > + > > + mutex_unlock(&state->lock); > > + > > + ret = icc_set_bw(state->icc_path, 0, 0); > > + if (ret) > > + dev_err(dev, "icc_set_bw failed with %d\n", ret); > > + > > + return ret ? -EAGAIN : 0; > > +} > > + > > +static int imx8mq_mipi_csi_pm_resume(struct device *dev, bool runtime) > > +{ > > + struct v4l2_subdev *sd = dev_get_drvdata(dev); > > + struct csi_state *state = mipi_sd_to_csi2_state(sd); > > + int ret = 0; > > + > > + ret = icc_set_bw(state->icc_path, 0, state->icc_path_bw); > > + if (ret) { > > + dev_err(dev, "icc_set_bw failed with %d\n", ret); > > + return ret; > > + } > > + > > + mutex_lock(&state->lock); > > + > > + if (!runtime && !(state->state & ST_SUSPENDED)) > > + goto unlock; > > + > > + if (!(state->state & ST_POWERED)) { > > + state->state |= ST_POWERED; > > + ret = imx8mq_mipi_csi_clk_enable(state); > > + } > > + if (state->state & ST_STREAMING) { > > + ret = imx8mq_mipi_csi_start_stream(state); > > + if (ret) > > + goto unlock; > > + } > > + > > + state->state &= ~ST_SUSPENDED; > > + > > +unlock: > > + mutex_unlock(&state->lock); > > + > > + return ret ? -EAGAIN : 0; > > +} > > + > > +static int __maybe_unused imx8mq_mipi_csi_suspend(struct device *dev) > > +{ > > + return imx8mq_mipi_csi_pm_suspend(dev, false); > > +} > > + > > +static int __maybe_unused imx8mq_mipi_csi_resume(struct device *dev) > > +{ > > + return imx8mq_mipi_csi_pm_resume(dev, false); > > +} > > + > > +static int __maybe_unused imx8mq_mipi_csi_runtime_suspend(struct device *dev) > > +{ > > + return imx8mq_mipi_csi_pm_suspend(dev, true); > > +} > > + > > +static int __maybe_unused imx8mq_mipi_csi_runtime_resume(struct device *dev) > > +{ > > + return imx8mq_mipi_csi_pm_resume(dev, true); > > +} > > + > > +static const struct dev_pm_ops imx8mq_mipi_csi_pm_ops = { > > + SET_RUNTIME_PM_OPS(imx8mq_mipi_csi_runtime_suspend, > > + imx8mq_mipi_csi_runtime_resume, > > + NULL) > > + SET_SYSTEM_SLEEP_PM_OPS(imx8mq_mipi_csi_suspend, imx8mq_mipi_csi_resume) > > +}; > > + > > +/* ----------------------------------------------------------------------------- > > + * Probe/remove & platform driver > > + */ > > + > > +static int imx8mq_mipi_csi_subdev_init(struct csi_state *state) > > +{ > > + struct v4l2_subdev *sd = &state->sd; > > + > > + v4l2_subdev_init(sd, &imx8mq_mipi_csi_subdev_ops); > > + sd->owner = THIS_MODULE; > > + snprintf(sd->name, sizeof(sd->name), "%s %s", > > + MIPI_CSI2_SUBDEV_NAME, dev_name(state->dev)); > > + > > + sd->flags |= V4L2_SUBDEV_FL_HAS_DEVNODE; > > + > > + sd->entity.function = MEDIA_ENT_F_VID_IF_BRIDGE; > > + sd->entity.ops = &imx8mq_mipi_csi_entity_ops; > > + > > + sd->dev = state->dev; > > + > > + state->csi2_fmt = &imx8mq_mipi_csi_formats[0]; > > + imx8mq_mipi_csi_init_cfg(sd, NULL); > > + > > + state->pads[MIPI_CSI2_PAD_SINK].flags = MEDIA_PAD_FL_SINK > > + | MEDIA_PAD_FL_MUST_CONNECT; > > + state->pads[MIPI_CSI2_PAD_SOURCE].flags = MEDIA_PAD_FL_SOURCE > > + | MEDIA_PAD_FL_MUST_CONNECT; > > + return media_entity_pads_init(&sd->entity, MIPI_CSI2_PADS_NUM, > > + state->pads); > > +} > > + > > +static void imx8mq_mipi_csi_release_icc(struct platform_device *pdev) > > +{ > > + struct v4l2_subdev *sd = dev_get_drvdata(&pdev->dev); > > + struct csi_state *state = mipi_sd_to_csi2_state(sd); > > + > > + icc_put(state->icc_path); > > +} > > + > > +static int imx8mq_mipi_csi_init_icc(struct platform_device *pdev) > > +{ > > + struct v4l2_subdev *sd = dev_get_drvdata(&pdev->dev); > > + struct csi_state *state = mipi_sd_to_csi2_state(sd); > > + > > + /* Optional interconnect request */ > > + state->icc_path = of_icc_get(&pdev->dev, "dram"); > > + if (IS_ERR_OR_NULL(state->icc_path)) > > + return PTR_ERR_OR_ZERO(state->icc_path); > > + > > + state->icc_path_bw = MBps_to_icc(700); > > + > > + return 0; > > +} > > + > > +static int imx8mq_mipi_csi_parse_dt(struct csi_state *state) > > +{ > > + struct device *dev = state->dev; > > + struct device_node *np = state->dev->of_node; > > + struct device_node *node; > > + phandle ph; > > + u32 out_val[2]; > > + int ret = 0; > > + > > + state->rst = devm_reset_control_array_get_exclusive(dev); > > + if (IS_ERR(state->rst)) { > > + dev_err(dev, "Failed to get reset: %pe\n", state->rst); > > + return PTR_ERR(state->rst); > > + } > > + > > + ret = of_property_read_u32_array(np, "fsl,mipi-phy-gpr", out_val, > > + ARRAY_SIZE(out_val)); > > + if (ret) { > > + dev_err(dev, "no fsl,mipi-phy-gpr property found: %d\n", ret); > > + return ret; > > + } > > + > > + ph = *out_val; > > + > > + node = of_find_node_by_phandle(ph); > > + if (!node) { > > + dev_err(dev, "Error finding node by phandle\n"); > > + return -ENODEV; > > + } > > + state->phy_gpr = syscon_node_to_regmap(node); > > + of_node_put(node); > > + if (IS_ERR(state->phy_gpr)) { > > + dev_err(dev, "failed to get gpr regmap: %pe\n", state->phy_gpr); > > + return PTR_ERR(state->phy_gpr); > > + } > > + > > + state->phy_gpr_reg = out_val[1]; > > + dev_dbg(dev, "phy gpr register set to 0x%x\n", state->phy_gpr_reg); > > + > > + return ret; > > +} > > + > > +static int imx8mq_mipi_csi_probe(struct platform_device *pdev) > > +{ > > + struct device *dev = &pdev->dev; > > + struct csi_state *state; > > + int ret; > > + > > + state = devm_kzalloc(dev, sizeof(*state), GFP_KERNEL); > > + if (!state) > > + return -ENOMEM; > > + > > + state->dev = dev; > > + > > + ret = imx8mq_mipi_csi_parse_dt(state); > > + if (ret < 0) { > > + dev_err(dev, "Failed to parse device tree: %d\n", ret); > > + return ret; > > + } > > + > > + /* Acquire resources. */ > > + state->regs = devm_platform_ioremap_resource(pdev, 0); > > + if (IS_ERR(state->regs)) > > + return PTR_ERR(state->regs); > > + > > + ret = imx8mq_mipi_csi_clk_get(state); > > + if (ret < 0) > > + return ret; > > + > > + platform_set_drvdata(pdev, &state->sd); > > + > > + mutex_init(&state->lock); > > + > > + ret = imx8mq_mipi_csi_subdev_init(state); > > + if (ret < 0) > > + goto mutex; > > + > > + ret = imx8mq_mipi_csi_init_icc(pdev); > > + if (ret) > > + goto mutex; > > + > > + /* Enable runtime PM. */ > > + pm_runtime_enable(dev); > > + if (!pm_runtime_enabled(dev)) { > > + ret = imx8mq_mipi_csi_pm_resume(dev, true); > > + if (ret < 0) > > + goto icc; > > + } > > + > > + ret = imx8mq_mipi_csi_async_register(state); > > + if (ret < 0) > > + goto cleanup; > > + > > + return 0; > > + > > +cleanup: > > + pm_runtime_disable(&pdev->dev); > > + imx8mq_mipi_csi_pm_suspend(&pdev->dev, true); > > + > > + media_entity_cleanup(&state->sd.entity); > > + v4l2_async_notifier_unregister(&state->notifier); > > + v4l2_async_notifier_cleanup(&state->notifier); > > + v4l2_async_unregister_subdev(&state->sd); > > +icc: > > + imx8mq_mipi_csi_release_icc(pdev); > > +mutex: > > + mutex_destroy(&state->lock); > > + > > + return ret; > > +} > > + > > +static int imx8mq_mipi_csi_remove(struct platform_device *pdev) > > +{ > > + struct v4l2_subdev *sd = platform_get_drvdata(pdev); > > + struct csi_state *state = mipi_sd_to_csi2_state(sd); > > + > > + v4l2_async_notifier_unregister(&state->notifier); > > + v4l2_async_notifier_cleanup(&state->notifier); > > + v4l2_async_unregister_subdev(&state->sd); > > + > > + pm_runtime_disable(&pdev->dev); > > + imx8mq_mipi_csi_pm_suspend(&pdev->dev, true); > > + media_entity_cleanup(&state->sd.entity); > > + mutex_destroy(&state->lock); > > + pm_runtime_set_suspended(&pdev->dev); > > + imx8mq_mipi_csi_release_icc(pdev); > > + > > + return 0; > > +} > > + > > +static const struct of_device_id imx8mq_mipi_csi_of_match[] = { > > + { .compatible = "fsl,imx8mq-mipi-csi2", }, > > + { /* sentinel */ }, > > +}; > > +MODULE_DEVICE_TABLE(of, imx8mq_mipi_csi_of_match); > > + > > +static struct platform_driver imx8mq_mipi_csi_driver = { > > + .probe = imx8mq_mipi_csi_probe, > > + .remove = imx8mq_mipi_csi_remove, > > + .driver = { > > + .of_match_table = imx8mq_mipi_csi_of_match, > > + .name = MIPI_CSI2_DRIVER_NAME, > > + .pm = &imx8mq_mipi_csi_pm_ops, > > + }, > > +}; > > + > > +module_platform_driver(imx8mq_mipi_csi_driver); > > + > > +MODULE_DESCRIPTION("i.MX8MQ MIPI CSI-2 receiver driver"); > > +MODULE_AUTHOR("Martin Kepplinger "); > > +MODULE_LICENSE("GPL v2"); > > +MODULE_ALIAS("platform:imx8mq-mipi-csi2"); -- Regards, Laurent Pinchart _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel