Received-SPF: pass (google.com: domain of linux-crypto-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
MIME-Version: 1.0
References: <266065918e47e8965bb6a0ab486da070278788e4.1641996057.git.geert+renesas@glider.be>
 <BY3PR18MB47375336D4AC1FE79D493945C6529@BY3PR18MB4737.namprd18.prod.outlook.com>
 <CAK8P3a2jLgtcuJ6MD7LmJzagiRQSk85eL3tiHDmB33i_n3CZ2w@mail.gmail.com>
 <BY3PR18MB4737E57C0DC616FFE73CC7AEC6529@BY3PR18MB4737.namprd18.prod.outlook.com>
 <CAK8P3a0Eeyt=rWG6CjYCKck-k5+uT5GkLuDG+S4GkQgD49-H1A@mail.gmail.com> <BY3PR18MB47371C7C66042D7982C00474C6539@BY3PR18MB4737.namprd18.prod.outlook.com>
In-Reply-To: <BY3PR18MB47371C7C66042D7982C00474C6539@BY3PR18MB4737.namprd18.prod.outlook.com>
From:   Arnd Bergmann <arnd@arndb.de>
Date:   Thu, 13 Jan 2022 12:10:18 +0100
Message-ID: <CAK8P3a0qxMDFie3SvciMNUOq0G6njqqqVdBOxrXexFWMv+0png@mail.gmail.com>
Subject: Re: [EXT] [PATCH] hwrng: cn10k - HW_RANDOM_CN10K should depend on ARCH_THUNDER
To:     Sunil Kovvuri Goutham <sgoutham@marvell.com>
Cc:     Arnd Bergmann <arnd@arndb.de>,
        Geert Uytterhoeven <geert+renesas@glider.be>,
        Herbert Xu <herbert@gondor.apana.org.au>,
        Bharat Bhushan <bbhushan2@marvell.com>,
        Joseph Longever <jlongever@marvell.com>,
        "linux-crypto@vger.kernel.org" <linux-crypto@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Content-Type: text/plain; charset="UTF-8"
Precedence: bulk

On Thu, Jan 13, 2022 at 10:42 AM Sunil Kovvuri Goutham
<sgoutham@marvell.com> wrote:
> >From: Arnd Bergmann <arnd@arndb.de>

> >
> >Ok, and those are the same family as the earlier cnMIPS based OCTEON
> >CN3xxx/CN5xxx/CN6xxx/CN7xxx/CNF7xxx and the later ARMv8.2
> >Fusion CNF9xxx, right?
>
> Yes, except the CNF9x.

Ok, what is CNF9x then?

> >> Octeon 10 and OcteonTx2 (9x series) are different families and not related to
> >> Armada as well.
>
> >I assume with '9x' you mean only the CN92xx/CN96xx/CN98xx/CN10x/DPU400
> >family here, not the OcteonTx2 CN91xx that is clearly part of the Marvell
> >Sheeva/Kirkwood/MVEBU/Armada family, and the CN99xx in the
> >Netlogic/Broadcom family.
>
> By 9x I mean CN96xx, CN98xx, CNF9xx, base architecture is same across all.

> >Is there anything that you can say about this product line? It looked like it
> >was derived from the cnMIPS/OcteonTX line, and it seems to share its
> >mystery (presumably ThunderX1-derived rather than Cortex or Vulcan)
> >ARMv8.2 core with the CNF9xxx.
>
> 9x core is Marvell (or Cavium) developed core and is next-gen of 8x.
> 10x and DPU400 have ARM cores.

I don't care about the CPU core, only about the SoC design for the family,
the old octeon family includes both cnMIPS and ThunderX cores already
and those are not even the same ISA, so going from custom ARMv8.2
cores to Neoverse ARMv9 is not relevant at all.

> CN91xx is Armada.

Ok

> >> But I am fine if ARCH_THUNDER is renamed to ARCH_OCTEON to include all.
>
> >I'd really prefer to have sensible names, so if there are six unrelated Marvell
> >SoC families (pxa/mmp, armada/mvebu/cn91xx, xlp/vulcan/thunderx2,
> >berlin/synaptics, cavium/octeon/thunderx/fusion, and whatever turned into
> >non-cavium cn92xx/cn96xx/cn98xx/cn10x/dpu400) instead of five, we should
> >probably have six ARCH_* names for those. (Note: for some other
> >manufacturers such as Broadcom or Mediatek, we do use just the company
> >name as the CONFIG_ARCH_* symbol, but I feel that for Marvell or NXP,
> >that ship has sailed long ago, based on the number of acquisitions and
> >spinoffs).
> >
> >Any suggestions for the name? Were these acquired from some other company?
>
> For OcteonTx2, CN10K and future silicons, ARCH_MARVELL_OCTEON  / ARCH_MRVL_OCTEON ?
> Different ARCH_* config for each family is not needed IMHO.

It does sound like these are all the same family then. Ideally we'd use the same
name as the MIPS based parts, but those currently use
CONFIG_CAVIUM_OCTEON_SOC, which doesn't fit the same naming scheme,
and we seem to have a duplicate set of drivers.

We usually pick shorter names, so I'd just leave it with CONFIG_ARCH_OCTEON
as I have in my current patch. That also means it's less likely to
require a rename
after the next acquisition or spinoff ;-)

        Arnd