2019-09-12 09:12:40

by Srinivas Kandagatla

[permalink] [raw]
Subject: [PATCH] soc: qcom: socinfo: add missing soc_id sysfs entry

looks like SoC ID is not exported to sysfs for some reason.
This patch adds it!

This is mostly used by userspace libraries like SNPE.

Signed-off-by: Srinivas Kandagatla <[email protected]>
---
drivers/soc/qcom/socinfo.c | 2 ++
1 file changed, 2 insertions(+)

diff --git a/drivers/soc/qcom/socinfo.c b/drivers/soc/qcom/socinfo.c
index 8dc86a74559b..876a3f6612a3 100644
--- a/drivers/soc/qcom/socinfo.c
+++ b/drivers/soc/qcom/socinfo.c
@@ -422,6 +422,8 @@ static int qcom_socinfo_probe(struct platform_device *pdev)
qs->attr.family = "Snapdragon";
qs->attr.machine = socinfo_machine(&pdev->dev,
le32_to_cpu(info->id));
+ qs->attr.soc_id = devm_kasprintf(&pdev->dev, GFP_KERNEL, "%u",
+ le32_to_cpu(info->id));
qs->attr.revision = devm_kasprintf(&pdev->dev, GFP_KERNEL, "%u.%u",
SOCINFO_MAJOR(le32_to_cpu(info->ver)),
SOCINFO_MINOR(le32_to_cpu(info->ver)));
--
2.21.0


2019-09-17 04:54:29

by Stephen Boyd

[permalink] [raw]
Subject: Re: [PATCH] soc: qcom: socinfo: add missing soc_id sysfs entry

Quoting Srinivas Kandagatla (2019-09-12 02:10:19)
> looks like SoC ID is not exported to sysfs for some reason.
> This patch adds it!
>
> This is mostly used by userspace libraries like SNPE.

What is SNPE?

>
> Signed-off-by: Srinivas Kandagatla <[email protected]>
> ---
> drivers/soc/qcom/socinfo.c | 2 ++
> 1 file changed, 2 insertions(+)

Hmm I wasn't aware this driver was merged.

>
> diff --git a/drivers/soc/qcom/socinfo.c b/drivers/soc/qcom/socinfo.c
> index 8dc86a74559b..876a3f6612a3 100644
> --- a/drivers/soc/qcom/socinfo.c
> +++ b/drivers/soc/qcom/socinfo.c
> @@ -422,6 +422,8 @@ static int qcom_socinfo_probe(struct platform_device *pdev)
> qs->attr.family = "Snapdragon";
> qs->attr.machine = socinfo_machine(&pdev->dev,
> le32_to_cpu(info->id));
> + qs->attr.soc_id = devm_kasprintf(&pdev->dev, GFP_KERNEL, "%u",
> + le32_to_cpu(info->id));
> qs->attr.revision = devm_kasprintf(&pdev->dev, GFP_KERNEL, "%u.%u",
> SOCINFO_MAJOR(le32_to_cpu(info->ver)),
> SOCINFO_MINOR(le32_to_cpu(info->ver)));

2019-09-26 09:09:43

by Jeffrey Hugo

[permalink] [raw]
Subject: Re: [PATCH] soc: qcom: socinfo: add missing soc_id sysfs entry

On Mon, Sep 16, 2019 at 3:44 PM Stephen Boyd <[email protected]> wrote:
>
> Quoting Srinivas Kandagatla (2019-09-12 02:10:19)
> > looks like SoC ID is not exported to sysfs for some reason.
> > This patch adds it!
> >
> > This is mostly used by userspace libraries like SNPE.
>
> What is SNPE?

Snapdragon Neural Processing Engine. Pronounced "snap-e". Its
basically the framework someone goes through to run a neural network
on a Qualcomm mobile SoC. SNPE can utilize various hardware resources
such as the applications CPU, GPU, and dedicated compute resources
such as a NSP, if available. Its been around for over a year, and
much more information can be found by just doing a simple search since
SNPE is pretty much a unique search term currently.

2019-09-26 18:02:54

by Stephen Boyd

[permalink] [raw]
Subject: Re: [PATCH] soc: qcom: socinfo: add missing soc_id sysfs entry

Quoting Jeffrey Hugo (2019-09-24 20:54:41)
> On Mon, Sep 16, 2019 at 3:44 PM Stephen Boyd <[email protected]> wrote:
> >
> > Quoting Srinivas Kandagatla (2019-09-12 02:10:19)
> > > looks like SoC ID is not exported to sysfs for some reason.
> > > This patch adds it!
> > >
> > > This is mostly used by userspace libraries like SNPE.
> >
> > What is SNPE?
>
> Snapdragon Neural Processing Engine. Pronounced "snap-e". Its
> basically the framework someone goes through to run a neural network
> on a Qualcomm mobile SoC. SNPE can utilize various hardware resources
> such as the applications CPU, GPU, and dedicated compute resources
> such as a NSP, if available. Its been around for over a year, and
> much more information can be found by just doing a simple search since
> SNPE is pretty much a unique search term currently.

I wouldn't mind if it was still spelled out instead of just as an
acronym. Who knows, a few years from now it may not be a unique acronym
and then taking the extra few seconds to write it out once would have
saved future effort.

2019-09-26 18:22:28

by Jeffrey Hugo

[permalink] [raw]
Subject: Re: [PATCH] soc: qcom: socinfo: add missing soc_id sysfs entry

On Thu, Sep 26, 2019 at 11:57 AM Stephen Boyd <[email protected]> wrote:
>
> Quoting Jeffrey Hugo (2019-09-24 20:54:41)
> > On Mon, Sep 16, 2019 at 3:44 PM Stephen Boyd <[email protected]> wrote:
> > >
> > > Quoting Srinivas Kandagatla (2019-09-12 02:10:19)
> > > > looks like SoC ID is not exported to sysfs for some reason.
> > > > This patch adds it!
> > > >
> > > > This is mostly used by userspace libraries like SNPE.
> > >
> > > What is SNPE?
> >
> > Snapdragon Neural Processing Engine. Pronounced "snap-e". Its
> > basically the framework someone goes through to run a neural network
> > on a Qualcomm mobile SoC. SNPE can utilize various hardware resources
> > such as the applications CPU, GPU, and dedicated compute resources
> > such as a NSP, if available. Its been around for over a year, and
> > much more information can be found by just doing a simple search since
> > SNPE is pretty much a unique search term currently.
>
> I wouldn't mind if it was still spelled out instead of just as an
> acronym. Who knows, a few years from now it may not be a unique acronym
> and then taking the extra few seconds to write it out once would have
> saved future effort.
>

Fair point.