2022-05-05 13:31:38

by Doug Anderson

[permalink] [raw]
Subject: Re: (subset) [PATCH] soc: qcom: socinfo: Add another ID for sc7180

Hi,

On Wed, May 4, 2022 at 10:21 AM Bjorn Andersson
<[email protected]> wrote:
>
> On Mon, 2 May 2022 17:33:45 -0700, Douglas Anderson wrote:
> > It appears the some sc7180 devices, like the one in my
> > sc7180-trogdor-homestar, report an ID of 407 instead of 425. Add
> > another ID into the list.
> >
> >
>
> Applied, thanks!
>
> [1/1] soc: qcom: socinfo: Add another ID for sc7180
> commit: c35886d605604e0b03cdd835ae3249dc1fe0cc2a

Hmm. Did you see the responses from Sai [1] about this? He seemed to
indicate that there might be some issue here because he thought 407
was supposed to be a different SoC. Are we sure we want to land this
patch while we're sorting it out?

[1] https://lore.kernel.org/r/[email protected]/


2022-05-05 23:16:51

by Bjorn Andersson

[permalink] [raw]
Subject: Re: (subset) [PATCH] soc: qcom: socinfo: Add another ID for sc7180

On Wed 04 May 10:51 PDT 2022, Doug Anderson wrote:

> Hi,
>
> On Wed, May 4, 2022 at 10:21 AM Bjorn Andersson
> <[email protected]> wrote:
> >
> > On Mon, 2 May 2022 17:33:45 -0700, Douglas Anderson wrote:
> > > It appears the some sc7180 devices, like the one in my
> > > sc7180-trogdor-homestar, report an ID of 407 instead of 425. Add
> > > another ID into the list.
> > >
> > >
> >
> > Applied, thanks!
> >
> > [1/1] soc: qcom: socinfo: Add another ID for sc7180
> > commit: c35886d605604e0b03cdd835ae3249dc1fe0cc2a
>
> Hmm. Did you see the responses from Sai [1] about this? He seemed to
> indicate that there might be some issue here because he thought 407
> was supposed to be a different SoC. Are we sure we want to land this
> patch while we're sorting it out?
>

I stared at the thread but didn't see his reply for some reason.

Congrats on the getting the random SoC in your device ;) As this was the
top patch on drivers-for-5.19, I just popped that off and pushed the
branch again.

Let me know when you've figured the numbering out (and if it turns out
to be the mobile id I wouldn't mind carrying that in the table)

Regards,
Bjorn

> [1] https://lore.kernel.org/r/[email protected]/