2021-12-13 10:29:55

by Daniel Palmer

[permalink] [raw]
Subject: [PATCH 2/3] dt-bindings: arm: mstar: Add compatible for Miyoo Mini

The Miyoo Mini is a SigmaStar SSD202D based retro emulation
device.

Add a compatible for it to the list of infinity2m devices.

Signed-off-by: Daniel Palmer <[email protected]>
Link: http://linux-chenxing.org/infinity2/miyoomini/
---
Documentation/devicetree/bindings/arm/mstar/mstar.yaml | 1 +
1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/arm/mstar/mstar.yaml b/Documentation/devicetree/bindings/arm/mstar/mstar.yaml
index a316eef1b728..8e1a0e99a60b 100644
--- a/Documentation/devicetree/bindings/arm/mstar/mstar.yaml
+++ b/Documentation/devicetree/bindings/arm/mstar/mstar.yaml
@@ -25,6 +25,7 @@ properties:
- enum:
- honestar,ssd201htv2 # Honestar SSD201_HT_V2 devkit
- m5stack,unitv2 # M5Stack UnitV2
+ - miyoo,miyoo-mini # Miyoo Mini
- const: mstar,infinity2m

- description: infinity3 boards
--
2.34.1



2021-12-14 16:09:38

by Daniel Palmer

[permalink] [raw]
Subject: Re: [PATCH 2/3] dt-bindings: arm: mstar: Add compatible for Miyoo Mini

Hi Romain,

On Wed, 15 Dec 2021 at 00:07, Romain Perier <[email protected]> wrote:
>> Signed-off-by: Daniel Palmer <[email protected]>
>> Link: http://linux-chenxing.org/infinity2/miyoomini/
>
>
> 'Link:' should not be used for that purpose. Usually it is used for referencing discussions
> or patches on lore.k.o . This is typically what I used in the last pull request I sent to Arnd for 5.15.

I sort of remember having this pointed out somewhere else and then
looking at the text in submitting patches and thinking it background
info seemed to be what Link was for with the note that if it's a link
to an on list discussion to use a link to lore:

`If related discussions or any other background information behind the
change can be found on the web, add ‘Link:’ tags pointing to it.
....
When linking to mailing list archives, preferably use the
lore.kernel.org message archiver service."

mmm I need to send a v2 anyhow as I messed up the vendor prefix commit
so I'll move the links out into the message.

Cheers,

Daniel

2021-12-15 20:12:29

by Rob Herring

[permalink] [raw]
Subject: Re: [PATCH 2/3] dt-bindings: arm: mstar: Add compatible for Miyoo Mini

On Mon, 13 Dec 2021 19:01:11 +0900, Daniel Palmer wrote:
> The Miyoo Mini is a SigmaStar SSD202D based retro emulation
> device.
>
> Add a compatible for it to the list of infinity2m devices.
>
> Signed-off-by: Daniel Palmer <[email protected]>
> Link: http://linux-chenxing.org/infinity2/miyoomini/
> ---
> Documentation/devicetree/bindings/arm/mstar/mstar.yaml | 1 +
> 1 file changed, 1 insertion(+)
>

Acked-by: Rob Herring <[email protected]>