* [PATCH 1/6] media: qcom: camss: Add support for TFE (Spectra 340)
@ 2025-04-16 12:09 ` Loic Poulain
2025-04-16 12:09 ` [PATCH 2/6] media: qcom: camss: Add CSID 340 support Loic Poulain
` (8 more replies)
0 siblings, 9 replies; 24+ messages in thread
From: Loic Poulain @ 2025-04-16 12:09 UTC (permalink / raw)
To: bryan.odonoghue, rfoss, konradybcio, andersson, krzk+dt, robh
Cc: linux-arm-msm, linux-media, devicetree, Loic Poulain
Add support for TFE (Thin Front End) found in QCM2290.
Signed-off-by: Loic Poulain <loic.poulain@oss.qualcomm.com>
---
drivers/media/platform/qcom/camss/Makefile | 1 +
.../media/platform/qcom/camss/camss-vfe-340.c | 281 ++++++++++++++++++
drivers/media/platform/qcom/camss/camss-vfe.h | 1 +
3 files changed, 283 insertions(+)
create mode 100644 drivers/media/platform/qcom/camss/camss-vfe-340.c
diff --git a/drivers/media/platform/qcom/camss/Makefile b/drivers/media/platform/qcom/camss/Makefile
index d26a9c24a430..719898f5d32b 100644
--- a/drivers/media/platform/qcom/camss/Makefile
+++ b/drivers/media/platform/qcom/camss/Makefile
@@ -17,6 +17,7 @@ qcom-camss-objs += \
camss-vfe-4-7.o \
camss-vfe-4-8.o \
camss-vfe-17x.o \
+ camss-vfe-340.o \
camss-vfe-480.o \
camss-vfe-680.o \
camss-vfe-780.o \
diff --git a/drivers/media/platform/qcom/camss/camss-vfe-340.c b/drivers/media/platform/qcom/camss/camss-vfe-340.c
new file mode 100644
index 000000000000..fc454d66e1d2
--- /dev/null
+++ b/drivers/media/platform/qcom/camss/camss-vfe-340.c
@@ -0,0 +1,281 @@
+// SPDX-License-Identifier: GPL-2.0
+/*
+ * Qualcomm MSM Camera Subsystem - VFE (Video Front End) Module 340 (TFE)
+ *
+ * Copyright (c) 2025 Qualcomm Technologies, Inc.
+ */
+
+#include <linux/delay.h>
+#include <linux/interrupt.h>
+#include <linux/io.h>
+#include <linux/iopoll.h>
+
+#include "camss.h"
+#include "camss-vfe.h"
+
+#define TFE_GLOBAL_RESET_CMD (0x014)
+#define TFE_GLOBAL_RESET_CMD_CORE BIT(0)
+
+#define TFE_REG_UPDATE_CMD (0x02C)
+
+#define TFE_IRQ_CMD (0x030)
+#define TFE_IRQ_CMD_CLEAR BIT(0)
+#define TFE_IRQ_MASK_0 (0x034)
+#define TFE_IRQ_MASK_0_RST_DONE BIT(0)
+#define TFE_IRQ_MASK_0_BUS_WR BIT(1)
+#define TFE_IRQ_MASK_1 (0x038)
+#define TFE_IRQ_MASK_2 (0x03c)
+#define TFE_IRQ_CLEAR_0 (0x040)
+
+#define TFE_IRQ_STATUS_0 (0x04c)
+
+#define BUS_REG(a) (0xa00 + (a))
+
+#define TFE_BUS_IRQ_MASK_0 BUS_REG(0x18)
+#define TFE_BUS_IRQ_MASK_RUP_DONE_ALL 0x000f
+#define TFE_BUS_IRQ_MASK_RUP_DONE(src) BIT(src)
+#define TFE_BUS_IRQ_MASK_BUF_DONE_ALL 0xff00
+#define TFE_BUS_IRQ_MASK_BUF_DONE(sg) BIT((sg) + 8)
+#define TFE_BUS_IRQ_MASK_0_CONS_VIOL BIT(28)
+#define TFE_BUS_IRQ_MASK_0_VIOL BIT(30)
+#define TFE_BUS_IRQ_MASK_0_IMG_VIOL BIT(31)
+
+#define TFE_BUS_IRQ_MASK_1 BUS_REG(0x1C)
+#define TFE_BUS_IRQ_CLEAR_0 BUS_REG(0x20)
+#define TFE_BUS_IRQ_STATUS_0 BUS_REG(0x28)
+#define TFE_BUS_IRQ_CMD BUS_REG(0x30)
+#define TFE_BUS_IRQ_CMD_CLEAR BIT(0)
+
+#define TFE_BUS_STATUS_CLEAR BUS_REG(0x60)
+#define TFE_BUS_VIOLATION_STATUS BUS_REG(0x64)
+#define TFE_BUS_OVERFLOW_STATUS BUS_REG(0x68)
+#define TFE_BUS_IMAGE_SZ_VIOLATION_STATUS BUS_REG(0x70)
+
+#define TFE_BUS_CLIENT_CFG(c) BUS_REG(0x200 + (c) * 0x100)
+#define TFE_BUS_CLIENT_CFG_EN BIT(0)
+#define TFE_BUS_CLIENT_CFG_MODE_FRAME BIT(16)
+#define TFE_BUS_IMAGE_ADDR(c) BUS_REG(0x204 + (c) * 0x100)
+#define TFE_BUS_FRAME_INCR(c) BUS_REG(0x208 + (c) * 0x100)
+#define TFE_BUS_IMAGE_CFG_0(c) BUS_REG(0x20C + (c) * 0x100)
+#define TFE_BUS_IMAGE_CFG_0_DEFAULT 0xffff
+#define TFE_BUS_IMAGE_CFG_1(c) BUS_REG(0x210 + (c) * 0x100)
+#define TFE_BUS_IMAGE_CFG_2(c) BUS_REG(0x214 + (c) * 0x100)
+#define TFE_BUS_IMAGE_CFG_2_DEFAULT 0xffff
+#define TFE_BUS_PACKER_CFG(c) BUS_REG(0x218 + (c) * 0x100)
+#define TFE_BUS_PACKER_CFG_FMT_PLAIN64 0xa
+#define TFE_BUS_IRQ_SUBSAMPLE_CFG_0(c) BUS_REG(0x230 + (c) * 0x100)
+#define TFE_BUS_IRQ_SUBSAMPLE_CFG_1(c) BUS_REG(0x234 + (c) * 0x100)
+#define TFE_BUS_FRAMEDROP_CFG_0(c) BUS_REG(0x238 + (c) * 0x100)
+#define TFE_BUS_FRAMEDROP_CFG_1(c) BUS_REG(0x23c + (c) * 0x100)
+
+#define RDI_CLIENT(n) (7 + (n))
+#define TFE_SOURCES_NUM 4
+#define TFE_SUBGROUPS_NUM 8
+#define TFE_CLIENTS_NUM 10
+
+static inline unsigned int __regupdate_iface(enum vfe_line_id line_id)
+{
+ switch (line_id) {
+ case VFE_LINE_RDI0:
+ return 1;
+ case VFE_LINE_RDI1:
+ return 2;
+ case VFE_LINE_RDI2:
+ return 3;
+ case VFE_LINE_PIX:
+ default:
+ return 0;
+ }
+}
+
+static inline unsigned int __regupdate_line(unsigned int iface)
+{
+ if (!iface)
+ return VFE_LINE_PIX;
+ if (iface < 4)
+ return VFE_LINE_RDI0 + (iface - 1);
+
+ return VFE_LINE_NONE;
+}
+
+static inline unsigned int __subgroup_line(unsigned int subgroup)
+{
+ switch (subgroup) {
+ case 5:
+ return VFE_LINE_RDI0;
+ case 6:
+ return VFE_LINE_RDI1;
+ case 7:
+ return VFE_LINE_RDI2;
+ default:
+ return VFE_LINE_PIX;
+ }
+
+ return VFE_LINE_NONE;
+}
+
+static void vfe_global_reset(struct vfe_device *vfe)
+{
+ writel_relaxed(TFE_IRQ_MASK_0_RST_DONE, vfe->base + TFE_IRQ_MASK_0);
+ writel(TFE_GLOBAL_RESET_CMD_CORE, vfe->base + TFE_GLOBAL_RESET_CMD);
+}
+
+static irqreturn_t vfe_isr(int irq, void *dev)
+{
+ struct vfe_device *vfe = dev;
+ u32 status;
+ int i;
+
+ status = readl_relaxed(vfe->base + TFE_IRQ_STATUS_0);
+ writel_relaxed(status, vfe->base + TFE_IRQ_CLEAR_0);
+ writel_relaxed(TFE_IRQ_CMD_CLEAR, vfe->base + TFE_IRQ_CMD);
+
+
+ if (status & TFE_IRQ_MASK_0_RST_DONE) {
+ dev_dbg(vfe->camss->dev, "VFE%u: Reset done!", vfe->id);
+ vfe_isr_reset_ack(vfe);
+ }
+
+ if (status & TFE_IRQ_MASK_0_BUS_WR) {
+ u32 bus_status = readl_relaxed(vfe->base + TFE_BUS_IRQ_STATUS_0);
+
+ writel_relaxed(bus_status, vfe->base + TFE_BUS_IRQ_CLEAR_0);
+ writel_relaxed(TFE_BUS_IRQ_CMD_CLEAR, vfe->base + TFE_BUS_IRQ_CMD);
+
+ for (i = 0; i < TFE_SOURCES_NUM; i++) {
+ if (bus_status & TFE_BUS_IRQ_MASK_RUP_DONE(i))
+ vfe->res->hw_ops->reg_update_clear(vfe, __regupdate_line(i));
+ }
+
+ for (i = 0; i < TFE_SUBGROUPS_NUM; i++) {
+ if (bus_status & TFE_BUS_IRQ_MASK_BUF_DONE(i))
+ vfe_buf_done(vfe, __subgroup_line(i));
+ }
+
+ if (bus_status & TFE_BUS_IRQ_MASK_0_CONS_VIOL)
+ dev_err_ratelimited(vfe->camss->dev, "VFE%u: Bad config violation",
+ vfe->id);
+
+ if (bus_status & TFE_BUS_IRQ_MASK_0_VIOL)
+ dev_err_ratelimited(vfe->camss->dev, "VFE%u: Input data violation",
+ vfe->id);
+
+ if (bus_status & TFE_BUS_IRQ_MASK_0_IMG_VIOL)
+ dev_err_ratelimited(vfe->camss->dev, "VFE%u: Image size violation",
+ vfe->id);
+ }
+
+ status = readl_relaxed(vfe->base + TFE_BUS_OVERFLOW_STATUS);
+ if (status) {
+ writel_relaxed(status, vfe->base + TFE_BUS_STATUS_CLEAR);
+ for (i = 0; i < TFE_CLIENTS_NUM; i++) {
+ if (status & BIT(i))
+ dev_err_ratelimited(vfe->camss->dev,
+ "VFE%u: bus overflow for client %u\n",
+ vfe->id, i);
+ }
+ }
+
+ return IRQ_HANDLED;
+}
+
+static int vfe_halt(struct vfe_device *vfe)
+{
+ /* rely on vfe_disable_output() to stop the VFE */
+ return 0;
+}
+
+static void vfe_enable_irq(struct vfe_device *vfe)
+{
+ writel_relaxed(TFE_IRQ_MASK_0_RST_DONE | TFE_IRQ_MASK_0_BUS_WR,
+ vfe->base + TFE_IRQ_MASK_0);
+ writel_relaxed(TFE_BUS_IRQ_MASK_RUP_DONE_ALL | TFE_BUS_IRQ_MASK_BUF_DONE_ALL |
+ TFE_BUS_IRQ_MASK_0_CONS_VIOL | TFE_BUS_IRQ_MASK_0_VIOL |
+ TFE_BUS_IRQ_MASK_0_IMG_VIOL, vfe->base + TFE_BUS_IRQ_MASK_0);
+}
+
+static void vfe_wm_update(struct vfe_device *vfe, u8 rdi, u32 addr,
+ struct vfe_line *line)
+{
+ u8 client = RDI_CLIENT(rdi);
+
+ writel_relaxed(addr, vfe->base + TFE_BUS_IMAGE_ADDR(client));
+}
+
+static void vfe_wm_start(struct vfe_device *vfe, u8 rdi, struct vfe_line *line)
+{
+ struct v4l2_pix_format_mplane *pix =
+ &line->video_out.active_fmt.fmt.pix_mp;
+ u32 stride = pix->plane_fmt[0].bytesperline;
+ u8 client = RDI_CLIENT(rdi);
+
+ /* Configuration for plain RDI frames */
+ writel_relaxed(TFE_BUS_IMAGE_CFG_0_DEFAULT, vfe->base + TFE_BUS_IMAGE_CFG_0(client));
+ writel_relaxed(0u, vfe->base + TFE_BUS_IMAGE_CFG_1(client));
+ writel_relaxed(TFE_BUS_IMAGE_CFG_2_DEFAULT, vfe->base + TFE_BUS_IMAGE_CFG_2(client));
+ writel_relaxed(stride * pix->height, vfe->base + TFE_BUS_FRAME_INCR(client));
+ writel_relaxed(TFE_BUS_PACKER_CFG_FMT_PLAIN64, vfe->base + TFE_BUS_PACKER_CFG(client));
+
+ /* No dropped frames, one irq per frame */
+ writel_relaxed(0, vfe->base + TFE_BUS_FRAMEDROP_CFG_0(client));
+ writel_relaxed(1, vfe->base + TFE_BUS_FRAMEDROP_CFG_1(client));
+ writel_relaxed(0, vfe->base + TFE_BUS_IRQ_SUBSAMPLE_CFG_0(client));
+ writel_relaxed(1, vfe->base + TFE_BUS_IRQ_SUBSAMPLE_CFG_1(client));
+
+ vfe_enable_irq(vfe);
+
+ writel(TFE_BUS_CLIENT_CFG_EN | TFE_BUS_CLIENT_CFG_MODE_FRAME,
+ vfe->base + TFE_BUS_CLIENT_CFG(client));
+
+ dev_dbg(vfe->camss->dev, "VFE%u: Started RDI%u width %u height %u stride %u\n",
+ vfe->id, rdi, pix->width, pix->height, stride);
+}
+
+static void vfe_wm_stop(struct vfe_device *vfe, u8 rdi)
+{
+ u8 client = RDI_CLIENT(rdi);
+
+ writel(0, vfe->base + TFE_BUS_CLIENT_CFG(client));
+
+ dev_dbg(vfe->camss->dev, "VFE%u: Stopped RDI%u\n", vfe->id, rdi);
+}
+
+static const struct camss_video_ops vfe_video_ops_520 = {
+ .queue_buffer = vfe_queue_buffer_v2,
+ .flush_buffers = vfe_flush_buffers,
+};
+
+static void vfe_subdev_init(struct device *dev, struct vfe_device *vfe)
+{
+ vfe->video_ops = vfe_video_ops_520;
+}
+
+static void vfe_reg_update(struct vfe_device *vfe, enum vfe_line_id line_id)
+{
+ vfe->reg_update |= BIT(__regupdate_iface(line_id));
+ writel_relaxed(vfe->reg_update, vfe->base + TFE_REG_UPDATE_CMD);
+}
+
+static inline void vfe_reg_update_clear(struct vfe_device *vfe,
+ enum vfe_line_id line_id)
+{
+ vfe->reg_update &= ~BIT(__regupdate_iface(line_id));
+}
+
+const struct vfe_hw_ops vfe_ops_340 = {
+ .global_reset = vfe_global_reset,
+ .hw_version = vfe_hw_version,
+ .isr = vfe_isr,
+ .pm_domain_off = vfe_pm_domain_off,
+ .pm_domain_on = vfe_pm_domain_on,
+ .subdev_init = vfe_subdev_init,
+ .vfe_disable = vfe_disable,
+ .vfe_enable = vfe_enable_v2,
+ .vfe_halt = vfe_halt,
+ .vfe_wm_start = vfe_wm_start,
+ .vfe_wm_stop = vfe_wm_stop,
+ .vfe_buf_done = vfe_buf_done,
+ .vfe_wm_update = vfe_wm_update,
+ .reg_update = vfe_reg_update,
+ .reg_update_clear = vfe_reg_update_clear,
+};
diff --git a/drivers/media/platform/qcom/camss/camss-vfe.h b/drivers/media/platform/qcom/camss/camss-vfe.h
index a23f666be753..9b138849caca 100644
--- a/drivers/media/platform/qcom/camss/camss-vfe.h
+++ b/drivers/media/platform/qcom/camss/camss-vfe.h
@@ -242,6 +242,7 @@ extern const struct vfe_hw_ops vfe_ops_4_1;
extern const struct vfe_hw_ops vfe_ops_4_7;
extern const struct vfe_hw_ops vfe_ops_4_8;
extern const struct vfe_hw_ops vfe_ops_170;
+extern const struct vfe_hw_ops vfe_ops_340;
extern const struct vfe_hw_ops vfe_ops_480;
extern const struct vfe_hw_ops vfe_ops_680;
extern const struct vfe_hw_ops vfe_ops_780;
--
2.34.1
^ permalink raw reply related [flat|nested] 24+ messages in thread
* [PATCH 2/6] media: qcom: camss: Add CSID 340 support
2025-04-16 12:09 ` [PATCH 1/6] media: qcom: camss: Add support for TFE (Spectra 340) Loic Poulain
@ 2025-04-16 12:09 ` Loic Poulain
2025-04-16 12:09 ` [PATCH 3/6] media: qcom: camss: csiphy-3ph: Add CSIPHY 2ph DPHY v2.0.1 init sequence Loic Poulain
` (7 subsequent siblings)
8 siblings, 0 replies; 24+ messages in thread
From: Loic Poulain @ 2025-04-16 12:09 UTC (permalink / raw)
To: bryan.odonoghue, rfoss, konradybcio, andersson, krzk+dt, robh
Cc: linux-arm-msm, linux-media, devicetree, Loic Poulain
Add support for CSID found in QCM2290, it's a simplified gen-2 version.
- There is no Test Pattern Generator (moved outside CSID)
- There is no subsampling (moved to CAMIF module)
Signed-off-by: Loic Poulain <loic.poulain@oss.qualcomm.com>
---
drivers/media/platform/qcom/camss/Makefile | 1 +
.../platform/qcom/camss/camss-csid-340.c | 190 ++++++++++++++++++
.../media/platform/qcom/camss/camss-csid.h | 1 +
3 files changed, 192 insertions(+)
create mode 100644 drivers/media/platform/qcom/camss/camss-csid-340.c
diff --git a/drivers/media/platform/qcom/camss/Makefile b/drivers/media/platform/qcom/camss/Makefile
index 719898f5d32b..3217bf40976d 100644
--- a/drivers/media/platform/qcom/camss/Makefile
+++ b/drivers/media/platform/qcom/camss/Makefile
@@ -6,6 +6,7 @@ qcom-camss-objs += \
camss-csid.o \
camss-csid-4-1.o \
camss-csid-4-7.o \
+ camss-csid-340.o \
camss-csid-680.o \
camss-csid-gen2.o \
camss-csid-780.o \
diff --git a/drivers/media/platform/qcom/camss/camss-csid-340.c b/drivers/media/platform/qcom/camss/camss-csid-340.c
new file mode 100644
index 000000000000..24f8f6fb6f4c
--- /dev/null
+++ b/drivers/media/platform/qcom/camss/camss-csid-340.c
@@ -0,0 +1,190 @@
+// SPDX-License-Identifier: GPL-2.0
+/*
+ * Qualcomm MSM Camera Subsystem - CSID (CSI Decoder) Module 340
+ *
+ * Copyright (c) 2025 Qualcomm Technologies, Inc.
+ */
+
+#include <linux/completion.h>
+#include <linux/interrupt.h>
+#include <linux/io.h>
+#include <linux/kernel.h>
+
+#include "camss.h"
+#include "camss-csid.h"
+#include "camss-csid-gen2.h"
+
+#define CSID_RST_STROBES (0x010)
+#define CSID_RST_SW_REGS BIT(0)
+#define CSID_RST_IRQ BIT(1)
+#define CSID_RST_IFE_CLK BIT(2)
+#define CSID_RST_PHY_CLK BIT(3)
+#define CSID_RST_CSID_CLK BIT(4)
+
+#define CSID_IRQ_STATUS (0x070)
+#define CSID_IRQ_MASK (0x074)
+#define CSID_IRQ_MASK_RST_DONE BIT(0)
+#define CSID_IRQ_CLEAR (0x078)
+#define CSID_IRQ_CMD (0x080)
+#define CSID_IRQ_CMD_CLEAR BIT(0)
+
+#define CSID_CSI2_RX_CFG0 (0x100)
+#define CSI2_RX_CFG0_NUM_ACTIVE_LANES 0
+#define CSI2_RX_CFG0_DL0_INPUT_SEL 4
+#define CSI2_RX_CFG0_DL1_INPUT_SEL 8
+#define CSI2_RX_CFG0_DL2_INPUT_SEL 12
+#define CSI2_RX_CFG0_DL3_INPUT_SEL 16
+#define CSI2_RX_CFG0_PHY_NUM_SEL 20
+#define CSI2_RX_CFG0_PHY_SEL_BASE_IDX 1
+#define CSI2_RX_CFG0_PHY_TYPE_SEL 24
+
+#define CSID_CSI2_RX_CFG1 (0x104)
+#define CSI2_RX_CFG1_PACKET_ECC_CORRECTION_EN BIT(0)
+#define CSI2_RX_CFG1_MISR_EN BIT(6)
+#define CSI2_RX_CFG1_CGC_MODE BIT(7)
+
+#define CSID_RDI_CFG0(rdi) (0x300 + 0x100 * (rdi))
+#define CSID_RDI_CFG0_BYTE_CNTR_EN BIT(0)
+#define CSID_RDI_CFG0_TIMESTAMP_EN BIT(1)
+#define CSID_RDI_CFG0_DECODE_FORMAT_NOP (0xf << 12)
+#define CSID_RDI_CFG0_DT(dt) ((dt) << 16)
+#define CSID_RDI_CFG0_VC(vc) ((vc) << 22)
+#define CSID_RDI_CFG0_DTID(dtid) ((dtid) << 22)
+#define CSID_RDI_CFG0_ENABLE BIT(31)
+
+#define CSID_RDI_CTRL(rdi) (0x308 + 0x100 * (rdi))
+#define CSID_RDI_CTRL_HALT_AT_FRAME_BOUNDARY 0
+#define CSID_RDI_CTRL_RESUME_AT_FRAME_BOUNDARY 1
+
+static void __csid_configure_rx(struct csid_device *csid,
+ struct csid_phy_config *phy, int vc)
+{
+ u32 val;
+
+ val = (phy->lane_cnt - 1) << CSI2_RX_CFG0_NUM_ACTIVE_LANES;
+ val |= phy->lane_assign << CSI2_RX_CFG0_DL0_INPUT_SEL;
+ val |= (phy->csiphy_id + CSI2_RX_CFG0_PHY_SEL_BASE_IDX) << CSI2_RX_CFG0_PHY_NUM_SEL;
+ writel_relaxed(val, csid->base + CSID_CSI2_RX_CFG0);
+
+ val = CSI2_RX_CFG1_PACKET_ECC_CORRECTION_EN;
+ writel_relaxed(val, csid->base + CSID_CSI2_RX_CFG1);
+}
+
+static void __csid_ctrl_rdi(struct csid_device *csid, int enable, u8 rdi)
+{
+ writel(!!enable, csid->base + CSID_RDI_CTRL(rdi));
+}
+
+static void __csid_configure_rdi_stream(struct csid_device *csid, u8 enable, u8 vc)
+{
+ struct v4l2_mbus_framefmt *input_format = &csid->fmt[MSM_CSID_PAD_FIRST_SRC + vc];
+ const struct csid_format_info *format = csid_get_fmt_entry(csid->res->formats->formats,
+ csid->res->formats->nformats,
+ input_format->code);
+ u8 lane_cnt = csid->phy.lane_cnt;
+ u8 dt_id;
+ u32 val;
+
+ if (!lane_cnt)
+ lane_cnt = 4;
+
+ /*
+ * DT_ID is a two bit bitfield that is concatenated with
+ * the four least significant bits of the five bit VC
+ * bitfield to generate an internal CID value.
+ *
+ * CSID_RDI_CFG0(vc)
+ * DT_ID : 28:27
+ * VC : 26:22
+ * DT : 21:16
+ *
+ * CID : VC 3:0 << 2 | DT_ID 1:0
+ */
+ dt_id = vc & 0x03;
+
+ val = CSID_RDI_CFG0_DECODE_FORMAT_NOP; /* only for RDI path */
+ val |= CSID_RDI_CFG0_DT(format->data_type);
+ val |= CSID_RDI_CFG0_VC(vc);
+ val |= CSID_RDI_CFG0_DTID(dt_id);
+
+ if (enable)
+ val |= CSID_RDI_CFG0_ENABLE;
+
+ dev_dbg(csid->camss->dev, "CSID%u: Stream %s (dt:0x%x vc=%u)\n",
+ csid->id, enable ? "enable" : "disable", format->data_type, vc);
+
+ writel_relaxed(val, csid->base + CSID_RDI_CFG0(vc));
+}
+
+static void csid_configure_stream(struct csid_device *csid, u8 enable)
+{
+ int i;
+
+ for (i = 0; i < MSM_CSID_MAX_SRC_STREAMS; i++) {
+ if (csid->phy.en_vc & BIT(i)) {
+ __csid_configure_rdi_stream(csid, enable, i);
+ __csid_configure_rx(csid, &csid->phy, i);
+ __csid_ctrl_rdi(csid, enable, i);
+ }
+ }
+}
+
+static int csid_reset(struct csid_device *csid)
+{
+ unsigned long time;
+
+ writel_relaxed(CSID_IRQ_MASK_RST_DONE, csid->base + CSID_IRQ_MASK);
+ writel_relaxed(CSID_IRQ_MASK_RST_DONE, csid->base + CSID_IRQ_CLEAR);
+ writel_relaxed(CSID_IRQ_CMD_CLEAR, csid->base + CSID_IRQ_CMD);
+
+ reinit_completion(&csid->reset_complete);
+
+ /* Reset with registers preserved */
+ writel(CSID_RST_IRQ | CSID_RST_IFE_CLK | CSID_RST_PHY_CLK | CSID_RST_CSID_CLK,
+ csid->base + CSID_RST_STROBES);
+
+ time = wait_for_completion_timeout(&csid->reset_complete,
+ msecs_to_jiffies(CSID_RESET_TIMEOUT_MS));
+ if (!time) {
+ dev_err(csid->camss->dev, "CSID%u: reset timeout\n", csid->id);
+ return -EIO;
+ }
+
+ dev_dbg(csid->camss->dev, "CSID%u: reset done\n", csid->id);
+
+ return 0;
+}
+
+static irqreturn_t csid_isr(int irq, void *dev)
+{
+ struct csid_device *csid = dev;
+ u32 val;
+
+ val = readl_relaxed(csid->base + CSID_IRQ_STATUS);
+ writel_relaxed(val, csid->base + CSID_IRQ_CLEAR);
+ writel_relaxed(CSID_IRQ_CMD_CLEAR, csid->base + CSID_IRQ_CMD);
+
+ if (val & CSID_IRQ_MASK_RST_DONE)
+ complete(&csid->reset_complete);
+ else
+ dev_warn_ratelimited(csid->camss->dev, "Spurious CSID interrupt\n");
+
+ return IRQ_HANDLED;
+}
+
+static int csid_configure_testgen_pattern(struct csid_device *csid, s32 val)
+{
+ return -EOPNOTSUPP; /* Not part of CSID */
+}
+
+static void csid_subdev_init(struct csid_device *csid) {}
+
+const struct csid_hw_ops csid_ops_340 = {
+ .configure_testgen_pattern = csid_configure_testgen_pattern,
+ .configure_stream = csid_configure_stream,
+ .hw_version = csid_hw_version,
+ .isr = csid_isr,
+ .reset = csid_reset,
+ .src_pad_code = csid_src_pad_code,
+ .subdev_init = csid_subdev_init,
+};
diff --git a/drivers/media/platform/qcom/camss/camss-csid.h b/drivers/media/platform/qcom/camss/camss-csid.h
index 9dc826d8c8f6..3399e92658d8 100644
--- a/drivers/media/platform/qcom/camss/camss-csid.h
+++ b/drivers/media/platform/qcom/camss/camss-csid.h
@@ -213,6 +213,7 @@ extern const struct csid_formats csid_formats_gen2;
extern const struct csid_hw_ops csid_ops_4_1;
extern const struct csid_hw_ops csid_ops_4_7;
+extern const struct csid_hw_ops csid_ops_340;
extern const struct csid_hw_ops csid_ops_680;
extern const struct csid_hw_ops csid_ops_gen2;
extern const struct csid_hw_ops csid_ops_780;
--
2.34.1
^ permalink raw reply related [flat|nested] 24+ messages in thread
* [PATCH 3/6] media: qcom: camss: csiphy-3ph: Add CSIPHY 2ph DPHY v2.0.1 init sequence
2025-04-16 12:09 ` [PATCH 1/6] media: qcom: camss: Add support for TFE (Spectra 340) Loic Poulain
2025-04-16 12:09 ` [PATCH 2/6] media: qcom: camss: Add CSID 340 support Loic Poulain
@ 2025-04-16 12:09 ` Loic Poulain
2025-04-16 22:36 ` Dmitry Baryshkov
2025-04-16 12:09 ` [PATCH 4/6] media: qcom: camss: add support for QCM2290 camss Loic Poulain
` (6 subsequent siblings)
8 siblings, 1 reply; 24+ messages in thread
From: Loic Poulain @ 2025-04-16 12:09 UTC (permalink / raw)
To: bryan.odonoghue, rfoss, konradybcio, andersson, krzk+dt, robh
Cc: linux-arm-msm, linux-media, devicetree, Loic Poulain
This is the CSI PHY version found in QCS2290/QCM2290 SoCs.
The table is extracted from downstream camera driver.
Signed-off-by: Loic Poulain <loic.poulain@oss.qualcomm.com>
---
.../qcom/camss/camss-csiphy-3ph-1-0.c | 89 +++++++++++++++++++
drivers/media/platform/qcom/camss/camss.h | 1 +
2 files changed, 90 insertions(+)
diff --git a/drivers/media/platform/qcom/camss/camss-csiphy-3ph-1-0.c b/drivers/media/platform/qcom/camss/camss-csiphy-3ph-1-0.c
index f732a76de93e..0e314ff9292f 100644
--- a/drivers/media/platform/qcom/camss/camss-csiphy-3ph-1-0.c
+++ b/drivers/media/platform/qcom/camss/camss-csiphy-3ph-1-0.c
@@ -319,6 +319,90 @@ csiphy_lane_regs lane_regs_sm8250[] = {
{0x0884, 0x01, 0x00, CSIPHY_DEFAULT_PARAMS},
};
+/* GEN2 2.0.1 2PH DPHY mode */
+static const struct
+csiphy_lane_regs lane_regs_qcm2290[] = {
+ {0x0030, 0x02, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x002C, 0x01, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x0034, 0x01, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x0028, 0x04, 0x00, CSIPHY_DNP_PARAMS},
+ {0x003C, 0xB8, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x001C, 0x0A, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x0000, 0xD7, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x0004, 0x08, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x0020, 0x00, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x0008, 0x04, 0x00, CSIPHY_SETTLE_CNT_LOWER_BYTE},
+ {0x000C, 0xFF, 0x00, CSIPHY_DNP_PARAMS},
+ {0x0010, 0x50, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x0038, 0x01, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x0060, 0x00, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x0064, 0x3F, 0x00, CSIPHY_DEFAULT_PARAMS},
+
+ {0x0730, 0x02, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x072C, 0x01, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x0734, 0x01, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x0728, 0x04, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x073C, 0xB8, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x071C, 0x0A, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x0700, 0xC0, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x0704, 0x08, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x0720, 0x00, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x0708, 0x04, 0x00, CSIPHY_SETTLE_CNT_LOWER_BYTE},
+ {0x070C, 0xFF, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x0710, 0x50, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x0738, 0x01, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x0760, 0x00, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x0764, 0x3F, 0x00, CSIPHY_DEFAULT_PARAMS},
+
+ {0x0230, 0x02, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x022C, 0x01, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x0234, 0x01, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x0228, 0x04, 0x00, CSIPHY_DNP_PARAMS},
+ {0x023C, 0xB8, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x021C, 0x0A, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x0200, 0xD7, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x0204, 0x08, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x0220, 0x00, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x0208, 0x04, 0x00, CSIPHY_SETTLE_CNT_LOWER_BYTE},
+ {0x020C, 0xFF, 0x00, CSIPHY_DNP_PARAMS},
+ {0x0210, 0x50, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x0238, 0x01, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x0260, 0x00, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x0264, 0x3F, 0x00, CSIPHY_DEFAULT_PARAMS},
+
+ {0x0430, 0x02, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x042C, 0x01, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x0434, 0x01, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x0428, 0x04, 0x00, CSIPHY_DNP_PARAMS},
+ {0x043C, 0xB8, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x041C, 0x0A, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x0400, 0xD7, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x0404, 0x08, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x0420, 0x00, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x0408, 0x04, 0x00, CSIPHY_SETTLE_CNT_LOWER_BYTE},
+ {0x040C, 0xFF, 0x00, CSIPHY_DNP_PARAMS},
+ {0x0410, 0x50, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x0438, 0x01, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x0460, 0x00, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x0464, 0x3F, 0x00, CSIPHY_DEFAULT_PARAMS},
+
+ {0x0630, 0x02, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x062C, 0x01, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x0634, 0x01, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x0628, 0x04, 0x00, CSIPHY_DNP_PARAMS},
+ {0x063C, 0xB8, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x061C, 0x0A, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x0600, 0xD7, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x0604, 0x08, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x0620, 0x00, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x0608, 0x04, 0x00, CSIPHY_SETTLE_CNT_LOWER_BYTE},
+ {0x060C, 0xFF, 0x00, CSIPHY_DNP_PARAMS},
+ {0x0610, 0x50, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x0638, 0x01, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x0660, 0x00, 0x00, CSIPHY_DEFAULT_PARAMS},
+ {0x0664, 0x3F, 0x00, CSIPHY_DEFAULT_PARAMS},
+};
+
/* GEN2 2.1.2 2PH DPHY mode */
static const struct
csiphy_lane_regs lane_regs_sm8550[] = {
@@ -744,6 +828,7 @@ static bool csiphy_is_gen2(u32 version)
bool ret = false;
switch (version) {
+ case CAMSS_2290:
case CAMSS_7280:
case CAMSS_8250:
case CAMSS_8280XP:
@@ -829,6 +914,10 @@ static int csiphy_init(struct csiphy_device *csiphy)
regs->lane_regs = &lane_regs_sdm845[0];
regs->lane_array_size = ARRAY_SIZE(lane_regs_sdm845);
break;
+ case CAMSS_2290:
+ regs->lane_regs = &lane_regs_qcm2290[0];
+ regs->lane_array_size = ARRAY_SIZE(lane_regs_qcm2290);
+ break;
case CAMSS_7280:
case CAMSS_8250:
regs->lane_regs = &lane_regs_sm8250[0];
diff --git a/drivers/media/platform/qcom/camss/camss.h b/drivers/media/platform/qcom/camss/camss.h
index 63c0afee154a..377707d91ff2 100644
--- a/drivers/media/platform/qcom/camss/camss.h
+++ b/drivers/media/platform/qcom/camss/camss.h
@@ -78,6 +78,7 @@ enum pm_domain {
enum camss_version {
CAMSS_660,
+ CAMSS_2290,
CAMSS_7280,
CAMSS_8x16,
CAMSS_8x53,
--
2.34.1
^ permalink raw reply related [flat|nested] 24+ messages in thread
* [PATCH 4/6] media: qcom: camss: add support for QCM2290 camss
2025-04-16 12:09 ` [PATCH 1/6] media: qcom: camss: Add support for TFE (Spectra 340) Loic Poulain
2025-04-16 12:09 ` [PATCH 2/6] media: qcom: camss: Add CSID 340 support Loic Poulain
2025-04-16 12:09 ` [PATCH 3/6] media: qcom: camss: csiphy-3ph: Add CSIPHY 2ph DPHY v2.0.1 init sequence Loic Poulain
@ 2025-04-16 12:09 ` Loic Poulain
2025-04-16 12:17 ` Bryan O'Donoghue
2025-04-16 12:09 ` [PATCH 5/6] media: dt-bindings: media: camss: Add qcom,qcm2290-camss binding Loic Poulain
` (5 subsequent siblings)
8 siblings, 1 reply; 24+ messages in thread
From: Loic Poulain @ 2025-04-16 12:09 UTC (permalink / raw)
To: bryan.odonoghue, rfoss, konradybcio, andersson, krzk+dt, robh
Cc: linux-arm-msm, linux-media, devicetree, Loic Poulain
The camera subsystem for QCM2290 which is based on Spectra 340.
Signed-off-by: Loic Poulain <loic.poulain@oss.qualcomm.com>
---
drivers/media/platform/qcom/camss/camss-vfe.c | 2 +
drivers/media/platform/qcom/camss/camss.c | 146 ++++++++++++++++++
2 files changed, 148 insertions(+)
diff --git a/drivers/media/platform/qcom/camss/camss-vfe.c b/drivers/media/platform/qcom/camss/camss-vfe.c
index 4bca6c3abaff..c575c9767492 100644
--- a/drivers/media/platform/qcom/camss/camss-vfe.c
+++ b/drivers/media/platform/qcom/camss/camss-vfe.c
@@ -340,6 +340,7 @@ static u32 vfe_src_pad_code(struct vfe_line *line, u32 sink_code,
}
break;
case CAMSS_660:
+ case CAMSS_2290:
case CAMSS_7280:
case CAMSS_8x96:
case CAMSS_8250:
@@ -1969,6 +1970,7 @@ static int vfe_bpl_align(struct vfe_device *vfe)
int ret = 8;
switch (vfe->camss->res->version) {
+ case CAMSS_2290:
case CAMSS_7280:
case CAMSS_8250:
case CAMSS_8280XP:
diff --git a/drivers/media/platform/qcom/camss/camss.c b/drivers/media/platform/qcom/camss/camss.c
index 06f42875702f..1e0eb2a650a3 100644
--- a/drivers/media/platform/qcom/camss/camss.c
+++ b/drivers/media/platform/qcom/camss/camss.c
@@ -515,6 +515,138 @@ static const struct camss_subdev_resources vfe_res_8x96[] = {
}
};
+static const struct camss_subdev_resources csiphy_res_2290[] = {
+ /* CSIPHY0 */
+ {
+ .regulators = { "vdda-phy", "vdda-pll" },
+ .clock = { "top_ahb", "ahb", "csiphy0", "csiphy0_timer" },
+ .clock_rate = { { 0 },
+ { 0 },
+ { 240000000, 341330000, 384000000 },
+ { 100000000, 200000000, 268800000 } },
+ .reg = { "csiphy0" },
+ .interrupt = { "csiphy0" },
+ .csiphy = {
+ .hw_ops = &csiphy_ops_3ph_1_0,
+ .formats = &csiphy_formats_sdm845
+ }
+ },
+
+ /* CSIPHY1 */
+ {
+ .regulators = { "vdda-phy", "vdda-pll" },
+ .clock = { "top_ahb", "ahb", "csiphy1", "csiphy1_timer" },
+ .clock_rate = { { 0 },
+ { 0 },
+ { 240000000, 341330000, 384000000 },
+ { 100000000, 200000000, 268800000 } },
+ .reg = { "csiphy1" },
+ .interrupt = { "csiphy1" },
+ .csiphy = {
+ .hw_ops = &csiphy_ops_3ph_1_0,
+ .formats = &csiphy_formats_sdm845
+ }
+ }
+};
+
+static const struct camss_subdev_resources csid_res_2290[] = {
+ /* CSID0 */
+ {
+ .regulators = {},
+ .clock = { "top_ahb", "ahb", "csi0", "vfe0_cphy_rx", "vfe0" },
+ .clock_rate = { { 0 },
+ { 0 },
+ { 192000000, 240000000, 384000000, 426400000 },
+ { 0 },
+ { 0 } },
+ .reg = { "csid0" },
+ .interrupt = { "csid0" },
+ .csid = {
+ .hw_ops = &csid_ops_340,
+ .parent_dev_ops = &vfe_parent_dev_ops,
+ .formats = &csid_formats_gen2
+ }
+ },
+
+ /* CSID1 */
+ {
+ .regulators = {},
+ .clock = { "top_ahb", "ahb", "csi1", "vfe1_cphy_rx", "vfe1" },
+ .clock_rate = { { 0 },
+ { 0 },
+ { 192000000, 240000000, 384000000, 426400000 },
+ { 0 },
+ { 0 } },
+ .reg = { "csid1" },
+ .interrupt = { "csid1" },
+ .csid = {
+ .hw_ops = &csid_ops_340,
+ .parent_dev_ops = &vfe_parent_dev_ops,
+ .formats = &csid_formats_gen2
+ }
+ }
+};
+
+static const struct camss_subdev_resources vfe_res_2290[] = {
+ /* VFE0 */
+ {
+ .regulators = {},
+ .clock = { "top_ahb", "ahb", "axi", "vfe0", "camnoc_rt_axi", "camnoc_nrt_axi" },
+ .clock_rate = { { 0 },
+ { 0 },
+ { 0 },
+ { 19200000, 153600000, 192000000, 256000000, 384000000, 460800000 },
+ { 0 },
+ { 0 }, },
+ .reg = { "vfe0" },
+ .interrupt = { "vfe0" },
+ .vfe = {
+ .line_num = 4,
+ .hw_ops = &vfe_ops_340,
+ .formats_rdi = &vfe_formats_rdi_845,
+ .formats_pix = &vfe_formats_pix_845
+ }
+ },
+
+ /* VFE1 */
+ {
+ .regulators = {},
+ .clock = { "top_ahb", "ahb", "axi", "vfe1", "camnoc_rt_axi", "camnoc_nrt_axi" },
+ .clock_rate = { { 0 },
+ { 0 },
+ { 0 },
+ { 19200000, 153600000, 192000000, 256000000, 384000000, 460800000 },
+ { 0 },
+ { 0 }, },
+ .reg = { "vfe1" },
+ .interrupt = { "vfe1" },
+ .vfe = {
+ .line_num = 4,
+ .hw_ops = &vfe_ops_340,
+ .formats_rdi = &vfe_formats_rdi_845,
+ .formats_pix = &vfe_formats_pix_845
+ }
+ },
+};
+
+static const struct resources_icc icc_res_2290[] = {
+ {
+ .name = "ahb",
+ .icc_bw_tbl.avg = 150000,
+ .icc_bw_tbl.peak = 300000,
+ },
+ {
+ .name = "hf_mnoc",
+ .icc_bw_tbl.avg = 2097152,
+ .icc_bw_tbl.peak = 2097152,
+ },
+ {
+ .name = "sf_mnoc",
+ .icc_bw_tbl.avg = 2097152,
+ .icc_bw_tbl.peak = 2097152,
+ },
+};
+
static const struct camss_subdev_resources csiphy_res_660[] = {
/* CSIPHY0 */
{
@@ -3753,6 +3885,19 @@ static const struct camss_resources msm8996_resources = {
.link_entities = camss_link_entities
};
+static const struct camss_resources qcm2290_resources = {
+ .version = CAMSS_2290,
+ .csiphy_res = csiphy_res_2290,
+ .csid_res = csid_res_2290,
+ .vfe_res = vfe_res_2290,
+ .icc_res = icc_res_2290,
+ .icc_path_num = ARRAY_SIZE(icc_res_2290),
+ .csiphy_num = ARRAY_SIZE(csiphy_res_2290),
+ .csid_num = ARRAY_SIZE(csid_res_2290),
+ .vfe_num = ARRAY_SIZE(vfe_res_2290),
+ .link_entities = camss_link_entities
+};
+
static const struct camss_resources sdm660_resources = {
.version = CAMSS_660,
.csiphy_res = csiphy_res_660,
@@ -3865,6 +4010,7 @@ static const struct of_device_id camss_dt_match[] = {
{ .compatible = "qcom,msm8916-camss", .data = &msm8916_resources },
{ .compatible = "qcom,msm8953-camss", .data = &msm8953_resources },
{ .compatible = "qcom,msm8996-camss", .data = &msm8996_resources },
+ { .compatible = "qcom,qcm2290-camss", .data = &qcm2290_resources },
{ .compatible = "qcom,sc7280-camss", .data = &sc7280_resources },
{ .compatible = "qcom,sc8280xp-camss", .data = &sc8280xp_resources },
{ .compatible = "qcom,sdm660-camss", .data = &sdm660_resources },
--
2.34.1
^ permalink raw reply related [flat|nested] 24+ messages in thread
* [PATCH 5/6] media: dt-bindings: media: camss: Add qcom,qcm2290-camss binding
2025-04-16 12:09 ` [PATCH 1/6] media: qcom: camss: Add support for TFE (Spectra 340) Loic Poulain
` (2 preceding siblings ...)
2025-04-16 12:09 ` [PATCH 4/6] media: qcom: camss: add support for QCM2290 camss Loic Poulain
@ 2025-04-16 12:09 ` Loic Poulain
2025-04-16 12:09 ` [PATCH 6/6] arm64: dts: qcom: qcm2290: Add CAMSS node Loic Poulain
` (4 subsequent siblings)
8 siblings, 0 replies; 24+ messages in thread
From: Loic Poulain @ 2025-04-16 12:09 UTC (permalink / raw)
To: bryan.odonoghue, rfoss, konradybcio, andersson, krzk+dt, robh
Cc: linux-arm-msm, linux-media, devicetree, Loic Poulain
Add bindings for qcom,qcm2290-camss in order to support the camera
subsystem found in the Qualcomm Robotics RB1 Platform (QRB2210).
Signed-off-by: Loic Poulain <loic.poulain@oss.qualcomm.com>
---
.../bindings/media/qcom,qcm2290-camss.yaml | 246 ++++++++++++++++++
1 file changed, 246 insertions(+)
create mode 100644 Documentation/devicetree/bindings/media/qcom,qcm2290-camss.yaml
diff --git a/Documentation/devicetree/bindings/media/qcom,qcm2290-camss.yaml b/Documentation/devicetree/bindings/media/qcom,qcm2290-camss.yaml
new file mode 100644
index 000000000000..dc53f445a90b
--- /dev/null
+++ b/Documentation/devicetree/bindings/media/qcom,qcm2290-camss.yaml
@@ -0,0 +1,246 @@
+# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/media/qcom,qcm2290-camss.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Qualcomm QCM2290 Camera Subsystem (CAMSS)
+
+maintainers:
+ - Loic Poulain <loic.poulain@oss.qualcomm.com>
+
+description:
+ The CAMSS IP is a CSI decoder and ISP present on Qualcomm platforms.
+
+properties:
+ compatible:
+ const: qcom,qcm2290-camss
+
+ reg:
+ maxItems: 6
+
+ reg-names:
+ items:
+ - const: csid0
+ - const: csid1
+ - const: csiphy0
+ - const: csiphy1
+ - const: vfe0
+ - const: vfe1
+
+ clocks:
+ maxItems: 15
+
+ clock-names:
+ items:
+ - const: ahb
+ - const: axi
+ - const: top_ahb
+ - const: csi0
+ - const: csi1
+ - const: csiphy0
+ - const: csiphy1
+ - const: csiphy0_timer
+ - const: csiphy1_timer
+ - const: vfe0
+ - const: vfe1
+ - const: vfe0_cphy_rx
+ - const: vfe1_cphy_rx
+ - const: camnoc_nrt_axi
+ - const: camnoc_rt_axi
+
+ interrupts:
+ maxItems: 6
+
+ interrupt-names:
+ items:
+ - const: csid0
+ - const: csid1
+ - const: csiphy0
+ - const: csiphy1
+ - const: vfe0
+ - const: vfe1
+
+ interconnects:
+ maxItems: 3
+
+ interconnect-names:
+ items:
+ - const: ahb
+ - const: hf_mnoc
+ - const: sf_mnoc
+
+ iommus:
+ maxItems: 4
+
+ power-domains:
+ items:
+ - description: GDSC CAMSS Block, Global Distributed Switch Controller.
+
+ vdda-phy-supply:
+ description:
+ Phandle to a 1.2V regulator supply to CSI PHYs.
+
+ vdda-pll-supply:
+ description:
+ Phandle to 1.8V regulator supply to CAMSS refclk pll block.
+
+ ports:
+ $ref: /schemas/graph.yaml#/properties/ports
+
+ description:
+ CSI input ports.
+
+ properties:
+ port@0:
+ $ref: /schemas/graph.yaml#/$defs/port-base
+ unevaluatedProperties: false
+ description:
+ Input port for receiving CSI data on CSIPHY 0.
+
+ properties:
+ endpoint:
+ $ref: video-interfaces.yaml#
+ unevaluatedProperties: false
+
+ properties:
+ data-lanes:
+ minItems: 1
+ maxItems: 4
+
+ required:
+ - data-lanes
+
+ port@1:
+ $ref: /schemas/graph.yaml#/$defs/port-base
+ unevaluatedProperties: false
+ description:
+ Input port for receiving CSI data on CSIPHY 1.
+
+ properties:
+ endpoint:
+ $ref: video-interfaces.yaml#
+ unevaluatedProperties: false
+
+ properties:
+ data-lanes:
+ minItems: 1
+ maxItems: 4
+
+ required:
+ - data-lanes
+
+required:
+ - compatible
+ - reg
+ - reg-names
+ - clocks
+ - clock-names
+ - interrupts
+ - interrupt-names
+ - interconnects
+ - interconnect-names
+ - iommus
+ - power-domains
+ - vdda-phy-supply
+ - vdda-pll-supply
+
+additionalProperties: false
+
+examples:
+ - |
+ #include <dt-bindings/clock/qcom,gcc-qcm2290.h>
+ #include <dt-bindings/interconnect/qcom,rpm-icc.h>
+ #include <dt-bindings/interconnect/qcom,qcm2290.h>
+ #include <dt-bindings/interrupt-controller/arm-gic.h>
+
+ soc {
+ #address-cells = <2>;
+ #size-cells = <2>;
+
+ camss: camss@5c52000 {
+ compatible = "qcom,qcm2290-camss";
+
+ reg = <0 0x5c52000 0 0x1000>,
+ <0 0x5c53000 0 0x1000>,
+ <0 0x5c6e000 0 0x1000>,
+ <0 0x5c75000 0 0x1000>,
+ <0 0x5c6f000 0 0x4000>,
+ <0 0x5c76000 0 0x4000>;
+ reg-names = "csid0",
+ "csid1",
+ "csiphy0",
+ "csiphy1",
+ "vfe0",
+ "vfe1";
+
+ interrupts = <GIC_SPI 210 IRQ_TYPE_EDGE_RISING>,
+ <GIC_SPI 212 IRQ_TYPE_EDGE_RISING>,
+ <GIC_SPI 72 IRQ_TYPE_EDGE_RISING>,
+ <GIC_SPI 73 IRQ_TYPE_EDGE_RISING>,
+ <GIC_SPI 211 IRQ_TYPE_EDGE_RISING>,
+ <GIC_SPI 213 IRQ_TYPE_EDGE_RISING>;
+ interrupt-names = "csid0",
+ "csid1",
+ "csiphy0",
+ "csiphy1",
+ "vfe0",
+ "vfe1";
+
+ power-domains = <&gcc GCC_CAMSS_TOP_GDSC>;
+
+ clocks = <&gcc GCC_CAMERA_AHB_CLK>,
+ <&gcc GCC_CAMSS_AXI_CLK>,
+ <&gcc GCC_CAMSS_TOP_AHB_CLK>,
+ <&gcc GCC_CAMSS_TFE_0_CSID_CLK>,
+ <&gcc GCC_CAMSS_TFE_1_CSID_CLK>,
+ <&gcc GCC_CAMSS_CPHY_0_CLK>,
+ <&gcc GCC_CAMSS_CPHY_1_CLK>,
+ <&gcc GCC_CAMSS_CSI0PHYTIMER_CLK>,
+ <&gcc GCC_CAMSS_CSI1PHYTIMER_CLK>,
+ <&gcc GCC_CAMSS_TFE_0_CLK>,
+ <&gcc GCC_CAMSS_TFE_1_CLK>,
+ <&gcc GCC_CAMSS_TFE_0_CPHY_RX_CLK>,
+ <&gcc GCC_CAMSS_TFE_1_CPHY_RX_CLK>,
+ <&gcc GCC_CAMSS_NRT_AXI_CLK>,
+ <&gcc GCC_CAMSS_RT_AXI_CLK>;
+ clock-names = "ahb",
+ "axi",
+ "top_ahb",
+ "csi0",
+ "csi1",
+ "csiphy0",
+ "csiphy1",
+ "csiphy0_timer",
+ "csiphy1_timer",
+ "vfe0",
+ "vfe1",
+ "vfe0_cphy_rx",
+ "vfe1_cphy_rx",
+ "camnoc_nrt_axi",
+ "camnoc_rt_axi";
+
+ iommus = <&apps_smmu 0x400 0x0>,
+ <&apps_smmu 0x800 0x0>,
+ <&apps_smmu 0x820 0x0>,
+ <&apps_smmu 0x840 0x0>;
+
+ interconnects = <&bimc MASTER_APPSS_PROC RPM_ALWAYS_TAG
+ &config_noc SLAVE_CAMERA_CFG RPM_ALWAYS_TAG>,
+ <&mmrt_virt MASTER_CAMNOC_HF RPM_ALWAYS_TAG
+ &bimc SLAVE_EBI1 RPM_ALWAYS_TAG>,
+ <&mmnrt_virt MASTER_CAMNOC_SF RPM_ALWAYS_TAG
+ &bimc SLAVE_EBI1 RPM_ALWAYS_TAG>;
+ interconnect-names = "ahb",
+ "hf_mnoc",
+ "sf_mnoc";
+
+ vdda-phy-supply = <&pm4125_l5>;
+ vdda-pll-supply = <&pm4125_l13>;
+
+ ports {
+ #address-cells = <1>;
+ #size-cells = <0>;
+ };
+ };
+ };
--
2.34.1
^ permalink raw reply related [flat|nested] 24+ messages in thread
* [PATCH 6/6] arm64: dts: qcom: qcm2290: Add CAMSS node
2025-04-16 12:09 ` [PATCH 1/6] media: qcom: camss: Add support for TFE (Spectra 340) Loic Poulain
` (3 preceding siblings ...)
2025-04-16 12:09 ` [PATCH 5/6] media: dt-bindings: media: camss: Add qcom,qcm2290-camss binding Loic Poulain
@ 2025-04-16 12:09 ` Loic Poulain
2025-04-16 14:46 ` Konrad Dybcio
2025-04-16 12:27 ` [PATCH 1/6] media: qcom: camss: Add support for TFE (Spectra 340) Bryan O'Donoghue
` (3 subsequent siblings)
8 siblings, 1 reply; 24+ messages in thread
From: Loic Poulain @ 2025-04-16 12:09 UTC (permalink / raw)
To: bryan.odonoghue, rfoss, konradybcio, andersson, krzk+dt, robh
Cc: linux-arm-msm, linux-media, devicetree, Loic Poulain
Add node for the QCM2290 camera subsystem.
Signed-off-by: Loic Poulain <loic.poulain@oss.qualcomm.com>
---
arch/arm64/boot/dts/qcom/qcm2290.dtsi | 93 +++++++++++++++++++++++++++
1 file changed, 93 insertions(+)
diff --git a/arch/arm64/boot/dts/qcom/qcm2290.dtsi b/arch/arm64/boot/dts/qcom/qcm2290.dtsi
index f0746123e594..4b81e721e50c 100644
--- a/arch/arm64/boot/dts/qcom/qcm2290.dtsi
+++ b/arch/arm64/boot/dts/qcom/qcm2290.dtsi
@@ -1579,6 +1579,99 @@ adreno_smmu: iommu@59a0000 {
#iommu-cells = <2>;
};
+ camss: camss@5c52000 {
+ compatible = "qcom,qcm2290-camss";
+
+ reg = <0 0x5c6e000 0 0x1000>,
+ <0 0x5c75000 0 0x1000>,
+ <0 0x5c52000 0 0x1000>,
+ <0 0x5c53000 0 0x1000>,
+ <0 0x5c6f000 0 0x4000>,
+ <0 0x5c76000 0 0x4000>;
+ reg-names = "csid0",
+ "csid1",
+ "csiphy0",
+ "csiphy1",
+ "vfe0",
+ "vfe1";
+
+ interrupts = <GIC_SPI 210 IRQ_TYPE_EDGE_RISING>,
+ <GIC_SPI 212 IRQ_TYPE_EDGE_RISING>,
+ <GIC_SPI 72 IRQ_TYPE_EDGE_RISING>,
+ <GIC_SPI 73 IRQ_TYPE_EDGE_RISING>,
+ <GIC_SPI 211 IRQ_TYPE_EDGE_RISING>,
+ <GIC_SPI 213 IRQ_TYPE_EDGE_RISING>;
+ interrupt-names = "csid0",
+ "csid1",
+ "csiphy0",
+ "csiphy1",
+ "vfe0",
+ "vfe1";
+
+ power-domains = <&gcc GCC_CAMSS_TOP_GDSC>;
+
+ clocks = <&gcc GCC_CAMERA_AHB_CLK>,
+ <&gcc GCC_CAMSS_AXI_CLK>,
+ <&gcc GCC_CAMSS_TOP_AHB_CLK>,
+ <&gcc GCC_CAMSS_TFE_0_CSID_CLK>,
+ <&gcc GCC_CAMSS_TFE_1_CSID_CLK>,
+ <&gcc GCC_CAMSS_CPHY_0_CLK>,
+ <&gcc GCC_CAMSS_CPHY_1_CLK>,
+ <&gcc GCC_CAMSS_CSI0PHYTIMER_CLK>,
+ <&gcc GCC_CAMSS_CSI1PHYTIMER_CLK>,
+ <&gcc GCC_CAMSS_TFE_0_CLK>,
+ <&gcc GCC_CAMSS_TFE_1_CLK>,
+ <&gcc GCC_CAMSS_TFE_0_CPHY_RX_CLK>,
+ <&gcc GCC_CAMSS_TFE_1_CPHY_RX_CLK>,
+ <&gcc GCC_CAMSS_NRT_AXI_CLK>,
+ <&gcc GCC_CAMSS_RT_AXI_CLK>;
+ clock-names = "ahb",
+ "axi",
+ "top_ahb",
+ "csi0",
+ "csi1",
+ "csiphy0",
+ "csiphy1",
+ "csiphy0_timer",
+ "csiphy1_timer",
+ "vfe0",
+ "vfe1",
+ "vfe0_cphy_rx",
+ "vfe1_cphy_rx",
+ "camnoc_nrt_axi",
+ "camnoc_rt_axi";
+
+ iommus = <&apps_smmu 0x400 0x0>,
+ <&apps_smmu 0x800 0x0>,
+ <&apps_smmu 0x820 0x0>,
+ <&apps_smmu 0x840 0x0>;
+
+ interconnects = <&bimc MASTER_APPSS_PROC RPM_ALWAYS_TAG
+ &config_noc SLAVE_CAMERA_CFG RPM_ALWAYS_TAG>,
+ <&mmrt_virt MASTER_CAMNOC_HF RPM_ALWAYS_TAG
+ &bimc SLAVE_EBI1 RPM_ALWAYS_TAG>,
+ <&mmnrt_virt MASTER_CAMNOC_SF RPM_ALWAYS_TAG
+ &bimc SLAVE_EBI1 RPM_ALWAYS_TAG>;
+ interconnect-names = "ahb",
+ "hf_mnoc",
+ "sf_mnoc";
+
+ status = "disabled";
+
+ ports {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ port@0 {
+ reg = <0>;
+ };
+
+ port@1 {
+ reg = <1>;
+ };
+ };
+ };
+
mdss: display-subsystem@5e00000 {
compatible = "qcom,qcm2290-mdss";
reg = <0x0 0x05e00000 0x0 0x1000>;
--
2.34.1
^ permalink raw reply related [flat|nested] 24+ messages in thread
* Re: [PATCH 4/6] media: qcom: camss: add support for QCM2290 camss
2025-04-16 12:09 ` [PATCH 4/6] media: qcom: camss: add support for QCM2290 camss Loic Poulain
@ 2025-04-16 12:17 ` Bryan O'Donoghue
2025-04-16 15:16 ` Loic Poulain
0 siblings, 1 reply; 24+ messages in thread
From: Bryan O'Donoghue @ 2025-04-16 12:17 UTC (permalink / raw)
To: Loic Poulain, rfoss, konradybcio, andersson, krzk+dt, robh
Cc: linux-arm-msm, linux-media, devicetree
On 16/04/2025 13:09, Loic Poulain wrote:
> The camera subsystem for QCM2290 which is based on Spectra 340.
>
> Signed-off-by: Loic Poulain <loic.poulain@oss.qualcomm.com>
> ---
> drivers/media/platform/qcom/camss/camss-vfe.c | 2 +
> drivers/media/platform/qcom/camss/camss.c | 146 ++++++++++++++++++
> 2 files changed, 148 insertions(+)
>
> diff --git a/drivers/media/platform/qcom/camss/camss-vfe.c b/drivers/media/platform/qcom/camss/camss-vfe.c
> index 4bca6c3abaff..c575c9767492 100644
> --- a/drivers/media/platform/qcom/camss/camss-vfe.c
> +++ b/drivers/media/platform/qcom/camss/camss-vfe.c
> @@ -340,6 +340,7 @@ static u32 vfe_src_pad_code(struct vfe_line *line, u32 sink_code,
> }
> break;
> case CAMSS_660:
> + case CAMSS_2290:
> case CAMSS_7280:
> case CAMSS_8x96:
> case CAMSS_8250:
> @@ -1969,6 +1970,7 @@ static int vfe_bpl_align(struct vfe_device *vfe)
> int ret = 8;
>
> switch (vfe->camss->res->version) {
> + case CAMSS_2290:
> case CAMSS_7280:
> case CAMSS_8250:
> case CAMSS_8280XP:
> diff --git a/drivers/media/platform/qcom/camss/camss.c b/drivers/media/platform/qcom/camss/camss.c
> index 06f42875702f..1e0eb2a650a3 100644
> --- a/drivers/media/platform/qcom/camss/camss.c
> +++ b/drivers/media/platform/qcom/camss/camss.c
> @@ -515,6 +515,138 @@ static const struct camss_subdev_resources vfe_res_8x96[] = {
> }
> };
>
> +static const struct camss_subdev_resources csiphy_res_2290[] = {
> + /* CSIPHY0 */
> + {
> + .regulators = { "vdda-phy", "vdda-pll" },
> + .clock = { "top_ahb", "ahb", "csiphy0", "csiphy0_timer" },
> + .clock_rate = { { 0 },
> + { 0 },
> + { 240000000, 341330000, 384000000 },
> + { 100000000, 200000000, 268800000 } },
> + .reg = { "csiphy0" },
> + .interrupt = { "csiphy0" },
> + .csiphy = {
> + .hw_ops = &csiphy_ops_3ph_1_0,
> + .formats = &csiphy_formats_sdm845
You'll need to specify and .id property
https://gitlab.freedesktop.org/linux-media/media-committers/-/commit/74cae7794341fa4f96fef0966f44471f7e2d322f
> + }
> + },
> +
> + /* CSIPHY1 */
> + {
> + .regulators = { "vdda-phy", "vdda-pll" },
> + .clock = { "top_ahb", "ahb", "csiphy1", "csiphy1_timer" },
> + .clock_rate = { { 0 },
> + { 0 },
> + { 240000000, 341330000, 384000000 },
> + { 100000000, 200000000, 268800000 } },
> + .reg = { "csiphy1" },
> + .interrupt = { "csiphy1" },
> + .csiphy = {
> + .hw_ops = &csiphy_ops_3ph_1_0,
> + .formats = &csiphy_formats_sdm845
> + }
> + }
> +};
> +
> +static const struct camss_subdev_resources csid_res_2290[] = {
> + /* CSID0 */
> + {
> + .regulators = {},
> + .clock = { "top_ahb", "ahb", "csi0", "vfe0_cphy_rx", "vfe0" },
> + .clock_rate = { { 0 },
> + { 0 },
> + { 192000000, 240000000, 384000000, 426400000 },
> + { 0 },
> + { 0 } },
> + .reg = { "csid0" },
> + .interrupt = { "csid0" },
> + .csid = {
> + .hw_ops = &csid_ops_340,
> + .parent_dev_ops = &vfe_parent_dev_ops,
> + .formats = &csid_formats_gen2
> + }
> + },
> +
> + /* CSID1 */
> + {
> + .regulators = {},
> + .clock = { "top_ahb", "ahb", "csi1", "vfe1_cphy_rx", "vfe1" },
> + .clock_rate = { { 0 },
> + { 0 },
> + { 192000000, 240000000, 384000000, 426400000 },
> + { 0 },
> + { 0 } },
> + .reg = { "csid1" },
> + .interrupt = { "csid1" },
> + .csid = {
> + .hw_ops = &csid_ops_340,
> + .parent_dev_ops = &vfe_parent_dev_ops,
> + .formats = &csid_formats_gen2
> + }
> + }
> +};
> +
> +static const struct camss_subdev_resources vfe_res_2290[] = {
> + /* VFE0 */
> + {
> + .regulators = {},
> + .clock = { "top_ahb", "ahb", "axi", "vfe0", "camnoc_rt_axi", "camnoc_nrt_axi" },
> + .clock_rate = { { 0 },
> + { 0 },
> + { 0 },
> + { 19200000, 153600000, 192000000, 256000000, 384000000, 460800000 },
> + { 0 },
> + { 0 }, },
> + .reg = { "vfe0" },
> + .interrupt = { "vfe0" },
> + .vfe = {
> + .line_num = 4,
> + .hw_ops = &vfe_ops_340,
> + .formats_rdi = &vfe_formats_rdi_845,
> + .formats_pix = &vfe_formats_pix_845
> + }
> + },
> +
> + /* VFE1 */
> + {
> + .regulators = {},
> + .clock = { "top_ahb", "ahb", "axi", "vfe1", "camnoc_rt_axi", "camnoc_nrt_axi" },
> + .clock_rate = { { 0 },
> + { 0 },
> + { 0 },
> + { 19200000, 153600000, 192000000, 256000000, 384000000, 460800000 },
> + { 0 },
> + { 0 }, },
> + .reg = { "vfe1" },
> + .interrupt = { "vfe1" },
> + .vfe = {
> + .line_num = 4,
> + .hw_ops = &vfe_ops_340,
> + .formats_rdi = &vfe_formats_rdi_845,
> + .formats_pix = &vfe_formats_pix_845
> + }
> + },
> +};
> +
> +static const struct resources_icc icc_res_2290[] = {
> + {
> + .name = "ahb",
> + .icc_bw_tbl.avg = 150000,
> + .icc_bw_tbl.peak = 300000,
> + },
> + {
> + .name = "hf_mnoc",
> + .icc_bw_tbl.avg = 2097152,
> + .icc_bw_tbl.peak = 2097152,
> + },
> + {
> + .name = "sf_mnoc",
> + .icc_bw_tbl.avg = 2097152,
> + .icc_bw_tbl.peak = 2097152,
> + },
> +};
I think you can get better numbers from downstream for the above.
> +
> static const struct camss_subdev_resources csiphy_res_660[] = {
> /* CSIPHY0 */
> {
> @@ -3753,6 +3885,19 @@ static const struct camss_resources msm8996_resources = {
> .link_entities = camss_link_entities
> };
>
> +static const struct camss_resources qcm2290_resources = {
> + .version = CAMSS_2290,
> + .csiphy_res = csiphy_res_2290,
> + .csid_res = csid_res_2290,
> + .vfe_res = vfe_res_2290,
> + .icc_res = icc_res_2290,
> + .icc_path_num = ARRAY_SIZE(icc_res_2290),
> + .csiphy_num = ARRAY_SIZE(csiphy_res_2290),
> + .csid_num = ARRAY_SIZE(csid_res_2290),
> + .vfe_num = ARRAY_SIZE(vfe_res_2290),
> + .link_entities = camss_link_entities
> +};
> +
> static const struct camss_resources sdm660_resources = {
> .version = CAMSS_660,
> .csiphy_res = csiphy_res_660,
> @@ -3865,6 +4010,7 @@ static const struct of_device_id camss_dt_match[] = {
> { .compatible = "qcom,msm8916-camss", .data = &msm8916_resources },
> { .compatible = "qcom,msm8953-camss", .data = &msm8953_resources },
> { .compatible = "qcom,msm8996-camss", .data = &msm8996_resources },
> + { .compatible = "qcom,qcm2290-camss", .data = &qcm2290_resources },
> { .compatible = "qcom,sc7280-camss", .data = &sc7280_resources },
> { .compatible = "qcom,sc8280xp-camss", .data = &sc8280xp_resources },
> { .compatible = "qcom,sdm660-camss", .data = &sdm660_resources },
Other than that LGTM.
---
bod
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 1/6] media: qcom: camss: Add support for TFE (Spectra 340)
2025-04-16 12:09 ` [PATCH 1/6] media: qcom: camss: Add support for TFE (Spectra 340) Loic Poulain
` (4 preceding siblings ...)
2025-04-16 12:09 ` [PATCH 6/6] arm64: dts: qcom: qcm2290: Add CAMSS node Loic Poulain
@ 2025-04-16 12:27 ` Bryan O'Donoghue
2025-04-16 12:30 ` Bryan O'Donoghue
` (2 subsequent siblings)
8 siblings, 0 replies; 24+ messages in thread
From: Bryan O'Donoghue @ 2025-04-16 12:27 UTC (permalink / raw)
To: Loic Poulain, rfoss, konradybcio, andersson, krzk+dt, robh
Cc: linux-arm-msm, linux-media, devicetree
On 16/04/2025 13:09, Loic Poulain wrote:
> Add support for TFE (Thin Front End) found in QCM2290.
>
> Signed-off-by: Loic Poulain <loic.poulain@oss.qualcomm.com>
> ---
> drivers/media/platform/qcom/camss/Makefile | 1 +
> .../media/platform/qcom/camss/camss-vfe-340.c | 281 ++++++++++++++++++
> drivers/media/platform/qcom/camss/camss-vfe.h | 1 +
> 3 files changed, 283 insertions(+)
> create mode 100644 drivers/media/platform/qcom/camss/camss-vfe-340.c
>
> diff --git a/drivers/media/platform/qcom/camss/Makefile b/drivers/media/platform/qcom/camss/Makefile
> index d26a9c24a430..719898f5d32b 100644
> --- a/drivers/media/platform/qcom/camss/Makefile
> +++ b/drivers/media/platform/qcom/camss/Makefile
> @@ -17,6 +17,7 @@ qcom-camss-objs += \
> camss-vfe-4-7.o \
> camss-vfe-4-8.o \
> camss-vfe-17x.o \
> + camss-vfe-340.o \
> camss-vfe-480.o \
> camss-vfe-680.o \
> camss-vfe-780.o \
> diff --git a/drivers/media/platform/qcom/camss/camss-vfe-340.c b/drivers/media/platform/qcom/camss/camss-vfe-340.c
> new file mode 100644
> index 000000000000..fc454d66e1d2
> --- /dev/null
> +++ b/drivers/media/platform/qcom/camss/camss-vfe-340.c
> @@ -0,0 +1,281 @@
> +// SPDX-License-Identifier: GPL-2.0
> +/*
> + * Qualcomm MSM Camera Subsystem - VFE (Video Front End) Module 340 (TFE)
> + *
> + * Copyright (c) 2025 Qualcomm Technologies, Inc.
> + */
> +
> +#include <linux/delay.h>
> +#include <linux/interrupt.h>
> +#include <linux/io.h>
> +#include <linux/iopoll.h>
> +
> +#include "camss.h"
> +#include "camss-vfe.h"
> +
> +#define TFE_GLOBAL_RESET_CMD (0x014)
> +#define TFE_GLOBAL_RESET_CMD_CORE BIT(0)
> +
> +#define TFE_REG_UPDATE_CMD (0x02C)
> +
> +#define TFE_IRQ_CMD (0x030)
> +#define TFE_IRQ_CMD_CLEAR BIT(0)
> +#define TFE_IRQ_MASK_0 (0x034)
> +#define TFE_IRQ_MASK_0_RST_DONE BIT(0)
> +#define TFE_IRQ_MASK_0_BUS_WR BIT(1)
> +#define TFE_IRQ_MASK_1 (0x038)
> +#define TFE_IRQ_MASK_2 (0x03c)
> +#define TFE_IRQ_CLEAR_0 (0x040)
> +
> +#define TFE_IRQ_STATUS_0 (0x04c)
> +
> +#define BUS_REG(a) (0xa00 + (a))
> +
> +#define TFE_BUS_IRQ_MASK_0 BUS_REG(0x18)
> +#define TFE_BUS_IRQ_MASK_RUP_DONE_ALL 0x000f
> +#define TFE_BUS_IRQ_MASK_RUP_DONE(src) BIT(src)
> +#define TFE_BUS_IRQ_MASK_BUF_DONE_ALL 0xff00
> +#define TFE_BUS_IRQ_MASK_BUF_DONE(sg) BIT((sg) + 8)
> +#define TFE_BUS_IRQ_MASK_0_CONS_VIOL BIT(28)
> +#define TFE_BUS_IRQ_MASK_0_VIOL BIT(30)
> +#define TFE_BUS_IRQ_MASK_0_IMG_VIOL BIT(31)
> +
> +#define TFE_BUS_IRQ_MASK_1 BUS_REG(0x1C)
> +#define TFE_BUS_IRQ_CLEAR_0 BUS_REG(0x20)
> +#define TFE_BUS_IRQ_STATUS_0 BUS_REG(0x28)
> +#define TFE_BUS_IRQ_CMD BUS_REG(0x30)
> +#define TFE_BUS_IRQ_CMD_CLEAR BIT(0)
> +
> +#define TFE_BUS_STATUS_CLEAR BUS_REG(0x60)
> +#define TFE_BUS_VIOLATION_STATUS BUS_REG(0x64)
> +#define TFE_BUS_OVERFLOW_STATUS BUS_REG(0x68)
> +#define TFE_BUS_IMAGE_SZ_VIOLATION_STATUS BUS_REG(0x70)
> +
> +#define TFE_BUS_CLIENT_CFG(c) BUS_REG(0x200 + (c) * 0x100)
> +#define TFE_BUS_CLIENT_CFG_EN BIT(0)
> +#define TFE_BUS_CLIENT_CFG_MODE_FRAME BIT(16)
> +#define TFE_BUS_IMAGE_ADDR(c) BUS_REG(0x204 + (c) * 0x100)
> +#define TFE_BUS_FRAME_INCR(c) BUS_REG(0x208 + (c) * 0x100)
> +#define TFE_BUS_IMAGE_CFG_0(c) BUS_REG(0x20C + (c) * 0x100)
> +#define TFE_BUS_IMAGE_CFG_0_DEFAULT 0xffff
> +#define TFE_BUS_IMAGE_CFG_1(c) BUS_REG(0x210 + (c) * 0x100)
> +#define TFE_BUS_IMAGE_CFG_2(c) BUS_REG(0x214 + (c) * 0x100)
> +#define TFE_BUS_IMAGE_CFG_2_DEFAULT 0xffff
> +#define TFE_BUS_PACKER_CFG(c) BUS_REG(0x218 + (c) * 0x100)
> +#define TFE_BUS_PACKER_CFG_FMT_PLAIN64 0xa
> +#define TFE_BUS_IRQ_SUBSAMPLE_CFG_0(c) BUS_REG(0x230 + (c) * 0x100)
> +#define TFE_BUS_IRQ_SUBSAMPLE_CFG_1(c) BUS_REG(0x234 + (c) * 0x100)
> +#define TFE_BUS_FRAMEDROP_CFG_0(c) BUS_REG(0x238 + (c) * 0x100)
> +#define TFE_BUS_FRAMEDROP_CFG_1(c) BUS_REG(0x23c + (c) * 0x100)
> +
> +#define RDI_CLIENT(n) (7 + (n))
Should be RDI_WM for consistency.
Please add a comment listing what those clients are
https://gitlab.freedesktop.org/linux-media/media-committers/-/commit/727970e9725c35366f501a890584e466ae34224a#c0063099c8260d8c4b3dc9ea650b6d0a4db47518_0_59
> +#define TFE_SOURCES_NUM 4
> +#define TFE_SUBGROUPS_NUM 8
> +#define TFE_CLIENTS_NUM 10
> +
> +static inline unsigned int __regupdate_iface(enum vfe_line_id line_id)
> +{
> + switch (line_id) {
> + case VFE_LINE_RDI0:
> + return 1;
> + case VFE_LINE_RDI1:
> + return 2;
> + case VFE_LINE_RDI2:
> + return 3;
> + case VFE_LINE_PIX:
> + default:
> + return 0;
> + }
An enum or a define instead of hard-coded values on the return please.
> +}
> +
> +static inline unsigned int __regupdate_line(unsigned int iface)
> +{
> + if (!iface)
> + return VFE_LINE_PIX;
> + if (iface < 4)
> + return VFE_LINE_RDI0 + (iface - 1);
> +
> + return VFE_LINE_NONE;
> +}
> +
> +static inline unsigned int __subgroup_line(unsigned int subgroup)
> +{
> + switch (subgroup) {
> + case 5:
> + return VFE_LINE_RDI0;
> + case 6:
> + return VFE_LINE_RDI1;
> + case 7:
> + return VFE_LINE_RDI2;
> + default:
> + return VFE_LINE_PIX;
> + }
> +
> + return VFE_LINE_NONE;
> +}
> +
> +static void vfe_global_reset(struct vfe_device *vfe)
> +{
> + writel_relaxed(TFE_IRQ_MASK_0_RST_DONE, vfe->base + TFE_IRQ_MASK_0);
> + writel(TFE_GLOBAL_RESET_CMD_CORE, vfe->base + TFE_GLOBAL_RESET_CMD);
Why is one relaxed and one not ?
> +}
> +
> +static irqreturn_t vfe_isr(int irq, void *dev)
> +{
> + struct vfe_device *vfe = dev;
> + u32 status;
> + int i;
> +
> + status = readl_relaxed(vfe->base + TFE_IRQ_STATUS_0);
> + writel_relaxed(status, vfe->base + TFE_IRQ_CLEAR_0);
> + writel_relaxed(TFE_IRQ_CMD_CLEAR, vfe->base + TFE_IRQ_CMD);
> +
> +
> + if (status & TFE_IRQ_MASK_0_RST_DONE) {
> + dev_dbg(vfe->camss->dev, "VFE%u: Reset done!", vfe->id);
> + vfe_isr_reset_ack(vfe);
> + }
> +
> + if (status & TFE_IRQ_MASK_0_BUS_WR) {
> + u32 bus_status = readl_relaxed(vfe->base + TFE_BUS_IRQ_STATUS_0);
> +
> + writel_relaxed(bus_status, vfe->base + TFE_BUS_IRQ_CLEAR_0);
> + writel_relaxed(TFE_BUS_IRQ_CMD_CLEAR, vfe->base + TFE_BUS_IRQ_CMD);
> +
> + for (i = 0; i < TFE_SOURCES_NUM; i++) {
> + if (bus_status & TFE_BUS_IRQ_MASK_RUP_DONE(i))
> + vfe->res->hw_ops->reg_update_clear(vfe, __regupdate_line(i));
> + }
> +
> + for (i = 0; i < TFE_SUBGROUPS_NUM; i++) {
> + if (bus_status & TFE_BUS_IRQ_MASK_BUF_DONE(i))
> + vfe_buf_done(vfe, __subgroup_line(i));
> + }
> +
> + if (bus_status & TFE_BUS_IRQ_MASK_0_CONS_VIOL)
> + dev_err_ratelimited(vfe->camss->dev, "VFE%u: Bad config violation",
> + vfe->id);
> +
> + if (bus_status & TFE_BUS_IRQ_MASK_0_VIOL)
> + dev_err_ratelimited(vfe->camss->dev, "VFE%u: Input data violation",
> + vfe->id);
> +
> + if (bus_status & TFE_BUS_IRQ_MASK_0_IMG_VIOL)
> + dev_err_ratelimited(vfe->camss->dev, "VFE%u: Image size violation",
> + vfe->id);
> + }
> +
> + status = readl_relaxed(vfe->base + TFE_BUS_OVERFLOW_STATUS);
> + if (status) {
> + writel_relaxed(status, vfe->base + TFE_BUS_STATUS_CLEAR);
> + for (i = 0; i < TFE_CLIENTS_NUM; i++) {
> + if (status & BIT(i))
> + dev_err_ratelimited(vfe->camss->dev,
> + "VFE%u: bus overflow for client %u\n",
> + vfe->id, i);
> + }
> + }
> +
> + return IRQ_HANDLED;
> +}
looks ok
> +
> +static int vfe_halt(struct vfe_device *vfe)
> +{
> + /* rely on vfe_disable_output() to stop the VFE */
> + return 0;
> +}
> +
> +static void vfe_enable_irq(struct vfe_device *vfe)
> +{
> + writel_relaxed(TFE_IRQ_MASK_0_RST_DONE | TFE_IRQ_MASK_0_BUS_WR,
> + vfe->base + TFE_IRQ_MASK_0);
> + writel_relaxed(TFE_BUS_IRQ_MASK_RUP_DONE_ALL | TFE_BUS_IRQ_MASK_BUF_DONE_ALL |
> + TFE_BUS_IRQ_MASK_0_CONS_VIOL | TFE_BUS_IRQ_MASK_0_VIOL |
> + TFE_BUS_IRQ_MASK_0_IMG_VIOL, vfe->base + TFE_BUS_IRQ_MASK_0);
> +}
> +
> +static void vfe_wm_update(struct vfe_device *vfe, u8 rdi, u32 addr,
> + struct vfe_line *line)
> +{
> + u8 client = RDI_CLIENT(rdi);
> +
> + writel_relaxed(addr, vfe->base + TFE_BUS_IMAGE_ADDR(client));
> +}
> +
> +static void vfe_wm_start(struct vfe_device *vfe, u8 rdi, struct vfe_line *line)
> +{
> + struct v4l2_pix_format_mplane *pix =
> + &line->video_out.active_fmt.fmt.pix_mp;
> + u32 stride = pix->plane_fmt[0].bytesperline;
> + u8 client = RDI_CLIENT(rdi);
> +
> + /* Configuration for plain RDI frames */
> + writel_relaxed(TFE_BUS_IMAGE_CFG_0_DEFAULT, vfe->base + TFE_BUS_IMAGE_CFG_0(client));
> + writel_relaxed(0u, vfe->base + TFE_BUS_IMAGE_CFG_1(client));
> + writel_relaxed(TFE_BUS_IMAGE_CFG_2_DEFAULT, vfe->base + TFE_BUS_IMAGE_CFG_2(client));
> + writel_relaxed(stride * pix->height, vfe->base + TFE_BUS_FRAME_INCR(client));
> + writel_relaxed(TFE_BUS_PACKER_CFG_FMT_PLAIN64, vfe->base + TFE_BUS_PACKER_CFG(client));
> +
> + /* No dropped frames, one irq per frame */
> + writel_relaxed(0, vfe->base + TFE_BUS_FRAMEDROP_CFG_0(client));
> + writel_relaxed(1, vfe->base + TFE_BUS_FRAMEDROP_CFG_1(client));
> + writel_relaxed(0, vfe->base + TFE_BUS_IRQ_SUBSAMPLE_CFG_0(client));
> + writel_relaxed(1, vfe->base + TFE_BUS_IRQ_SUBSAMPLE_CFG_1(client));
> +
> + vfe_enable_irq(vfe);
> +
> + writel(TFE_BUS_CLIENT_CFG_EN | TFE_BUS_CLIENT_CFG_MODE_FRAME,
> + vfe->base + TFE_BUS_CLIENT_CFG(client));
> +
> + dev_dbg(vfe->camss->dev, "VFE%u: Started RDI%u width %u height %u stride %u\n",
> + vfe->id, rdi, pix->width, pix->height, stride);
> +}
> +
> +static void vfe_wm_stop(struct vfe_device *vfe, u8 rdi)
> +{
> + u8 client = RDI_CLIENT(rdi);
> +
> + writel(0, vfe->base + TFE_BUS_CLIENT_CFG(client));
> +
> + dev_dbg(vfe->camss->dev, "VFE%u: Stopped RDI%u\n", vfe->id, rdi);
> +}
> +
> +static const struct camss_video_ops vfe_video_ops_520 = {
> + .queue_buffer = vfe_queue_buffer_v2,
> + .flush_buffers = vfe_flush_buffers,
> +};
> +
> +static void vfe_subdev_init(struct device *dev, struct vfe_device *vfe)
> +{
> + vfe->video_ops = vfe_video_ops_520;
> +}
> +
> +static void vfe_reg_update(struct vfe_device *vfe, enum vfe_line_id line_id)
> +{
> + vfe->reg_update |= BIT(__regupdate_iface(line_id));
> + writel_relaxed(vfe->reg_update, vfe->base + TFE_REG_UPDATE_CMD);
> +}
> +
> +static inline void vfe_reg_update_clear(struct vfe_device *vfe,
> + enum vfe_line_id line_id)
> +{
> + vfe->reg_update &= ~BIT(__regupdate_iface(line_id));
> +}
> +
> +const struct vfe_hw_ops vfe_ops_340 = {
> + .global_reset = vfe_global_reset,
> + .hw_version = vfe_hw_version,
> + .isr = vfe_isr,
> + .pm_domain_off = vfe_pm_domain_off,
> + .pm_domain_on = vfe_pm_domain_on,
> + .subdev_init = vfe_subdev_init,
> + .vfe_disable = vfe_disable,
> + .vfe_enable = vfe_enable_v2,
> + .vfe_halt = vfe_halt,
> + .vfe_wm_start = vfe_wm_start,
> + .vfe_wm_stop = vfe_wm_stop,
> + .vfe_buf_done = vfe_buf_done,
> + .vfe_wm_update = vfe_wm_update,
> + .reg_update = vfe_reg_update,
> + .reg_update_clear = vfe_reg_update_clear,
> +};
> diff --git a/drivers/media/platform/qcom/camss/camss-vfe.h b/drivers/media/platform/qcom/camss/camss-vfe.h
> index a23f666be753..9b138849caca 100644
> --- a/drivers/media/platform/qcom/camss/camss-vfe.h
> +++ b/drivers/media/platform/qcom/camss/camss-vfe.h
> @@ -242,6 +242,7 @@ extern const struct vfe_hw_ops vfe_ops_4_1;
> extern const struct vfe_hw_ops vfe_ops_4_7;
> extern const struct vfe_hw_ops vfe_ops_4_8;
> extern const struct vfe_hw_ops vfe_ops_170;
> +extern const struct vfe_hw_ops vfe_ops_340;
> extern const struct vfe_hw_ops vfe_ops_480;
> extern const struct vfe_hw_ops vfe_ops_680;
> extern const struct vfe_hw_ops vfe_ops_780;
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 1/6] media: qcom: camss: Add support for TFE (Spectra 340)
2025-04-16 12:09 ` [PATCH 1/6] media: qcom: camss: Add support for TFE (Spectra 340) Loic Poulain
` (5 preceding siblings ...)
2025-04-16 12:27 ` [PATCH 1/6] media: qcom: camss: Add support for TFE (Spectra 340) Bryan O'Donoghue
@ 2025-04-16 12:30 ` Bryan O'Donoghue
2025-04-16 13:04 ` Loic Poulain
2025-04-16 14:31 ` Konrad Dybcio
2025-04-16 22:33 ` Dmitry Baryshkov
8 siblings, 1 reply; 24+ messages in thread
From: Bryan O'Donoghue @ 2025-04-16 12:30 UTC (permalink / raw)
To: Loic Poulain, bryan.odonoghue, rfoss, konradybcio, andersson,
krzk+dt, robh
Cc: linux-arm-msm, linux-media, devicetree
On 16/04/2025 13:09, Loic Poulain wrote:
> Add support for TFE (Thin Front End) found in QCM2290.
>
> Signed-off-by: Loic Poulain <loic.poulain@oss.qualcomm.com>
Maybe I didn't receive but missing cover-letter for me.
In v2 could you please give an example of how you have tested in your
cover letter, if not already done so.
---
bod
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 1/6] media: qcom: camss: Add support for TFE (Spectra 340)
2025-04-16 12:30 ` Bryan O'Donoghue
@ 2025-04-16 13:04 ` Loic Poulain
0 siblings, 0 replies; 24+ messages in thread
From: Loic Poulain @ 2025-04-16 13:04 UTC (permalink / raw)
To: Bryan O'Donoghue
Cc: rfoss, konradybcio, andersson, krzk+dt, robh, linux-arm-msm,
linux-media, devicetree
On Wed, Apr 16, 2025 at 2:30 PM Bryan O'Donoghue
<bryan.odonoghue@linaro.org> wrote:
>
> On 16/04/2025 13:09, Loic Poulain wrote:
> > Add support for TFE (Thin Front End) found in QCM2290.
> >
> > Signed-off-by: Loic Poulain <loic.poulain@oss.qualcomm.com>
> Maybe I didn't receive but missing cover-letter for me.
>
> In v2 could you please give an example of how you have tested in your
> cover letter, if not already done so.
Yes will do! Thanks for the reactivity!
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 1/6] media: qcom: camss: Add support for TFE (Spectra 340)
2025-04-16 12:09 ` [PATCH 1/6] media: qcom: camss: Add support for TFE (Spectra 340) Loic Poulain
` (6 preceding siblings ...)
2025-04-16 12:30 ` Bryan O'Donoghue
@ 2025-04-16 14:31 ` Konrad Dybcio
2025-04-16 22:33 ` Dmitry Baryshkov
8 siblings, 0 replies; 24+ messages in thread
From: Konrad Dybcio @ 2025-04-16 14:31 UTC (permalink / raw)
To: Loic Poulain, bryan.odonoghue, rfoss, konradybcio, andersson,
krzk+dt, robh
Cc: linux-arm-msm, linux-media, devicetree
On 4/16/25 2:09 PM, Loic Poulain wrote:
> Add support for TFE (Thin Front End) found in QCM2290.
>
> Signed-off-by: Loic Poulain <loic.poulain@oss.qualcomm.com>
> ---
> drivers/media/platform/qcom/camss/Makefile | 1 +
> .../media/platform/qcom/camss/camss-vfe-340.c | 281 ++++++++++++++++++
> drivers/media/platform/qcom/camss/camss-vfe.h | 1 +
> 3 files changed, 283 insertions(+)
> create mode 100644 drivers/media/platform/qcom/camss/camss-vfe-340.c
>
> diff --git a/drivers/media/platform/qcom/camss/Makefile b/drivers/media/platform/qcom/camss/Makefile
> index d26a9c24a430..719898f5d32b 100644
> --- a/drivers/media/platform/qcom/camss/Makefile
> +++ b/drivers/media/platform/qcom/camss/Makefile
> @@ -17,6 +17,7 @@ qcom-camss-objs += \
> camss-vfe-4-7.o \
> camss-vfe-4-8.o \
> camss-vfe-17x.o \
> + camss-vfe-340.o \
> camss-vfe-480.o \
> camss-vfe-680.o \
> camss-vfe-780.o \
> diff --git a/drivers/media/platform/qcom/camss/camss-vfe-340.c b/drivers/media/platform/qcom/camss/camss-vfe-340.c
> new file mode 100644
> index 000000000000..fc454d66e1d2
> --- /dev/null
> +++ b/drivers/media/platform/qcom/camss/camss-vfe-340.c
> @@ -0,0 +1,281 @@
> +// SPDX-License-Identifier: GPL-2.0
> +/*
> + * Qualcomm MSM Camera Subsystem - VFE (Video Front End) Module 340 (TFE)
> + *
> + * Copyright (c) 2025 Qualcomm Technologies, Inc.
> + */
> +
> +#include <linux/delay.h>
> +#include <linux/interrupt.h>
> +#include <linux/io.h>
> +#include <linux/iopoll.h>
> +
> +#include "camss.h"
> +#include "camss-vfe.h"
> +
> +#define TFE_GLOBAL_RESET_CMD (0x014)
> +#define TFE_GLOBAL_RESET_CMD_CORE BIT(0)
> +
> +#define TFE_REG_UPDATE_CMD (0x02C)
Please uniformly use lowercase hex
> +
> +#define TFE_IRQ_CMD (0x030)
> +#define TFE_IRQ_CMD_CLEAR BIT(0)
> +#define TFE_IRQ_MASK_0 (0x034)
> +#define TFE_IRQ_MASK_0_RST_DONE BIT(0)
> +#define TFE_IRQ_MASK_0_BUS_WR BIT(1)
> +#define TFE_IRQ_MASK_1 (0x038)
> +#define TFE_IRQ_MASK_2 (0x03c)
> +#define TFE_IRQ_CLEAR_0 (0x040)
> +
> +#define TFE_IRQ_STATUS_0 (0x04c)
> +
> +#define BUS_REG(a) (0xa00 + (a))
> +
> +#define TFE_BUS_IRQ_MASK_0 BUS_REG(0x18)
> +#define TFE_BUS_IRQ_MASK_RUP_DONE_ALL 0x000f
> +#define TFE_BUS_IRQ_MASK_RUP_DONE(src) BIT(src)
> +#define TFE_BUS_IRQ_MASK_BUF_DONE_ALL 0xff00
You can use GENMASK to define the fields and FIELD_PREP(field, x) to
fill it out
[...]
> +static inline unsigned int __regupdate_iface(enum vfe_line_id line_id)
> +{
> + switch (line_id) {
> + case VFE_LINE_RDI0:
> + return 1;
> + case VFE_LINE_RDI1:
> + return 2;
> + case VFE_LINE_RDI2:
> + return 3;
> + case VFE_LINE_PIX:
> + default:
> + return 0;
Warning?
> + }
> +}
> +
> +static inline unsigned int __regupdate_line(unsigned int iface)
> +{
> + if (!iface)
> + return VFE_LINE_PIX;
> + if (iface < 4)
> + return VFE_LINE_RDI0 + (iface - 1);
> +
> + return VFE_LINE_NONE;
> +}
> +
> +static inline unsigned int __subgroup_line(unsigned int subgroup)
> +{
> + switch (subgroup) {
> + case 5:
> + return VFE_LINE_RDI0;
> + case 6:
> + return VFE_LINE_RDI1;
> + case 7:
> + return VFE_LINE_RDI2;
> + default:
> + return VFE_LINE_PIX;
> + }
> +
> + return VFE_LINE_NONE;
> +}
All these translations looks a little fragile.. Not sure if it's in the
scope of this series, but adding an op to do this (or a static map)
sounds reasonable
[...]
> +static void vfe_wm_start(struct vfe_device *vfe, u8 rdi, struct vfe_line *line)
> +{
> + struct v4l2_pix_format_mplane *pix =
> + &line->video_out.active_fmt.fmt.pix_mp;
weird linebreak
Konrad
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 6/6] arm64: dts: qcom: qcm2290: Add CAMSS node
2025-04-16 12:09 ` [PATCH 6/6] arm64: dts: qcom: qcm2290: Add CAMSS node Loic Poulain
@ 2025-04-16 14:46 ` Konrad Dybcio
2025-04-16 15:01 ` Loic Poulain
2025-04-16 16:16 ` Loic Poulain
0 siblings, 2 replies; 24+ messages in thread
From: Konrad Dybcio @ 2025-04-16 14:46 UTC (permalink / raw)
To: Loic Poulain, bryan.odonoghue, rfoss, konradybcio, andersson,
krzk+dt, robh
Cc: linux-arm-msm, linux-media, devicetree
On 4/16/25 2:09 PM, Loic Poulain wrote:
> Add node for the QCM2290 camera subsystem.
>
> Signed-off-by: Loic Poulain <loic.poulain@oss.qualcomm.com>
> ---
> arch/arm64/boot/dts/qcom/qcm2290.dtsi | 93 +++++++++++++++++++++++++++
> 1 file changed, 93 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/qcom/qcm2290.dtsi b/arch/arm64/boot/dts/qcom/qcm2290.dtsi
> index f0746123e594..4b81e721e50c 100644
> --- a/arch/arm64/boot/dts/qcom/qcm2290.dtsi
> +++ b/arch/arm64/boot/dts/qcom/qcm2290.dtsi
> @@ -1579,6 +1579,99 @@ adreno_smmu: iommu@59a0000 {
> #iommu-cells = <2>;
> };
>
> + camss: camss@5c52000 {
@5c6e000
(first reg entry)
> + compatible = "qcom,qcm2290-camss";
> +
> + reg = <0 0x5c6e000 0 0x1000>,
> + <0 0x5c75000 0 0x1000>,
> + <0 0x5c52000 0 0x1000>,
> + <0 0x5c53000 0 0x1000>,
> + <0 0x5c6f000 0 0x4000>,
> + <0 0x5c76000 0 0x4000>;
> + reg-names = "csid0",
> + "csid1",
> + "csiphy0",
> + "csiphy1",
> + "vfe0",
> + "vfe1";
we also have a pair of TPGs at 0x5c6[68]000 - I think it would be good to
describe them from the get-go
> +
> + interrupts = <GIC_SPI 210 IRQ_TYPE_EDGE_RISING>,
> + <GIC_SPI 212 IRQ_TYPE_EDGE_RISING>,
> + <GIC_SPI 72 IRQ_TYPE_EDGE_RISING>,
> + <GIC_SPI 73 IRQ_TYPE_EDGE_RISING>,
> + <GIC_SPI 211 IRQ_TYPE_EDGE_RISING>,
> + <GIC_SPI 213 IRQ_TYPE_EDGE_RISING>;
the two TPGs would have IRQs 309 and 310
[...]
> + interconnects = <&bimc MASTER_APPSS_PROC RPM_ALWAYS_TAG
> + &config_noc SLAVE_CAMERA_CFG RPM_ALWAYS_TAG>,
This one should get a RPM_ACTIVE_TAG instead, on both endpoints
Konrad
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 6/6] arm64: dts: qcom: qcm2290: Add CAMSS node
2025-04-16 14:46 ` Konrad Dybcio
@ 2025-04-16 15:01 ` Loic Poulain
2025-04-16 15:02 ` Konrad Dybcio
2025-04-16 15:03 ` Bryan O'Donoghue
2025-04-16 16:16 ` Loic Poulain
1 sibling, 2 replies; 24+ messages in thread
From: Loic Poulain @ 2025-04-16 15:01 UTC (permalink / raw)
To: Konrad Dybcio
Cc: bryan.odonoghue, rfoss, konradybcio, andersson, krzk+dt, robh,
linux-arm-msm, linux-media, devicetree
On Wed, Apr 16, 2025 at 4:46 PM Konrad Dybcio
<konrad.dybcio@oss.qualcomm.com> wrote:
>
> On 4/16/25 2:09 PM, Loic Poulain wrote:
> > Add node for the QCM2290 camera subsystem.
> >
> > Signed-off-by: Loic Poulain <loic.poulain@oss.qualcomm.com>
> > ---
> > arch/arm64/boot/dts/qcom/qcm2290.dtsi | 93 +++++++++++++++++++++++++++
> > 1 file changed, 93 insertions(+)
> >
> > diff --git a/arch/arm64/boot/dts/qcom/qcm2290.dtsi b/arch/arm64/boot/dts/qcom/qcm2290.dtsi
> > index f0746123e594..4b81e721e50c 100644
> > --- a/arch/arm64/boot/dts/qcom/qcm2290.dtsi
> > +++ b/arch/arm64/boot/dts/qcom/qcm2290.dtsi
> > @@ -1579,6 +1579,99 @@ adreno_smmu: iommu@59a0000 {
> > #iommu-cells = <2>;
> > };
> >
> > + camss: camss@5c52000 {
>
> @5c6e000
> (first reg entry)
Ah right, I reordered reg to be aligned with other camss drivers.
Should I keep that order (and update node name) or reorder with phy
lower addresses first (PHYs)?
>
> > + compatible = "qcom,qcm2290-camss";
> > +
> > + reg = <0 0x5c6e000 0 0x1000>,
> > + <0 0x5c75000 0 0x1000>,
> > + <0 0x5c52000 0 0x1000>,
> > + <0 0x5c53000 0 0x1000>,
> > + <0 0x5c6f000 0 0x4000>,
> > + <0 0x5c76000 0 0x4000>;
> > + reg-names = "csid0",
> > + "csid1",
> > + "csiphy0",
> > + "csiphy1",
> > + "vfe0",
> > + "vfe1";
>
> we also have a pair of TPGs at 0x5c6[68]000 - I think it would be good to
> describe them from the get-go
Indeed, will do.
>
> > +
> > + interrupts = <GIC_SPI 210 IRQ_TYPE_EDGE_RISING>,
> > + <GIC_SPI 212 IRQ_TYPE_EDGE_RISING>,
> > + <GIC_SPI 72 IRQ_TYPE_EDGE_RISING>,
> > + <GIC_SPI 73 IRQ_TYPE_EDGE_RISING>,
> > + <GIC_SPI 211 IRQ_TYPE_EDGE_RISING>,
> > + <GIC_SPI 213 IRQ_TYPE_EDGE_RISING>;
>
> the two TPGs would have IRQs 309 and 310
>
> [...]
>
> > + interconnects = <&bimc MASTER_APPSS_PROC RPM_ALWAYS_TAG
> > + &config_noc SLAVE_CAMERA_CFG RPM_ALWAYS_TAG>,
>
> This one should get a RPM_ACTIVE_TAG instead, on both endpoints
ok, thanks.
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 6/6] arm64: dts: qcom: qcm2290: Add CAMSS node
2025-04-16 15:01 ` Loic Poulain
@ 2025-04-16 15:02 ` Konrad Dybcio
2025-04-16 15:04 ` Bryan O'Donoghue
2025-04-16 15:03 ` Bryan O'Donoghue
1 sibling, 1 reply; 24+ messages in thread
From: Konrad Dybcio @ 2025-04-16 15:02 UTC (permalink / raw)
To: Loic Poulain
Cc: bryan.odonoghue, rfoss, konradybcio, andersson, krzk+dt, robh,
linux-arm-msm, linux-media, devicetree
On 4/16/25 5:01 PM, Loic Poulain wrote:
> On Wed, Apr 16, 2025 at 4:46 PM Konrad Dybcio
> <konrad.dybcio@oss.qualcomm.com> wrote:
>>
>> On 4/16/25 2:09 PM, Loic Poulain wrote:
>>> Add node for the QCM2290 camera subsystem.
>>>
>>> Signed-off-by: Loic Poulain <loic.poulain@oss.qualcomm.com>
>>> ---
>>> arch/arm64/boot/dts/qcom/qcm2290.dtsi | 93 +++++++++++++++++++++++++++
>>> 1 file changed, 93 insertions(+)
>>>
>>> diff --git a/arch/arm64/boot/dts/qcom/qcm2290.dtsi b/arch/arm64/boot/dts/qcom/qcm2290.dtsi
>>> index f0746123e594..4b81e721e50c 100644
>>> --- a/arch/arm64/boot/dts/qcom/qcm2290.dtsi
>>> +++ b/arch/arm64/boot/dts/qcom/qcm2290.dtsi
>>> @@ -1579,6 +1579,99 @@ adreno_smmu: iommu@59a0000 {
>>> #iommu-cells = <2>;
>>> };
>>>
>>> + camss: camss@5c52000 {
>>
>> @5c6e000
>> (first reg entry)
>
> Ah right, I reordered reg to be aligned with other camss drivers.
> Should I keep that order (and update node name) or reorder with phy
> lower addresses first (PHYs)?
That seems to be wildly different across a number of socs, so let's
just say this is okay as is, as we're not gonna find any reasonable
order in here
Konrad
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 6/6] arm64: dts: qcom: qcm2290: Add CAMSS node
2025-04-16 15:01 ` Loic Poulain
2025-04-16 15:02 ` Konrad Dybcio
@ 2025-04-16 15:03 ` Bryan O'Donoghue
1 sibling, 0 replies; 24+ messages in thread
From: Bryan O'Donoghue @ 2025-04-16 15:03 UTC (permalink / raw)
To: Loic Poulain, Konrad Dybcio
Cc: rfoss, konradybcio, andersson, krzk+dt, robh, linux-arm-msm,
linux-media, devicetree
On 16/04/2025 16:01, Loic Poulain wrote:
>> @5c6e000
>> (first reg entry)
> Ah right, I reordered reg to be aligned with other camss drivers.
> Should I keep that order (and update node name) or reorder with phy
> lower addresses first (PHYs)?
>
>>> + compatible = "qcom,qcm2290-camss";
>>> +
>>> + reg = <0 0x5c6e000 0 0x1000>,
>>> + <0 0x5c75000 0 0x1000>,
>>> + <0 0x5c52000 0 0x1000>,
>>> + <0 0x5c53000 0 0x1000>,
>>> + <0 0x5c6f000 0 0x4000>,
>>> + <0 0x5c76000 0 0x4000>;
>>> + reg-names = "csid0",
>>> + "csid1",
>>> + "csiphy0",
>>> + "csiphy1",
>>> + "vfe0",
>>> + "vfe1";
Please keep consistent with the last committed yaml for CAMSS.
---
bod
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 6/6] arm64: dts: qcom: qcm2290: Add CAMSS node
2025-04-16 15:02 ` Konrad Dybcio
@ 2025-04-16 15:04 ` Bryan O'Donoghue
0 siblings, 0 replies; 24+ messages in thread
From: Bryan O'Donoghue @ 2025-04-16 15:04 UTC (permalink / raw)
To: Konrad Dybcio, Loic Poulain
Cc: rfoss, konradybcio, andersson, krzk+dt, robh, linux-arm-msm,
linux-media, devicetree
On 16/04/2025 16:02, Konrad Dybcio wrote:
>>> @5c6e000
>>> (first reg entry)
>> Ah right, I reordered reg to be aligned with other camss drivers.
>> Should I keep that order (and update node name) or reorder with phy
>> lower addresses first (PHYs)?
> That seems to be wildly different across a number of socs, so let's
> just say this is okay as is, as we're not gonna find any reasonable
> order in here
eek no please - we've been trying to make this consistent.
Please do as was done with sm8550. My stress levels can't take another
change.
---
bod
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 4/6] media: qcom: camss: add support for QCM2290 camss
2025-04-16 12:17 ` Bryan O'Donoghue
@ 2025-04-16 15:16 ` Loic Poulain
2025-04-16 15:38 ` Bryan O'Donoghue
0 siblings, 1 reply; 24+ messages in thread
From: Loic Poulain @ 2025-04-16 15:16 UTC (permalink / raw)
To: Bryan O'Donoghue
Cc: rfoss, konradybcio, andersson, krzk+dt, robh, linux-arm-msm,
linux-media, devicetree
On Wed, Apr 16, 2025 at 2:17 PM Bryan O'Donoghue
<bryan.odonoghue@linaro.org> wrote:
>
> On 16/04/2025 13:09, Loic Poulain wrote:
> > The camera subsystem for QCM2290 which is based on Spectra 340.
> >
> > Signed-off-by: Loic Poulain <loic.poulain@oss.qualcomm.com>
> > ---
> > drivers/media/platform/qcom/camss/camss-vfe.c | 2 +
> > drivers/media/platform/qcom/camss/camss.c | 146 ++++++++++++++++++
> > 2 files changed, 148 insertions(+)
> >
> > diff --git a/drivers/media/platform/qcom/camss/camss-vfe.c b/drivers/media/platform/qcom/camss/camss-vfe.c
> > index 4bca6c3abaff..c575c9767492 100644
> > --- a/drivers/media/platform/qcom/camss/camss-vfe.c
> > +++ b/drivers/media/platform/qcom/camss/camss-vfe.c
[...]
> > +
> > +static const struct resources_icc icc_res_2290[] = {
> > + {
> > + .name = "ahb",
> > + .icc_bw_tbl.avg = 150000,
> > + .icc_bw_tbl.peak = 300000,
> > + },
> > + {
> > + .name = "hf_mnoc",
> > + .icc_bw_tbl.avg = 2097152,
> > + .icc_bw_tbl.peak = 2097152,
> > + },
> > + {
> > + .name = "sf_mnoc",
> > + .icc_bw_tbl.avg = 2097152,
> > + .icc_bw_tbl.peak = 2097152,
> > + },
> > +};
>
> I think you can get better numbers from downstream for the above.
So I'm not sure how to get the 'correct' values? I've not executed the
downstream driver to get the output of aggregated votes. The only
clear references I see is: `CAM_CPAS_AXI_MIN_CAMNOC_IB_BW
(3000000000UL)` as well as `camnoc-axi-min-ib-bw = <3000000000>;`
in the devicetree, which would give us 3000000kBps.
Regards,
Loic
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 4/6] media: qcom: camss: add support for QCM2290 camss
2025-04-16 15:16 ` Loic Poulain
@ 2025-04-16 15:38 ` Bryan O'Donoghue
0 siblings, 0 replies; 24+ messages in thread
From: Bryan O'Donoghue @ 2025-04-16 15:38 UTC (permalink / raw)
To: Loic Poulain
Cc: rfoss, konradybcio, andersson, krzk+dt, robh, linux-arm-msm,
linux-media, devicetree
On 16/04/2025 16:16, Loic Poulain wrote:
>> I think you can get better numbers from downstream for the above.
> So I'm not sure how to get the 'correct' values? I've not executed the
> downstream driver to get the output of aggregated votes. The only
> clear references I see is: `CAM_CPAS_AXI_MIN_CAMNOC_IB_BW
> (3000000000UL)` as well as `camnoc-axi-min-ib-bw = <3000000000>;`
> in the devicetree, which would give us 3000000kBps.
>
> Regards,
> Loic
Yes, something like that from downstream is the best reference.
https://android.googlesource.com/kernel/msm/+/f51b78f14bcda2ea59c26e5fe54d88f277ef242c%5E1..f51b78f14bcda2ea59c26e5fe54d88f277ef242c/
* for your SoC
---
bod
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 6/6] arm64: dts: qcom: qcm2290: Add CAMSS node
2025-04-16 14:46 ` Konrad Dybcio
2025-04-16 15:01 ` Loic Poulain
@ 2025-04-16 16:16 ` Loic Poulain
2025-04-16 23:38 ` Bryan O'Donoghue
1 sibling, 1 reply; 24+ messages in thread
From: Loic Poulain @ 2025-04-16 16:16 UTC (permalink / raw)
To: Konrad Dybcio
Cc: bryan.odonoghue, rfoss, konradybcio, andersson, krzk+dt, robh,
linux-arm-msm, linux-media, devicetree
On Wed, Apr 16, 2025 at 4:46 PM Konrad Dybcio
<konrad.dybcio@oss.qualcomm.com> wrote:
>
> On 4/16/25 2:09 PM, Loic Poulain wrote:
> > Add node for the QCM2290 camera subsystem.
> >
> > Signed-off-by: Loic Poulain <loic.poulain@oss.qualcomm.com>
> > ---
> > arch/arm64/boot/dts/qcom/qcm2290.dtsi | 93 +++++++++++++++++++++++++++
> > 1 file changed, 93 insertions(+)
> >
> > diff --git a/arch/arm64/boot/dts/qcom/qcm2290.dtsi b/arch/arm64/boot/dts/qcom/qcm2290.dtsi
> > index f0746123e594..4b81e721e50c 100644
> > --- a/arch/arm64/boot/dts/qcom/qcm2290.dtsi
> > +++ b/arch/arm64/boot/dts/qcom/qcm2290.dtsi
> > @@ -1579,6 +1579,99 @@ adreno_smmu: iommu@59a0000 {
> > #iommu-cells = <2>;
> > };
> >
> > + camss: camss@5c52000 {
>
> @5c6e000
> (first reg entry)
>
> > + compatible = "qcom,qcm2290-camss";
> > +
> > + reg = <0 0x5c6e000 0 0x1000>,
> > + <0 0x5c75000 0 0x1000>,
> > + <0 0x5c52000 0 0x1000>,
> > + <0 0x5c53000 0 0x1000>,
> > + <0 0x5c6f000 0 0x4000>,
> > + <0 0x5c76000 0 0x4000>;
> > + reg-names = "csid0",
> > + "csid1",
> > + "csiphy0",
> > + "csiphy1",
> > + "vfe0",
> > + "vfe1";
>
> we also have a pair of TPGs at 0x5c6[68]000 - I think it would be good to
> describe them from the get-go
If we want to be ready for TPG, we may also want a camss 'top' reg
(0x05C11000), since it seems to be required for configuring CSID input
mux (TPG vs CSIPHY).
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 1/6] media: qcom: camss: Add support for TFE (Spectra 340)
2025-04-16 12:09 ` [PATCH 1/6] media: qcom: camss: Add support for TFE (Spectra 340) Loic Poulain
` (7 preceding siblings ...)
2025-04-16 14:31 ` Konrad Dybcio
@ 2025-04-16 22:33 ` Dmitry Baryshkov
8 siblings, 0 replies; 24+ messages in thread
From: Dmitry Baryshkov @ 2025-04-16 22:33 UTC (permalink / raw)
To: Loic Poulain
Cc: bryan.odonoghue, rfoss, konradybcio, andersson, krzk+dt, robh,
linux-arm-msm, linux-media, devicetree
On Wed, Apr 16, 2025 at 02:09:03PM +0200, Loic Poulain wrote:
> Add support for TFE (Thin Front End) found in QCM2290.
>
> Signed-off-by: Loic Poulain <loic.poulain@oss.qualcomm.com>
> ---
> drivers/media/platform/qcom/camss/Makefile | 1 +
> .../media/platform/qcom/camss/camss-vfe-340.c | 281 ++++++++++++++++++
> drivers/media/platform/qcom/camss/camss-vfe.h | 1 +
> 3 files changed, 283 insertions(+)
> create mode 100644 drivers/media/platform/qcom/camss/camss-vfe-340.c
>
> +
> +static inline void vfe_reg_update_clear(struct vfe_device *vfe,
No need to use inline here, it is a callback that will be called by the
pointer.
> + enum vfe_line_id line_id)
> +{
> + vfe->reg_update &= ~BIT(__regupdate_iface(line_id));
> +}
> +
> +const struct vfe_hw_ops vfe_ops_340 = {
> + .global_reset = vfe_global_reset,
> + .hw_version = vfe_hw_version,
> + .isr = vfe_isr,
> + .pm_domain_off = vfe_pm_domain_off,
> + .pm_domain_on = vfe_pm_domain_on,
> + .subdev_init = vfe_subdev_init,
> + .vfe_disable = vfe_disable,
> + .vfe_enable = vfe_enable_v2,
> + .vfe_halt = vfe_halt,
> + .vfe_wm_start = vfe_wm_start,
> + .vfe_wm_stop = vfe_wm_stop,
> + .vfe_buf_done = vfe_buf_done,
> + .vfe_wm_update = vfe_wm_update,
> + .reg_update = vfe_reg_update,
> + .reg_update_clear = vfe_reg_update_clear,
> +};
> diff --git a/drivers/media/platform/qcom/camss/camss-vfe.h b/drivers/media/platform/qcom/camss/camss-vfe.h
> index a23f666be753..9b138849caca 100644
> --- a/drivers/media/platform/qcom/camss/camss-vfe.h
> +++ b/drivers/media/platform/qcom/camss/camss-vfe.h
> @@ -242,6 +242,7 @@ extern const struct vfe_hw_ops vfe_ops_4_1;
> extern const struct vfe_hw_ops vfe_ops_4_7;
> extern const struct vfe_hw_ops vfe_ops_4_8;
> extern const struct vfe_hw_ops vfe_ops_170;
> +extern const struct vfe_hw_ops vfe_ops_340;
> extern const struct vfe_hw_ops vfe_ops_480;
> extern const struct vfe_hw_ops vfe_ops_680;
> extern const struct vfe_hw_ops vfe_ops_780;
> --
> 2.34.1
>
--
With best wishes
Dmitry
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 3/6] media: qcom: camss: csiphy-3ph: Add CSIPHY 2ph DPHY v2.0.1 init sequence
2025-04-16 12:09 ` [PATCH 3/6] media: qcom: camss: csiphy-3ph: Add CSIPHY 2ph DPHY v2.0.1 init sequence Loic Poulain
@ 2025-04-16 22:36 ` Dmitry Baryshkov
2025-04-17 8:26 ` Loic Poulain
0 siblings, 1 reply; 24+ messages in thread
From: Dmitry Baryshkov @ 2025-04-16 22:36 UTC (permalink / raw)
To: Loic Poulain
Cc: bryan.odonoghue, rfoss, konradybcio, andersson, krzk+dt, robh,
linux-arm-msm, linux-media, devicetree
On Wed, Apr 16, 2025 at 02:09:05PM +0200, Loic Poulain wrote:
> This is the CSI PHY version found in QCS2290/QCM2290 SoCs.
> The table is extracted from downstream camera driver.
>
> Signed-off-by: Loic Poulain <loic.poulain@oss.qualcomm.com>
> ---
> .../qcom/camss/camss-csiphy-3ph-1-0.c | 89 +++++++++++++++++++
> drivers/media/platform/qcom/camss/camss.h | 1 +
> 2 files changed, 90 insertions(+)
>
> diff --git a/drivers/media/platform/qcom/camss/camss-csiphy-3ph-1-0.c b/drivers/media/platform/qcom/camss/camss-csiphy-3ph-1-0.c
> index f732a76de93e..0e314ff9292f 100644
> --- a/drivers/media/platform/qcom/camss/camss-csiphy-3ph-1-0.c
> +++ b/drivers/media/platform/qcom/camss/camss-csiphy-3ph-1-0.c
> @@ -319,6 +319,90 @@ csiphy_lane_regs lane_regs_sm8250[] = {
> {0x0884, 0x01, 0x00, CSIPHY_DEFAULT_PARAMS},
> };
>
> +/* GEN2 2.0.1 2PH DPHY mode */
> +static const struct
> +csiphy_lane_regs lane_regs_qcm2290[] = {
> + {0x0030, 0x02, 0x00, CSIPHY_DEFAULT_PARAMS},
> + {0x002C, 0x01, 0x00, CSIPHY_DEFAULT_PARAMS},
lowercase hex, please.
> + {0x0034, 0x01, 0x00, CSIPHY_DEFAULT_PARAMS},
--
With best wishes
Dmitry
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 6/6] arm64: dts: qcom: qcm2290: Add CAMSS node
2025-04-16 16:16 ` Loic Poulain
@ 2025-04-16 23:38 ` Bryan O'Donoghue
0 siblings, 0 replies; 24+ messages in thread
From: Bryan O'Donoghue @ 2025-04-16 23:38 UTC (permalink / raw)
To: Loic Poulain, Konrad Dybcio
Cc: rfoss, konradybcio, andersson, krzk+dt, robh, linux-arm-msm,
linux-media, devicetree
On 16/04/2025 17:16, Loic Poulain wrote:
>> we also have a pair of TPGs at 0x5c6[68]000 - I think it would be good to
>> describe them from the get-go
> If we want to be ready for TPG, we may also want a camss 'top' reg
> (0x05C11000), since it seems to be required for configuring CSID input
> mux (TPG vs CSIPHY).
There's no harm in including the TPG in the reg list.
---
bod
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 3/6] media: qcom: camss: csiphy-3ph: Add CSIPHY 2ph DPHY v2.0.1 init sequence
2025-04-16 22:36 ` Dmitry Baryshkov
@ 2025-04-17 8:26 ` Loic Poulain
2025-04-25 19:46 ` Konrad Dybcio
0 siblings, 1 reply; 24+ messages in thread
From: Loic Poulain @ 2025-04-17 8:26 UTC (permalink / raw)
To: Dmitry Baryshkov
Cc: bryan.odonoghue, rfoss, konradybcio, andersson, krzk+dt, robh,
linux-arm-msm, linux-media, devicetree
Hi Dmitry,
On Thu, Apr 17, 2025 at 12:36 AM Dmitry Baryshkov
<dmitry.baryshkov@oss.qualcomm.com> wrote:
>
> On Wed, Apr 16, 2025 at 02:09:05PM +0200, Loic Poulain wrote:
> > This is the CSI PHY version found in QCS2290/QCM2290 SoCs.
> > The table is extracted from downstream camera driver.
> >
> > Signed-off-by: Loic Poulain <loic.poulain@oss.qualcomm.com>
> > ---
> > .../qcom/camss/camss-csiphy-3ph-1-0.c | 89 +++++++++++++++++++
> > drivers/media/platform/qcom/camss/camss.h | 1 +
> > 2 files changed, 90 insertions(+)
> >
> > diff --git a/drivers/media/platform/qcom/camss/camss-csiphy-3ph-1-0.c b/drivers/media/platform/qcom/camss/camss-csiphy-3ph-1-0.c
> > index f732a76de93e..0e314ff9292f 100644
> > --- a/drivers/media/platform/qcom/camss/camss-csiphy-3ph-1-0.c
> > +++ b/drivers/media/platform/qcom/camss/camss-csiphy-3ph-1-0.c
> > @@ -319,6 +319,90 @@ csiphy_lane_regs lane_regs_sm8250[] = {
> > {0x0884, 0x01, 0x00, CSIPHY_DEFAULT_PARAMS},
> > };
> >
> > +/* GEN2 2.0.1 2PH DPHY mode */
> > +static const struct
> > +csiphy_lane_regs lane_regs_qcm2290[] = {
> > + {0x0030, 0x02, 0x00, CSIPHY_DEFAULT_PARAMS},
> > + {0x002C, 0x01, 0x00, CSIPHY_DEFAULT_PARAMS},
>
> lowercase hex, please.
I don't mind, but all other phy tables use upper case hex.
Regards,
Loic
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 3/6] media: qcom: camss: csiphy-3ph: Add CSIPHY 2ph DPHY v2.0.1 init sequence
2025-04-17 8:26 ` Loic Poulain
@ 2025-04-25 19:46 ` Konrad Dybcio
0 siblings, 0 replies; 24+ messages in thread
From: Konrad Dybcio @ 2025-04-25 19:46 UTC (permalink / raw)
To: Loic Poulain, Dmitry Baryshkov
Cc: bryan.odonoghue, rfoss, konradybcio, andersson, krzk+dt, robh,
linux-arm-msm, linux-media, devicetree
On 4/17/25 10:26 AM, Loic Poulain wrote:
> Hi Dmitry,
>
> On Thu, Apr 17, 2025 at 12:36 AM Dmitry Baryshkov
> <dmitry.baryshkov@oss.qualcomm.com> wrote:
>>
>> On Wed, Apr 16, 2025 at 02:09:05PM +0200, Loic Poulain wrote:
>>> This is the CSI PHY version found in QCS2290/QCM2290 SoCs.
>>> The table is extracted from downstream camera driver.
>>>
>>> Signed-off-by: Loic Poulain <loic.poulain@oss.qualcomm.com>
>>> ---
>>> .../qcom/camss/camss-csiphy-3ph-1-0.c | 89 +++++++++++++++++++
>>> drivers/media/platform/qcom/camss/camss.h | 1 +
>>> 2 files changed, 90 insertions(+)
>>>
>>> diff --git a/drivers/media/platform/qcom/camss/camss-csiphy-3ph-1-0.c b/drivers/media/platform/qcom/camss/camss-csiphy-3ph-1-0.c
>>> index f732a76de93e..0e314ff9292f 100644
>>> --- a/drivers/media/platform/qcom/camss/camss-csiphy-3ph-1-0.c
>>> +++ b/drivers/media/platform/qcom/camss/camss-csiphy-3ph-1-0.c
>>> @@ -319,6 +319,90 @@ csiphy_lane_regs lane_regs_sm8250[] = {
>>> {0x0884, 0x01, 0x00, CSIPHY_DEFAULT_PARAMS},
>>> };
>>>
>>> +/* GEN2 2.0.1 2PH DPHY mode */
>>> +static const struct
>>> +csiphy_lane_regs lane_regs_qcm2290[] = {
>>> + {0x0030, 0x02, 0x00, CSIPHY_DEFAULT_PARAMS},
>>> + {0x002C, 0x01, 0x00, CSIPHY_DEFAULT_PARAMS},
>>
>> lowercase hex, please.
>
> I don't mind, but all other phy tables use upper case hex.
Ugh, we've been trying to get some unity in place but that's an
eternal WIP
Konrad
^ permalink raw reply [flat|nested] 24+ messages in thread
end of thread, other threads:[~2025-04-25 19:46 UTC | newest]
Thread overview: 24+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <yD7X4MYItg2wLMb5iPs1JXadfzhFB7wSFqo_hFbs6K72VbLmTtoOrnwcLJrP4WBvndDUd2eklJl3R4GRWLbl6Q==@protonmail.internalid>
2025-04-16 12:09 ` [PATCH 1/6] media: qcom: camss: Add support for TFE (Spectra 340) Loic Poulain
2025-04-16 12:09 ` [PATCH 2/6] media: qcom: camss: Add CSID 340 support Loic Poulain
2025-04-16 12:09 ` [PATCH 3/6] media: qcom: camss: csiphy-3ph: Add CSIPHY 2ph DPHY v2.0.1 init sequence Loic Poulain
2025-04-16 22:36 ` Dmitry Baryshkov
2025-04-17 8:26 ` Loic Poulain
2025-04-25 19:46 ` Konrad Dybcio
2025-04-16 12:09 ` [PATCH 4/6] media: qcom: camss: add support for QCM2290 camss Loic Poulain
2025-04-16 12:17 ` Bryan O'Donoghue
2025-04-16 15:16 ` Loic Poulain
2025-04-16 15:38 ` Bryan O'Donoghue
2025-04-16 12:09 ` [PATCH 5/6] media: dt-bindings: media: camss: Add qcom,qcm2290-camss binding Loic Poulain
2025-04-16 12:09 ` [PATCH 6/6] arm64: dts: qcom: qcm2290: Add CAMSS node Loic Poulain
2025-04-16 14:46 ` Konrad Dybcio
2025-04-16 15:01 ` Loic Poulain
2025-04-16 15:02 ` Konrad Dybcio
2025-04-16 15:04 ` Bryan O'Donoghue
2025-04-16 15:03 ` Bryan O'Donoghue
2025-04-16 16:16 ` Loic Poulain
2025-04-16 23:38 ` Bryan O'Donoghue
2025-04-16 12:27 ` [PATCH 1/6] media: qcom: camss: Add support for TFE (Spectra 340) Bryan O'Donoghue
2025-04-16 12:30 ` Bryan O'Donoghue
2025-04-16 13:04 ` Loic Poulain
2025-04-16 14:31 ` Konrad Dybcio
2025-04-16 22:33 ` Dmitry Baryshkov
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox