LinuxLists.cc - [PATCH v2 0/5] docs: arm: Improvements to Marvell SoC documentation

2021-02-03 23:56:07

Subject: [PATCH v2 0/5] docs: arm: Improvements to Marvell SoC documentation

Hi,

please consider applying the patches chained to this message.

The objective is to deal with the a large amount of dead links to
material that often comes handy in marvel.rst; and improve some details
along the way.

The most important change since v1 is the removal of "[PATCH 1/5] docs:
arm: marvell: turn the automatic links into labels" patch and replacing
the URLs inline instead. Detailed changelogs in individual patches.

Thank you
Lubo

2021-02-03 23:56:18

by Lubomir Rintel

[permalink] [raw]

Subject: [PATCH v2 5/5] docs: arm: marvell: rename marvel.rst to marvell.rst

This company is not the superheroes you're looking for.

Signed-off-by: Lubomir Rintel <[email protected]>
---
Documentation/arm/{marvel.rst => marvell.rst} | 0
1 file changed, 0 insertions(+), 0 deletions(-)
rename Documentation/arm/{marvel.rst => marvell.rst} (100%)

diff --git a/Documentation/arm/marvel.rst b/Documentation/arm/marvell.rst
similarity index 100%
rename from Documentation/arm/marvel.rst
rename to Documentation/arm/marvell.rst
--
2.29.2

2021-02-03 23:56:38

by Lubomir Rintel

[permalink] [raw]

Subject: [PATCH v2 1/5] docs: arm: marvell: drop some dead links

Just remove these; there's good chance there wasn't anything useful
there anyway.

Signed-off-by: Lubomir Rintel <[email protected]>

---
Changes since v1:
- Adjust for removal of "[PATCH 1/5] docs: arm: marvell: turn the automatic
links into labels"
- Split off the hunk that fixes 38x functional spec link

Documentation/arm/marvel.rst | 25 -------------------------
1 file changed, 25 deletions(-)

diff --git a/Documentation/arm/marvel.rst b/Documentation/arm/marvel.rst
index 16ab2eb085b86..502a1b89a2c85 100644
--- a/Documentation/arm/marvel.rst
+++ b/Documentation/arm/marvel.rst
@@ -63,8 +63,6 @@ Kirkwood family
- Product Brief : http://www.marvell.com/embedded-processors/kirkwood/assets/88F6281-004_ver1.pdf
- Hardware Spec : http://www.marvell.com/embedded-processors/kirkwood/assets/HW_88F6281_OpenSource.pdf
- Functional Spec: http://www.marvell.com/embedded-processors/kirkwood/assets/FS_88F6180_9x_6281_OpenSource.pdf
- Homepage:
- http://www.marvell.com/embedded-processors/kirkwood/
Core:
Feroceon 88fr131 ARMv5 compatible
Linux kernel mach directory:
@@ -126,7 +124,6 @@ EBU Armada family
- 88F6820 Armada 385
- 88F6828 Armada 388

- - Product infos: http://www.marvell.com/embedded-processors/armada-38x/
- Functional Spec: https://marvellcorp.wufoo.com/forms/marvell-armada-38x-functional-specifications/

Core:
@@ -136,8 +133,6 @@ EBU Armada family
- 88F6920 Armada 390
- 88F6928 Armada 398

- - Product infos: http://www.marvell.com/embedded-processors/armada-39x/
-
Core:
ARM Cortex-A9

@@ -179,9 +174,6 @@ EBU Armada family ARMv8
Core:
ARM Cortex A53 (ARMv8)

- Homepage:
- http://www.marvell.com/embedded-processors/armada-3700/
-
Product Brief:
http://www.marvell.com/embedded-processors/assets/PB-88F3700-FNL.pdf

@@ -194,9 +186,6 @@ EBU Armada family ARMv8

Core: ARM Cortex A72

- Homepage:
- http://www.marvell.com/embedded-processors/armada-70xx/
-
Product Brief:
- http://www.marvell.com/embedded-processors/assets/Armada7020PB-Jan2016.pdf
- http://www.marvell.com/embedded-processors/assets/Armada7040PB-Jan2016.pdf
@@ -210,9 +199,6 @@ EBU Armada family ARMv8
Core:
ARM Cortex A72

- Homepage:
- http://www.marvell.com/embedded-processors/armada-80xx/
-
Product Brief:
- http://www.marvell.com/embedded-processors/assets/Armada8020PB-Jan2016.pdf
- http://www.marvell.com/embedded-processors/assets/Armada8040PB-Jan2016.pdf
@@ -229,9 +215,6 @@ Avanta family
- 88F6550
- 88F6560

- Homepage:
- http://www.marvell.com/broadband/
-
Product Brief:
http://www.marvell.com/broadband/assets/Marvell_Avanta_88F6510_305_060-001_product_brief.pdf

@@ -251,9 +234,6 @@ Storage family
Armada SP:
- 88RC1580

- Product infos:
- http://www.marvell.com/storage/armada-sp/
-
Core:
Sheeva ARMv7 comatible Quad-core PJ4C

@@ -274,9 +254,6 @@ Dove family (application processor)
Functional Spec:
http://www.marvell.com/application-processors/armada-500/assets/Armada-510-Functional-Spec.pdf

- Homepage:
- http://www.marvell.com/application-processors/armada-500/
-
Core:
ARMv7 compatible

@@ -348,7 +325,6 @@ MMP/MMP2/MMP3 family (communication processor)
- Application processor only
- Core: ARMv5 compatible Marvell PJ1 88sv331 (Mohawk)
- PXA910/PXA920
- - Homepage : http://www.marvell.com/communication-processors/pxa910/
- Product Brief : http://www.marvell.com/communication-processors/pxa910/assets/Marvell_PXA910_Platform-001_PB_final.pdf
- Application processor with Communication processor
- Core: ARMv5 compatible Marvell PJ1 88sv331 (Mohawk)
@@ -394,7 +370,6 @@ Berlin family (Multimedia Solutions)
- Flavors:
- 88DE3010, Armada 1000 (no Linux support)
- Core: Marvell PJ1 (ARMv5TE), Dual-core
- - Product Brief: http://www.marvell.com.cn/digital-entertainment/assets/armada_1000_pb.pdf
- 88DE3005, Armada 1500 Mini
- Design name: BG2CD
- Core: ARM Cortex-A9, PL310 L2CC
--
2.29.2

2021-02-03 23:57:24

by Lubomir Rintel

[permalink] [raw]

Subject: [PATCH v2 2/5] docs: arm: marvell: fix 38x functional spec link

This one went away, but the document is still available.

Signed-off-by: Lubomir Rintel <[email protected]>

---
Changes since v1:
- Split this off from "[PATCH] docs: arm: marvell: drop some dead links"

Documentation/arm/marvel.rst | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/arm/marvel.rst b/Documentation/arm/marvel.rst
index 502a1b89a2c85..ab0c38dc108a8 100644
--- a/Documentation/arm/marvel.rst
+++ b/Documentation/arm/marvel.rst
@@ -124,7 +124,7 @@ EBU Armada family
- 88F6820 Armada 385
- 88F6828 Armada 388

- - Functional Spec: https://marvellcorp.wufoo.com/forms/marvell-armada-38x-functional-specifications/
+ - Functional Spec: https://www.marvell.com/content/dam/marvell/en/public-collateral/embedded-processors/marvell-embedded-processors-armada-38x-functional-specifications-2015-11.pdf

Core:
ARM Cortex-A9
--
2.29.2

2021-02-03 23:57:59

by Lubomir Rintel

[permalink] [raw]

Subject: [PATCH v2 3/5] docs: arm: marvell: replace stale links with archive links

2021-02-03 23:59:09

by Lubomir Rintel

[permalink] [raw]

Subject: [PATCH v2 4/5] docs: arm: marvell: clarify some unimportant Armada 6x0 details

MMP2 is used in XO-1.75 and MMP3 is now supported in mainline.

Signed-off-by: Lubomir Rintel <[email protected]>
---
Documentation/arm/marvel.rst | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/Documentation/arm/marvel.rst b/Documentation/arm/marvel.rst
index bcf3f4e3e8faf..d83917f226376 100644
--- a/Documentation/arm/marvel.rst
+++ b/Documentation/arm/marvel.rst
@@ -329,13 +329,12 @@ MMP/MMP2/MMP3 family (communication processor)
- Product Brief : https://archive.org/download/marvell-pxa910-pb/Marvell_PXA910_Platform-001_PB.pdf
- Application processor with Communication processor
- Core: ARMv5 compatible Marvell PJ1 88sv331 (Mohawk)
- - PXA688, a.k.a. MMP2, a.k.a Armada 610
+ - PXA688, a.k.a. MMP2, a.k.a Armada 610 (OLPC XO-1.75)
- Product Brief : https://web.archive.org/web/20111102023255/http://www.marvell.com/application-processors/armada-600/assets/armada610_pb.pdf
- Application processor only
- Core: ARMv7 compatible Sheeva PJ4 88sv581x core
- - PXA2128, a.k.a. MMP3 (OLPC XO4, Linux support not upstream)
+ - PXA2128, a.k.a. MMP3, a.k.a Armada 620 (OLPC XO-4)
- Product Brief : https://web.archive.org/web/20120824055155/http://www.marvell.com/application-processors/armada/pxa2128/assets/Marvell-ARMADA-PXA2128-SoC-PB.pdf
-
- Application processor only
- Core: Dual-core ARMv7 compatible Sheeva PJ4C core
- PXA960/PXA968/PXA978 (Linux support not upstream)
--
2.29.2

2021-02-04 21:30:54

by Jonathan Corbet

[permalink] [raw]

Subject: Re: [PATCH v2 1/5] docs: arm: marvell: drop some dead links

Lubomir Rintel <[email protected]> writes:

> Just remove these; there's good chance there wasn't anything useful
> there anyway.
>
> Signed-off-by: Lubomir Rintel <[email protected]>
>
> ---
> Changes since v1:
> - Adjust for removal of "[PATCH 1/5] docs: arm: marvell: turn the automatic
> links into labels"
> - Split off the hunk that fixes 38x functional spec link
>
> Documentation/arm/marvel.rst | 25 -------------------------
> 1 file changed, 25 deletions(-)
>
> diff --git a/Documentation/arm/marvel.rst b/Documentation/arm/marvel.rst
> index 16ab2eb085b86..502a1b89a2c85 100644
> --- a/Documentation/arm/marvel.rst
> +++ b/Documentation/arm/marvel.rst
> @@ -63,8 +63,6 @@ Kirkwood family
> - Product Brief : http://www.marvell.com/embedded-processors/kirkwood/assets/88F6281-004_ver1.pdf
> - Hardware Spec : http://www.marvell.com/embedded-processors/kirkwood/assets/HW_88F6281_OpenSource.pdf
> - Functional Spec: http://www.marvell.com/embedded-processors/kirkwood/assets/FS_88F6180_9x_6281_OpenSource.pdf
> - Homepage:
> - http://www.marvell.com/embedded-processors/kirkwood/
> Core:
> Feroceon 88fr131 ARMv5 compatible
> Linux kernel mach directory:
> @@ -126,7 +124,6 @@ EBU Armada family
> - 88F6820 Armada 385
> - 88F6828 Armada 388
>
> - - Product infos: http://www.marvell.com/embedded-processors/armada-38x/

So these URLs do still exist in the Wayback machine; the above is
https://web.archive.org/web/20180829171124/http://www.marvell.com/embedded-processors/armada-38x/
for example. If we delete the links, we make it harder for any
interested person to ever find them. Assuming that we want to keep
information about these product families in the documentation at all,
I'd think that we would want to have the online information as well. So
I'd replace these with wayback links, or else just leave them as they
are so that sufficiently motivated people can look them up themselves...

Make sense?

Thanks,

jon

2021-02-04 21:35:02

by Jonathan Corbet

[permalink] [raw]

Subject: Re: [PATCH v2 5/5] docs: arm: marvell: rename marvel.rst to marvell.rst

Lubomir Rintel <[email protected]> writes:

> This company is not the superheroes you're looking for.
>
> Signed-off-by: Lubomir Rintel <[email protected]>
> ---
> Documentation/arm/{marvel.rst => marvell.rst} | 0
> 1 file changed, 0 insertions(+), 0 deletions(-)
> rename Documentation/arm/{marvel.rst => marvell.rst} (100%)
>
> diff --git a/Documentation/arm/marvel.rst b/Documentation/arm/marvell.rst
> similarity index 100%
> rename from Documentation/arm/marvel.rst
> rename to Documentation/arm/marvell.rst

This is already fixed in docs-next.

Thanks,

jon

2021-02-04 21:35:35

by Jonathan Corbet

[permalink] [raw]

Subject: Re: [PATCH v2 0/5] docs: arm: Improvements to Marvell SoC documentation

Lubomir Rintel <[email protected]> writes:

> Hi,
>
> please consider applying the patches chained to this message.
>
> The objective is to deal with the a large amount of dead links to
> material that often comes handy in marvel.rst; and improve some details
> along the way.
>
> The most important change since v1 is the removal of "[PATCH 1/5] docs:
> arm: marvell: turn the automatic links into labels" patch and replacing
> the URLs inline instead. Detailed changelogs in individual patches.

So I've commented on parts 1 and 5. If you send me versions of 2-4
based on docs-next, I'll apply them; for part 1, I'd like to see wayback
machine links or a good reason why we should just delete them even
though those links exist.

Thanks,

jon

2021-02-04 22:02:49

by Andrew Lunn

[permalink] [raw]

Subject: Re: [PATCH v2 1/5] docs: arm: marvell: drop some dead links

On Thu, Feb 04, 2021 at 02:29:11PM -0700, Jonathan Corbet wrote:
> Lubomir Rintel <[email protected]> writes:
>
> > Just remove these; there's good chance there wasn't anything useful
> > there anyway.
> >
> > Signed-off-by: Lubomir Rintel <[email protected]>
> >
> > ---
> > Changes since v1:
> > - Adjust for removal of "[PATCH 1/5] docs: arm: marvell: turn the automatic
> > links into labels"
> > - Split off the hunk that fixes 38x functional spec link
> >
> > Documentation/arm/marvel.rst | 25 -------------------------
> > 1 file changed, 25 deletions(-)
> >
> > diff --git a/Documentation/arm/marvel.rst b/Documentation/arm/marvel.rst
> > index 16ab2eb085b86..502a1b89a2c85 100644
> > --- a/Documentation/arm/marvel.rst
> > +++ b/Documentation/arm/marvel.rst
> > @@ -63,8 +63,6 @@ Kirkwood family
> > - Product Brief : http://www.marvell.com/embedded-processors/kirkwood/assets/88F6281-004_ver1.pdf
> > - Hardware Spec : http://www.marvell.com/embedded-processors/kirkwood/assets/HW_88F6281_OpenSource.pdf
> > - Functional Spec: http://www.marvell.com/embedded-processors/kirkwood/assets/FS_88F6180_9x_6281_OpenSource.pdf
> > - Homepage:
> > - http://www.marvell.com/embedded-processors/kirkwood/
> > Core:
> > Feroceon 88fr131 ARMv5 compatible
> > Linux kernel mach directory:
> > @@ -126,7 +124,6 @@ EBU Armada family
> > - 88F6820 Armada 385
> > - 88F6828 Armada 388
> >
> > - - Product infos: http://www.marvell.com/embedded-processors/armada-38x/
>
> So these URLs do still exist in the Wayback machine; the above is
> https://web.archive.org/web/20180829171124/http://www.marvell.com/embedded-processors/armada-38x/
> for example. If we delete the links, we make it harder for any
> interested person to ever find them. Assuming that we want to keep
> information about these product families in the documentation at all,
> I'd think that we would want to have the online information as well. So
> I'd replace these with wayback links, or else just leave them as they
> are so that sufficiently motivated people can look them up themselves...

The kirkwood processors are old, but they were used in NAS
devices. They tend to have a long life time, and people do put Debian
on them. Armada-38x is definitely still in use in NAS boxes.

If we can have working links, that would be great.

Andrew