mbox series

[Bionic,SRU,0/3] CPPC: bug fixes

Message ID 20181016185237.6196-1-manoj.iyer@canonical.com
Headers show
Series CPPC: bug fixes | expand

Message

Manoj Iyer Oct. 16, 2018, 6:52 p.m. UTC
Please consider the following patch set for SRU in Bionic for CPPC bug 
fixes from upstream, these patches are clean cherry-picks. 

CPPC bug fixes: https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1796949

The CPPC feature needs to be explicitly enabled in Firmware (default is
disabled) for these patches to be in effect. 

I have build a kernel and regression tested on Cavium Thunder X2 CN99XX 
system. For test kernel, and detailed test results. please refer to the 
comments in the bug report.

Comments

Kleber Sacilotto de Souza Oct. 17, 2018, 8:09 a.m. UTC | #1
On 10/16/18 20:52, Manoj Iyer wrote:
> Please consider the following patch set for SRU in Bionic for CPPC bug 
> fixes from upstream, these patches are clean cherry-picks. 
> 
> CPPC bug fixes: https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1796949
> 
> The CPPC feature needs to be explicitly enabled in Firmware (default is
> disabled) for these patches to be in effect. 
> 
> I have build a kernel and regression tested on Cavium Thunder X2 CN99XX 
> system. For test kernel, and detailed test results. please refer to the 
> comments in the bug report.
> 
> 
> 

All clean cherry-picks, limited to platform driver and tested.

My only comment is regarding the missing S-O-B on patch 1/3, which can
be fixed while applying it.


Acked-by: Kleber Sacilotto de Souza <kleber.souza@canonical.com>
Stefan Bader Oct. 17, 2018, 8:43 a.m. UTC | #2
On 16.10.18 20:52, Manoj Iyer wrote:
> Please consider the following patch set for SRU in Bionic for CPPC bug 
> fixes from upstream, these patches are clean cherry-picks. 
> 
> CPPC bug fixes: https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1796949
> 
> The CPPC feature needs to be explicitly enabled in Firmware (default is
> disabled) for these patches to be in effect. 
> 
> I have build a kernel and regression tested on Cavium Thunder X2 CN99XX 
> system. For test kernel, and detailed test results. please refer to the 
> comments in the bug report.
> 
> 
> 
Acked-by: Stefan Bader <stefan.bader@canonical.com>

As Kleber mentioned the s-o-b for patch #1 needs to be added. Also, though it
does not strictly matter, we should probably use the form

https://bugs.launchpad.net/bugs/#

for all BugLinks we do. Since I am not sure how flexible the scraper is it might
actually matter.

-Stefan
Khalid Elmously Oct. 23, 2018, 6:30 a.m. UTC | #3
Added missing S-O-B


On 2018-10-16 13:52:34 , Manoj Iyer wrote:
> Please consider the following patch set for SRU in Bionic for CPPC bug 
> fixes from upstream, these patches are clean cherry-picks. 
> 
> CPPC bug fixes: https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1796949
> 
> The CPPC feature needs to be explicitly enabled in Firmware (default is
> disabled) for these patches to be in effect. 
> 
> I have build a kernel and regression tested on Cavium Thunder X2 CN99XX 
> system. For test kernel, and detailed test results. please refer to the 
> comments in the bug report.
> 
> 
> 
> -- 
> kernel-team mailing list
> kernel-team@lists.ubuntu.com
> https://lists.ubuntu.com/mailman/listinfo/kernel-team
Kleber Sacilotto de Souza Oct. 23, 2018, 7:24 a.m. UTC | #4
On 10/23/18 08:30, Khaled Elmously wrote:
> Added missing S-O-B

The BugLink url hasn't been fixed at first, so I fixed it on all 3
patches and force pushed the branch.


Thanks,
Kleber

> 
> 
> On 2018-10-16 13:52:34 , Manoj Iyer wrote:
>> Please consider the following patch set for SRU in Bionic for CPPC bug 
>> fixes from upstream, these patches are clean cherry-picks. 
>>
>> CPPC bug fixes: https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1796949
>>
>> The CPPC feature needs to be explicitly enabled in Firmware (default is
>> disabled) for these patches to be in effect. 
>>
>> I have build a kernel and regression tested on Cavium Thunder X2 CN99XX 
>> system. For test kernel, and detailed test results. please refer to the 
>> comments in the bug report.
>>
>>
>>
>> -- 
>> kernel-team mailing list
>> kernel-team@lists.ubuntu.com
>> https://lists.ubuntu.com/mailman/listinfo/kernel-team
>