Re: [PATCH v2 8/8] platform: x86: intel_bxtwc_tmu: remove first level irq unmask

[Andy Shevchenko]

On Mon, Apr 24, 2017 at 3:24 PM, Lee Jones  wrote:
> On Mon, 24 Apr 2017, Andy Shevchenko wrote:
>
>> On Mon, Apr 24, 2017 at 12:24 PM, Lee Jones  wrote:
>> > On Sat, 22 Apr 2017, Andy Shevchenko wrote:
>> >
>> >> On Sat, Apr 22, 2017 at 1:34 AM, sathyanarayanan kuppuswamy
>> >>  wrote:
>> >>
>> >> > Thanks for brining it up. I was planning to ask either Andy or Lee 
>> >> > regarding
>> >> > this issue after all patches in the series are reviewed.
>> >>
>> >> Darren, I'm planning to review this soon.
>> >>
>> >> P.S. We have few series flying around regarding to Intel PMIC(s): my
>> >> for Kconfig naming, Hans' for Crystal Cove (touches Kconfig as well),
>> >> and Sathya's series. I hope Lee can collect them in proper order.
>> >
>> > That could be easier said than done.
>>
>> Some of them are already in your tree, then apply
>> - my Kconfig patch which makes Broxton Whiskey Cove to be built
>> independently of Crystal Cove (I didn't notice that patch in your
>> tree, though some of your mail told it had been applied)
>> - Hans' series regarding CherryTrail Whiskey Cove support (if it has all 
>> ACKs)
>> - this series after my review.
>>
>> Does it sound doable?
>
> My Inbox is now empty, so all patches are now either applied or
> waiting further Acks/Actions.

I reviewed the series and found it suitable to apply.
The GPIO Whiskey Cove patch will do a conflict (in GPIO tree is
another patch that adds GPIO IRQ masks in a way by using GENMASK(),
so, we might avoid this doing the same in patch 5 here).

Reviewed-by: Andy Shevchenko 

for entire series.

> I'm pretty certain that this set is not going in before v4.13.

It is now on your side :-)

-- 
With Best Regards,
Andy Shevchenko

Re: [PATCH v2 8/8] platform: x86: intel_bxtwc_tmu: remove first level irq unmask

[Andy Shevchenko]

On Mon, Apr 24, 2017 at 3:24 PM, Lee Jones  wrote:
> On Mon, 24 Apr 2017, Andy Shevchenko wrote:
>
>> On Mon, Apr 24, 2017 at 12:24 PM, Lee Jones  wrote:
>> > On Sat, 22 Apr 2017, Andy Shevchenko wrote:
>> >
>> >> On Sat, Apr 22, 2017 at 1:34 AM, sathyanarayanan kuppuswamy
>> >>  wrote:
>> >>
>> >> > Thanks for brining it up. I was planning to ask either Andy or Lee 
>> >> > regarding
>> >> > this issue after all patches in the series are reviewed.
>> >>
>> >> Darren, I'm planning to review this soon.
>> >>
>> >> P.S. We have few series flying around regarding to Intel PMIC(s): my
>> >> for Kconfig naming, Hans' for Crystal Cove (touches Kconfig as well),
>> >> and Sathya's series. I hope Lee can collect them in proper order.
>> >
>> > That could be easier said than done.
>>
>> Some of them are already in your tree, then apply
>> - my Kconfig patch which makes Broxton Whiskey Cove to be built
>> independently of Crystal Cove (I didn't notice that patch in your
>> tree, though some of your mail told it had been applied)
>> - Hans' series regarding CherryTrail Whiskey Cove support (if it has all 
>> ACKs)
>> - this series after my review.
>>
>> Does it sound doable?
>
> My Inbox is now empty, so all patches are now either applied or
> waiting further Acks/Actions.

I reviewed the series and found it suitable to apply.
The GPIO Whiskey Cove patch will do a conflict (in GPIO tree is
another patch that adds GPIO IRQ masks in a way by using GENMASK(),
so, we might avoid this doing the same in patch 5 here).

Reviewed-by: Andy Shevchenko 

for entire series.

> I'm pretty certain that this set is not going in before v4.13.

It is now on your side :-)

-- 
With Best Regards,
Andy Shevchenko

Re: [PATCH v2 8/8] platform: x86: intel_bxtwc_tmu: remove first level irq unmask

[Lee Jones]

On Mon, 24 Apr 2017, Andy Shevchenko wrote:

> On Mon, Apr 24, 2017 at 12:24 PM, Lee Jones  wrote:
> > On Sat, 22 Apr 2017, Andy Shevchenko wrote:
> >
> >> On Sat, Apr 22, 2017 at 1:34 AM, sathyanarayanan kuppuswamy
> >>  wrote:
> >>
> >> > Thanks for brining it up. I was planning to ask either Andy or Lee 
> >> > regarding
> >> > this issue after all patches in the series are reviewed.
> >>
> >> Darren, I'm planning to review this soon.
> >>
> >> P.S. We have few series flying around regarding to Intel PMIC(s): my
> >> for Kconfig naming, Hans' for Crystal Cove (touches Kconfig as well),
> >> and Sathya's series. I hope Lee can collect them in proper order.
> >
> > That could be easier said than done.
> 
> Some of them are already in your tree, then apply
> - my Kconfig patch which makes Broxton Whiskey Cove to be built
> independently of Crystal Cove (I didn't notice that patch in your
> tree, though some of your mail told it had been applied)
> - Hans' series regarding CherryTrail Whiskey Cove support (if it has all ACKs)
> - this series after my review.
> 
> Does it sound doable?

My Inbox is now empty, so all patches are now either applied or
waiting further Acks/Actions.

I'm pretty certain that this set is not going in before v4.13.

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog

Re: [PATCH v2 8/8] platform: x86: intel_bxtwc_tmu: remove first level irq unmask

[Lee Jones]

On Mon, 24 Apr 2017, Andy Shevchenko wrote:

> On Mon, Apr 24, 2017 at 12:24 PM, Lee Jones  wrote:
> > On Sat, 22 Apr 2017, Andy Shevchenko wrote:
> >
> >> On Sat, Apr 22, 2017 at 1:34 AM, sathyanarayanan kuppuswamy
> >>  wrote:
> >>
> >> > Thanks for brining it up. I was planning to ask either Andy or Lee 
> >> > regarding
> >> > this issue after all patches in the series are reviewed.
> >>
> >> Darren, I'm planning to review this soon.
> >>
> >> P.S. We have few series flying around regarding to Intel PMIC(s): my
> >> for Kconfig naming, Hans' for Crystal Cove (touches Kconfig as well),
> >> and Sathya's series. I hope Lee can collect them in proper order.
> >
> > That could be easier said than done.
> 
> Some of them are already in your tree, then apply
> - my Kconfig patch which makes Broxton Whiskey Cove to be built
> independently of Crystal Cove (I didn't notice that patch in your
> tree, though some of your mail told it had been applied)
> - Hans' series regarding CherryTrail Whiskey Cove support (if it has all ACKs)
> - this series after my review.
> 
> Does it sound doable?

My Inbox is now empty, so all patches are now either applied or
waiting further Acks/Actions.

I'm pretty certain that this set is not going in before v4.13.

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog

Re: [PATCH v2 8/8] platform: x86: intel_bxtwc_tmu: remove first level irq unmask

[Andy Shevchenko]

On Mon, Apr 24, 2017 at 12:24 PM, Lee Jones  wrote:
> On Sat, 22 Apr 2017, Andy Shevchenko wrote:
>
>> On Sat, Apr 22, 2017 at 1:34 AM, sathyanarayanan kuppuswamy
>>  wrote:
>>
>> > Thanks for brining it up. I was planning to ask either Andy or Lee 
>> > regarding
>> > this issue after all patches in the series are reviewed.
>>
>> Darren, I'm planning to review this soon.
>>
>> P.S. We have few series flying around regarding to Intel PMIC(s): my
>> for Kconfig naming, Hans' for Crystal Cove (touches Kconfig as well),
>> and Sathya's series. I hope Lee can collect them in proper order.
>
> That could be easier said than done.

Some of them are already in your tree, then apply
- my Kconfig patch which makes Broxton Whiskey Cove to be built
independently of Crystal Cove (I didn't notice that patch in your
tree, though some of your mail told it had been applied)
- Hans' series regarding CherryTrail Whiskey Cove support (if it has all ACKs)
- this series after my review.

Does it sound doable?

-- 
With Best Regards,
Andy Shevchenko

Re: [PATCH v2 8/8] platform: x86: intel_bxtwc_tmu: remove first level irq unmask

[Andy Shevchenko]

On Mon, Apr 24, 2017 at 12:24 PM, Lee Jones  wrote:
> On Sat, 22 Apr 2017, Andy Shevchenko wrote:
>
>> On Sat, Apr 22, 2017 at 1:34 AM, sathyanarayanan kuppuswamy
>>  wrote:
>>
>> > Thanks for brining it up. I was planning to ask either Andy or Lee 
>> > regarding
>> > this issue after all patches in the series are reviewed.
>>
>> Darren, I'm planning to review this soon.
>>
>> P.S. We have few series flying around regarding to Intel PMIC(s): my
>> for Kconfig naming, Hans' for Crystal Cove (touches Kconfig as well),
>> and Sathya's series. I hope Lee can collect them in proper order.
>
> That could be easier said than done.

Some of them are already in your tree, then apply
- my Kconfig patch which makes Broxton Whiskey Cove to be built
independently of Crystal Cove (I didn't notice that patch in your
tree, though some of your mail told it had been applied)
- Hans' series regarding CherryTrail Whiskey Cove support (if it has all ACKs)
- this series after my review.

Does it sound doable?

-- 
With Best Regards,
Andy Shevchenko

Re: [PATCH v2 8/8] platform: x86: intel_bxtwc_tmu: remove first level irq unmask

[Lee Jones]

On Fri, 21 Apr 2017, Darren Hart wrote:

> On Fri, Apr 14, 2017 at 04:26:00PM -0700, 
> sathyanarayanan.kuppusw...@linux.intel.com wrote:
> > From: Kuppuswamy Sathyanarayanan 
> > 
> > 
> > Currently in WCOVE PMIC mfd driver, all second level irq chips
> 
> By currently I believe you mean after the earlier patch in this series is
> applied, correct? This one is dependent on the previous one?
> 
> > are chained to the respective first level irqs. So there is no
> > need for explicitly unmasking the first level irq in this
> > driver. This patches removes this level 1 irq unmask support.
> > 
> > Signed-off-by: Kuppuswamy Sathyanarayanan 
> > 
> 
> For platform drivers x86:
> 
> Reviewed-by: Darren Hart (VMware) 
> 
> Are you working with a specific maintainers to pull this in as a series? With 
> so
> many subsystems, we need to coordinate to make sure we don't make a mess for
> Linus. Given the interdependencies, I'd recommend someone pull the series in 
> as
> a whole - maybe into MFD? Lee, do you have a preference?

I am happy to take the set, however I think v4.12 is an unlikely
candidate, since there still have quite a few patches which are yet to
receive review and the merge-window opens in 6 days.

Once we have all the required Acks, I'll push the set into -next where
it can sit for a good soak test until v4.13.

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog

Re: [PATCH v2 8/8] platform: x86: intel_bxtwc_tmu: remove first level irq unmask

[Lee Jones]

On Fri, 21 Apr 2017, Darren Hart wrote:

> On Fri, Apr 14, 2017 at 04:26:00PM -0700, 
> sathyanarayanan.kuppusw...@linux.intel.com wrote:
> > From: Kuppuswamy Sathyanarayanan 
> > 
> > 
> > Currently in WCOVE PMIC mfd driver, all second level irq chips
> 
> By currently I believe you mean after the earlier patch in this series is
> applied, correct? This one is dependent on the previous one?
> 
> > are chained to the respective first level irqs. So there is no
> > need for explicitly unmasking the first level irq in this
> > driver. This patches removes this level 1 irq unmask support.
> > 
> > Signed-off-by: Kuppuswamy Sathyanarayanan 
> > 
> 
> For platform drivers x86:
> 
> Reviewed-by: Darren Hart (VMware) 
> 
> Are you working with a specific maintainers to pull this in as a series? With 
> so
> many subsystems, we need to coordinate to make sure we don't make a mess for
> Linus. Given the interdependencies, I'd recommend someone pull the series in 
> as
> a whole - maybe into MFD? Lee, do you have a preference?

I am happy to take the set, however I think v4.12 is an unlikely
candidate, since there still have quite a few patches which are yet to
receive review and the merge-window opens in 6 days.

Once we have all the required Acks, I'll push the set into -next where
it can sit for a good soak test until v4.13.

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog

Re: [PATCH v2 8/8] platform: x86: intel_bxtwc_tmu: remove first level irq unmask

[Lee Jones]

On Sat, 22 Apr 2017, Andy Shevchenko wrote:

> On Sat, Apr 22, 2017 at 1:34 AM, sathyanarayanan kuppuswamy
>  wrote:
> 
> > Thanks for brining it up. I was planning to ask either Andy or Lee regarding
> > this issue after all patches in the series are reviewed.
> 
> Darren, I'm planning to review this soon.
> 
> P.S. We have few series flying around regarding to Intel PMIC(s): my
> for Kconfig naming, Hans' for Crystal Cove (touches Kconfig as well),
> and Sathya's series. I hope Lee can collect them in proper order.

That could be easier said than done.

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog

Re: [PATCH v2 8/8] platform: x86: intel_bxtwc_tmu: remove first level irq unmask

[Lee Jones]

On Sat, 22 Apr 2017, Andy Shevchenko wrote:

> On Sat, Apr 22, 2017 at 1:34 AM, sathyanarayanan kuppuswamy
>  wrote:
> 
> > Thanks for brining it up. I was planning to ask either Andy or Lee regarding
> > this issue after all patches in the series are reviewed.
> 
> Darren, I'm planning to review this soon.
> 
> P.S. We have few series flying around regarding to Intel PMIC(s): my
> for Kconfig naming, Hans' for Crystal Cove (touches Kconfig as well),
> and Sathya's series. I hope Lee can collect them in proper order.

That could be easier said than done.

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog

Re: [PATCH v2 8/8] platform: x86: intel_bxtwc_tmu: remove first level irq unmask

[Andy Shevchenko]

On Sat, Apr 22, 2017 at 1:34 AM, sathyanarayanan kuppuswamy
 wrote:

> Thanks for brining it up. I was planning to ask either Andy or Lee regarding
> this issue after all patches in the series are reviewed.

Darren, I'm planning to review this soon.

P.S. We have few series flying around regarding to Intel PMIC(s): my
for Kconfig naming, Hans' for Crystal Cove (touches Kconfig as well),
and Sathya's series. I hope Lee can collect them in proper order.

-- 
With Best Regards,
Andy Shevchenko

Re: [PATCH v2 8/8] platform: x86: intel_bxtwc_tmu: remove first level irq unmask

[Andy Shevchenko]

On Sat, Apr 22, 2017 at 1:34 AM, sathyanarayanan kuppuswamy
 wrote:

> Thanks for brining it up. I was planning to ask either Andy or Lee regarding
> this issue after all patches in the series are reviewed.

Darren, I'm planning to review this soon.

P.S. We have few series flying around regarding to Intel PMIC(s): my
for Kconfig naming, Hans' for Crystal Cove (touches Kconfig as well),
and Sathya's series. I hope Lee can collect them in proper order.

-- 
With Best Regards,
Andy Shevchenko

Re: [PATCH v2 8/8] platform: x86: intel_bxtwc_tmu: remove first level irq unmask

[sathyanarayanan kuppuswamy]

Hi Darren,

Thanks for the review.


On 04/21/2017 03:00 PM, Darren Hart wrote:

On Fri, Apr 14, 2017 at 04:26:00PM -0700, 
sathyanarayanan.kuppusw...@linux.intel.com wrote:

From: Kuppuswamy Sathyanarayanan 

Currently in WCOVE PMIC mfd driver, all second level irq chips

By currently I believe you mean after the earlier patch in this series is
applied, correct?

Yes.

This one is dependent on the previous one?
Yes, one of my previous patch in this series fixes this problem in MFD 
driver.



are chained to the respective first level irqs. So there is no
need for explicitly unmasking the first level irq in this
driver. This patches removes this level 1 irq unmask support.

Signed-off-by: Kuppuswamy Sathyanarayanan 


For platform drivers x86:

Reviewed-by: Darren Hart (VMware) 

Are you working with a specific maintainers to pull this in as a series?
Thanks for brining it up. I was planning to ask either Andy or Lee 
regarding this issue after all patches in the series are reviewed.

With so
many subsystems, we need to coordinate to make sure we don't make a mess for
Linus. Given the interdependencies, I'd recommend someone pull the series in as
a whole - maybe into MFD? Lee, do you have a preference?



--
Sathyanarayanan Kuppuswamy
Android kernel developer

Re: [PATCH v2 8/8] platform: x86: intel_bxtwc_tmu: remove first level irq unmask

[sathyanarayanan kuppuswamy]

Hi Darren,

Thanks for the review.


On 04/21/2017 03:00 PM, Darren Hart wrote:

On Fri, Apr 14, 2017 at 04:26:00PM -0700, 
sathyanarayanan.kuppusw...@linux.intel.com wrote:

From: Kuppuswamy Sathyanarayanan 

Currently in WCOVE PMIC mfd driver, all second level irq chips

By currently I believe you mean after the earlier patch in this series is
applied, correct?

Yes.

This one is dependent on the previous one?
Yes, one of my previous patch in this series fixes this problem in MFD 
driver.



are chained to the respective first level irqs. So there is no
need for explicitly unmasking the first level irq in this
driver. This patches removes this level 1 irq unmask support.

Signed-off-by: Kuppuswamy Sathyanarayanan 


For platform drivers x86:

Reviewed-by: Darren Hart (VMware) 

Are you working with a specific maintainers to pull this in as a series?
Thanks for brining it up. I was planning to ask either Andy or Lee 
regarding this issue after all patches in the series are reviewed.

With so
many subsystems, we need to coordinate to make sure we don't make a mess for
Linus. Given the interdependencies, I'd recommend someone pull the series in as
a whole - maybe into MFD? Lee, do you have a preference?



--
Sathyanarayanan Kuppuswamy
Android kernel developer

Re: [PATCH v2 8/8] platform: x86: intel_bxtwc_tmu: remove first level irq unmask

[Darren Hart]

On Fri, Apr 14, 2017 at 04:26:00PM -0700, 
sathyanarayanan.kuppusw...@linux.intel.com wrote:
> From: Kuppuswamy Sathyanarayanan 
> 
> Currently in WCOVE PMIC mfd driver, all second level irq chips

By currently I believe you mean after the earlier patch in this series is
applied, correct? This one is dependent on the previous one?

> are chained to the respective first level irqs. So there is no
> need for explicitly unmasking the first level irq in this
> driver. This patches removes this level 1 irq unmask support.
> 
> Signed-off-by: Kuppuswamy Sathyanarayanan 
> 

For platform drivers x86:

Reviewed-by: Darren Hart (VMware) 

Are you working with a specific maintainers to pull this in as a series? With so
many subsystems, we need to coordinate to make sure we don't make a mess for
Linus. Given the interdependencies, I'd recommend someone pull the series in as
a whole - maybe into MFD? Lee, do you have a preference?

-- 
Darren Hart
VMware Open Source Technology Center

Re: [PATCH v2 8/8] platform: x86: intel_bxtwc_tmu: remove first level irq unmask

[Darren Hart]

On Fri, Apr 14, 2017 at 04:26:00PM -0700, 
sathyanarayanan.kuppusw...@linux.intel.com wrote:
> From: Kuppuswamy Sathyanarayanan 
> 
> Currently in WCOVE PMIC mfd driver, all second level irq chips

By currently I believe you mean after the earlier patch in this series is
applied, correct? This one is dependent on the previous one?

> are chained to the respective first level irqs. So there is no
> need for explicitly unmasking the first level irq in this
> driver. This patches removes this level 1 irq unmask support.
> 
> Signed-off-by: Kuppuswamy Sathyanarayanan 
> 

For platform drivers x86:

Reviewed-by: Darren Hart (VMware) 

Are you working with a specific maintainers to pull this in as a series? With so
many subsystems, we need to coordinate to make sure we don't make a mess for
Linus. Given the interdependencies, I'd recommend someone pull the series in as
a whole - maybe into MFD? Lee, do you have a preference?

-- 
Darren Hart
VMware Open Source Technology Center