Linux-ACPI Archive on lore.kernel.org
 help / color / Atom feed
* [PATCH 1/2] i2c: designware: Add ACPI HID for Hisilicon Hip08-Lite I2C controller
@ 2020-02-02 11:34 Hanjun Guo
  2020-02-02 11:34 ` [PATCH 2/2] ACPI / APD: Add clock frequency " Hanjun Guo
  2020-02-02 19:52 ` [PATCH 1/2] i2c: designware: Add ACPI HID " Andy Shevchenko
  0 siblings, 2 replies; 4+ messages in thread
From: Hanjun Guo @ 2020-02-02 11:34 UTC (permalink / raw)
  To: Rafael J. Wysocki, Jarkko Nikula, Andy Shevchenko, Mika Westerberg
  Cc: linux-i2c, linux-acpi, linuxarm, Sheng Feng, Hanjun Guo

Add ACPI HID HISI02A3 for Hisilicon Hip08 Lite, which has different
clock frequency from Hip08 for I2C controller.

Tested-by: Sheng Feng <fengsheng5@huawei.com>
Signed-off-by: Hanjun Guo <guohanjun@huawei.com>
---
 drivers/i2c/busses/i2c-designware-platdrv.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/i2c/busses/i2c-designware-platdrv.c b/drivers/i2c/busses/i2c-designware-platdrv.c
index 16dd338..3b7d58c 100644
--- a/drivers/i2c/busses/i2c-designware-platdrv.c
+++ b/drivers/i2c/busses/i2c-designware-platdrv.c
@@ -130,6 +130,7 @@ static int dw_i2c_acpi_configure(struct platform_device *pdev)
 	{ "APMC0D0F", 0 },
 	{ "HISI02A1", 0 },
 	{ "HISI02A2", 0 },
+	{ "HISI02A3", 0 },
 	{ }
 };
 MODULE_DEVICE_TABLE(acpi, dw_i2c_acpi_match);
-- 
1.7.12.4


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

* [PATCH 2/2] ACPI / APD: Add clock frequency for Hisilicon Hip08-Lite I2C controller
  2020-02-02 11:34 [PATCH 1/2] i2c: designware: Add ACPI HID for Hisilicon Hip08-Lite I2C controller Hanjun Guo
@ 2020-02-02 11:34 ` " Hanjun Guo
  2020-02-02 19:52 ` [PATCH 1/2] i2c: designware: Add ACPI HID " Andy Shevchenko
  1 sibling, 0 replies; 4+ messages in thread
From: Hanjun Guo @ 2020-02-02 11:34 UTC (permalink / raw)
  To: Rafael J. Wysocki, Jarkko Nikula, Andy Shevchenko, Mika Westerberg
  Cc: linux-i2c, linux-acpi, linuxarm, Sheng Feng, Hanjun Guo

I2C clock frequency of Designware ip for Hisilicon Hip08 Lite
is 125M, use a new ACPI HID to enable it.

Tested-by: Sheng Feng <fengsheng5@huawei.com>
Signed-off-by: Hanjun Guo <guohanjun@huawei.com>
---
 drivers/acpi/acpi_apd.c | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/drivers/acpi/acpi_apd.c b/drivers/acpi/acpi_apd.c
index 71511ae..ba2612e 100644
--- a/drivers/acpi/acpi_apd.c
+++ b/drivers/acpi/acpi_apd.c
@@ -161,6 +161,11 @@ static int st_misc_setup(struct apd_private_data *pdata)
 	.fixed_clk_rate = 250000000,
 };
 
+static const struct apd_device_desc hip08_lite_i2c_desc = {
+	.setup = acpi_apd_setup,
+	.fixed_clk_rate = 125000000,
+};
+
 static const struct apd_device_desc thunderx2_i2c_desc = {
 	.setup = acpi_apd_setup,
 	.fixed_clk_rate = 125000000,
@@ -243,6 +248,7 @@ static int acpi_apd_create_device(struct acpi_device *adev,
 	{ "CAV9007",  APD_ADDR(thunderx2_i2c_desc) },
 	{ "HISI02A1", APD_ADDR(hip07_i2c_desc) },
 	{ "HISI02A2", APD_ADDR(hip08_i2c_desc) },
+	{ "HISI02A3", APD_ADDR(hip08_lite_i2c_desc) },
 	{ "HISI0173", APD_ADDR(hip08_spi_desc) },
 	{ "NXP0001", APD_ADDR(nxp_i2c_desc) },
 #endif
-- 
1.7.12.4


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

* Re: [PATCH 1/2] i2c: designware: Add ACPI HID for Hisilicon Hip08-Lite I2C controller
  2020-02-02 11:34 [PATCH 1/2] i2c: designware: Add ACPI HID for Hisilicon Hip08-Lite I2C controller Hanjun Guo
  2020-02-02 11:34 ` [PATCH 2/2] ACPI / APD: Add clock frequency " Hanjun Guo
@ 2020-02-02 19:52 ` " Andy Shevchenko
  2020-02-03  1:28   ` Hanjun Guo
  1 sibling, 1 reply; 4+ messages in thread
From: Andy Shevchenko @ 2020-02-02 19:52 UTC (permalink / raw)
  To: Hanjun Guo
  Cc: Rafael J. Wysocki, Jarkko Nikula, Mika Westerberg, linux-i2c,
	linux-acpi, linuxarm, Sheng Feng

On Sun, Feb 02, 2020 at 07:34:14PM +0800, Hanjun Guo wrote:
> Add ACPI HID HISI02A3 for Hisilicon Hip08 Lite, which has different
> clock frequency from Hip08 for I2C controller.

The series is good for me, but I think you should reverse the order of the
patches in it.

> Tested-by: Sheng Feng <fengsheng5@huawei.com>
> Signed-off-by: Hanjun Guo <guohanjun@huawei.com>
> ---
>  drivers/i2c/busses/i2c-designware-platdrv.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/i2c/busses/i2c-designware-platdrv.c b/drivers/i2c/busses/i2c-designware-platdrv.c
> index 16dd338..3b7d58c 100644
> --- a/drivers/i2c/busses/i2c-designware-platdrv.c
> +++ b/drivers/i2c/busses/i2c-designware-platdrv.c
> @@ -130,6 +130,7 @@ static int dw_i2c_acpi_configure(struct platform_device *pdev)
>  	{ "APMC0D0F", 0 },
>  	{ "HISI02A1", 0 },
>  	{ "HISI02A2", 0 },
> +	{ "HISI02A3", 0 },
>  	{ }
>  };
>  MODULE_DEVICE_TABLE(acpi, dw_i2c_acpi_match);
> -- 
> 1.7.12.4
> 

-- 
With Best Regards,
Andy Shevchenko



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

* Re: [PATCH 1/2] i2c: designware: Add ACPI HID for Hisilicon Hip08-Lite I2C controller
  2020-02-02 19:52 ` [PATCH 1/2] i2c: designware: Add ACPI HID " Andy Shevchenko
@ 2020-02-03  1:28   ` Hanjun Guo
  0 siblings, 0 replies; 4+ messages in thread
From: Hanjun Guo @ 2020-02-03  1:28 UTC (permalink / raw)
  To: Andy Shevchenko
  Cc: Rafael J. Wysocki, Jarkko Nikula, Mika Westerberg, linux-i2c,
	linux-acpi, linuxarm, Sheng Feng

On 2020/2/3 3:52, Andy Shevchenko wrote:
> On Sun, Feb 02, 2020 at 07:34:14PM +0800, Hanjun Guo wrote:
>> Add ACPI HID HISI02A3 for Hisilicon Hip08 Lite, which has different
>> clock frequency from Hip08 for I2C controller.
> 
> The series is good for me, but I think you should reverse the order of the
> patches in it.

I will send a updated version as you suggested.

Thanks
Hanjun


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

end of thread, back to index

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-02-02 11:34 [PATCH 1/2] i2c: designware: Add ACPI HID for Hisilicon Hip08-Lite I2C controller Hanjun Guo
2020-02-02 11:34 ` [PATCH 2/2] ACPI / APD: Add clock frequency " Hanjun Guo
2020-02-02 19:52 ` [PATCH 1/2] i2c: designware: Add ACPI HID " Andy Shevchenko
2020-02-03  1:28   ` Hanjun Guo

Linux-ACPI Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/linux-acpi/0 linux-acpi/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 linux-acpi linux-acpi/ https://lore.kernel.org/linux-acpi \
		linux-acpi@vger.kernel.org
	public-inbox-index linux-acpi

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.kernel.vger.linux-acpi


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git