[PULL REQUEST] i2c/fix-registration for v7.2

[PULL REQUEST] i2c/fix-registration for v7.2

[Wolfram Sang]

[-- Attachment #1: Type: text/plain, Size: 1618 bytes --]

Hi Andi,

here are the latest patches from Johan. I reviewed them, tested them as
far as I could, and already let them sit in next for a while. If there
are problems with these patches, I'll take care. Please pull, so Johan
can continue work on the life cycle issues.

Thank you and all the best,

   Wolfram


The following changes since commit 254f49634ee16a731174d2ae34bc50bd5f45e731:

  Linux 7.1-rc1 (2026-04-26 14:19:00 -0700)

are available in the Git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/wsa/linux.git i2c/fix-registration

for you to fetch changes up to 1640403fd38e9c0a1d9d70965d36410b263b6e2c:

  i2c: core: clean up adapter registration error label (2026-05-30 23:57:19 +0200)

----------------------------------------------------------------
Johan Hovold (10):
      i2c: core: fix irq domain leak on adapter registration failure
      i2c: core: fix hang on adapter registration failure
      i2c: core: fix NULL-deref on adapter registration failure
      i2c: core: fix adapter probe deferral loop
      i2c: core: fix adapter debugfs creation
      i2c: core: disable runtime PM on adapter registration failure
      i2c: core: fix adapter registration race
      i2c: core: fix adapter deregistration race
      i2c: core: clean up bus id allocation
      i2c: core: clean up adapter registration error label

 drivers/i2c/i2c-core-base.c | 173 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++-------------------------------------------------------------------------
 1 file changed, 100 insertions(+), 73 deletions(-)



[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: [PULL REQUEST] i2c/fix-registration for v7.2

[Andi Shyti]

Hi Wolfram,

> here are the latest patches from Johan. I reviewed them, tested them as
> far as I could, and already let them sit in next for a while. If there
> are problems with these patches, I'll take care. Please pull, so Johan
> can continue work on the life cycle issues.
> 
> Thank you and all the best,

Thank you, I really appreciate this pull request!

> Johan Hovold (10):
>       i2c: core: fix irq domain leak on adapter registration failure
>       i2c: core: fix hang on adapter registration failure
>       i2c: core: fix NULL-deref on adapter registration failure
>       i2c: core: fix adapter probe deferral loop
>       i2c: core: fix adapter debugfs creation
>       i2c: core: disable runtime PM on adapter registration failure
>       i2c: core: fix adapter registration race
>       i2c: core: fix adapter deregistration race

I cherry-picked until here in i2c-host-fixes.

>       i2c: core: clean up bus id allocation
>       i2c: core: clean up adapter registration error label

I cherry-picked these last two in i2c-host-next.

Thanks,
Andi

Re: [PULL REQUEST] i2c/fix-registration for v7.2

[Wolfram Sang]

[-- Attachment #1: Type: text/plain, Size: 1078 bytes --]

Hi Andi,

> > Johan Hovold (10):
> >       i2c: core: fix irq domain leak on adapter registration failure
> >       i2c: core: fix hang on adapter registration failure
> >       i2c: core: fix NULL-deref on adapter registration failure
> >       i2c: core: fix adapter probe deferral loop
> >       i2c: core: fix adapter debugfs creation
> >       i2c: core: disable runtime PM on adapter registration failure
> >       i2c: core: fix adapter registration race
> >       i2c: core: fix adapter deregistration race
> 
> I cherry-picked until here in i2c-host-fixes.

Please don't use 'fixes'. These changes are too intrusive to be added so
late in the cycle. Stable maintainers can decide if they want to
backport this. While -next gets some testing, a lot more testing is done
once it hits Linus' tree. If we cause a regression now, we won't have
another rc to fix it. If a regression shows up during the mergewindow,
we have multiple options to deal with it (fix it, revert it). Next
mergewindow is totally fine for this.

All the best,

   Wolfram


[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: [PULL REQUEST] i2c/fix-registration for v7.2

[Andi Shyti]

Hi Wolfram,

> > > Johan Hovold (10):
> > >       i2c: core: fix irq domain leak on adapter registration failure
> > >       i2c: core: fix hang on adapter registration failure
> > >       i2c: core: fix NULL-deref on adapter registration failure
> > >       i2c: core: fix adapter probe deferral loop
> > >       i2c: core: fix adapter debugfs creation
> > >       i2c: core: disable runtime PM on adapter registration failure
> > >       i2c: core: fix adapter registration race
> > >       i2c: core: fix adapter deregistration race
> > 
> > I cherry-picked until here in i2c-host-fixes.
> 
> Please don't use 'fixes'. These changes are too intrusive to be added so
> late in the cycle. Stable maintainers can decide if they want to
> backport this. While -next gets some testing, a lot more testing is done
> once it hits Linus' tree. If we cause a regression now, we won't have
> another rc to fix it. If a regression shows up during the mergewindow,
> we have multiple options to deal with it (fix it, revert it). Next
> mergewindow is totally fine for this.

yes, I did spend some time looking at these patches and i was in
doubt where to merge them. At the end I decided to split the
series.

Now they are all in the non fixes branch.

Thanks,
Andi

Re: [PULL REQUEST] i2c/fix-registration for v7.2

[Wolfram Sang]

[-- Attachment #1: Type: text/plain, Size: 348 bytes --]


> yes, I did spend some time looking at these patches and i was in
> doubt where to merge them. At the end I decided to split the
> series.

My suggestion: When in doubt, tend to be conservative in changing
something. Fixes can always be backported, but regression are a real
pain.

> Now they are all in the non fixes branch.

Great, thank you!


[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]