Hi Bryan,
I wanted to let you know that I will likely not be able to review this
v3 in a timely manner. v2 was quite fine overall, I trust that you've
taken my review comments into account in this new version. Maybe someone
else could give it another look, in order to get it merged in v6.7 ?
On Wed, Aug 30, 2023 at 04:16:05PM +0100, Bryan O'Donoghue wrote:
> V3:
> - Adds Reviewed-by where indicated - Laurent
> - Adds a new patch for genpd cleanup. TBH I completely missed this so thanks ! - Laurent
> - "media: qcom: camss: Fix V4L2 async notifier error path" stays the same fixes spalt in -next
> Fixes: 51397a4ec75d ("media: qcom: Initialise V4L2 async notifier later")
> - I like the suggesting of using a common fix for vfe-17x and vfe-480 however, I believe
> we need to support multiple write-master/RDI => VCs in 17x which currently we only do
> in vfe-480 so sharing the code between the two here right now, is n't possible.
> - Included other suggestions on vfe-17x and vfe-480 - Laurent
> - I didn't change the val |= 1 << CSI2_RX_CFG1_VC_MODE to BIT(2)
> The reason for that is all of the code uses this odd bit-shifting and I'd rather do
> the conversion from shifting to BIT(x) as a distinct series instead of piecemeal - bod
>
> V2:
> - Amends commit log for TPG fix to cover dropping of fixed
> VC when setting up a TPG - Konrad
>
> - Leaves GENMASK etc out. I'm happy to do a "make it pretty"
> series later on. - bod
>
> Link: https://git.codelinaro.org/bryan.odonoghue/kernel/-/commits/Bugfix-series-v2?ref_type=tags
>
> V1:
> - Drops dt_id = vc * 4 in favour of a patch in a later series - Hans
> Link: https://git.codelinaro.org/bryan.odonoghue/kernel/-/commit/d4c382c5d6ee153b410a01e172b3e811011d0b14
> - Adds Konrad's Acked-by as indicated
>
> V0:
> This series covers a number of Fixes: all of which are for application to
> stable as well as -next with the exception of the second patch which is a
> fix for a SHA that is still in -next.
>
> Link: https://git.codelinaro.org/bryan.odonoghue/kernel/-/tree/linux-next-23-08-07-db410c-rb3-camss-dts-v3
>
> This series is part of a larger set of fixes, improvements developed/found
> when adding a new SoC.
>
> Link: https://git.codelinaro.org/bryan.odonoghue/kernel/-/tree/lenovo-x13s-v6.5-rc4-x13s-camss-patches
>
> First pass on that larger series is to get all of the current Fixes: in the
> branch out.
>
> Andrey Konovalov (1):
> media: qcom: camss: Fix csid-gen2 for test pattern generator
>
> Bryan O'Donoghue (9):
> media: qcom: camss: Fix pm_domain_on sequence in probe
> media: qcom: camss: Fix V4L2 async notifier error path
> media: qcom: camss: Fix genpd cleanup
> media: qcom: camss: Fix vfe_get() error jump
> media: qcom: camss: Fix VFE-17x vfe_disable_output()
> media: qcom: camss: Fix VFE-480 vfe_disable_output()
> media: qcom: camss: Fix missing vfe_lite clocks check
> media: qcom: camss: Fix invalid clock enable bit disjunction
> media: qcom: camss: Fix set CSI2_RX_CFG1_VC_MODE when VC is greater
> than 3
>
> .../platform/qcom/camss/camss-csid-gen2.c | 11 ++--
> .../qcom/camss/camss-csiphy-3ph-1-0.c | 2 +-
> .../media/platform/qcom/camss/camss-vfe-170.c | 22 +-------
> .../media/platform/qcom/camss/camss-vfe-480.c | 22 +-------
> drivers/media/platform/qcom/camss/camss-vfe.c | 5 +-
> drivers/media/platform/qcom/camss/camss.c | 55 +++++++++++--------
> 6 files changed, 46 insertions(+), 71 deletions(-)
--
Regards,
Laurent Pinchart
