Re: [PATCH 7/8] media: qcom: camss: drive streams-aware transmitters through the streams API

[Hitesh Patel <hitesh@ebytelogic.com>]

Hi Loic, Bryan,

> I would like to point out that Gjorgji has recently posted a CAMSS
> stream API series [1]. Would that series address your use case, or are
> there requirements that it does not cover?

Thanks for the pointer, I had missed it. From the cover letter it
covers what patches 6-8 here do, and does it in the right place: the
first/last-stream handling inside the CSID instead of a refcount in
camss-video, and CSID routing for multi-VC sources. Our use case is two
cameras behind a GMSL deserializer on SC7280, both as two virtual
channels on one CSIPHY and as one camera per CSIPHY, with the cameras
started and stopped independently.

Plan from my side:

- I will apply Gjorgji's series on our tree, enable streams for the
  SC7280 resources and run our two-camera tests on RB3 Gen2. I will
  report the results on his thread, including anything that does not
  work, and review the CSID/CSIPHY parts while at it.

- If it covers the use case, v2 of this series will be only the three
  VFE fixes plus the link frequency and CSIPHY link patches, and I will
  send the SC7280 enablement and any fixes on top of his series rather
  than keep 6-8.

> It'd be great if you and Gjorgji could cross-review each other's
> contributions.

Yes, will do. Gjorgji, if you want to try the fixes on SM8250, the
three VFE ones (3/8, 4/8, 5/8) are independent of everything else and
are the ones that matter for two RDIs streaming on one VFE.

Regards,
Hitesh