From: Adam Ford <aford173@gmail.com>
To: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Cc: linux-media@vger.kernel.org, Rui Miguel Silva <rmfrfs@gmail.com>,
Paul Elder <paul.elder@ideasonboard.com>,
Martin Kepplinger <martin.kepplinger@puri.sm>,
kernel@pengutronix.de, linux-imx@nxp.com
Subject: Re: [PATCH v1 2/6] media: imx: imx7-media-csi: Simplify imx7_csi_video_init_format()
Date: Thu, 26 Jan 2023 21:19:28 -0600 [thread overview]
Message-ID: <CAHCN7x+VFoQskRGSSRxLPkE0=LGZ+vA=23Cc04NBTUyR03WNUw@mail.gmail.com> (raw)
In-Reply-To: <20230127022715.27234-3-laurent.pinchart@ideasonboard.com>
On Thu, Jan 26, 2023 at 8:27 PM Laurent Pinchart
<laurent.pinchart@ideasonboard.com> wrote:
>
> The imx7_csi_video_init_format() function instantiates a
> v4l2_subdev_format on the stack, to only use the .format field of that
> structure. Replace it with a v4l2_mbus_framefmt instance.
>
> Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> ---
With this series and the CSIS series you posted earlier, I get a ton
of splat and the ov5640 camera doesn't appear in the media information
with media-ctrl -p
12.386980] lr : imx7_csi_probe+0x26c/0x380 [imx7_media_csi]
[ 12.387010] sp : ffff80000afd3900
[ 12.387013] x29: ffff80000afd3900 x28: 0000000000000000 x27: 0000000000000000
[ 12.387025] x26: ffff8000012ae180 x25: 0000000000000001 x24: ffff000005bb8340
[ 12.387033] x23: ffff000005bb8450 x22: ffff000005bb80a8 x21: ffff8000012ac4f8
[ 12.387040] x20: 0000000000000000 x19: ffff000005bb8080 x18: ffffffffffffffff
[ 12.387048] x17: 0000000000000000
[ 12.393690] Bluetooth: HCI UART protocol QCA registered
[ 12.397321] x16: 0000000000000000 x15: 64656d3d4d455453
[ 12.397327] x14: ffff80000a56d220 x13: 0000000000000040 x12: 0000000000000228
[ 12.397335] x11: 0000000000000000 x10: 0000000000000000 x9 : 000001e000000280
[ 12.397342] x8 : 0000000100002006 x7 : 0002000100000008 x6 : 0000000000000002
[ 12.397350] x5 : 0000000000000000 x4 : 0000000000000000 x3 : 0000000000000000
[ 12.397357] x2 : ffff000005bb84c8 x1 : 0000000000000000 x0 : ffff000005bb8450
[ 12.397365] Call trace:
[ 12.397368] imx7_csi_init_cfg+0x64/0x9c [imx7_media_csi]
[ 12.397385] platform_probe+0x68/0xe0
[ 12.406436] Bluetooth: HCI UART protocol Marvell registered
[ 12.413735] really_probe+0xbc/0x2dc
[ 12.413743] __driver_probe_device+0x78/0xe0
[ 12.413748] driver_probe_device+0xd8/0x160
[ 12.413754] __driver_attach+0x94/0x19c
[ 12.413759] bus_for_each_dev+0x70/0xd0
[ 12.413764] driver_attach+0x24/0x30
[ 12.413769] bus_add_driver+0x154/0x20c
[ 12.413774] driver_register+0x78/0x130
[ 12.413780] __platform_driver_register+0x28/0x34
[ 12.413786] imx7_csi_driver_init+0x20/0x1000 [imx7_media_csi]
[ 12.413803] do_one_initcall+0x50/0x1d0
[ 12.413810] do_init_module+0x48/0x1d0
[ 12.413817] load_module+0x193c/0x1cb0
[ 12.413822] __do_sys_finit_module+0xa8/0x100
[ 12.413828] __arm64_sys_finit_module+0x20/0x30
[ 12.413834] invoke_syscall+0x48/0x114
[ 12.413842] el0_svc_common.constprop.0+0xd4/0xfc
[ 12.413848] do_el0_svc+0x3c/0xc0
[ 12.413854] el0_svc+0x2c/0x84
[ 12.413863] el0t_64_sync_handler+0xbc/0x140
[ 12.624336] el0t_64_sync+0x190/0x194
[ 12.628002] ---[ end trace 0000000000000000 ]---
[ 12.633012] Unable to handle kernel NULL pointer dereference at
virtual address 0000000000000000
[ 12.641948] Mem abort info:
[ 12.644812] ESR = 0x0000000096000044
[ 12.648652] EC = 0x25: DABT (current EL), IL = 32 bits
[ 12.654047] SET = 0, FnV = 0
[ 12.654923] imx8m-ddrc-devfreq 3d400000.memory-controller: failed
to init firmware freq info: -19
[ 12.657176] EA = 0, S1PTW = 0
[ 12.669382] FSC = 0x04: level 0 translation fault
[ 12.674349] Data abort info:
[ 12.677284] ISV = 0, ISS = 0x00000044
[ 12.681169] CM = 0, WnR = 1
[ 12.684189] user pgtable: 4k pages, 48-bit VAs, pgdp=000000004597e000
[ 12.690698] [0000000000000000] pgd=0000000000000000, p4d=0000000000000000
[ 12.697570] Internal error: Oops: 0000000096000044 [#1] PREEMPT SMP
[ 12.703848] Modules linked in: imx8m_ddrc v4l2_h264
fsl_imx8_ddr_perf hci_uart cfg80211 imx7_media_csi(+) v4l2_mem2mem
btqca videobuf2_dma_contig videobuf2_memops btbcm videobuf2_v4l2
imx_mipi_csis etnaviv videobuf2_common gpu_sched bluetooth
snd_soc_wm8962 clk_bd718x7 ecdh_generic ecc rfkill rtc_pcf85363 at24
caam error spi_imx snd_soc_fsl_sai rtc_snvs snvs_pwrkey
snd_soc_fsl_utils imx_pcm_dma imx8mm_thermal imx_cpufreq_dt imx_sdma
ov5640 v4l2_fwnode v4l2_async videodev mc fuse drm ipv6
[ 12.747111] CPU: 0 PID: 161 Comm: systemd-udevd Tainted: G W
6.2.0-rc3-30330-gb58b9dd3fb9e-dirty #3
[ 12.757549] Hardware name: Beacon EmbeddedWorks i.MX8M Mini
Development Kit (DT)
[ 12.764945] pstate: 60000005 (nZCv daif -PAN -UAO -TCO -DIT -SSBS BTYPE=--)
[ 12.771908] pc : imx7_csi_init_cfg+0x70/0x9c [imx7_media_csi]
[ 12.777674] lr : imx7_csi_probe+0x26c/0x380 [imx7_media_csi]
[ 12.783344] sp : ffff80000afd3900
[ 12.786655] x29: ffff80000afd3900 x28: 0000000000000000 x27: 0000000000000000
[ 12.793796] x26: ffff8000012ae180 x25: 0000000000000001 x24: ffff000005bb8340
[ 12.800934] x23: ffff000005bb8450 x22: ffff000005bb80a8 x21: ffff8000012ac4f8
[ 12.808072] x20: 0000000000000000 x19: ffff000005bb8080 x18: ffffffffffffffff
[ 12.815215] x17: 0000000000000000 x16: 0000000000000000 x15: 64656d3d4d455453
[ 12.822354] x14: ffff80000a56d220 x13: 0000000000000040 x12: 0000000000000228
[ 12.829497] x11: 0000000000000000 x10: 0000000000000000 x9 : 000001e000000280
[ 12.836636] x8 : 0000000100002006 x7 : 0002000100000008 x6 : 0000000000000002
[ 12.843778] x5 : 0000000000000000 x4 : 0000000000000000 x3 : 0000000000000050
[ 12.850918] x2 : 0000000000000000 x1 : 0000000000000000 x0 : ffff000005bb8450
[ 12.858058] Call trace:
[ 12.860503] imx7_csi_init_cfg+0x70/0x9c [imx7_media_csi]
[ 12.865915] platform_probe+0x68/0xe0
[ 12.869584] really_probe+0xbc/0x2dc
[ 12.873160] __driver_probe_device+0x78/0xe0
[ 12.877434] driver_probe_device+0xd8/0x160
[ 12.881618] __driver_attach+0x94/0x19c
[ 12.885456] bus_for_each_dev+0x70/0xd0
[ 12.889293] driver_attach+0x24/0x30
[ 12.892868] bus_add_driver+0x154/0x20c
[ 12.896707] driver_register+0x78/0x130
[ 12.900545] __platform_driver_register+0x28/0x34
[ 12.905255] imx7_csi_driver_init+0x20/0x1000 [imx7_media_csi]
[ 12.911099] do_one_initcall+0x50/0x1d0
[ 12.914937] do_init_module+0x48/0x1d0
[ 12.918691] load_module+0x193c/0x1cb0
[ 12.922442] __do_sys_finit_module+0xa8/0x100
[ 12.926802] __arm64_sys_finit_module+0x20/0x30
[ 12.931336] invoke_syscall+0x48/0x114
[ 12.935090] el0_svc_common.constprop.0+0xd4/0xfc
[ 12.939796] do_el0_svc+0x3c/0xc0
[ 12.943114] el0_svc+0x2c/0x84
[ 12.946174] el0t_64_sync_handler+0xbc/0x140
[ 12.950446] el0t_64_sync+0x190/0x194
[ 12.954114] Code: b5fffe81 d4210000 d2800002 91014063 (a9002049)
[ 12.960209] ---[ end trace 0000000000000000 ]---
The media information:
root@beacon-imx8mm-kit:~# media-ctl -p
Media controller API version 6.2.0
Media device information
------------------------
driver imx7-csi
model imx-media
serial
bus info platform:32e20000.csi
hw revision 0x0
driver version 6.2.0
Device topology
- entity 1: csi (2 pads, 1 link)
type V4L2 subdev subtype Unknown flags 0
pad0: Sink
pad1: Source
-> "csi capture":0 [ENABLED,IMMUTABLE]
- entity 4: csi capture (1 pad, 1 link)
type Node subtype V4L flags 0
device node name /dev/video0
pad0: Sink
<- "csi":1 [ENABLED,IMMUTABLE]
I confirmed the ov5640 camera enumerated:
I'm going to roll back this latest series to verify whether or not
this series caused the splat.
adam
> drivers/media/platform/nxp/imx7-media-csi.c | 18 ++++++++----------
> 1 file changed, 8 insertions(+), 10 deletions(-)
>
> diff --git a/drivers/media/platform/nxp/imx7-media-csi.c b/drivers/media/platform/nxp/imx7-media-csi.c
> index be3c1494cfb3..e96bee4e5921 100644
> --- a/drivers/media/platform/nxp/imx7-media-csi.c
> +++ b/drivers/media/platform/nxp/imx7-media-csi.c
> @@ -1598,17 +1598,15 @@ static struct imx7_csi_vb2_buffer *imx7_csi_video_next_buf(struct imx7_csi *csi)
>
> static int imx7_csi_video_init_format(struct imx7_csi *csi)
> {
> - struct v4l2_subdev_format fmt_src = {
> - .pad = IMX7_CSI_PAD_SRC,
> - .which = V4L2_SUBDEV_FORMAT_ACTIVE,
> - };
> - fmt_src.format.code = IMX7_CSI_DEF_MBUS_CODE;
> - fmt_src.format.width = IMX7_CSI_DEF_PIX_WIDTH;
> - fmt_src.format.height = IMX7_CSI_DEF_PIX_HEIGHT;
> + struct v4l2_mbus_framefmt format = { };
>
> - imx7_csi_mbus_fmt_to_pix_fmt(&csi->vdev_fmt, &fmt_src.format, NULL);
> - csi->vdev_compose.width = fmt_src.format.width;
> - csi->vdev_compose.height = fmt_src.format.height;
> + format.code = IMX7_CSI_DEF_MBUS_CODE;
> + format.width = IMX7_CSI_DEF_PIX_WIDTH;
> + format.height = IMX7_CSI_DEF_PIX_HEIGHT;
> +
> + imx7_csi_mbus_fmt_to_pix_fmt(&csi->vdev_fmt, &format, NULL);
> + csi->vdev_compose.width = format.width;
> + csi->vdev_compose.height = format.height;
>
> csi->vdev_cc = imx7_csi_find_pixel_format(csi->vdev_fmt.pixelformat);
>
> --
> Regards,
>
> Laurent Pinchart
>
next prev parent reply other threads:[~2023-01-27 3:19 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-01-27 2:27 [PATCH v1 0/6] media: nxp: imx7-media-csi: Move to subdev active state Laurent Pinchart
2023-01-27 2:27 ` [PATCH v1 1/6] media: imx: imx7-media-csi: Drop imx7_csi.cc field Laurent Pinchart
2023-01-27 2:27 ` [PATCH v1 2/6] media: imx: imx7-media-csi: Simplify imx7_csi_video_init_format() Laurent Pinchart
2023-01-27 3:19 ` Adam Ford [this message]
2023-01-27 6:57 ` Laurent Pinchart
2023-01-27 11:07 ` Adam Ford
2023-01-27 11:20 ` Adam Ford
2023-01-29 2:36 ` Laurent Pinchart
2023-01-27 2:27 ` [PATCH v1 3/6] media: imx: imx7-media-csi: Drop unneeded check when starting streaming Laurent Pinchart
2023-01-27 2:27 ` [PATCH v1 4/6] media: imx: imx7-media-csi: Drop unneeded src_sd check Laurent Pinchart
2023-01-27 2:27 ` [PATCH v1 5/6] media: imx: imx7-media-csi: Drop unneeded pad checks Laurent Pinchart
2023-01-27 2:27 ` [PATCH v1 6/6] media: imx: imx7-media-csi: Use V4L2 subdev active state Laurent Pinchart
2023-01-27 2:41 ` [PATCH v1 0/6] media: nxp: imx7-media-csi: Move to " Adam Ford
2023-01-27 11:41 ` Martin Kepplinger
2023-01-29 2:35 ` Laurent Pinchart
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='CAHCN7x+VFoQskRGSSRxLPkE0=LGZ+vA=23Cc04NBTUyR03WNUw@mail.gmail.com' \
--to=aford173@gmail.com \
--cc=kernel@pengutronix.de \
--cc=laurent.pinchart@ideasonboard.com \
--cc=linux-imx@nxp.com \
--cc=linux-media@vger.kernel.org \
--cc=martin.kepplinger@puri.sm \
--cc=paul.elder@ideasonboard.com \
--cc=rmfrfs@gmail.com \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.