linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [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).