public inbox for linux-i2c@vger.kernel.org
 help / color / mirror / Atom feed
From: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
To: Mario Limonciello <mario.limonciello@amd.com>,
	Andy Shevchenko <andriy.shevchenko@linux.intel.com>,
	Andi Shyti <andi.shyti@kernel.org>,
	Jarkko Nikula <jarkko.nikula@linux.intel.com>,
	linux-i2c@vger.kernel.org, linux-kernel@vger.kernel.org
Cc: Mika Westerberg <mika.westerberg@linux.intel.com>,
	Jan Dabros <jsd@semihalf.com>,
	Philipp Zabel <p.zabel@pengutronix.de>,
	Narasimhan.V@amd.com, Borislav Petkov <bp@alien8.de>,
	Kim Phillips <kim.phillips@amd.com>
Subject: [PATCH v1 0/5] i2c: designware: Cleanups (part 4)
Date: Wed, 25 Sep 2024 15:44:18 +0300	[thread overview]
Message-ID: <20240925124916.976173-1-andriy.shevchenko@linux.intel.com> (raw)

This is the subset of the patches [1] that should not affect any
functionality. Here are:
- tiding up the probe and remove functions
- dead and redundant code removal
- spelling fixes

In any case this is Cc'ed to AMD who reported a problem in [1]
presumably in the patch that is *not* included here.

Link: https://lore.kernel.org/linux-i2c/20231207141653.2785124-1-andriy.shevchenko@linux.intel.com/ [1]

This is assumed to be the last straightforward patch series to clean up
the driver. The rest is directly related or dependent on the reported
problem and needs more thinking and work. Perhaps I can do it in the
future.

Andy Shevchenko (5):
  i2c: designware: Use temporary variable for struct device
  i2c: designware: Get rid of redundant 'else'
  i2c: designware: Remove 'cond' from i2c_dw_scl_hcnt()
  i2c: designware: Use sda_hold_time variable name everywhere
  i2c: designware: Fix spelling and other issues in the comments

 drivers/i2c/busses/i2c-designware-amdpsp.c  | 10 ++--
 drivers/i2c/busses/i2c-designware-common.c  | 66 +++++++--------------
 drivers/i2c/busses/i2c-designware-core.h    | 12 ++--
 drivers/i2c/busses/i2c-designware-master.c  | 17 +++---
 drivers/i2c/busses/i2c-designware-pcidrv.c  | 39 ++++++------
 drivers/i2c/busses/i2c-designware-platdrv.c | 52 ++++++++--------
 drivers/i2c/busses/i2c-designware-slave.c   |  6 +-
 7 files changed, 90 insertions(+), 112 deletions(-)

-- 
2.43.0.rc1.1336.g36b5255a03ac


             reply	other threads:[~2024-09-25 12:49 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-09-25 12:44 Andy Shevchenko [this message]
2024-09-25 12:44 ` [PATCH v1 1/5] i2c: designware: Use temporary variable for struct device Andy Shevchenko
2024-09-25 12:44 ` [PATCH v1 2/5] i2c: designware: Get rid of redundant 'else' Andy Shevchenko
2024-09-25 12:44 ` [PATCH v1 3/5] i2c: designware: Remove 'cond' from i2c_dw_scl_hcnt() Andy Shevchenko
2024-09-25 12:44 ` [PATCH v1 4/5] i2c: designware: Use sda_hold_time variable name everywhere Andy Shevchenko
2024-09-25 12:44 ` [PATCH v1 5/5] i2c: designware: Fix spelling and other issues in the comments Andy Shevchenko
2024-09-25 13:37 ` [PATCH v1 0/5] i2c: designware: Cleanups (part 4) Jarkko Nikula
2024-09-26 20:03 ` Andi Shyti

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20240925124916.976173-1-andriy.shevchenko@linux.intel.com \
    --to=andriy.shevchenko@linux.intel.com \
    --cc=Narasimhan.V@amd.com \
    --cc=andi.shyti@kernel.org \
    --cc=bp@alien8.de \
    --cc=jarkko.nikula@linux.intel.com \
    --cc=jsd@semihalf.com \
    --cc=kim.phillips@amd.com \
    --cc=linux-i2c@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mario.limonciello@amd.com \
    --cc=mika.westerberg@linux.intel.com \
    --cc=p.zabel@pengutronix.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox