[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <1d31a521-f5c6-48f0-1202-2f19919bca6b@quicinc.com>
Date: Thu, 19 Jan 2023 15:05:25 +0530
From: Naman Jain <quic_namajain@...cinc.com>
To: Trilok Soni <quic_tsoni@...cinc.com>,
Dmitry Baryshkov <dmitry.baryshkov@...aro.org>,
Bjorn Andersson <andersson@...nel.org>,
"Andy Gross" <agross@...nel.org>,
Konrad Dybcio <konrad.dybcio@...ainline.org>
CC: <linux-arm-msm@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
<quic_pkondeti@...cinc.com>
Subject: Re: [PATCH 2/2] soc: qcom: socinfo: Add sysfs attributes for fields
in v2-v6
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
Powered by blists - more mailing lists