* Re: [PATCH] docs: Change reference that was to removed file
[not found] <1611051980-3834-1-git-send-email-milan.lakhani@codethink.co.uk>
@ 2021-01-20 0:01 ` Jonathan Corbet
2021-01-20 1:07 ` Rob Herring
0 siblings, 1 reply; 3+ messages in thread
From: Jonathan Corbet @ 2021-01-20 0:01 UTC (permalink / raw)
To: Milan Lakhani, lukas.bulwahn, sudip.mukherjee, linux-safety,
linux-kernel, linux-doc, harryxiyou, alex.shi
Cc: Milan Lakhani, Rob Herring
Milan Lakhani <milan.lakhani@codethink.co.uk> writes:
[CC += Rob]
> There were two references to
> Documentation/devicetree/booting-without-of.rst for dtb format
> information. This file has been removed, so I have changed the links to
> Documentation/devicetree/usage-model.rst for this information, which
> itself points to https://elinux.org/Device_Tree_Usage for an overview of
> the device tree data format.
>
> Signed-off-by: Milan Lakhani <milan.lakhani@codethink.co.uk>
> ---
> Documentation/arm/booting.rst | 2 +-
> Documentation/translations/zh_CN/arm/Booting | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/Documentation/arm/booting.rst b/Documentation/arm/booting.rst
> index a226345..5392d01 100644
> --- a/Documentation/arm/booting.rst
> +++ b/Documentation/arm/booting.rst
> @@ -128,7 +128,7 @@ it. The recommended placement is in the first 16KiB of RAM.
>
> The boot loader must load a device tree image (dtb) into system ram
> at a 64bit aligned address and initialize it with the boot data. The
> -dtb format is documented in Documentation/devicetree/booting-without-of.rst.
> +dtb format is documented in Documentation/devicetree/usage-model.rst.
Except that ... usage-model.rst doesn't actually document the DTB
format. Perhaps we should instead substitute a link that does what's
promised here?
Thanks,
jon
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] docs: Change reference that was to removed file
2021-01-20 0:01 ` [PATCH] docs: Change reference that was to removed file Jonathan Corbet
@ 2021-01-20 1:07 ` Rob Herring
2021-01-20 14:46 ` [linux-safety] " Milan Lakhani
0 siblings, 1 reply; 3+ messages in thread
From: Rob Herring @ 2021-01-20 1:07 UTC (permalink / raw)
To: Jonathan Corbet
Cc: Milan Lakhani, Lukas Bulwahn, sudip.mukherjee, linux-safety,
linux-kernel, Linux Doc Mailing List, Harry Wei, Alex Shi
On Tue, Jan 19, 2021 at 6:01 PM Jonathan Corbet <corbet@lwn.net> wrote:
>
> Milan Lakhani <milan.lakhani@codethink.co.uk> writes:
>
> [CC += Rob]
>
> > There were two references to
> > Documentation/devicetree/booting-without-of.rst for dtb format
> > information. This file has been removed, so I have changed the links to
> > Documentation/devicetree/usage-model.rst for this information, which
> > itself points to https://elinux.org/Device_Tree_Usage for an overview of
> > the device tree data format.
> >
> > Signed-off-by: Milan Lakhani <milan.lakhani@codethink.co.uk>
> > ---
> > Documentation/arm/booting.rst | 2 +-
> > Documentation/translations/zh_CN/arm/Booting | 2 +-
> > 2 files changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/Documentation/arm/booting.rst b/Documentation/arm/booting.rst
> > index a226345..5392d01 100644
> > --- a/Documentation/arm/booting.rst
> > +++ b/Documentation/arm/booting.rst
> > @@ -128,7 +128,7 @@ it. The recommended placement is in the first 16KiB of RAM.
> >
> > The boot loader must load a device tree image (dtb) into system ram
> > at a 64bit aligned address and initialize it with the boot data. The
> > -dtb format is documented in Documentation/devicetree/booting-without-of.rst.
> > +dtb format is documented in Documentation/devicetree/usage-model.rst.
>
> Except that ... usage-model.rst doesn't actually document the DTB
> format. Perhaps we should instead substitute a link that does what's
> promised here?
Yes, it's in the DT spec. I'd point to
'https://www.devicetree.org/specifications/' so it's always going to
show the latest. And that's the link we have elsewhere in the kernel.
usage-model.rst is also something needing updating...
Rob
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [linux-safety] [PATCH] docs: Change reference that was to removed file
2021-01-20 1:07 ` Rob Herring
@ 2021-01-20 14:46 ` Milan Lakhani
0 siblings, 0 replies; 3+ messages in thread
From: Milan Lakhani @ 2021-01-20 14:46 UTC (permalink / raw)
To: Rob Herring, Jonathan Corbet
Cc: Lukas Bulwahn, sudip.mukherjee, linux-safety, linux-kernel,
Linux Doc Mailing List, Harry Wei, Alex Shi
Thanks Rob, Jon, I've submitted a new patch pointing the two broken
links and also usage-model.rst all to
https://www.devicetree.org.specifications/.
Milan
On 20/01/2021 01:07, Rob Herring wrote:
> On Tue, Jan 19, 2021 at 6:01 PM Jonathan Corbet <corbet@lwn.net> wrote:
>> Milan Lakhani <milan.lakhani@codethink.co.uk> writes:
>>
>> [CC += Rob]
>>
>>> There were two references to
>>> Documentation/devicetree/booting-without-of.rst for dtb format
>>> information. This file has been removed, so I have changed the links to
>>> Documentation/devicetree/usage-model.rst for this information, which
>>> itself points to https://elinux.org/Device_Tree_Usage for an overview of
>>> the device tree data format.
>>>
>>> Signed-off-by: Milan Lakhani <milan.lakhani@codethink.co.uk>
>>> ---
>>> Documentation/arm/booting.rst | 2 +-
>>> Documentation/translations/zh_CN/arm/Booting | 2 +-
>>> 2 files changed, 2 insertions(+), 2 deletions(-)
>>>
>>> diff --git a/Documentation/arm/booting.rst b/Documentation/arm/booting.rst
>>> index a226345..5392d01 100644
>>> --- a/Documentation/arm/booting.rst
>>> +++ b/Documentation/arm/booting.rst
>>> @@ -128,7 +128,7 @@ it. The recommended placement is in the first 16KiB of RAM.
>>>
>>> The boot loader must load a device tree image (dtb) into system ram
>>> at a 64bit aligned address and initialize it with the boot data. The
>>> -dtb format is documented in Documentation/devicetree/booting-without-of.rst.
>>> +dtb format is documented in Documentation/devicetree/usage-model.rst.
>> Except that ... usage-model.rst doesn't actually document the DTB
>> format. Perhaps we should instead substitute a link that does what's
>> promised here?
> Yes, it's in the DT spec. I'd point to
> 'https://www.devicetree.org/specifications/' so it's always going to
> show the latest. And that's the link we have elsewhere in the kernel.
>
> usage-model.rst is also something needing updating...
>
> Rob
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#235): https://lists.elisa.tech/g/linux-safety/message/235
> Mute This Topic: https://lists.elisa.tech/mt/79948363/4617862
> Group Owner: linux-safety+owner@lists.elisa.tech
> Unsubscribe: https://lists.elisa.tech/g/linux-safety/unsub [milan.lakhani@codethink.co.uk]
> -=-=-=-=-=-=-=-=-=-=-=-
>
>
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-01-20 15:07 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <1611051980-3834-1-git-send-email-milan.lakhani@codethink.co.uk>
2021-01-20 0:01 ` [PATCH] docs: Change reference that was to removed file Jonathan Corbet
2021-01-20 1:07 ` Rob Herring
2021-01-20 14:46 ` [linux-safety] " Milan Lakhani
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 NNTP newsgroup(s).