Re: [PATCH v3 0/5] platform/x86: intel_scu_ipc: Avoid working around IO and cleanups

Ilpo Järvinen
2024-10-22T08:57:34Z

On Mon, 21 Oct 2024 16:34:28 +0300, Andy Shevchenko wrote:

> The first patch avoids using a workaround for IO (which seems unneeded).
> The rest is a batch of cleanups. Has been tested on Intel Merrifield
> (thanks, Ferry).
>
> v3:
> - split patches 1 & 2 to two patches each (Ilpo)
> - preserve a comment for reading buffer (Ilpo)
> - inherited tags for new patches as the result is the same code wise
>
> [...]

Thank you for your contribution, it has been applied to my local
review-ilpo branch. Note it will show up in the public
platform-drivers-x86/review-ilpo branch only once I've pushed my
local branch there, which might take a while.

The list of commits applied:
[1/5] platform/x86: intel_scu_ipc: Unify the flow in pwr_reg_rdwr()
commit: 6bd35252102e2b79acb54c74527c81c09f690759
[2/5] platform/x86: intel_scu_ipc: Replace workaround by 32-bit IO
commit: 8b142950a5a718d55b09f08900306c75ceed2cf9
[3/5] platform/x86: intel_scu_ipc: Simplify code with cleanup helpers
commit: ce44b96261a6af318a9e28a16f0d9749e89179a6
[4/5] platform/x86: intel_scu_ipc: Convert to check for errors first
commit: acf1b04c6890aef9bb356195221e75ce3862f84e
[5/5] platform/x86: intel_scu_ipc: Save a copy of the entire struct intel_scu_ipc_data
commit: f02fcc7c12a61cfcb65937a570915672fa33ce73

--
i.