Re: [PATCH V6 1/6] regmap: irq: dispose all virtual irq before removing domain

[Javier Martinez Canillas]

Hello Laxman,

On 02/09/2016 10:58 AM, Laxman Dewangan wrote:
> On Tuesday 09 February 2016 06:57 PM, Javier Martinez Canillas wrote:
> > Hello Laxman,
> >
> > On 02/09/2016 09:28 AM, Laxman Dewangan wrote:
> > > It is require to dispose all virtual irq of hwirq on chip
> > > created on given irq domain before removing this irq domain.
> > > Hence dispose all mapped irqs before deleting the irq domains
> > > in regmap_del_irq_chip();
> > >
> > > Signed-off-by: Laxman Dewangan <ldewangan@xxxxxxxxxx>
> >
> > I believe this patch could be picked separately and not made part of
> > this series since is fixing a bug that happens on most drivers using
> > the regmap-irq API. This will avoid cross-subsystem churn for people.
> >
> > Your patch 6/6 does not introduce a regression since the bug already
> > exists in the MFD driver, it just makes it more noticeable since it
> > is easier to unbind the max77686 RTC driver than the MFD one.
>
>
> If we dont have fix then rtc unbind/bind creates issue on S2R. Although it was issue on tot but the issue visible with my patch only.
>
> So if you test my 2 to 6 without 1, you will see issue.

Yes I know that but my point is that this will also happen in mainline if
the MFD device is unbind. Is just that it's harder to do so since at least
in the Chromebooks the regulators are used for the panel and backlight so
you will need a serial console.
 
> So to avoid bisect issue in functionality wise, this should go on sequence. This is my view.

The problem with patch series touching different subsystems is that the
maintainers have to agree on how to handle the possible conflicts so it
is easier for them if you split the patches and post them separately if
the are really no dependencies.

My view is that this fixes a regmap-irq core bug that is present in all
the drivers using regmap-irq that remove the IRQ chip after mapping IRQs
so it's really not related to your RTC series.

But of course I'm not a subsystem maintainer so is up to Alexandre and
Mark to decide that. I was just giving my opinion.

Best regards,
--
Javier Martinez Canillas
Open Source Group
Samsung Research America