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 v2: - dropped stray change (Mika) - elaborated conditional changes in the commit message (Mika) - gathered tags (Mika, Ferry) Andy Shevchenko (5): platform/x86: intel_scu_ipc: Unify the flow in pwr_reg_rdwr() platform/x86: intel_scu_ipc: Replace workaround by 32-bit IO platform/x86: intel_scu_ipc: Simplify code with cleanup helpers platform/x86: intel_scu_ipc: Convert to check for errors first platform/x86: intel_scu_ipc: Save a copy of the entire struct intel_scu_ipc_data drivers/platform/x86/intel_scu_ipc.c | 142 ++++++++++++--------------- 1 file changed, 61 insertions(+), 81 deletions(-) -- 2.43.0.rc1.1336.g36b5255a03ac