Linux I2C development
 help / color / mirror / Atom feed
* [PATCH v2 1/1] i2c: designware: Add ACPI ID MSFT0006 for Microsoft Cobalt SoCs
@ 2026-11-09 21:01 Siddharth Chintamaneni
  2026-11-09 21:30 ` Andy Shevchenko
  0 siblings, 1 reply; 4+ messages in thread
From: Siddharth Chintamaneni @ 2026-11-09 21:01 UTC (permalink / raw)
  To: Andi Shyti, Mika Westerberg, Rafael J . Wysocki
  Cc: Andy Shevchenko, Len Brown, linux-i2c, linux-acpi, linux-kernel,
	apais, rlmenge, hargar, Siddharth Chintamaneni, Jarred White,
	Jeff Kirsher

Add the MSFT0006 ACPI ID for the DesignWare I2C controller integrated
on Microsoft Cobalt SoCs, along with its 300 MHz input clock description
in acpi_apd.c.

Without an explicit MSFT0006 entry, ACPI device matching falls back to
its _CID (APMC0D0F), which configures a 100 MHz input clock instead of
the controller's actual 300 MHz clock.

Acked-by: Jarred White <jarredwhite@linux.microsoft.com>
Acked-by: Jeff Kirsher <jeff.kirsher@microsoft.com>
Signed-off-by: Siddharth Chintamaneni <sidchintamaneni@gmail.com>
---
 drivers/acpi/acpi_apd.c                     | 6 ++++++
 drivers/i2c/busses/i2c-designware-platdrv.c | 1 +
 2 files changed, 7 insertions(+)

diff --git a/drivers/acpi/acpi_apd.c b/drivers/acpi/acpi_apd.c
index ea226fd987567..66a7607752d9b 100644
--- a/drivers/acpi/acpi_apd.c
+++ b/drivers/acpi/acpi_apd.c
@@ -198,6 +198,11 @@ static const struct apd_device_desc hjmc_i2c_desc = {
 	.fixed_clk_rate = 200 * HZ_PER_MHZ,
 };
 
+static const struct apd_device_desc msft_i2c_desc = {
+	.setup = acpi_apd_setup,
+	.fixed_clk_rate = 300 * HZ_PER_MHZ,
+};
+
 #endif /* CONFIG_ARM64 */
 
 #endif
@@ -272,6 +277,7 @@ static const struct acpi_device_id acpi_apd_device_ids[] = {
 	{ "HJMC3001", APD_ADDR(hjmc_i2c_desc) },
 	{ "LECA0002", APD_ADDR(leca_spi_desc) },
 	{ "LECA0003", APD_ADDR(leca_i2c_desc) },
+	{ "MSFT0006", APD_ADDR(msft_i2c_desc) },
 	{ "NXP0001", APD_ADDR(nxp_i2c_desc) },
 #endif
 	{ }
diff --git a/drivers/i2c/busses/i2c-designware-platdrv.c b/drivers/i2c/busses/i2c-designware-platdrv.c
index 13dd7c50486a4..cf364f3c26199 100644
--- a/drivers/i2c/busses/i2c-designware-platdrv.c
+++ b/drivers/i2c/busses/i2c-designware-platdrv.c
@@ -280,6 +280,7 @@ static const struct acpi_device_id dw_i2c_acpi_match[] = {
 	{ .id = "INT3433" },
 	{ .id = "INTC10EF" },
 	{ .id = "LECA0003" },
+	{ .id = "MSFT0006" },
 	{ }
 };
 MODULE_DEVICE_TABLE(acpi, dw_i2c_acpi_match);
-- 
2.45.4


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH v2 1/1] i2c: designware: Add ACPI ID MSFT0006 for Microsoft Cobalt SoCs
  2026-11-09 21:01 [PATCH v2 1/1] i2c: designware: Add ACPI ID MSFT0006 for Microsoft Cobalt SoCs Siddharth Chintamaneni
@ 2026-11-09 21:30 ` Andy Shevchenko
  2026-11-10 20:15   ` Siddharth Chintamaneni
  0 siblings, 1 reply; 4+ messages in thread
From: Andy Shevchenko @ 2026-11-09 21:30 UTC (permalink / raw)
  To: Siddharth Chintamaneni
  Cc: Andi Shyti, Mika Westerberg, Rafael J . Wysocki, Len Brown,
	linux-i2c, linux-acpi, linux-kernel, apais, rlmenge, hargar,
	Jarred White, Jeff Kirsher

On Mon, Nov 09, 2026 at 09:01:46PM +0000, Siddharth Chintamaneni wrote:
> Add the MSFT0006 ACPI ID for the DesignWare I2C controller integrated
> on Microsoft Cobalt SoCs, along with its 300 MHz input clock description
> in acpi_apd.c.
> 
> Without an explicit MSFT0006 entry, ACPI device matching falls back to
> its _CID (APMC0D0F), which configures a 100 MHz input clock instead of
> the controller's actual 300 MHz clock.
> 
> Acked-by: Jarred White <jarredwhite@linux.microsoft.com>
> Acked-by: Jeff Kirsher <jeff.kirsher@microsoft.com>
> Signed-off-by: Siddharth Chintamaneni <sidchintamaneni@gmail.com>
> ---

Where is the changelog?
Haven't I given a tag already?

-- 
With Best Regards,
Andy Shevchenko



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH v2 1/1] i2c: designware: Add ACPI ID MSFT0006 for Microsoft Cobalt SoCs
  2026-11-09 21:30 ` Andy Shevchenko
@ 2026-11-10 20:15   ` Siddharth Chintamaneni
  2026-11-10 21:05     ` Andy Shevchenko
  0 siblings, 1 reply; 4+ messages in thread
From: Siddharth Chintamaneni @ 2026-11-10 20:15 UTC (permalink / raw)
  To: Andy Shevchenko
  Cc: Andi Shyti, Mika Westerberg, Rafael J . Wysocki, Len Brown,
	linux-i2c, linux-acpi, linux-kernel, apais, rlmenge, hargar,
	Jarred White, Jeff Kirsher

On Mon, 9 Nov 2026 at 13:30, Andy Shevchenko
<andriy.shevchenko@linux.intel.com> wrote:
>
> On Mon, Nov 09, 2026 at 09:01:46PM +0000, Siddharth Chintamaneni wrote:
> > Add the MSFT0006 ACPI ID for the DesignWare I2C controller integrated
> > on Microsoft Cobalt SoCs, along with its 300 MHz input clock description
> > in acpi_apd.c.
> >
> > Without an explicit MSFT0006 entry, ACPI device matching falls back to
> > its _CID (APMC0D0F), which configures a 100 MHz input clock instead of
> > the controller's actual 300 MHz clock.
> >
> > Acked-by: Jarred White <jarredwhite@linux.microsoft.com>
> > Acked-by: Jeff Kirsher <jeff.kirsher@microsoft.com>
> > Signed-off-by: Siddharth Chintamaneni <sidchintamaneni@gmail.com>
> > ---
>
> Where is the changelog?

Oh I just dropped cc tags.

> Haven't I given a tag already?

no, I thought you were suggesting that I repost it.

>
> --
> With Best Regards,
> Andy Shevchenko
>
>

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH v2 1/1] i2c: designware: Add ACPI ID MSFT0006 for Microsoft Cobalt SoCs
  2026-11-10 20:15   ` Siddharth Chintamaneni
@ 2026-11-10 21:05     ` Andy Shevchenko
  0 siblings, 0 replies; 4+ messages in thread
From: Andy Shevchenko @ 2026-11-10 21:05 UTC (permalink / raw)
  To: Siddharth Chintamaneni
  Cc: Andi Shyti, Mika Westerberg, Rafael J . Wysocki, Len Brown,
	linux-i2c, linux-acpi, linux-kernel, apais, rlmenge, hargar,
	Jarred White, Jeff Kirsher

On Tue, Nov 10, 2026 at 12:15:10PM -0800, Siddharth Chintamaneni wrote:
> On Mon, 9 Nov 2026 at 13:30, Andy Shevchenko
> <andriy.shevchenko@linux.intel.com> wrote:
> >
> > On Mon, Nov 09, 2026 at 09:01:46PM +0000, Siddharth Chintamaneni wrote:
> > > Add the MSFT0006 ACPI ID for the DesignWare I2C controller integrated
> > > on Microsoft Cobalt SoCs, along with its 300 MHz input clock description
> > > in acpi_apd.c.
> > >
> > > Without an explicit MSFT0006 entry, ACPI device matching falls back to
> > > its _CID (APMC0D0F), which configures a 100 MHz input clock instead of
> > > the controller's actual 300 MHz clock.
> > >
> > > Acked-by: Jarred White <jarredwhite@linux.microsoft.com>
> > > Acked-by: Jeff Kirsher <jeff.kirsher@microsoft.com>
> > > Signed-off-by: Siddharth Chintamaneni <sidchintamaneni@gmail.com>
> > > ---
> >
> > Where is the changelog?
> 
> Oh I just dropped cc tags.
> 
> > Haven't I given a tag already?
> 
> no, I thought you were suggesting that I repost it.

Okay, it's fine to me
Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>

-- 
With Best Regards,
Andy Shevchenko



^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2026-11-10 21:06 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-11-09 21:01 [PATCH v2 1/1] i2c: designware: Add ACPI ID MSFT0006 for Microsoft Cobalt SoCs Siddharth Chintamaneni
2026-11-09 21:30 ` Andy Shevchenko
2026-11-10 20:15   ` Siddharth Chintamaneni
2026-11-10 21:05     ` Andy Shevchenko

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox