Re: [PATCH 2/2] soc/tegra: fuse: speedo-tegra210: Support revision B01

[Jon Hunter]

On 21/07/2026 18:31, Aaron Kling wrote:
> On Tue, Jul 21, 2026 at 4:46 AM Jon Hunter <jonathanh@xxxxxxxxxx> wrote:
> > On 18/07/2026 00:19, Aaron Kling via B4 Relay wrote:
> > > From: Aaron Kling <webgeek1234@xxxxxxxxx>
> > >
> > > Tegra210B01 has a separate set of skus and speedo tables
> > >
> > > Signed-off-by: Aaron Kling <webgeek1234@xxxxxxxxx>
> > > ---
> > >    drivers/soc/tegra/fuse/speedo-tegra210.c | 22 ++++++++++++++++++++--
> > >    1 file changed, 20 insertions(+), 2 deletions(-)
> > >
> > > diff --git a/drivers/soc/tegra/fuse/speedo-tegra210.c b/drivers/soc/tegra/fuse/speedo-tegra210.c
> > > index 06c2bcbee5734..699751f4bd0c0 100644
> > > --- a/drivers/soc/tegra/fuse/speedo-tegra210.c
> > > +++ b/drivers/soc/tegra/fuse/speedo-tegra210.c
> > > @@ -65,7 +65,25 @@ static void __init rev_sku_to_speedo_ids(struct tegra_sku_info *sku_info,
> > >        sku_info->gpu_speedo_id = 0;
> > >        *threshold = THRESHOLD_INDEX_0;
> > >
> > > -     if (sku_info->revision >= TEGRA_REVISION_A02) {
> > > +     if (sku_info->revision >= TEGRA_REVISION_B01) {
> > > +             sku_info->gpu_speedo_id = 1;
> > > +
> > > +             switch (sku) {
> > > +             case 0x00: /* Engineering SKU */
> > > +             case 0x01: /* Engineering SKU */
> > > +             case 0x83:
> > > +                     break;
> > > +
> > > +             case 0x87:
> > > +                     sku_info->cpu_speedo_id = 3;
> > > +                     break;
> > > +
> > > +             default:
> > > +                     pr_err("Tegra210B01: unknown SKU %#04x\n", sku);
> > > +                     /* Using the default for the error case */
> > > +                     break;
> > > +             }
> > > +     } else if (sku_info->revision >= TEGRA_REVISION_A02) {
> > >                switch (sku) {
> > >                case 0x00: /* Engineering SKU */
> > >                case 0x01: /* Engineering SKU */
> > > @@ -154,7 +172,7 @@ void __init tegra210_init_speedo_data(struct tegra_sku_info *sku_info)
> > >        speedo_revision = get_speedo_revision();
> > >        pr_info("Speedo Revision %u\n", speedo_revision);
> > >
> > > -     if (speedo_revision >= 3) {
> > > +     if (sku_info->revision == TEGRA_REVISION_B01 || speedo_revision >= 3) {
> > >                sku_info->cpu_speedo_value = cpu_speedo[0];
> > >                sku_info->gpu_speedo_value = cpu_speedo[2];
> > >                sku_info->soc_speedo_value = soc_speedo[0];
> >
> > I am guessing that this is derived/ported from some kernel that NVIDIA
> > has released in the past. It would be nice to include a reference to
> > whatever change was used to create this.
>
> It is loosely based on [0], but restructured to not change nearly as
> many lines. How would you want this referenced? Something like a
> 'based on downstream 5.10' blurb in the commit message or a direct
> reference to the commit somehow?

Commit d93f8399053dcf117ff56a3029ff08c0e36f4b75 uses ...

Based-on: 
https://lore.kernel.org/r/7703024f-8882-9eec-a122-599871728a89@xxxxxxxxx
Based-on-patch-by: Heiner Kallweit <hkallweit1@xxxxxxxxx>

So this would be fine.

Jon

--
nvpublic