* [PATCH v5 0/2] i2c: designware-platdrv: simplify reset control and error handling
@ 2026-01-23 10:05 Artem Shimko
2026-01-23 11:13 ` Andy Shevchenko
0 siblings, 1 reply; 6+ messages in thread
From: Artem Shimko @ 2026-01-23 10:05 UTC (permalink / raw)
To: andi.shyti, p.zabel, mika.westerberg, andriy.shevchenko, jsd
Cc: Artem Shimko, linux-i2c, linux-kernel
Hello maintainers and reviewers,
This series refactors the DesignWare platform I2C driver to simplify
reset control management and improve error handling:
Patch 1 replaces the manual reset control acquisition and deassertion
with devm_reset_control_get_optional_exclusive_deasserted(), which
combines both operations in a single call and leverages automatic
resource management.
Patch 2 follows up by refactoring the probe error handling to eliminate
goto statements in favor of direct returns with dev_err_probe(), resulting
in cleaner and more linear control flow.
Both patches maintain functional equivalence while improving code
readability and maintainability.
Regards,
artem
Artem Shimko (2):
i2c: designware-platdrv: simplify reset control
i2c: designware-platdrv: refactor error handling and simplify cleanup
drivers/i2c/busses/i2c-designware-platdrv.c | 41 +++++++--------------
1 file changed, 13 insertions(+), 28 deletions(-)
--
2.43.0
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v5 0/2] i2c: designware-platdrv: simplify reset control and error handling
2026-01-23 10:05 [PATCH v5 0/2] i2c: designware-platdrv: simplify reset control and error handling Artem Shimko
@ 2026-01-23 11:13 ` Andy Shevchenko
2026-01-23 11:14 ` Andy Shevchenko
0 siblings, 1 reply; 6+ messages in thread
From: Andy Shevchenko @ 2026-01-23 11:13 UTC (permalink / raw)
To: Artem Shimko
Cc: andi.shyti, p.zabel, mika.westerberg, jsd, linux-i2c,
linux-kernel
On Fri, Jan 23, 2026 at 01:05:34PM +0300, Artem Shimko wrote:
> Hello maintainers and reviewers,
>
> This series refactors the DesignWare platform I2C driver to simplify
> reset control management and improve error handling:
>
> Patch 1 replaces the manual reset control acquisition and deassertion
> with devm_reset_control_get_optional_exclusive_deasserted(), which
> combines both operations in a single call and leverages automatic
> resource management.
>
> Patch 2 follows up by refactoring the probe error handling to eliminate
> goto statements in favor of direct returns with dev_err_probe(), resulting
> in cleaner and more linear control flow.
>
> Both patches maintain functional equivalence while improving code
> readability and maintainability.
You send a series as _three_ detached emails. Forgot using
git format-patch --thread ...
?
--
With Best Regards,
Andy Shevchenko
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v5 0/2] i2c: designware-platdrv: simplify reset control and error handling
2026-01-23 11:13 ` Andy Shevchenko
@ 2026-01-23 11:14 ` Andy Shevchenko
2026-01-23 11:35 ` Artem Shimko
0 siblings, 1 reply; 6+ messages in thread
From: Andy Shevchenko @ 2026-01-23 11:14 UTC (permalink / raw)
To: Artem Shimko
Cc: andi.shyti, p.zabel, mika.westerberg, jsd, linux-i2c,
linux-kernel
On Fri, Jan 23, 2026 at 01:13:16PM +0200, Andy Shevchenko wrote:
> On Fri, Jan 23, 2026 at 01:05:34PM +0300, Artem Shimko wrote:
> > Hello maintainers and reviewers,
> >
> > This series refactors the DesignWare platform I2C driver to simplify
> > reset control management and improve error handling:
> >
> > Patch 1 replaces the manual reset control acquisition and deassertion
> > with devm_reset_control_get_optional_exclusive_deasserted(), which
> > combines both operations in a single call and leverages automatic
> > resource management.
> >
> > Patch 2 follows up by refactoring the probe error handling to eliminate
> > goto statements in favor of direct returns with dev_err_probe(), resulting
> > in cleaner and more linear control flow.
> >
> > Both patches maintain functional equivalence while improving code
> > readability and maintainability.
>
> You send a series as _three_ detached emails. Forgot using
>
> git format-patch --thread ...
>
> ?
On top of that, this is marked as v5, where the changelog for all previous
iterations?
--
With Best Regards,
Andy Shevchenko
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v5 0/2] i2c: designware-platdrv: simplify reset control and error handling
2026-01-23 11:14 ` Andy Shevchenko
@ 2026-01-23 11:35 ` Artem Shimko
2026-01-23 14:10 ` Andy Shevchenko
0 siblings, 1 reply; 6+ messages in thread
From: Artem Shimko @ 2026-01-23 11:35 UTC (permalink / raw)
To: Andy Shevchenko
Cc: andi.shyti, p.zabel, mika.westerberg, jsd, linux-i2c,
linux-kernel
On Fri, Jan 23, 2026 at 2:14 PM Andy Shevchenko
<andriy.shevchenko@linux.intel.com> wrote:
> On top of that, this is marked as v5, where the changelog for all previous
> iterations?
Hi Andy,
Got it, yeah... Thanks!
Could you please advise, should I send it again with changelog as v6?
Regards,
artem
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v5 0/2] i2c: designware-platdrv: simplify reset control and error handling
2026-01-23 11:35 ` Artem Shimko
@ 2026-01-23 14:10 ` Andy Shevchenko
2026-01-23 16:03 ` Artem Shimko
0 siblings, 1 reply; 6+ messages in thread
From: Andy Shevchenko @ 2026-01-23 14:10 UTC (permalink / raw)
To: Artem Shimko
Cc: andi.shyti, p.zabel, mika.westerberg, jsd, linux-i2c,
linux-kernel
On Fri, Jan 23, 2026 at 02:35:44PM +0300, Artem Shimko wrote:
> On Fri, Jan 23, 2026 at 2:14 PM Andy Shevchenko
> <andriy.shevchenko@linux.intel.com> wrote:
> > On top of that, this is marked as v5, where the changelog for all previous
> > iterations?
>
> Got it, yeah... Thanks!
>
> Could you please advise, should I send it again with changelog as v6?
Wait a bit (24h + ) for others to have a chance to comment.
--
With Best Regards,
Andy Shevchenko
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v5 0/2] i2c: designware-platdrv: simplify reset control and error handling
2026-01-23 14:10 ` Andy Shevchenko
@ 2026-01-23 16:03 ` Artem Shimko
0 siblings, 0 replies; 6+ messages in thread
From: Artem Shimko @ 2026-01-23 16:03 UTC (permalink / raw)
To: Andy Shevchenko
Cc: andi.shyti, p.zabel, mika.westerberg, jsd, linux-i2c,
linux-kernel
On Fri, Jan 23, 2026 at 5:10 PM Andy Shevchenko
<andriy.shevchenko@linux.intel.com> wrote:
> Wait a bit (24h + ) for others to have a chance to comment.
Got it, thanks. =)
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2026-01-23 16:03 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-01-23 10:05 [PATCH v5 0/2] i2c: designware-platdrv: simplify reset control and error handling Artem Shimko
2026-01-23 11:13 ` Andy Shevchenko
2026-01-23 11:14 ` Andy Shevchenko
2026-01-23 11:35 ` Artem Shimko
2026-01-23 14:10 ` Andy Shevchenko
2026-01-23 16:03 ` Artem Shimko
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox