[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <8d097a0486e94257952600bf6d20975d@SFHDAG5NODE1.st.com>
Date: Thu, 3 Oct 2019 10:05:46 +0000
From: Gerald BAEZA <gerald.baeza@...com>
To: Alexandre TORGUE <alexandre.torgue@...com>,
Jonathan Corbet <corbet@....net>
CC: "mcoquelin.stm32@...il.com" <mcoquelin.stm32@...il.com>,
"linux-doc@...r.kernel.org" <linux-doc@...r.kernel.org>,
"linux-stm32@...md-mailman.stormreply.com"
<linux-stm32@...md-mailman.stormreply.com>,
"linux-arm-kernel@...ts.infradead.org"
<linux-arm-kernel@...ts.infradead.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: RE: [PATCH] Documentation: add link to stm32mp157 docs
Hi Jonathan
> From: Alexandre TORGUE <alexandre.torgue@...com>
> Hi Jonathan,
>
> On 8/27/19 3:48 PM, Jonathan Corbet wrote:
> > On Tue, 27 Aug 2019 12:19:32 +0000
> > Gerald BAEZA <gerald.baeza@...com> wrote:
> >
> >> Link to the online stm32mp157 documentation added in the overview.
> >>
> >> Signed-off-by: Gerald Baeza <gerald.baeza@...com>
> >> ---
> >> Documentation/arm/stm32/stm32mp157-overview.rst | 6 ++++++
> >> 1 file changed, 6 insertions(+)
> >>
> >> diff --git a/Documentation/arm/stm32/stm32mp157-overview.rst
> >> b/Documentation/arm/stm32/stm32mp157-overview.rst
> >> index f62fdc8..8d5a476 100644
> >> --- a/Documentation/arm/stm32/stm32mp157-overview.rst
> >> +++ b/Documentation/arm/stm32/stm32mp157-overview.rst
> >> @@ -14,6 +14,12 @@ It features:
> >> - Standard connectivity, widely inherited from the STM32 MCU family
> >> - Comprehensive security support
> >>
> >> +Resources
> >> +---------
> >> +
> >> +Datasheet and reference manual are publicly available on ST website:
> >> +.. _STM32MP157:
> >> +https://www.st.com/en/microcontrollers-
> microprocessors/stm32mp157.ht
> >> +ml
> >> +
> >
> > Adding the URL is a fine idea. But you don't need the extra syntax to
> > create a link if you're not going to actually make a link out of it.
> > So I'd take the ".. _STM32MP157:" part out and life will be good.
> >
>
> We also did it for older stm32 product. Idea was to not have the "full"
> address but just a shortcut of the link when html file is read. It maybe makes
> no sens ? (if yes we will have to update older stm32 overview :))
Example in https://www.kernel.org/doc/html/latest/arm/stm32/stm32h743-overview.html
Do you agree to continue like this ?
Best regards,
GĂ©rald
> thanks
> Alex
>
>
> > Thanks,
> >
> > jon
> >
Powered by blists - more mailing lists