Re: [PATCH] firmware: smccc: Export both soc_id functions

[Martin Botka]

The patches using this are still WIP but the basic point is that H616 
has 2 die revisions (And those have
couple of other versions themselves) that require different uV and some 
frequencies are blacklisted on those
versions.

The driver is using these functions to read the revision (Not the 
version. It just looked like the logical
thing to export both and Andre confirmed. We can ofc drop the version 
symbol export if need be) and reads
trough the OPP table and selects the supported frequencies and uV for 
the chip its running on :)

I will try to get the cpufreq patches in order in the upcoming days and 
I will Cc you in those as well +
post a link to them in this thread :)

Cheers,
Martin

On Wed, Aug 30 2023 at 03:05:09 PM +01:00:00, Sudeep Holla 
<sudeep.holla@xxxxxxx> wrote:
> On Wed, Aug 30, 2023 at 2:24 PM Martin Botka 
> <martin.botka@xxxxxxxxxxxxxx> wrote:
> > arm_smccc_get_soc_id_version and arm_smccc_get_soc_id_revision
> >  need to be exported so they can be used by modules.
> >  Currently sun50i cpu freq driver is planning to use these functions.
>
> I am not against exporting these but please post along with the 
> patches using these
> exported symbols so that no one complains about you exporting the 
> symbols mentioning
> some patches that may or may not appear in the future.
>
> --
> Regards,
> Sudeep