MIME-Version: 1.0
References: <20211213100112.1791192-1-daniel@0x0f.com> <20211213100112.1791192-3-daniel@0x0f.com>
 <CABgxDo+8VK+HQVfts6gxLnm1xW5fBog5rEfvoLN+tjk6KYwMTw@mail.gmail.com>
In-Reply-To: <CABgxDo+8VK+HQVfts6gxLnm1xW5fBog5rEfvoLN+tjk6KYwMTw@mail.gmail.com>
From:   Daniel Palmer <daniel@0x0f.com>
Date:   Wed, 15 Dec 2021 01:12:37 +0900
Message-ID: <CAFr9PXmihQSeDagG7Yt7kfvpJxurjiLsnPPB68uvKbQh52m7eg@mail.gmail.com>
Subject: Re: [PATCH 2/3] dt-bindings: arm: mstar: Add compatible for Miyoo Mini
To:     Romain Perier <romain.perier@gmail.com>
Cc:     devicetree <devicetree@vger.kernel.org>,
        linux-arm-kernel <linux-arm-kernel@lists.infradead.org>,
        Rob Herring <robh+dt@kernel.org>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

Hi Romain,

On Wed, 15 Dec 2021 at 00:07, Romain Perier <romain.perier@gmail.com> wrote=
:
>> Signed-off-by: Daniel Palmer <daniel@0x0f.com>
>> Link: http://linux-chenxing.org/infinity2/miyoomini/
>
>
> 'Link:' should not be used for that purpose. Usually it is used for refer=
encing discussions
> or patches on lore.k.o . This is typically what I used in the last pull r=
equest 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 =E2=80=98Link:=E2=80=99 tags pointing t=
o 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