Linux-RISC-V Archive mirror
 help / color / mirror / Atom feed
* [PATCH v6 0/2] add initial Milk-V Duo S board support
@ 2024-04-21  5:57 michael.opdenacker
  2024-04-21  5:57 ` [PATCH v6 1/2] dt-bindings: riscv: add Milk-V Duo S board compatibles michael.opdenacker
  2024-04-21  5:57 ` [PATCH v6 2/2] riscv: dts: sophgo: add initial Milk-V Duo S board support michael.opdenacker
  0 siblings, 2 replies; 8+ messages in thread
From: michael.opdenacker @ 2024-04-21  5:57 UTC (permalink / raw
  To: Paul Walmsley, Palmer Dabbelt, Albert Ou, Chen Wang,
	Inochi Amaoto
  Cc: Michael Opdenacker, linux-riscv

From: Michael Opdenacker <michael.opdenacker@bootlin.com>

This adds an initial device tree for the Milk-V Duo S board.
Last tested on linux-next 20240419

Patch produced against the "for-next" branch of the sophgo "linux" tree:
https://github.com/sophgo/linux.git

- Changes in V6
  - Change board compatible from "milkv,duos" to "milkv,duo-s"
  - Rename sg2000-milkv-duos.dts to cv1812h-milkv-duo-s.dts
  - Remove redundant line from board DTS:
    device_type = "memory";
    It was already set in cv1812h.dtsi

- Changes in V5
  - Fix the commit message describing the board DTS.
    MMC is not supported yet.

- Changes in V4
  - Remove mmc device (&sdhci0) as not working yet with linux-next
  - Use "sophgo,cv1812h" compatible as on the Huashan Pi board.
    Should I call the DTS cv1812h-milkv-duos.dts instead
    of sg2000-milkv-duos.dts?

- Changes in V3
  - Fix enum in devicetree/bindings/riscv/sophgo.yaml

- Changes in V2:
  - Add a first patch to register the new compatible string
    for this board.
  - Use a correct SoC compatible string
  - Add "disable-wp" (disable write protect) property
    to enable write access to the external SD card.

Michael Opdenacker (2):
  dt-bindings: riscv: add Milk-V Duo S board compatibles
  riscv: dts: sophgo: add initial Milk-V Duo S board support

 .../devicetree/bindings/riscv/sophgo.yaml     |  1 +
 arch/riscv/boot/dts/sophgo/Makefile           |  1 +
 .../boot/dts/sophgo/cv1812h-milkv-duo-s.dts   | 33 +++++++++++++++++++
 3 files changed, 35 insertions(+)
 create mode 100644 arch/riscv/boot/dts/sophgo/cv1812h-milkv-duo-s.dts

-- 
2.34.1


_______________________________________________
linux-riscv mailing list
linux-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-riscv

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [PATCH v6 1/2] dt-bindings: riscv: add Milk-V Duo S board compatibles
  2024-04-21  5:57 [PATCH v6 0/2] add initial Milk-V Duo S board support michael.opdenacker
@ 2024-04-21  5:57 ` michael.opdenacker
  2024-04-21 14:12   ` Krzysztof Kozlowski
  2024-04-21  5:57 ` [PATCH v6 2/2] riscv: dts: sophgo: add initial Milk-V Duo S board support michael.opdenacker
  1 sibling, 1 reply; 8+ messages in thread
From: michael.opdenacker @ 2024-04-21  5:57 UTC (permalink / raw
  To: Conor Dooley, Rob Herring, Krzysztof Kozlowski, Chen Wang,
	Inochi Amaoto, Paul Walmsley, Palmer Dabbelt, Albert Ou, Chao Wei
  Cc: Michael Opdenacker, linux-riscv, devicetree, linux-kernel

From: Michael Opdenacker <michael.opdenacker@bootlin.com>

Document the compatible strings for the Milk-V Duo S board[1] which uses
the SOPHGO SG2000 SoC, compatible with the CV1812H SoC[2].

Link: https://milkv.io/duo-s [1]
Link: https://en.sophgo.com/product/introduce/cv181xH.html [2]

Signed-off-by: Michael Opdenacker <michael.opdenacker@bootlin.com>
---
 Documentation/devicetree/bindings/riscv/sophgo.yaml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/riscv/sophgo.yaml b/Documentation/devicetree/bindings/riscv/sophgo.yaml
index 9bc813dad098..79546aa21021 100644
--- a/Documentation/devicetree/bindings/riscv/sophgo.yaml
+++ b/Documentation/devicetree/bindings/riscv/sophgo.yaml
@@ -24,6 +24,7 @@ properties:
           - const: sophgo,cv1800b
       - items:
           - enum:
+              - milkv,duo-s
               - sophgo,huashan-pi
           - const: sophgo,cv1812h
       - items:
-- 
2.34.1


_______________________________________________
linux-riscv mailing list
linux-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-riscv

^ permalink raw reply related	[flat|nested] 8+ messages in thread

* [PATCH v6 2/2] riscv: dts: sophgo: add initial Milk-V Duo S board support
  2024-04-21  5:57 [PATCH v6 0/2] add initial Milk-V Duo S board support michael.opdenacker
  2024-04-21  5:57 ` [PATCH v6 1/2] dt-bindings: riscv: add Milk-V Duo S board compatibles michael.opdenacker
@ 2024-04-21  5:57 ` michael.opdenacker
  1 sibling, 0 replies; 8+ messages in thread
From: michael.opdenacker @ 2024-04-21  5:57 UTC (permalink / raw
  To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Chen Wang,
	Inochi Amaoto, Paul Walmsley, Palmer Dabbelt, Albert Ou
  Cc: Michael Opdenacker, devicetree, linux-riscv, linux-kernel

From: Michael Opdenacker <michael.opdenacker@bootlin.com>

This adds initial support for the Milk-V Duo S board
(https://milkv.io/duo-s), enabling the serial port,
making it possible to boot Linux to the command line.

Link: https://lore.kernel.org/linux-riscv/171266958507.1032617.9460749136730849811.robh@kernel.org/T/#t

Signed-off-by: Michael Opdenacker <michael.opdenacker@bootlin.com>
---
 arch/riscv/boot/dts/sophgo/Makefile           |  1 +
 .../boot/dts/sophgo/cv1812h-milkv-duo-s.dts   | 33 +++++++++++++++++++
 2 files changed, 34 insertions(+)
 create mode 100644 arch/riscv/boot/dts/sophgo/cv1812h-milkv-duo-s.dts

diff --git a/arch/riscv/boot/dts/sophgo/Makefile b/arch/riscv/boot/dts/sophgo/Makefile
index 57ad82a61ea6..a410f6af03fc 100644
--- a/arch/riscv/boot/dts/sophgo/Makefile
+++ b/arch/riscv/boot/dts/sophgo/Makefile
@@ -1,4 +1,5 @@
 # SPDX-License-Identifier: GPL-2.0
 dtb-$(CONFIG_ARCH_SOPHGO) += cv1800b-milkv-duo.dtb
 dtb-$(CONFIG_ARCH_SOPHGO) += cv1812h-huashan-pi.dtb
+dtb-$(CONFIG_ARCH_SOPHGO) += cv1812h-milkv-duo-s.dtb
 dtb-$(CONFIG_ARCH_SOPHGO) += sg2042-milkv-pioneer.dtb
diff --git a/arch/riscv/boot/dts/sophgo/cv1812h-milkv-duo-s.dts b/arch/riscv/boot/dts/sophgo/cv1812h-milkv-duo-s.dts
new file mode 100644
index 000000000000..0f2991d8c8f5
--- /dev/null
+++ b/arch/riscv/boot/dts/sophgo/cv1812h-milkv-duo-s.dts
@@ -0,0 +1,33 @@
+// SPDX-License-Identifier: (GPL-2.0 OR MIT)
+/*
+ * Copyright (C) 2024 Michael Opdenacker <michael.opdenacker@bootlin.com>
+ */
+
+/dts-v1/;
+
+#include "cv1812h.dtsi"
+
+/ {
+	model = "Milk-V Duo S";
+	compatible = "milkv,duo-s", "sophgo,cv1812h";
+
+	aliases {
+		serial0 = &uart0;
+	};
+
+	chosen {
+		stdout-path = "serial0:115200n8";
+	};
+
+	memory@80000000 {
+		reg = <0x80000000 0x20000000>;
+	};
+};
+
+&osc {
+	clock-frequency = <25000000>;
+};
+
+&uart0 {
+	status = "okay";
+};
-- 
2.34.1


_______________________________________________
linux-riscv mailing list
linux-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-riscv

^ permalink raw reply related	[flat|nested] 8+ messages in thread

* Re: [PATCH v6 1/2] dt-bindings: riscv: add Milk-V Duo S board compatibles
  2024-04-21  5:57 ` [PATCH v6 1/2] dt-bindings: riscv: add Milk-V Duo S board compatibles michael.opdenacker
@ 2024-04-21 14:12   ` Krzysztof Kozlowski
  2024-04-21 18:55     ` Michael Opdenacker
  0 siblings, 1 reply; 8+ messages in thread
From: Krzysztof Kozlowski @ 2024-04-21 14:12 UTC (permalink / raw
  To: michael.opdenacker, Conor Dooley, Rob Herring,
	Krzysztof Kozlowski, Chen Wang, Inochi Amaoto, Paul Walmsley,
	Palmer Dabbelt, Albert Ou, Chao Wei
  Cc: linux-riscv, devicetree, linux-kernel

On 21/04/2024 07:57, michael.opdenacker@bootlin.com wrote:
> From: Michael Opdenacker <michael.opdenacker@bootlin.com>
> 
> Document the compatible strings for the Milk-V Duo S board[1] which uses
> the SOPHGO SG2000 SoC, compatible with the CV1812H SoC[2].
> 
> Link: https://milkv.io/duo-s [1]
> Link: https://en.sophgo.com/product/introduce/cv181xH.html [2]
> 
> Signed-off-by: Michael Opdenacker <michael.opdenacker@bootlin.com>
> ---

Where is the changelog? It's v6!


>  Documentation/devicetree/bindings/riscv/sophgo.yaml | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/Documentation/devicetree/bindings/riscv/sophgo.yaml b/Documentation/devicetree/bindings/riscv/sophgo.yaml
> index 9bc813dad098..79546aa21021 100644
> --- a/Documentation/devicetree/bindings/riscv/sophgo.yaml
> +++ b/Documentation/devicetree/bindings/riscv/sophgo.yaml
> @@ -24,6 +24,7 @@ properties:
>            - const: sophgo,cv1800b
>        - items:
>            - enum:
> +              - milkv,duo-s

? It was duo... and duos... now weirdly duo-s without explanation.

>                - sophgo,huashan-pi
>            - const: sophgo,cv1812h

It was CV1800 before... and you were asked for SoC compatible.

Please reach to your Bootlin folks to explain you how to send new
versions of patchsets and how to use compatibles.

Best regards,
Krzysztof


_______________________________________________
linux-riscv mailing list
linux-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-riscv

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH v6 1/2] dt-bindings: riscv: add Milk-V Duo S board compatibles
  2024-04-21 14:12   ` Krzysztof Kozlowski
@ 2024-04-21 18:55     ` Michael Opdenacker
  2024-04-22  4:24       ` Krzysztof Kozlowski
  0 siblings, 1 reply; 8+ messages in thread
From: Michael Opdenacker @ 2024-04-21 18:55 UTC (permalink / raw
  To: Krzysztof Kozlowski, Conor Dooley, Rob Herring,
	Krzysztof Kozlowski, Chen Wang, Inochi Amaoto, Paul Walmsley,
	Palmer Dabbelt, Albert Ou, Chao Wei
  Cc: michael.opdenacker, linux-riscv, devicetree, linux-kernel

Hi Krzysztof

On 4/21/24 at 16:12, Krzysztof Kozlowski wrote:
> On 21/04/2024 07:57, michael.opdenacker@bootlin.com wrote:
>> From: Michael Opdenacker <michael.opdenacker@bootlin.com>
>>
>> Document the compatible strings for the Milk-V Duo S board[1] which uses
>> the SOPHGO SG2000 SoC, compatible with the CV1812H SoC[2].
>>
>> Link: https://milkv.io/duo-s [1]
>> Link: https://en.sophgo.com/product/introduce/cv181xH.html [2]
>>
>> Signed-off-by: Michael Opdenacker <michael.opdenacker@bootlin.com>
>> ---
> Where is the changelog? It's v6!

Oops, it was in my cover letter 
(https://lore.kernel.org/linux-riscv/20240421055710.143617-1-michael.opdenacker@bootlin.com/T/#t), 
which you didn't get because of get-maintainer.pl. As the recipients are 
patch dependent, am I supposed to describe the changes in all patches 
and not only in the cover letter?

Cheers
Michael.

-- 
Michael Opdenacker, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com


_______________________________________________
linux-riscv mailing list
linux-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-riscv

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH v6 1/2] dt-bindings: riscv: add Milk-V Duo S board compatibles
  2024-04-21 18:55     ` Michael Opdenacker
@ 2024-04-22  4:24       ` Krzysztof Kozlowski
  2024-04-22  6:23         ` Michael Opdenacker
  0 siblings, 1 reply; 8+ messages in thread
From: Krzysztof Kozlowski @ 2024-04-22  4:24 UTC (permalink / raw
  To: Michael Opdenacker, Conor Dooley, Rob Herring,
	Krzysztof Kozlowski, Chen Wang, Inochi Amaoto, Paul Walmsley,
	Palmer Dabbelt, Albert Ou, Chao Wei
  Cc: linux-riscv, devicetree, linux-kernel

On 21/04/2024 20:55, Michael Opdenacker wrote:
> Hi Krzysztof
> 
> On 4/21/24 at 16:12, Krzysztof Kozlowski wrote:
>> On 21/04/2024 07:57, michael.opdenacker@bootlin.com wrote:
>>> From: Michael Opdenacker <michael.opdenacker@bootlin.com>
>>>
>>> Document the compatible strings for the Milk-V Duo S board[1] which uses
>>> the SOPHGO SG2000 SoC, compatible with the CV1812H SoC[2].
>>>
>>> Link: https://milkv.io/duo-s [1]
>>> Link: https://en.sophgo.com/product/introduce/cv181xH.html [2]
>>>
>>> Signed-off-by: Michael Opdenacker <michael.opdenacker@bootlin.com>
>>> ---
>> Where is the changelog? It's v6!
> 
> Oops, it was in my cover letter 
> (https://lore.kernel.org/linux-riscv/20240421055710.143617-1-michael.opdenacker@bootlin.com/T/#t), 
> which you didn't get because of get-maintainer.pl. As the recipients are 
> patch dependent, am I supposed to describe the changes in all patches 
> and not only in the cover letter?

If you do not send the changelog to everyone, then yes, please describe
respective changes in each patch.

It's your choice, but if you decide not to send cover letter to some
folks, then *it does not exist* for such folks.

Best regards,
Krzysztof


_______________________________________________
linux-riscv mailing list
linux-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-riscv

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH v6 1/2] dt-bindings: riscv: add Milk-V Duo S board compatibles
  2024-04-22  4:24       ` Krzysztof Kozlowski
@ 2024-04-22  6:23         ` Michael Opdenacker
  2024-04-22  7:49           ` Cristian Ciocaltea
  0 siblings, 1 reply; 8+ messages in thread
From: Michael Opdenacker @ 2024-04-22  6:23 UTC (permalink / raw
  To: Krzysztof Kozlowski, Conor Dooley, Rob Herring,
	Krzysztof Kozlowski, Chen Wang, Inochi Amaoto, Paul Walmsley,
	Palmer Dabbelt, Albert Ou, Chao Wei
  Cc: michael.opdenacker, linux-riscv, devicetree, linux-kernel


On 4/22/24 at 06:24, Krzysztof Kozlowski wrote:
> On 21/04/2024 20:55, Michael Opdenacker wrote:
>> Hi Krzysztof
>>
>> On 4/21/24 at 16:12, Krzysztof Kozlowski wrote:
>>> On 21/04/2024 07:57, michael.opdenacker@bootlin.com wrote:
>>>> From: Michael Opdenacker <michael.opdenacker@bootlin.com>
>>>>
>>>> Document the compatible strings for the Milk-V Duo S board[1] which uses
>>>> the SOPHGO SG2000 SoC, compatible with the CV1812H SoC[2].
>>>>
>>>> Link: https://milkv.io/duo-s [1]
>>>> Link: https://en.sophgo.com/product/introduce/cv181xH.html [2]
>>>>
>>>> Signed-off-by: Michael Opdenacker <michael.opdenacker@bootlin.com>
>>>> ---
>>> Where is the changelog? It's v6!
>> Oops, it was in my cover letter
>> (https://lore.kernel.org/linux-riscv/20240421055710.143617-1-michael.opdenacker@bootlin.com/T/#t),
>> which you didn't get because of get-maintainer.pl. As the recipients are
>> patch dependent, am I supposed to describe the changes in all patches
>> and not only in the cover letter?
> If you do not send the changelog to everyone, then yes, please describe
> respective changes in each patch.

Ok, will do, thanks!

>
> It's your choice, but if you decide not to send cover letter to some
> folks, then *it does not exist* for such folks.

Well, actually, I shouldn't have let get-maintainer.pl decide for me. Is 
there a way to send the cover-letter to the union of people who will 
receive the individual patches?
Cheers
Michael.

-- 
Michael Opdenacker, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com


_______________________________________________
linux-riscv mailing list
linux-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-riscv

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH v6 1/2] dt-bindings: riscv: add Milk-V Duo S board compatibles
  2024-04-22  6:23         ` Michael Opdenacker
@ 2024-04-22  7:49           ` Cristian Ciocaltea
  0 siblings, 0 replies; 8+ messages in thread
From: Cristian Ciocaltea @ 2024-04-22  7:49 UTC (permalink / raw
  To: Michael Opdenacker, Krzysztof Kozlowski, Conor Dooley,
	Rob Herring, Krzysztof Kozlowski, Chen Wang, Inochi Amaoto,
	Paul Walmsley, Palmer Dabbelt, Albert Ou, Chao Wei
  Cc: linux-riscv, devicetree, linux-kernel

Hi Michael,

On 4/22/24 9:23 AM, Michael Opdenacker wrote:
> 
> On 4/22/24 at 06:24, Krzysztof Kozlowski wrote:
>> On 21/04/2024 20:55, Michael Opdenacker wrote:
>>> Hi Krzysztof
>>>
>>> On 4/21/24 at 16:12, Krzysztof Kozlowski wrote:
>>>> On 21/04/2024 07:57, michael.opdenacker@bootlin.com wrote:
>>>>> From: Michael Opdenacker <michael.opdenacker@bootlin.com>
>>>>>
>>>>> Document the compatible strings for the Milk-V Duo S board[1] which
>>>>> uses
>>>>> the SOPHGO SG2000 SoC, compatible with the CV1812H SoC[2].
>>>>>
>>>>> Link: https://milkv.io/duo-s [1]
>>>>> Link: https://en.sophgo.com/product/introduce/cv181xH.html [2]
>>>>>
>>>>> Signed-off-by: Michael Opdenacker <michael.opdenacker@bootlin.com>
>>>>> ---
>>>> Where is the changelog? It's v6!
>>> Oops, it was in my cover letter
>>> (https://lore.kernel.org/linux-riscv/20240421055710.143617-1-michael.opdenacker@bootlin.com/T/#t),
>>> which you didn't get because of get-maintainer.pl. As the recipients are
>>> patch dependent, am I supposed to describe the changes in all patches
>>> and not only in the cover letter?
>> If you do not send the changelog to everyone, then yes, please describe
>> respective changes in each patch.
> 
> Ok, will do, thanks!
> 
>>
>> It's your choice, but if you decide not to send cover letter to some
>> folks, then *it does not exist* for such folks.
> 
> Well, actually, I shouldn't have let get-maintainer.pl decide for me. Is
> there a way to send the cover-letter to the union of people who will
> receive the individual patches?

You may call get-maintainer.pl once for all the patches in the series
(e.g. use a glob pattern to select all needed files) and then manually
edit the cover letter to provide the 'To:' and 'Cc:' fields.

Additionally, you could use '--to-cover --cc-cover' arguments passed to
'git send-email' so that all those recipients will receive all patches.

Alternatively, you may try the b4 utility [1], which is able to automate
most of the manual steps.

Hope this helps!

Regards,
Cristian

[1]: https://b4.docs.kernel.org/en/latest/contributor/prep.html

_______________________________________________
linux-riscv mailing list
linux-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-riscv

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2024-04-22  7:49 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-04-21  5:57 [PATCH v6 0/2] add initial Milk-V Duo S board support michael.opdenacker
2024-04-21  5:57 ` [PATCH v6 1/2] dt-bindings: riscv: add Milk-V Duo S board compatibles michael.opdenacker
2024-04-21 14:12   ` Krzysztof Kozlowski
2024-04-21 18:55     ` Michael Opdenacker
2024-04-22  4:24       ` Krzysztof Kozlowski
2024-04-22  6:23         ` Michael Opdenacker
2024-04-22  7:49           ` Cristian Ciocaltea
2024-04-21  5:57 ` [PATCH v6 2/2] riscv: dts: sophgo: add initial Milk-V Duo S board support michael.opdenacker

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).