mfd: intel_soc_pmic_bxtwc: A few cleanups

[PATCH v1 0/4] mfd: intel_soc_pmic_bxtwc: A few cleanups

Posted by Andy Shevchenko 1 month, 1 week ago

While working on the fixes series against the driver I have noticed some
places that can be improved using better APIs or approaches. Hence this
mini-series. The plus statistics is due to the patch 2 that brought
missed inclusions to the code (instead of using a proxy header).

Have been tested on Intel Joule device.

Andy Shevchenko (4):
  mfd: intel_soc_pmic_bxtwc: Switch to use ATTRIBUTE_GROUPS()
  mfd: intel_soc_pmic_bxtwc: Don't use "proxy" headers
  mfd: intel_soc_pmic_bxtwc: Use temporary variable for struct device
  mfd: intel_soc_pmic_bxtwc: Deduplicate error messages

 drivers/mfd/intel_soc_pmic_bxtwc.c | 45 +++++++++++++++++-------------
 1 file changed, 26 insertions(+), 19 deletions(-)

-- 
2.43.0.rc1.1336.g36b5255a03ac

Re: [PATCH v1 0/4] mfd: intel_soc_pmic_bxtwc: A few cleanups

Posted by Lee Jones 3 weeks, 5 days ago

On Wed, 16 Oct 2024 13:49:50 +0300, Andy Shevchenko wrote:
> While working on the fixes series against the driver I have noticed some
> places that can be improved using better APIs or approaches. Hence this
> mini-series. The plus statistics is due to the patch 2 that brought
> missed inclusions to the code (instead of using a proxy header).
> 
> Have been tested on Intel Joule device.
> 
> [...]

Applied, thanks!

[1/4] mfd: intel_soc_pmic_bxtwc: Switch to use ATTRIBUTE_GROUPS()
      commit: 1f9e418aafc6ee52aad25b85f1e799f031f8de67
[2/4] mfd: intel_soc_pmic_bxtwc: Don't use "proxy" headers
      commit: d2e77347ce9a4d70165f4e97d6a4133e48e678cc
[3/4] mfd: intel_soc_pmic_bxtwc: Use temporary variable for struct device
      commit: 3d6b7374f0d8b5f8945185e901a6f5e7ae887a16
[4/4] mfd: intel_soc_pmic_bxtwc: Deduplicate error messages
      commit: 4a8b3d48dad2ddbbb150602606c65bef3c2b3f8e

--
Lee Jones [李琼斯]