mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: "Luca Weiss" <luca.weiss@fairphone.com>
To: "Bryan O'Donoghue" <bryan.odonoghue@linaro.org>,
	"Vikram Sharma" <quic_vikramsa@quicinc.com>,
	"Robert Foss" <rfoss@kernel.org>,
	"Todor Tomov" <todor.too@gmail.com>,
	"Mauro Carvalho Chehab" <mchehab@kernel.org>,
	"Rob Herring" <robh@kernel.org>,
	"Krzysztof Kozlowski" <krzk+dt@kernel.org>,
	"Conor Dooley" <conor+dt@kernel.org>,
	"Kapatrala Syed" <akapatra@quicinc.com>,
	"Hariram Purushothaman" <hariramp@quicinc.com>,
	"Bjorn Andersson" <andersson@kernel.org>,
	"Konrad Dybcio" <konradybcio@kernel.org>,
	"Hans Verkuil" <hverkuil-cisco@xs4all.nl>,
	cros-qcom-dts-watchers@chromium.org,
	"Catalin Marinas" <catalin.marinas@arm.com>,
	"Will Deacon" <will@kernel.org>,
	"Matti Lehtimäki" <matti.lehtimaki@gmail.com>
Cc: <linux-arm-msm@vger.kernel.org>, <linux-media@vger.kernel.org>,
	<devicetree@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
	<linux-arm-kernel@lists.infradead.org>,
	"Suresh Vankadara" <quic_svankada@quicinc.com>,
	"Trishansh Bhardwaj" <quic_tbhardwa@quicinc.com>,
	<stable@vger.kernel.org>,
	"Hariram Purushothaman" <quic_hariramp@quicinc.com>
Subject: Re: [PATCH 00/10] (no cover subject)
Date: Tue, 01 Oct 2024 16:22:50 +0200	[thread overview]
Message-ID: <D4KJCLCDGQ96.251XO3OG6DVB6@fairphone.com> (raw)
In-Reply-To: <c912f2da-519c-4bdc-a5cb-e19c3aa63ea8@linaro.org>

On Tue Oct 1, 2024 at 2:49 PM CEST, Bryan O'Donoghue wrote:
> On 01/10/2024 12:39, Luca Weiss wrote:
>
> > And v4l-subdev5 is msm_csid0 on my device.
>
> <snip>
>
> > 
> > - entity 16: msm_csid0 (5 pads, 22 links, 0 routes)
> >               type V4L2 subdev subtype Unknown flags 0
> >               device node name /dev/v4l-subdev5
> >          pad0: Sink
> >                  [stream:0 fmt:SRGGB10_1X10/4056x3040 field:none colorspace:srgb]
> >                  <- "msm_csiphy0":1 []
> >                  <- "msm_csiphy1":1 []
> >                  <- "msm_csiphy2":1 []
> >                  <- "msm_csiphy3":1 []
> >                  <- "msm_csiphy4":1 []
> >          pad1: Source
> >                  [stream:0 fmt:SRGGB10_1X10/4056x3040 field:none colorspace:srgb]
> >                  -> "msm_vfe0_rdi0":0 [ENABLED]
> >                  -> "msm_vfe1_rdi0":0 []
> >                  -> "msm_vfe2_rdi0":0 []
> >                  -> "msm_vfe3_rdi0":0 []
> >                  -> "msm_vfe4_rdi0":0 []
>
> <snip>
>
> media-ctl --reset
> yavta --no-query -w '0x009f0903 2' /dev/v4l-subdev5
> yavta --list /dev/v4l-subdev5
> media-ctl -V '"msm_csid0":0[fmt:SRGGB10/4056x3040]'
> media-ctl -V '"msm_vfe0_rdi0":0[fmt:SRGGB10/4056x3040]'
> media-ctl -l '"msm_csid0":1->"msm_vfe0_rdi0":0[1]'
> media-ctl -d /dev/media0 -p
>
> That command list and this
>
> yavta -B capture-mplane -c -I -n 5 -f SRGGB10P -s 4056x3040 -F /dev/video0
>
> should work.

Yeah, unfortunately this is still hanging... Let's also see what Vikram
sees on their board.

fairphone-fp5:~$ yavta -B capture-mplane -c -I -n 5 -f SRGGB10P -s 4056x3040 -F /dev/video0
Device /dev/video0 opened.
Device `Qualcomm Camera Subsystem' on `platform:acb3000.camss' (driver 'qcom-camss') supports video, capture, with mplanes.
Video format set: SRGGB10P (41415270) 4056x3040 field none, 1 planes: 
 * Stride 5072, buffer size 15418880
Video format: SRGGB10P (41415270) 4056x3040 field none, 1 planes: 
 * Stride 5072, buffer size 15418880
5 buffers requested.
length: 1 offset: 3442938648 timestamp type/source: mono/EoF
Buffer 0/0 mapped at address 0xffff85e00000.
length: 1 offset: 3442938648 timestamp type/source: mono/EoF
Buffer 1/0 mapped at address 0xffff84e08000.
length: 1 offset: 3442938648 timestamp type/source: mono/EoF
Buffer 2/0 mapped at address 0xffff83e10000.
length: 1 offset: 3442938648 timestamp type/source: mono/EoF
Buffer 3/0 mapped at address 0xffff82e18000.
length: 1 offset: 3442938648 timestamp type/source: mono/EoF
Buffer 4/0 mapped at address 0xffff81e20000.

> I have to test Vladimir's two patches. I'll verify rb5 TPG while I'm at 
> it, perhaps the error is not sdm670 specific.

FWIW this is not sdm670 but sc7280/qcm6490 here :) But I didn't follow
the sdm670 thread so maybe you mean something there.

>
> That said last time I tested it, it worked and no changes have gone in, 
> in the meantime.

I also had the test pattern working on a 6.8-based kernel on this device
with camss patches from Matti Lehtimäki.

Regards
Luca

>
> ---
> bod


  reply	other threads:[~2024-10-01 14:22 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-09-04 11:10 Vikram Sharma
2024-09-04 11:10 ` [PATCH 01/10] media: dt-bindings: media: camss: Add qcom,sc7280-camss binding Vikram Sharma
2024-09-04 11:28   ` Krzysztof Kozlowski
2024-09-05  6:47   ` Krzysztof Kozlowski
2024-09-04 11:10 ` [PATCH 02/10] media: dt-bindings: media: qcs6490-rb3gen2-vision-mezzanine: Add dt bindings Vikram Sharma
2024-09-04 11:29   ` Krzysztof Kozlowski
2024-09-04 11:10 ` [PATCH 03/10] media: qcom: camss: Fix potential crash if domain attach fails Vikram Sharma
2024-09-05 12:13   ` Konrad Dybcio
2024-09-04 11:10 ` [PATCH 04/10] media: qcom: camss: Sort CAMSS version enums and compatible strings Vikram Sharma
2024-09-04 14:44   ` Bryan O'Donoghue
2024-09-04 11:10 ` [PATCH 05/10] media: qcom: camss: Add support for camss driver on SC7280 Vikram Sharma
2024-09-04 11:10 ` [PATCH 06/10] media: qcom: camss: Add camss_link_entities_v2 Vikram Sharma
2024-09-04 14:50   ` Bryan O'Donoghue
2024-09-04 11:10 ` [PATCH 07/10] arm64: dts: qcom: sc7280: Add support for camss Vikram Sharma
2024-09-04 11:31   ` Krzysztof Kozlowski
2024-09-04 11:10 ` [PATCH 08/10] arm64: dts: qcom: qcs6490-rb3gen2-vision-mezzanine: Enable IMX577 sensor Vikram Sharma
2024-09-04 11:32   ` Krzysztof Kozlowski
2024-09-04 11:10 ` [PATCH 09/10] arm64: dts: qcom: sc7280: Add default and suspend states for GPIO Vikram Sharma
2024-09-04 11:33   ` Krzysztof Kozlowski
2024-09-04 11:10 ` [PATCH 10/10] arm64: defconfig: Enable camcc driver for SC7280 Vikram Sharma
2024-09-04 11:33   ` Krzysztof Kozlowski
2024-09-04 11:21 ` [PATCH 00/10] (no cover subject) Bryan O'Donoghue
2024-09-04 11:22   ` Bryan O'Donoghue
2024-09-04 13:36 ` Rob Herring (Arm)
2024-09-30 10:52 ` Luca Weiss
2024-09-30 11:54   ` Bryan O'Donoghue
2024-10-01  8:24     ` Luca Weiss
2024-10-01  8:51       ` Vikram Sharma
2024-10-01  9:30       ` Bryan O'Donoghue
2024-10-01 11:39         ` Luca Weiss
2024-10-01 12:49           ` Bryan O'Donoghue
2024-10-01 14:22             ` Luca Weiss [this message]
2024-10-01 14:58               ` Bryan O'Donoghue

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=D4KJCLCDGQ96.251XO3OG6DVB6@fairphone.com \
    --to=luca.weiss@fairphone.com \
    --cc=akapatra@quicinc.com \
    --cc=andersson@kernel.org \
    --cc=bryan.odonoghue@linaro.org \
    --cc=catalin.marinas@arm.com \
    --cc=conor+dt@kernel.org \
    --cc=cros-qcom-dts-watchers@chromium.org \
    --cc=devicetree@vger.kernel.org \
    --cc=hariramp@quicinc.com \
    --cc=hverkuil-cisco@xs4all.nl \
    --cc=konradybcio@kernel.org \
    --cc=krzk+dt@kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-media@vger.kernel.org \
    --cc=matti.lehtimaki@gmail.com \
    --cc=mchehab@kernel.org \
    --cc=quic_hariramp@quicinc.com \
    --cc=quic_svankada@quicinc.com \
    --cc=quic_tbhardwa@quicinc.com \
    --cc=quic_vikramsa@quicinc.com \
    --cc=rfoss@kernel.org \
    --cc=robh@kernel.org \
    --cc=stable@vger.kernel.org \
    --cc=todor.too@gmail.com \
    --cc=will@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

all inboxes | Powered by JetHome®