linux-kernel - RE: [PATCH v7 0/7] firmware: support i.MX95 SCMI BBM/MISC Extenstion

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID:
 <PAXPR04MB845929B61EF637D67BDEAB0088B12@PAXPR04MB8459.eurprd04.prod.outlook.com>
Date: Wed, 31 Jul 2024 15:28:56 +0000
From: Peng Fan <peng.fan@....com>
To: Sudeep Holla <sudeep.holla@....com>, "Peng Fan (OSS)"
	<peng.fan@....nxp.com>
CC: Cristian Marussi <cristian.marussi@....com>, Rob Herring
	<robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley
	<conor+dt@...nel.org>, Shawn Guo <shawnguo@...nel.org>, Sascha Hauer
	<s.hauer@...gutronix.de>, Pengutronix Kernel Team <kernel@...gutronix.de>,
	Fabio Estevam <festevam@...il.com>, Alexandre Belloni
	<alexandre.belloni@...tlin.com>, Dmitry Torokhov <dmitry.torokhov@...il.com>,
	"arm-scmi@...r.kernel.org" <arm-scmi@...r.kernel.org>,
	"linux-arm-kernel@...ts.infradead.org"
	<linux-arm-kernel@...ts.infradead.org>, "devicetree@...r.kernel.org"
	<devicetree@...r.kernel.org>, "linux-kernel@...r.kernel.org"
	<linux-kernel@...r.kernel.org>, "imx@...ts.linux.dev" <imx@...ts.linux.dev>,
	"linux-rtc@...r.kernel.org" <linux-rtc@...r.kernel.org>,
	"linux-input@...r.kernel.org" <linux-input@...r.kernel.org>
Subject: RE: [PATCH v7 0/7] firmware: support i.MX95 SCMI BBM/MISC Extenstion

> Subject: Re: [PATCH v7 0/7] firmware: support i.MX95 SCMI BBM/MISC
> Extenstion
> 
> On Wed, Jul 31, 2024 at 08:56:04PM +0800, Peng Fan (OSS) wrote:
> > i.MX95 System Manager Firmware source:
> >
> 
> > To generate html from the repo: make html
> >
> > i.MX95 System Manager Firmware support vendor extension
> protocol:
> > - Battery Backed Module(BBM) Protocol
> >   This protocol is intended provide access to the battery-backed
> module.
> >   This contains persistent storage (GPR), an RTC, and the ON/OFF
> button.
> >   The protocol can also provide access to similar functions
> implemented via
> >   external board components. The BBM protocol provides functions to:
> >
> >   - Describe the protocol version.
> >   - Discover implementation attributes.
> >   - Read/write GPR
> >   - Discover the RTCs available in the system.
> >   - Read/write the RTC time in seconds and ticks
> >   - Set an alarm (per LM) in seconds
> >   - Get notifications on RTC update, alarm, or rollover.
> >   - Get notification on ON/OFF button activity.
> >
> > - MISC Protocol for misc settings
> >   This includes controls that are misc settings/actions that must be
> >   exposed from the SM to agents. They are device specific and are
> usually
> >   define to access bit fields in various mix block control modules,
> >   IOMUX_GPR, and other GPR/CSR owned by the SM.
> >   This protocol supports the following functions:
> >
> >   - Describe the protocol version.
> >   - Discover implementation attributes.
> >   - Set/Get a control.
> >   - Initiate an action on a control.
> >   - Obtain platform (i.e. SM) build information.
> >   - Obtain ROM passover data.
> >   - Read boot/shutdown/reset information for the LM or the system.
> >
> > This patchset is to support the two protocols and users that use the
> > protocols. The upper protocol infomation is also included in patch 1
> >
> > Signed-off-by: Peng Fan <peng.fan@....com>
> >
> > Changes in v7:
> > - Just correct R-b tag from Rob to drop quotes "", and rebased
> > - Link to v6:
> 
> I specifically asked you to avoid re-spinning just for this reason within
> 20 mins from your response, yet you managed to churn it within that
> time 🙁.
> I must respond at flash speed to avoid such things in the future 😢.
> 

The V6 patchset was posted at 2024-07-18, and Krzysztof raised that
R-b tag comment at 2024-7-21, so just after I posted the reply and
started doing the new version, since it is a simple update, it takes
short time to do v7. Since 6.11-rc1 merge window is closed, I think
a clean patchset would be good for maintainers to apply.

As I replied in the v6, I not intend to bring trouble for your
reviewing. I will wait more time before doing a new version.

Sorry, again!

Thanks,
Peng.

> --
> Regards,
> Sudeep