linux-arm-msm.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Manivannan Sadhasivam <mani@kernel.org>
To: Jeffrey Hugo <quic_jhugo@quicinc.com>
Cc: Manivannan Sadhasivam <mani@kernel.org>,
	quic_pkanojiy@quicinc.com, quic_carlv@quicinc.com,
	mhi@lists.linux.dev, linux-arm-msm@vger.kernel.org
Subject: Re: [PATCH] bus: mhi: host: Read PK HASH dynamically
Date: Tue, 30 Jan 2024 23:51:45 +0530	[thread overview]
Message-ID: <20240130182145.GC4218@thinkpad> (raw)
In-Reply-To: <6966f6a7-9fb0-0766-3b69-af82c723d349@quicinc.com>

On Tue, Jan 30, 2024 at 09:05:06AM -0700, Jeffrey Hugo wrote:
> On 1/30/2024 1:21 AM, Manivannan Sadhasivam wrote:
> > On Fri, Dec 08, 2023 at 09:59:38AM -0700, Jeffrey Hugo wrote:
> > > The OEM PK HASH registers in the BHI region are read once during firmware
> > > load (boot), cached, and displayed on demand via sysfs. This has a few
> > > problems - if firmware load is skipped, the registers will not be read and
> > > if the register values change over the life of the device the local cache
> > > will be out of sync.
> > > 
> > > Qualcomm Cloud AI 100 can expose both these problems. It is possible for
> > > mhi_async_power_up() to be invoked while the device is in AMSS EE, which
> > > would bypass firmware loading. Also, Qualcomm Cloud AI 100 has 5 PK HASH
> > > slots which can be dynamically provisioned while the device is active,
> > > which would result in the values changing and users may want to know what
> > > keys are active.
> > > 
> > > Address these concerns by reading the PK HASH registers on-demand during
> > > the sysfs read. This will result in showing the most current information.
> > > 
> > > Signed-off-by: Jeffrey Hugo <quic_jhugo@quicinc.com>
> > 
> > Applied to mhi-next!
> > 
> > - Mani
> 
> I hope you applied v2 since this is a reply to v1.  I don't see mhi-next
> with this change published yet, so I can't check.
> 

Sorry, I did apply v2.

- Mani

-- 
மணிவண்ணன் சதாசிவம்

      reply	other threads:[~2024-01-30 18:21 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-12-08 16:59 [PATCH] bus: mhi: host: Read PK HASH dynamically Jeffrey Hugo
2024-01-04  6:46 ` Manivannan Sadhasivam
2024-01-04 15:54   ` Jeffrey Hugo
2024-01-30  8:21 ` Manivannan Sadhasivam
2024-01-30 16:05   ` Jeffrey Hugo
2024-01-30 18:21     ` Manivannan Sadhasivam [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20240130182145.GC4218@thinkpad \
    --to=mani@kernel.org \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=mhi@lists.linux.dev \
    --cc=quic_carlv@quicinc.com \
    --cc=quic_jhugo@quicinc.com \
    --cc=quic_pkanojiy@quicinc.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).