Re: [PATCH 2/2] soc: qcom: socinfo: Add sysfs attributes for fields in v2-v6

[Naman Jain]

Hi Trilok,

On 1/12/2023 5:28 AM, Trilok Soni wrote:
> On 1/11/2023 3:19 PM, Dmitry Baryshkov wrote:
> > On 11/01/2023 10:21, Naman Jain wrote:
> > > Add support in sysfs custom attributes for fields in socinfo version
> > > v2-v6. This is to support SoC based operations in userland scripts
> > > and test scripts. Also, add name mappings for hw-platform type to
> > > make the sysfs information more descriptive.
> >
> > Please include a patch documenting your additions to 
> > Documentation/ABI/testing/sysfs-devices-soc. Please describe usecases 
> > for new attributes and their applicability to non-Qualcomm boards.
> >
> > Note, that testing scripts can access debugfs entries without any 
> > issues.
>
> The commit text mentions the "userland" scripts and it could mean the 
> product OS like Android or Yocto having the applications using these 
> /sysfs entries. Naman, please clarify if the vendor application layer 
> in the Android is using these Entries to make decisions based on the 
> platforms / soc information?


That may have been a wrong choice of word here, if that is what it 
means. The use of these interfaces is in post boot shell scripts, and 
userspace services. As these are Qualcomm specific, I don't think we 
have any examples in Android. I have now mentioned the use-cases of 
sysfs interface, in my reply on Dmitry's email on PATCH 2/2. Have added 
you there to avoid duplication.


> ---Trilok Soni


Thanks,

Naman Jain