linux-kernel - Re: [PATCH v3] soc: tegra: fuse: speedo-tegra210: Update speedo ids

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <CALHNRZ-43RHsjiw1NJAxc3qAqHEoEYongtB0MzsfP5q93H7_XA@mail.gmail.com>
Date: Mon, 22 Sep 2025 12:16:28 -0500
From: Aaron Kling <webgeek1234@...il.com>
To: Jon Hunter <jonathanh@...dia.com>
Cc: Thierry Reding <thierry.reding@...il.com>, Joseph Lo <josephl@...dia.com>, 
	Stephen Boyd <sboyd@...nel.org>, Thierry Reding <treding@...dia.com>, linux-tegra@...r.kernel.org, 
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH v3] soc: tegra: fuse: speedo-tegra210: Update speedo ids

On Fri, Sep 19, 2025 at 4:55 AM Jon Hunter <jonathanh@...dia.com> wrote:
>
>
> On 18/09/2025 18:19, Aaron Kling wrote:
> > On Thu, Sep 18, 2025 at 4:45 AM Jon Hunter <jonathanh@...dia.com> wrote:
> >>
> >>
> >> On 04/09/2025 02:58, Aaron Kling via B4 Relay wrote:
> >>> From: Aaron Kling <webgeek1234@...il.com>
> >>>
> >>> Existing code only sets cpu and gpu speedo ids 0 and 1. The cpu dvfs
> >>> code supports 11 ids and nouveau supports 5. This aligns with what the
> >>> downstream vendor kernel supports. Align skus with the downstream list.
> >>
> >> Do you have a reference for the downstream kernel change you are
> >> referring to? I have found this change [0]. However, this does not quite
> >> align with what you have in this patch.
> >
> > This is based on L4T r32.7.6 [0], which builds up the list over
> > several commits, so I can't link to just one. The first revision only
> > added sku's that I had specifically verified. Mikko suggested to just
> > import everything from downstream and to simplify the conditionals.
> > And that's this revision.
>
> ...
>
> > [0] https://nv-tegra.nvidia.com/r/plugins/gitiles/linux-4.9/+/refs/tags/tegra-l4t-r32.7.6_good/drivers/soc/tegra/fuse/speedo-tegra210.c#72
>
>
> Thanks! I saw Mikko's comments on V2 and that makes sense. The problem I
> have is that comparing this with the above, it is not clear that these
> are equivalent. The above is using the 'a02' chip version for setting
> the speedo IDs but this is using the speedo revision. Now it might turn
> out this is equivalent, but it is not obvious to me. Ideally we would
> end up with something similar to the above.

Thanks for noticing this discrepancy. I will fix and verify, then send
a new revision.

Aaron