[PATCH 00/10] media: rcar-fcp, vsp1: Enable on R-Car X5H

Paul Elder posted 10 patches 1 week ago
There is a newer version of this series
arch/arm64/boot/dts/renesas/r8a78000.dtsi        | 171 +++++++++++++++++++++++
drivers/clk/renesas/r8a78000-cpg.c               |   2 +
drivers/media/platform/renesas/rcar-fcp.c        | 162 +++++++++++++++++++--
drivers/media/platform/renesas/vsp1/vsp1_drv.c   |  13 +-
drivers/media/platform/renesas/vsp1/vsp1_regs.h  |   3 +-
drivers/pmdomain/renesas/r8a78000-mdlc.c         |  34 ++++-
include/dt-bindings/clock/renesas,r8a78000-cpg.h |   2 +
7 files changed, 372 insertions(+), 15 deletions(-)
[PATCH 00/10] media: rcar-fcp, vsp1: Enable on R-Car X5H
Posted by Paul Elder 1 week ago
This series is based on Geert's
"topic/rcar-x5h-ironhide-scmi-cpg-mdlc-remapping-v3" branch [0]. A
branch including this series is available here [1].

This series enables the FCPV[BI] and the VSP[BI] blocks on the Renesas
R-Car X5H SoC.

This has been tested on an Ironhide with the vsp-tests test suite [2]:
148 tests: 142 passed, 0 failed, 6 skipped

There was some odd behavior with the clocks, where from what I
understood was that the clocks for the FCPV[BI] and VSP[BI] are
always-on, but the tests would hang unless I explicitly enabled...  the
always-on clock (or fsleep). Then after I removed the patch that
explicitly enabled the always-on clock everything ran fine. Just
reporting that this happened just in case it's relevant.

[0] https://git.kernel.org/pub/scm/linux/kernel/git/geert/renesas-drivers.git/log/?h=topic/rcar-x5h-ironhide-scmi-cpg-mdlc-remapping-v3
[1] https://git.ideasonboard.com/epaul/linux/src/branch/epaul/v7.3-rc1/x5h/vsp/fcp
[2] https://git.ideasonboard.com/renesas/vsp-tests/commit/900168f58c9d96be0592c2593f4d8f34edc9c69e

Signed-off-by: Paul Elder <paul.elder+renesas@ideasonboard.com>
---
Paul Elder (10):
      dt-bindings: clock: renesas,r8a78000-cpg: Add CPG_SGD4_VIO_DP_{TX,OTHER}
      clk: renesas: r8a78000: Add clock for FCPV and VSP for SCP FW SDKv4.36
      pmdomain: renesas: r8a78000: Add maps for VIO for SCP FW SDKv4.36
      arm64: renesas: r8a78000: Add FCPV[BI] nodes
      arm64: renesas: r8a78000: Add VSP[BI] nodes
      media: rcar-fcp: Check device revision at probe time
      media: rcar-fcp: Handle resets
      media: vsp1: Support full VSPB on R-Car M3-W, M3-N and E3
      media: vsp1: Document X5H SoC ID
      media: rcar-fcp: Add R-Car X5H support

 arch/arm64/boot/dts/renesas/r8a78000.dtsi        | 171 +++++++++++++++++++++++
 drivers/clk/renesas/r8a78000-cpg.c               |   2 +
 drivers/media/platform/renesas/rcar-fcp.c        | 162 +++++++++++++++++++--
 drivers/media/platform/renesas/vsp1/vsp1_drv.c   |  13 +-
 drivers/media/platform/renesas/vsp1/vsp1_regs.h  |   3 +-
 drivers/pmdomain/renesas/r8a78000-mdlc.c         |  34 ++++-
 include/dt-bindings/clock/renesas,r8a78000-cpg.h |   2 +
 7 files changed, 372 insertions(+), 15 deletions(-)
---
base-commit: 02c5d61866607bce0b3271e7c1b838c53dbedc55
change-id: 20260917-epaul-v7-3-rc1-x5h-vsp-fcp-8afdeb2740f7

Best regards,
-- 
Paul Elder <paul.elder+renesas@ideasonboard.com>
Re: [PATCH 00/10] media: rcar-fcp, vsp1: Enable on R-Car X5H
Posted by Laurent Pinchart 1 week ago
On Thu, Sep 17, 2026 at 09:04:39PM +0900, Paul Elder wrote:
> This series is based on Geert's
> "topic/rcar-x5h-ironhide-scmi-cpg-mdlc-remapping-v3" branch [0]. A
> branch including this series is available here [1].

The series contains a mix of patches ready for upstream and patches that
are not. The cover letter should explain this better to avoid wasting
reviewers' time. Please organize the patches better in v2, mark the ones
not ready for uptreaming with '[DNI]' or a similar prefix, and explain
the rationale in the cover letter.

> This series enables the FCPV[BI] and the VSP[BI] blocks on the Renesas
> R-Car X5H SoC.
> 
> This has been tested on an Ironhide with the vsp-tests test suite [2]:
> 148 tests: 142 passed, 0 failed, 6 skipped
> 
> There was some odd behavior with the clocks, where from what I
> understood was that the clocks for the FCPV[BI] and VSP[BI] are
> always-on, but the tests would hang unless I explicitly enabled...  the
> always-on clock (or fsleep). Then after I removed the patch that
> explicitly enabled the always-on clock everything ran fine. Just
> reporting that this happened just in case it's relevant.
> 
> [0] https://git.kernel.org/pub/scm/linux/kernel/git/geert/renesas-drivers.git/log/?h=topic/rcar-x5h-ironhide-scmi-cpg-mdlc-remapping-v3
> [1] https://git.ideasonboard.com/epaul/linux/src/branch/epaul/v7.3-rc1/x5h/vsp/fcp
> [2] https://git.ideasonboard.com/renesas/vsp-tests/commit/900168f58c9d96be0592c2593f4d8f34edc9c69e
> 
> Signed-off-by: Paul Elder <paul.elder+renesas@ideasonboard.com>
> ---
> Paul Elder (10):
>       dt-bindings: clock: renesas,r8a78000-cpg: Add CPG_SGD4_VIO_DP_{TX,OTHER}
>       clk: renesas: r8a78000: Add clock for FCPV and VSP for SCP FW SDKv4.36
>       pmdomain: renesas: r8a78000: Add maps for VIO for SCP FW SDKv4.36
>       arm64: renesas: r8a78000: Add FCPV[BI] nodes
>       arm64: renesas: r8a78000: Add VSP[BI] nodes
>       media: rcar-fcp: Check device revision at probe time
>       media: rcar-fcp: Handle resets
>       media: vsp1: Support full VSPB on R-Car M3-W, M3-N and E3
>       media: vsp1: Document X5H SoC ID
>       media: rcar-fcp: Add R-Car X5H support
> 
>  arch/arm64/boot/dts/renesas/r8a78000.dtsi        | 171 +++++++++++++++++++++++
>  drivers/clk/renesas/r8a78000-cpg.c               |   2 +
>  drivers/media/platform/renesas/rcar-fcp.c        | 162 +++++++++++++++++++--
>  drivers/media/platform/renesas/vsp1/vsp1_drv.c   |  13 +-
>  drivers/media/platform/renesas/vsp1/vsp1_regs.h  |   3 +-
>  drivers/pmdomain/renesas/r8a78000-mdlc.c         |  34 ++++-
>  include/dt-bindings/clock/renesas,r8a78000-cpg.h |   2 +
>  7 files changed, 372 insertions(+), 15 deletions(-)
> ---
> base-commit: 02c5d61866607bce0b3271e7c1b838c53dbedc55
> change-id: 20260917-epaul-v7-3-rc1-x5h-vsp-fcp-8afdeb2740f7

-- 
Regards,

Laurent Pinchart
Re: [PATCH 00/10] media: rcar-fcp, vsp1: Enable on R-Car X5H
Posted by Laurent Pinchart 1 week ago
On Thu, Sep 17, 2026 at 09:04:39PM +0900, Paul Elder wrote:
> This series is based on Geert's
> "topic/rcar-x5h-ironhide-scmi-cpg-mdlc-remapping-v3" branch [0]. A
> branch including this series is available here [1].
> 
> This series enables the FCPV[BI] and the VSP[BI] blocks on the Renesas
> R-Car X5H SoC.
> 
> This has been tested on an Ironhide with the vsp-tests test suite [2]:
> 148 tests: 142 passed, 0 failed, 6 skipped
> 
> There was some odd behavior with the clocks, where from what I
> understood was that the clocks for the FCPV[BI] and VSP[BI] are
> always-on, but the tests would hang unless I explicitly enabled...  the
> always-on clock (or fsleep). Then after I removed the patch that
> explicitly enabled the always-on clock everything ran fine. Just
> reporting that this happened just in case it's relevant.

Well, it's always worrying, but as you can't reproduce it now it's hard
to debug it. Let's keep an eye on that during further VSP work.

> [0] https://git.kernel.org/pub/scm/linux/kernel/git/geert/renesas-drivers.git/log/?h=topic/rcar-x5h-ironhide-scmi-cpg-mdlc-remapping-v3
> [1] https://git.ideasonboard.com/epaul/linux/src/branch/epaul/v7.3-rc1/x5h/vsp/fcp
> [2] https://git.ideasonboard.com/renesas/vsp-tests/commit/900168f58c9d96be0592c2593f4d8f34edc9c69e
> 
> Signed-off-by: Paul Elder <paul.elder+renesas@ideasonboard.com>
> ---
> Paul Elder (10):
>       dt-bindings: clock: renesas,r8a78000-cpg: Add CPG_SGD4_VIO_DP_{TX,OTHER}
>       clk: renesas: r8a78000: Add clock for FCPV and VSP for SCP FW SDKv4.36
>       pmdomain: renesas: r8a78000: Add maps for VIO for SCP FW SDKv4.36
>       arm64: renesas: r8a78000: Add FCPV[BI] nodes
>       arm64: renesas: r8a78000: Add VSP[BI] nodes
>       media: rcar-fcp: Check device revision at probe time
>       media: rcar-fcp: Handle resets
>       media: vsp1: Support full VSPB on R-Car M3-W, M3-N and E3
>       media: vsp1: Document X5H SoC ID
>       media: rcar-fcp: Add R-Car X5H support
> 
>  arch/arm64/boot/dts/renesas/r8a78000.dtsi        | 171 +++++++++++++++++++++++
>  drivers/clk/renesas/r8a78000-cpg.c               |   2 +
>  drivers/media/platform/renesas/rcar-fcp.c        | 162 +++++++++++++++++++--
>  drivers/media/platform/renesas/vsp1/vsp1_drv.c   |  13 +-
>  drivers/media/platform/renesas/vsp1/vsp1_regs.h  |   3 +-
>  drivers/pmdomain/renesas/r8a78000-mdlc.c         |  34 ++++-
>  include/dt-bindings/clock/renesas,r8a78000-cpg.h |   2 +
>  7 files changed, 372 insertions(+), 15 deletions(-)
> ---
> base-commit: 02c5d61866607bce0b3271e7c1b838c53dbedc55
> change-id: 20260917-epaul-v7-3-rc1-x5h-vsp-fcp-8afdeb2740f7

-- 
Regards,

Laurent Pinchart