[v1] platform/x86: intel_scu_ipc: Avoid working around IO and cleanups

[PATCH v1 0/3] platform/x86: intel_scu_ipc: Avoid working around IO and cleanups

Andy Shevchenko posted 3 patches 1 month, 1 week ago

Diff against v2 v3
Download series mbox

There is a newer version of this series

drivers/platform/x86/intel_scu_ipc.c | 142 ++++++++++++---------------
1 file changed, 61 insertions(+), 81 deletions(-)

Expand all Fold all

[PATCH v1 0/3] platform/x86: intel_scu_ipc: Avoid working around IO and cleanups

Posted by Andy Shevchenko 1 month, 1 week ago

The first patch avoids using a workaround for IO (which seems unneeded).
The rest is a batch of cleanups. Cc'ed to Ferry in hope of testing on
Intel Merrifield (the main platform that uses these APIs).

Andy Shevchenko (3):
  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: 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

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

Posted by Ferry Toth 1 month, 1 week ago

Hi

Op 16-10-2024 om 13:48 schreef Andy Shevchenko:
> The first patch avoids using a workaround for IO (which seems unneeded).
> The rest is a batch of cleanups. Cc'ed to Ferry in hope of testing on
> Intel Merrifield (the main platform that uses these APIs).
>
> Andy Shevchenko (3):
>    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: 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(-)
>
Tested iio_info (mrfld_bcove_adc), host/gadget switch, power button.

Tested-by: Ferry Toth <fntoth@gmail.com> (Intel Edison-Arduino)