2022-10-17 21:43:55

by Cristian Ciocaltea

[permalink] [raw]
Subject: [PATCH v4 0/3] Enable initial support for StarFive VisionFive V1 SBC

The StarFive VisionFive V1 SBC [1] is similar with the already supported
BeagleV Starlight Beta board, both being based on the StarFive JH7100 SoC.

In addition to documenting the necessary compatibles, this patch series
moves most of the content from jh7100-beaglev-starlight.dts to a new file
jh7100-common.dtsi, to be shared between the two boards.

No other changes are required in order to successfully boot the board.

[1] https://github.com/starfive-tech/VisionFive

Changes in v4:
- Rebased onto v6.1.0-rc1
- Fixed a conflict in patch 2/3 as pinctrl-starfive.h has been renamed
to pinctrl-starfive-jh7100.h

Changes in v3:
- Added Reviewed-by tag from Krzysztof in patch 1/3
- Optimized patch 2/3 by enabling copy detection on "git format-patch",
as indicated by Krzysztof

Changes in v2:
- Simplified documentation by using 'enum' instead of 'const' in
patch 1/3, according to Conor's review
- Added Reviewed-by tags from Conor

Cristian Ciocaltea (3):
dt-bindings: riscv: starfive: Add StarFive VisionFive V1 board
riscv: dts: starfive: Add common DT for JH7100 based boards
riscv: dts: starfive: Add StarFive VisionFive V1 device tree

.../devicetree/bindings/riscv/starfive.yaml | 4 +-
arch/riscv/boot/dts/starfive/Makefile | 2 +-
.../dts/starfive/jh7100-beaglev-starlight.dts | 153 +-----------------
...aglev-starlight.dts => jh7100-common.dtsi} | 3 -
.../jh7100-starfive-visionfive-v1.dts | 20 +++
5 files changed, 25 insertions(+), 157 deletions(-)
copy arch/riscv/boot/dts/starfive/{jh7100-beaglev-starlight.dts => jh7100-common.dtsi} (96%)
create mode 100644 arch/riscv/boot/dts/starfive/jh7100-starfive-visionfive-v1.dts


base-commit: 9abf2313adc1ca1b6180c508c25f22f9395cc780
--
2.38.0


2022-10-17 21:44:47

by Cristian Ciocaltea

[permalink] [raw]
Subject: [PATCH v4 1/3] dt-bindings: riscv: starfive: Add StarFive VisionFive V1 board

Document the compatibles for StarFive VisionFive V1 SBC [1].
The board is based on the StarFive JH7100 SoC.

[1] https://github.com/starfive-tech/VisionFive

Signed-off-by: Cristian Ciocaltea <[email protected]>
Reviewed-by: Conor Dooley <[email protected]>
Reviewed-by: Krzysztof Kozlowski <[email protected]>
---
Documentation/devicetree/bindings/riscv/starfive.yaml | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/riscv/starfive.yaml b/Documentation/devicetree/bindings/riscv/starfive.yaml
index 5b36243fd674..5d3fcee52d59 100644
--- a/Documentation/devicetree/bindings/riscv/starfive.yaml
+++ b/Documentation/devicetree/bindings/riscv/starfive.yaml
@@ -19,7 +19,9 @@ properties:
compatible:
oneOf:
- items:
- - const: beagle,beaglev-starlight-jh7100-r0
+ - enum:
+ - beagle,beaglev-starlight-jh7100-r0
+ - starfive,visionfive-v1
- const: starfive,jh7100

additionalProperties: true
--
2.38.0

2022-10-18 08:26:55

by Matthias Brugger

[permalink] [raw]
Subject: Re: [PATCH v4 1/3] dt-bindings: riscv: starfive: Add StarFive VisionFive V1 board

On Tue, Oct 18, 2022 at 12:05:40AM +0300, Cristian Ciocaltea wrote:
> Document the compatibles for StarFive VisionFive V1 SBC [1].
> The board is based on the StarFive JH7100 SoC.
>
> [1] https://github.com/starfive-tech/VisionFive
>
> Signed-off-by: Cristian Ciocaltea <[email protected]>
> Reviewed-by: Conor Dooley <[email protected]>
> Reviewed-by: Krzysztof Kozlowski <[email protected]>

Reviewed-by: Matthias Brugger <[email protected]>

> ---
> Documentation/devicetree/bindings/riscv/starfive.yaml | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/Documentation/devicetree/bindings/riscv/starfive.yaml b/Documentation/devicetree/bindings/riscv/starfive.yaml
> index 5b36243fd674..5d3fcee52d59 100644
> --- a/Documentation/devicetree/bindings/riscv/starfive.yaml
> +++ b/Documentation/devicetree/bindings/riscv/starfive.yaml
> @@ -19,7 +19,9 @@ properties:
> compatible:
> oneOf:
> - items:
> - - const: beagle,beaglev-starlight-jh7100-r0
> + - enum:
> + - beagle,beaglev-starlight-jh7100-r0
> + - starfive,visionfive-v1
> - const: starfive,jh7100
>
> additionalProperties: true
> --
> 2.38.0
>
>
> _______________________________________________
> linux-riscv mailing list
> [email protected]
> http://lists.infradead.org/mailman/listinfo/linux-riscv

2022-11-16 18:00:09

by Conor Dooley

[permalink] [raw]
Subject: Re: [PATCH v4 0/3] Enable initial support for StarFive VisionFive V1 SBC

On Tue, Oct 18, 2022 at 12:05:39AM +0300, Cristian Ciocaltea wrote:
> The StarFive VisionFive V1 SBC [1] is similar with the already supported
> BeagleV Starlight Beta board, both being based on the StarFive JH7100 SoC.
>
> In addition to documenting the necessary compatibles, this patch series
> moves most of the content from jh7100-beaglev-starlight.dts to a new file
> jh7100-common.dtsi, to be shared between the two boards.
>
> No other changes are required in order to successfully boot the board.
>
> [1] https://github.com/starfive-tech/VisionFive

Hey Cristian,

Apologies for the delay here, we've been sorting out re-routing RISC-V
DT stuff via the SoC tree. That should be sorted now, so I have picked
the series up. Should be at:
https://git.kernel.org/conor/c/a3fe69160c7328a284ebc711ccb689f3beec42dc

Thanks,
Conor.

>
> Changes in v4:
> - Rebased onto v6.1.0-rc1
> - Fixed a conflict in patch 2/3 as pinctrl-starfive.h has been renamed
> to pinctrl-starfive-jh7100.h
>
> Changes in v3:
> - Added Reviewed-by tag from Krzysztof in patch 1/3
> - Optimized patch 2/3 by enabling copy detection on "git format-patch",
> as indicated by Krzysztof
>
> Changes in v2:
> - Simplified documentation by using 'enum' instead of 'const' in
> patch 1/3, according to Conor's review
> - Added Reviewed-by tags from Conor
>
> Cristian Ciocaltea (3):
> dt-bindings: riscv: starfive: Add StarFive VisionFive V1 board
> riscv: dts: starfive: Add common DT for JH7100 based boards
> riscv: dts: starfive: Add StarFive VisionFive V1 device tree
>
> .../devicetree/bindings/riscv/starfive.yaml | 4 +-
> arch/riscv/boot/dts/starfive/Makefile | 2 +-
> .../dts/starfive/jh7100-beaglev-starlight.dts | 153 +-----------------
> ...aglev-starlight.dts => jh7100-common.dtsi} | 3 -
> .../jh7100-starfive-visionfive-v1.dts | 20 +++
> 5 files changed, 25 insertions(+), 157 deletions(-)
> copy arch/riscv/boot/dts/starfive/{jh7100-beaglev-starlight.dts => jh7100-common.dtsi} (96%)
> create mode 100644 arch/riscv/boot/dts/starfive/jh7100-starfive-visionfive-v1.dts
>
>
> base-commit: 9abf2313adc1ca1b6180c508c25f22f9395cc780
> --
> 2.38.0
>

2022-11-16 21:46:14

by Cristian Ciocaltea

[permalink] [raw]
Subject: Re: [PATCH v4 0/3] Enable initial support for StarFive VisionFive V1 SBC

On 11/16/22 19:50, Conor Dooley wrote:
> On Tue, Oct 18, 2022 at 12:05:39AM +0300, Cristian Ciocaltea wrote:
>> The StarFive VisionFive V1 SBC [1] is similar with the already supported
>> BeagleV Starlight Beta board, both being based on the StarFive JH7100 SoC.
>>
>> In addition to documenting the necessary compatibles, this patch series
>> moves most of the content from jh7100-beaglev-starlight.dts to a new file
>> jh7100-common.dtsi, to be shared between the two boards.
>>
>> No other changes are required in order to successfully boot the board.
>>
>> [1] https://github.com/starfive-tech/VisionFive
>
> Hey Cristian,
>
> Apologies for the delay here, we've been sorting out re-routing RISC-V
> DT stuff via the SoC tree. That should be sorted now, so I have picked
> the series up. Should be at:
> https://git.kernel.org/conor/c/a3fe69160c7328a284ebc711ccb689f3beec42dc

Hi Conor,

No problem, thanks for taking care of this!

Regards,
Cristian

> Thanks,
> Conor.
>
>>
>> Changes in v4:
>> - Rebased onto v6.1.0-rc1
>> - Fixed a conflict in patch 2/3 as pinctrl-starfive.h has been renamed
>> to pinctrl-starfive-jh7100.h
>>
>> Changes in v3:
>> - Added Reviewed-by tag from Krzysztof in patch 1/3
>> - Optimized patch 2/3 by enabling copy detection on "git format-patch",
>> as indicated by Krzysztof
>>
>> Changes in v2:
>> - Simplified documentation by using 'enum' instead of 'const' in
>> patch 1/3, according to Conor's review
>> - Added Reviewed-by tags from Conor
>>
>> Cristian Ciocaltea (3):
>> dt-bindings: riscv: starfive: Add StarFive VisionFive V1 board
>> riscv: dts: starfive: Add common DT for JH7100 based boards
>> riscv: dts: starfive: Add StarFive VisionFive V1 device tree
>>
>> .../devicetree/bindings/riscv/starfive.yaml | 4 +-
>> arch/riscv/boot/dts/starfive/Makefile | 2 +-
>> .../dts/starfive/jh7100-beaglev-starlight.dts | 153 +-----------------
>> ...aglev-starlight.dts => jh7100-common.dtsi} | 3 -
>> .../jh7100-starfive-visionfive-v1.dts | 20 +++
>> 5 files changed, 25 insertions(+), 157 deletions(-)
>> copy arch/riscv/boot/dts/starfive/{jh7100-beaglev-starlight.dts => jh7100-common.dtsi} (96%)
>> create mode 100644 arch/riscv/boot/dts/starfive/jh7100-starfive-visionfive-v1.dts
>>
>>
>> base-commit: 9abf2313adc1ca1b6180c508c25f22f9395cc780
>> --
>> 2.38.0
>>