Re: [PATCH v2 2/2] platform/x86/intel: pmc/core: Add Raptor Lake support to pmc core driver

From: Kammela, Gayatri
Date: Wed Oct 26 2022 - 10:48:29 EST



On 10/24/2022 2:46 AM, Hans de Goede wrote:
Hi,

On 10/14/22 09:50, Jiri Slaby wrote:
On 19. 09. 22, 13:41, Hans de Goede wrote:

On 9/13/22 00:33, Gayatri Kammela wrote:
Add Raptor Lake client parts (both RPL and RPL_S) support to pmc core
driver. Raptor Lake client parts reuse all the Alder Lake PCH IPs.

Cc: Srinivas Pandruvada <srinivas.pandruvada@xxxxxxxxx>
Cc: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
Cc: David Box <david.e.box@xxxxxxxxx>
Acked-by: Rajneesh Bhardwaj <irenic.rajneesh@xxxxxxxxx>
Acked-by: Hans de Goede <hdegoede@xxxxxxxxxx>
Signed-off-by: Gayatri Kammela <gayatri.kammela@xxxxxxxxxxxxxxx>
---
Changes since v1:
1) Added a dependency patch- ea902bcc1943f7539200ec464de3f54335588774 :
  "x86/cpu: Add new Raptor Lake CPU model number".
2) Rebased the above patch on v6.0-rc1 with "Acked-by" from Hans and
  Rajneesh.
I still cannot take this, since patch 1/2 is *already merged* through
another tree, so me cherry-picking it leads to potential conflicts.

As I have already explained twice you need to submit this upstream
throuh the same tree which has the original merge of patch 1/2.
Hi, friendly ping: has this ever happened, Gayatri?
Since the necessary change are in 6.1-rc1 now, I have merged this
patch now and I'll include this in the pdx86 fixes pull-req which
I'm preparing for 6.1-rc3:

Thank you for your patch, I've applied this patch to my review-hans
branch:
https://git.kernel.org/pub/scm/linux/kernel/git/pdx86/platform-drivers-x86.git/log/?h=review-hans

Note it will show up in my review-hans branch once I've pushed my
local branch there, which might take a while.

Once I've run some tests on this branch the patches there will be
added to the platform-drivers-x86/for-next branch and eventually
will be included in the pdx86 pull-request to Linus for the next
merge-window.

Regards,

Hans

hi Hans,

Thank you for merging this patch. I was planning on sending this patch to combine with other series as this could be dependency. Apologies for my late reply. Thank you!