From: Pratyush Yadav <p.yadav@ti.com>
To: Tomi Valkeinen <tomi.valkeinen@ideasonboard.com>
Cc: Maxime Ripard <mripard@kernel.org>,
Mauro Carvalho Chehab <mchehab@kernel.org>,
Rob Herring <robh+dt@kernel.org>,
Kishon Vijay Abraham I <kishon@ti.com>,
Vinod Koul <vkoul@kernel.org>,
Peter Ujfalusi <peter.ujfalusi@gmail.com>,
Steve Longerbeam <slongerbeam@gmail.com>,
Benoit Parrot <bparrot@ti.com>, <linux-media@vger.kernel.org>,
<devicetree@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
<linux-phy@lists.infradead.org>, <dmaengine@vger.kernel.org>,
Vignesh Raghavendra <vigneshr@ti.com>,
Chunfeng Yun <chunfeng.yun@mediatek.com>,
Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Subject: Re: [PATCH v2 00/18] CSI2RX support on J721E
Date: Thu, 3 Jun 2021 18:22:53 +0530 [thread overview]
Message-ID: <20210603125251.nmzibyvfzkkxfbtj@ti.com> (raw)
In-Reply-To: <83bcd60a-2a45-59b2-8ebe-26ad5d828965@ideasonboard.com>
On 27/05/21 04:23PM, Tomi Valkeinen wrote:
> Hi Pratyush,
>
> On 26/05/2021 18:22, Pratyush Yadav wrote:
> > Hi,
> >
> > This series adds support for CSI2 capture on J721E. It includes some
> > fixes to the Cadence CSI2RX driver, adds Rx support to Cadence DPHY
> > driver, and finally adds the TI CSI2RX wrapper driver.
> >
> > Tested on TI's J721E with OV5640 sensor.
>
> I also see this after a few captures:
Can you share the application/command you are using to test? I used
yavta to test and didn't see any problems after leaving the stream on
for around 10 minutes.
>
> [ 84.115503] ------------[ cut here ]------------
> [ 84.120144] DMA-API: ti-udma 31150000.dma-controller: mapping sg segment longer than device claims to support [len=1900544] [max=65536]
> [ 84.132376] WARNING: CPU: 1 PID: 594 at kernel/dma/debug.c:1172 debug_dma_map_sg+0x304/0x390
> [ 84.140804] Modules linked in: ov5640 ti_cal j721e_csi2rx cdns_csi2rx cdns_dphy v4l2_fwnode tidss ti_tfp410 tc358767 display_connector cdns_mhdp8546 panel_simple drm_kms_helper d
> rm drm_panel_orientation_quirks cfbfillrect cfbimgblt cfbcopyarea phy_j721e_wiz phy_cadence_torrent
> [ 84.165298] CPU: 1 PID: 594 Comm: cam-mplex.py Not tainted 5.13.0-rc1-00206-g98bb91e95a28-dirty #5
> [ 84.174236] Hardware name: Texas Instruments K3 J721E SoC (DT)
> [ 84.180051] pstate: 60000005 (nZCv daif -PAN -UAO -TCO BTYPE=--)
> [ 84.186041] pc : debug_dma_map_sg+0x304/0x390
> [ 84.190383] lr : debug_dma_map_sg+0x304/0x390
> [ 84.194725] sp : ffff800014d0f730
> [ 84.198026] x29: ffff800014d0f730 x28: ffff000801544680 x27: ffffffffffffffff
> [ 84.205148] x26: 0000000000000000 x25: 0000000000000002 x24: 0000000000000001
> [ 84.212269] x23: ffff80001163abe0 x22: 0000000000000000 x21: 0000000000000001
> [ 84.219390] x20: ffff000801fa3010 x19: ffff0008075c7580 x18: 0000000000000000
> [ 84.226510] x17: 0000000000000000 x16: 0000000000000000 x15: 0000000000000030
> [ 84.233630] x14: 6e61687420726567 x13: 6e6f6c20746e656d x12: 7420736d69616c63
> [ 84.240751] x11: 2065636976656420 x10: ffff8000116b18f8 x9 : ffff8000100eb920
> [ 84.247871] x8 : ffff8000116598f8 x7 : ffff8000116b18f8 x6 : 0000000000000001
> [ 84.254991] x5 : 0000000000000001 x4 : 0000000000000001 x3 : 0000000000000000
> [ 84.262111] x2 : 0000000000000000 x1 : 0000000000000000 x0 : ffff0008066f8000
> [ 84.269230] Call trace:
> [ 84.271665] debug_dma_map_sg+0x304/0x390
> [ 84.275660] dma_map_sg_attrs+0x70/0xb0
> [ 84.279486] drm_gem_map_dma_buf+0x6c/0xf0 [drm]
> [ 84.284177] __map_dma_buf+0x28/0x80
> [ 84.287742] dma_buf_map_attachment+0xe4/0x200
> [ 84.292172] vb2_dc_map_dmabuf+0x3c/0x150
> [ 84.296171] __prepare_dmabuf+0x1dc/0x514
> [ 84.300168] __buf_prepare+0x1a0/0x25c
> [ 84.303903] vb2_core_qbuf+0x3d4/0x72c
> [ 84.307638] vb2_qbuf+0x9c/0xf4
> [ 84.310767] vb2_ioctl_qbuf+0x68/0x7c
> [ 84.314416] v4l_qbuf+0x54/0x70
> [ 84.317545] __video_do_ioctl+0x194/0x400
> [ 84.321541] video_usercopy+0x19c/0x910
> [ 84.325362] video_ioctl2+0x24/0x40
> [ 84.328837] v4l2_ioctl+0x4c/0x70
> [ 84.332141] __arm64_sys_ioctl+0xb4/0xfc
> [ 84.336053] invoke_syscall+0x50/0x120
> [ 84.339791] el0_svc_common.constprop.0+0x68/0x104
> [ 84.344569] do_el0_svc+0x30/0x9c
> [ 84.347872] el0_svc+0x2c/0x54
> [ 84.350916] el0_sync_handler+0x1a8/0x1ac
> [ 84.354911] el0_sync+0x198/0x1c0
> [ 84.358215] irq event stamp: 0
> [ 84.361256] hardirqs last enabled at (0): [<0000000000000000>] 0x0
> [ 84.367507] hardirqs last disabled at (0): [<ffff80001004fe2c>] copy_process+0x44c/0x1800
> [ 84.375667] softirqs last enabled at (0): [<ffff80001004fe2c>] copy_process+0x44c/0x1800
> [ 84.383824] softirqs last disabled at (0): [<0000000000000000>] 0x0
> [ 84.390073] ---[ end trace af3448c784059129 ]---
--
Regards,
Pratyush Yadav
Texas Instruments Inc.
next prev parent reply other threads:[~2021-06-03 12:53 UTC|newest]
Thread overview: 48+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-05-26 15:22 [PATCH v2 00/18] CSI2RX support on J721E Pratyush Yadav
2021-05-26 15:22 ` [PATCH v2 01/18] phy: Distinguish between Rx and Tx for MIPI D-PHY with submodes Pratyush Yadav
2021-05-26 15:22 ` [PATCH v2 02/18] phy: cdns-dphy: Prepare for Rx support Pratyush Yadav
2021-05-26 15:22 ` [PATCH v2 03/18] phy: cdns-dphy: Allow setting mode Pratyush Yadav
2021-05-26 15:22 ` [PATCH v2 04/18] phy: cdns-dphy: Add Rx support Pratyush Yadav
2021-05-28 6:09 ` Tomi Valkeinen
2021-05-26 15:22 ` [PATCH v2 05/18] media: ov5640: Use runtime PM to control sensor power Pratyush Yadav
2021-05-28 6:44 ` Tomi Valkeinen
2021-05-28 7:25 ` Pratyush Yadav
2021-05-26 15:22 ` [PATCH v2 06/18] media: cadence: csi2rx: Add external DPHY support Pratyush Yadav
2021-05-28 7:23 ` Tomi Valkeinen
2021-05-28 7:30 ` Pratyush Yadav
2021-05-26 15:22 ` [PATCH v2 07/18] media: cadence: csi2rx: Soft reset the streams before starting capture Pratyush Yadav
2021-05-26 15:22 ` [PATCH v2 08/18] media: cadence: csi2rx: Set the STOP bit when stopping a stream Pratyush Yadav
2021-05-26 15:22 ` [PATCH v2 09/18] media: cadence: csi2rx: Fix stream data configuration Pratyush Yadav
2021-05-26 15:23 ` [PATCH v2 10/18] media: cadence: csi2rx: Populate subdev devnode Pratyush Yadav
2021-05-28 7:16 ` Tomi Valkeinen
2021-05-28 7:24 ` Pratyush Yadav
2021-05-28 7:35 ` Tomi Valkeinen
2021-06-03 12:42 ` Pratyush Yadav
2021-05-26 15:23 ` [PATCH v2 11/18] dmaengine: ti: k3-psil-j721e: Add entry for CSI2RX Pratyush Yadav
2021-05-31 6:51 ` Péter Ujfalusi
2021-05-31 9:16 ` Pratyush Yadav
2021-05-26 15:23 ` [PATCH v2 12/18] media: Re-structure TI platform drivers Pratyush Yadav
2021-05-28 8:29 ` Tomi Valkeinen
2021-05-26 15:23 ` [PATCH v2 13/18] media: ti: Add CSI2RX support for J721E Pratyush Yadav
2021-05-27 13:29 ` Tomi Valkeinen
2021-06-03 12:49 ` Pratyush Yadav
2021-06-04 8:04 ` Tomi Valkeinen
2021-05-26 15:23 ` [PATCH v2 14/18] media: dt-bindings: Add DT bindings for TI J721E CSI2RX driver Pratyush Yadav
2021-05-27 14:23 ` Rob Herring
2021-06-03 12:43 ` Pratyush Yadav
2021-05-26 15:23 ` [PATCH v2 15/18] media: dt-bindings: Convert Cadence CSI2RX binding to YAML Pratyush Yadav
2021-06-01 17:45 ` Rob Herring
2021-06-03 12:23 ` Pratyush Yadav
2021-05-26 15:23 ` [PATCH v2 16/18] phy: dt-bindings: Convert Cadence DPHY " Pratyush Yadav
2021-06-02 19:51 ` Rob Herring
2021-05-26 15:23 ` [PATCH v2 17/18] phy: dt-bindings: cdns,dphy: make clocks optional Pratyush Yadav
2021-06-02 13:27 ` Maxime Ripard
2021-06-03 8:16 ` Pratyush Yadav
2021-06-02 20:03 ` Rob Herring
2021-05-26 15:23 ` [PATCH v2 18/18] phy: dt-bindings: cdns,dphy: add power-domains property Pratyush Yadav
2021-06-02 20:03 ` Rob Herring
2021-05-27 12:42 ` [PATCH v2 00/18] CSI2RX support on J721E Tomi Valkeinen
2021-06-03 12:54 ` Pratyush Yadav
2021-05-27 13:23 ` Tomi Valkeinen
2021-06-03 12:52 ` Pratyush Yadav [this message]
2021-06-04 7:54 ` Tomi Valkeinen
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20210603125251.nmzibyvfzkkxfbtj@ti.com \
--to=p.yadav@ti.com \
--cc=bparrot@ti.com \
--cc=chunfeng.yun@mediatek.com \
--cc=devicetree@vger.kernel.org \
--cc=dmaengine@vger.kernel.org \
--cc=kishon@ti.com \
--cc=laurent.pinchart@ideasonboard.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=linux-phy@lists.infradead.org \
--cc=mchehab@kernel.org \
--cc=mripard@kernel.org \
--cc=peter.ujfalusi@gmail.com \
--cc=robh+dt@kernel.org \
--cc=slongerbeam@gmail.com \
--cc=tomi.valkeinen@ideasonboard.com \
--cc=vigneshr@ti.com \
--cc=vkoul@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).