Thread (2 messages) 2 messages, 2 authors, 2021-01-28

Re: [PATCH v3 05/22] media: camss: Refactor VFE HW version support

From: Nicolas Boichat <hidden>
Date: 2021-01-28 00:20:39
Also in: linux-arm-msm, linux-devicetree, linux-media, lkml

On Wed, Jan 27, 2021 at 10:56 PM Robert Foss [off-list ref] wrote:
quoted hunk
In order to support Qualcomm ISP hardware architectures that diverge
from older architectures, the VFE subdevice driver needs to be refactored
to better abstract the different ISP architectures.

Gen1 represents the CAMSS ISP architecture. The ISP architecture developed
after CAMSS, Titan, will be referred to as Gen2.

Signed-off-by: Robert Foss <redacted>
---
[snip]
diff --git a/drivers/media/platform/qcom/camss/camss-vfe-4-8.c b/drivers/media/platform/qcom/camss/camss-vfe-4-8.c
new file mode 100644
index 000000000000..153e0e20664e
--- /dev/null
+++ b/drivers/media/platform/qcom/camss/camss-vfe-4-8.c
[snip]
+/*
+ * vfe_isr - VFE module interrupt handler
+ * @irq: Interrupt line
+ * @dev: VFE device
+ *
+ * Return IRQ_HANDLED on success
+ */
+static irqreturn_t vfe_isr(int irq, void *dev)
+{
+       struct vfe_device *vfe = dev;
+       u32 value0, value1;
+       int i, j;
+
+       vfe->ops->isr_read(vfe, &value0, &value1);
+
+       trace_printk("VFE: status0 = 0x%08x, status1 = 0x%08x\n",
+                    value0, value1);
Please do not use trace_printk in production code [1,2], it is only
meant for debug use. Consider using trace events, or dev_dbg.

[1] https://elixir.bootlin.com/linux/v5.8/source/kernel/trace/trace.c#L3158
[2] https://elixir.bootlin.com/linux/v5.8/source/include/linux/kernel.h#L766
[snip]
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help