* [PATCH] Documentation: arm: marvell: Add 88F6825 model into list
@ 2021-08-14 12:48 Pali Rohár
2021-08-14 15:28 ` Andrew Lunn
2021-08-24 19:27 ` Jonathan Corbet
0 siblings, 2 replies; 6+ messages in thread
From: Pali Rohár @ 2021-08-14 12:48 UTC (permalink / raw)
To: Jonathan Corbet, Thomas Petazzoni, Andrew Lunn; +Cc: linux-doc, linux-kernel
88F6825 is just 88F6820 but without encryption acceleration hardware and is
used e.g. in DTS file arch/arm/boot/dts/armada-385-clearfog-gtr.dtsi
Signed-off-by: Pali Rohár <pali@kernel.org>
---
Depends on patch: https://lore.kernel.org/linux-doc/20210625215437.2156-1-pali@kernel.org/
---
Documentation/arm/marvell.rst | 1 +
1 file changed, 1 insertion(+)
diff --git a/Documentation/arm/marvell.rst b/Documentation/arm/marvell.rst
index 85169bc3f538..56bb592dbd0c 100644
--- a/Documentation/arm/marvell.rst
+++ b/Documentation/arm/marvell.rst
@@ -140,6 +140,7 @@ EBU Armada family
- 88F6821 Armada 382
- 88F6W21 Armada 383
- 88F6820 Armada 385
+ - 88F6825
- 88F6828 Armada 388
- Product infos: https://web.archive.org/web/20181006144616/http://www.marvell.com/embedded-processors/armada-38x/
--
2.20.1
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [PATCH] Documentation: arm: marvell: Add 88F6825 model into list
2021-08-14 12:48 [PATCH] Documentation: arm: marvell: Add 88F6825 model into list Pali Rohár
@ 2021-08-14 15:28 ` Andrew Lunn
2021-08-14 15:33 ` Pali Rohár
2021-08-24 19:27 ` Jonathan Corbet
1 sibling, 1 reply; 6+ messages in thread
From: Andrew Lunn @ 2021-08-14 15:28 UTC (permalink / raw)
To: Pali Rohár
Cc: Jonathan Corbet, Thomas Petazzoni, linux-doc, linux-kernel
On Sat, Aug 14, 2021 at 02:48:05PM +0200, Pali Rohár wrote:
> 88F6825 is just 88F6820 but without encryption acceleration hardware and is
> used e.g. in DTS file arch/arm/boot/dts/armada-385-clearfog-gtr.dtsi
>
> Signed-off-by: Pali Rohár <pali@kernel.org>
>
> ---
> Depends on patch: https://lore.kernel.org/linux-doc/20210625215437.2156-1-pali@kernel.org/
> ---
> Documentation/arm/marvell.rst | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/Documentation/arm/marvell.rst b/Documentation/arm/marvell.rst
> index 85169bc3f538..56bb592dbd0c 100644
> --- a/Documentation/arm/marvell.rst
> +++ b/Documentation/arm/marvell.rst
> @@ -140,6 +140,7 @@ EBU Armada family
> - 88F6821 Armada 382
> - 88F6W21 Armada 383
> - 88F6820 Armada 385
> + - 88F6825
Hi Pali
Does it have the marketing name of Armada 385?
Andrew
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH] Documentation: arm: marvell: Add 88F6825 model into list
2021-08-14 15:28 ` Andrew Lunn
@ 2021-08-14 15:33 ` Pali Rohár
2021-08-14 15:39 ` Andrew Lunn
0 siblings, 1 reply; 6+ messages in thread
From: Pali Rohár @ 2021-08-14 15:33 UTC (permalink / raw)
To: Andrew Lunn; +Cc: Jonathan Corbet, Thomas Petazzoni, linux-doc, linux-kernel
On Saturday 14 August 2021 17:28:56 Andrew Lunn wrote:
> On Sat, Aug 14, 2021 at 02:48:05PM +0200, Pali Rohár wrote:
> > 88F6825 is just 88F6820 but without encryption acceleration hardware and is
> > used e.g. in DTS file arch/arm/boot/dts/armada-385-clearfog-gtr.dtsi
> >
> > Signed-off-by: Pali Rohár <pali@kernel.org>
> >
> > ---
> > Depends on patch: https://lore.kernel.org/linux-doc/20210625215437.2156-1-pali@kernel.org/
> > ---
> > Documentation/arm/marvell.rst | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/Documentation/arm/marvell.rst b/Documentation/arm/marvell.rst
> > index 85169bc3f538..56bb592dbd0c 100644
> > --- a/Documentation/arm/marvell.rst
> > +++ b/Documentation/arm/marvell.rst
> > @@ -140,6 +140,7 @@ EBU Armada family
> > - 88F6821 Armada 382
> > - 88F6W21 Armada 383
> > - 88F6820 Armada 385
> > + - 88F6825
>
> Hi Pali
>
> Does it have the marketing name of Armada 385?
No, there is no marking. The only name in Marvell documents and also in
DTS files and on wikis/internet is just 88F6825.
I found only this statement from Marvell:
"The 88F6825 device is a member of the ARMADA® 380, 385, and 388 Family
of devices."
>
> Andrew
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] Documentation: arm: marvell: Add 88F6825 model into list
2021-08-14 15:33 ` Pali Rohár
@ 2021-08-14 15:39 ` Andrew Lunn
2021-08-14 15:44 ` Pali Rohár
0 siblings, 1 reply; 6+ messages in thread
From: Andrew Lunn @ 2021-08-14 15:39 UTC (permalink / raw)
To: Pali Rohár
Cc: Jonathan Corbet, Thomas Petazzoni, linux-doc, linux-kernel
On Sat, Aug 14, 2021 at 05:33:07PM +0200, Pali Rohár wrote:
> On Saturday 14 August 2021 17:28:56 Andrew Lunn wrote:
> > On Sat, Aug 14, 2021 at 02:48:05PM +0200, Pali Rohár wrote:
> > > 88F6825 is just 88F6820 but without encryption acceleration hardware and is
> > > used e.g. in DTS file arch/arm/boot/dts/armada-385-clearfog-gtr.dtsi
> > >
> > > Signed-off-by: Pali Rohár <pali@kernel.org>
> > >
> > > ---
> > > Depends on patch: https://lore.kernel.org/linux-doc/20210625215437.2156-1-pali@kernel.org/
> > > ---
> > > Documentation/arm/marvell.rst | 1 +
> > > 1 file changed, 1 insertion(+)
> > >
> > > diff --git a/Documentation/arm/marvell.rst b/Documentation/arm/marvell.rst
> > > index 85169bc3f538..56bb592dbd0c 100644
> > > --- a/Documentation/arm/marvell.rst
> > > +++ b/Documentation/arm/marvell.rst
> > > @@ -140,6 +140,7 @@ EBU Armada family
> > > - 88F6821 Armada 382
> > > - 88F6W21 Armada 383
> > > - 88F6820 Armada 385
> > > + - 88F6825
> >
> > Hi Pali
> >
> > Does it have the marketing name of Armada 385?
>
> No, there is no marking. The only name in Marvell documents and also in
> DTS files and on wikis/internet is just 88F6825.
>
> I found only this statement from Marvell:
>
> "The 88F6825 device is a member of the ARMADA® 380, 385, and 388 Family
> of devices."
O.K, thanks
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
Andrew
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH] Documentation: arm: marvell: Add 88F6825 model into list
2021-08-14 15:39 ` Andrew Lunn
@ 2021-08-14 15:44 ` Pali Rohár
0 siblings, 0 replies; 6+ messages in thread
From: Pali Rohár @ 2021-08-14 15:44 UTC (permalink / raw)
To: Andrew Lunn; +Cc: Jonathan Corbet, Thomas Petazzoni, linux-doc, linux-kernel
On Saturday 14 August 2021 17:39:18 Andrew Lunn wrote:
> On Sat, Aug 14, 2021 at 05:33:07PM +0200, Pali Rohár wrote:
> > On Saturday 14 August 2021 17:28:56 Andrew Lunn wrote:
> > > On Sat, Aug 14, 2021 at 02:48:05PM +0200, Pali Rohár wrote:
> > > > 88F6825 is just 88F6820 but without encryption acceleration hardware and is
> > > > used e.g. in DTS file arch/arm/boot/dts/armada-385-clearfog-gtr.dtsi
> > > >
> > > > Signed-off-by: Pali Rohár <pali@kernel.org>
> > > >
> > > > ---
> > > > Depends on patch: https://lore.kernel.org/linux-doc/20210625215437.2156-1-pali@kernel.org/
> > > > ---
> > > > Documentation/arm/marvell.rst | 1 +
> > > > 1 file changed, 1 insertion(+)
> > > >
> > > > diff --git a/Documentation/arm/marvell.rst b/Documentation/arm/marvell.rst
> > > > index 85169bc3f538..56bb592dbd0c 100644
> > > > --- a/Documentation/arm/marvell.rst
> > > > +++ b/Documentation/arm/marvell.rst
> > > > @@ -140,6 +140,7 @@ EBU Armada family
> > > > - 88F6821 Armada 382
> > > > - 88F6W21 Armada 383
> > > > - 88F6820 Armada 385
> > > > + - 88F6825
> > >
> > > Hi Pali
> > >
> > > Does it have the marketing name of Armada 385?
> >
> > No, there is no marking. The only name in Marvell documents and also in
> > DTS files and on wikis/internet is just 88F6825.
> >
> > I found only this statement from Marvell:
> >
> > "The 88F6825 device is a member of the ARMADA® 380, 385, and 388 Family
> > of devices."
>
> O.K, thanks
Normally I do not care about such thing... but as kernel already
supports this SoC and also have DTS file for some device, it
indicates that it is in use and supported... so mentioning this SoC in
documentation is a good idea, even it does not have "code name".
> Reviewed-by: Andrew Lunn <andrew@lunn.ch>
>
> Andrew
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] Documentation: arm: marvell: Add 88F6825 model into list
2021-08-14 12:48 [PATCH] Documentation: arm: marvell: Add 88F6825 model into list Pali Rohár
2021-08-14 15:28 ` Andrew Lunn
@ 2021-08-24 19:27 ` Jonathan Corbet
1 sibling, 0 replies; 6+ messages in thread
From: Jonathan Corbet @ 2021-08-24 19:27 UTC (permalink / raw)
To: Pali Rohár, Thomas Petazzoni, Andrew Lunn; +Cc: linux-doc, linux-kernel
Pali Rohár <pali@kernel.org> writes:
> 88F6825 is just 88F6820 but without encryption acceleration hardware and is
> used e.g. in DTS file arch/arm/boot/dts/armada-385-clearfog-gtr.dtsi
>
> Signed-off-by: Pali Rohár <pali@kernel.org>
>
> ---
> Depends on patch: https://lore.kernel.org/linux-doc/20210625215437.2156-1-pali@kernel.org/
> ---
> Documentation/arm/marvell.rst | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/Documentation/arm/marvell.rst b/Documentation/arm/marvell.rst
> index 85169bc3f538..56bb592dbd0c 100644
> --- a/Documentation/arm/marvell.rst
> +++ b/Documentation/arm/marvell.rst
> @@ -140,6 +140,7 @@ EBU Armada family
> - 88F6821 Armada 382
> - 88F6W21 Armada 383
> - 88F6820 Armada 385
> + - 88F6825
> - 88F6828 Armada 388
Applied, thanks.
jon
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2021-08-24 19:27 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-08-14 12:48 [PATCH] Documentation: arm: marvell: Add 88F6825 model into list Pali Rohár
2021-08-14 15:28 ` Andrew Lunn
2021-08-14 15:33 ` Pali Rohár
2021-08-14 15:39 ` Andrew Lunn
2021-08-14 15:44 ` Pali Rohár
2021-08-24 19:27 ` Jonathan Corbet
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).