[PATCH v2 0/2] i2c: Add ACPI support for HJMC01

[PATCH v2 0/2] i2c: Add ACPI support for HJMC01

[hunter.yu]

Hi, everyone:

HJMC01 is an ARM-based server processor that contain a
Designware I2C controller. This series add ACPI support
for the Designware i2c controller.

v1->v2:
Adjust the order of ACPI HID according to the alphabetical order

hunter.yu (2):
  ACPI: APD: Add clock frequency for HJMC01 I2C controller
  i2c: designware: Add a new ACPI HID for HJMC01 I2C controller

 drivers/acpi/acpi_apd.c                     | 6 ++++++
 drivers/i2c/busses/i2c-designware-platdrv.c | 1 +
 2 files changed, 7 insertions(+)

-- 
2.43.5

[PATCH v2 1/2] ACPI: APD: Add clock frequency for HJMC01 I2C controller

[hunter.yu]

I2C clock frequency for HJMC01 is 200M, define a new ACPI
HID for it.

Signed-off-by: hunter.yu <hunter.yu@hj-micro.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 800f97868448..229e80a19664 100644
--- a/drivers/acpi/acpi_apd.c
+++ b/drivers/acpi/acpi_apd.c
@@ -181,6 +181,11 @@ static const struct apd_device_desc hip08_spi_desc = {
 	.setup = acpi_apd_setup,
 	.fixed_clk_rate = 250000000,
 };
+
+static const struct apd_device_desc hjmc_i2c_desc = {
+	.setup = acpi_apd_setup,
+	.fixed_clk_rate = 200000000,
+};
 #endif /* CONFIG_ARM64 */
 
 #endif
@@ -251,6 +256,7 @@ static const struct acpi_device_id acpi_apd_device_ids[] = {
 	{ "HISI02A2", APD_ADDR(hip08_i2c_desc) },
 	{ "HISI02A3", APD_ADDR(hip08_lite_i2c_desc) },
 	{ "HISI0173", APD_ADDR(hip08_spi_desc) },
+	{ "HJMC3001", APD_ADDR(hjmc_i2c_desc) },
 	{ "NXP0001", APD_ADDR(nxp_i2c_desc) },
 #endif
 	{ }
-- 
2.43.5

Re: [PATCH v2 1/2] ACPI: APD: Add clock frequency for HJMC01 I2C controller

[Andi Shyti]

On Thu, Sep 26, 2024 at 10:40:05AM GMT, hunter.yu wrote:
> I2C clock frequency for HJMC01 is 200M, define a new ACPI
> HID for it.
> 
> Signed-off-by: hunter.yu <hunter.yu@hj-micro.com>

Do you want your name to be hunter.yu or Hunter Yu? I prefer the
second and if you browse the git log, you can see that everyone
uses Name Surnmae.

Andi

Re: [PATCH v2 1/2] ACPI: APD: Add clock frequency for HJMC01 I2C controller

[Rafael J. Wysocki]

On Thu, Oct 3, 2024 at 12:13 AM Andi Shyti <andi.shyti@kernel.org> wrote:
>
> On Thu, Sep 26, 2024 at 10:40:05AM GMT, hunter.yu wrote:
> > I2C clock frequency for HJMC01 is 200M, define a new ACPI
> > HID for it.
> >
> > Signed-off-by: hunter.yu <hunter.yu@hj-micro.com>
>
> Do you want your name to be hunter.yu or Hunter Yu? I prefer the
> second and if you browse the git log, you can see that everyone
> uses Name Surname.

It must be a real name as per submitting-patches.rst

The S-o-b is meaningless otherwise.

Re: [PATCH v2 1/2] ACPI: APD: Add clock frequency for HJMC01 I2C controller

[Andy Shevchenko]

On Wed, Oct 16, 2024 at 10:45:26PM +0200, Rafael J. Wysocki wrote:
> On Thu, Oct 3, 2024 at 12:13 AM Andi Shyti <andi.shyti@kernel.org> wrote:
> > On Thu, Sep 26, 2024 at 10:40:05AM GMT, hunter.yu wrote:
> > > I2C clock frequency for HJMC01 is 200M, define a new ACPI
> > > HID for it.
> > >
> > > Signed-off-by: hunter.yu <hunter.yu@hj-micro.com>
> >
> > Do you want your name to be hunter.yu or Hunter Yu? I prefer the
> > second and if you browse the git log, you can see that everyone
> > uses Name Surname.
> 
> It must be a real name as per submitting-patches.rst

Hasn't this been relaxed last year by the d4563201f33a ("Documentation:
simplify and clarify DCO contribution example language")?

> The S-o-b is meaningless otherwise.

-- 
With Best Regards,
Andy Shevchenko

Re: [PATCH v2 1/2] ACPI: APD: Add clock frequency for HJMC01 I2C controller

[Andy Shevchenko]

On Thu, Oct 17, 2024 at 12:26:18PM +0300, Andy Shevchenko wrote:
> On Wed, Oct 16, 2024 at 10:45:26PM +0200, Rafael J. Wysocki wrote:
> > On Thu, Oct 3, 2024 at 12:13 AM Andi Shyti <andi.shyti@kernel.org> wrote:
> > > On Thu, Sep 26, 2024 at 10:40:05AM GMT, hunter.yu wrote:
> > > > I2C clock frequency for HJMC01 is 200M, define a new ACPI
> > > > HID for it.
> > > >
> > > > Signed-off-by: hunter.yu <hunter.yu@hj-micro.com>
> > >
> > > Do you want your name to be hunter.yu or Hunter Yu? I prefer the
> > > second and if you browse the git log, you can see that everyone
> > > uses Name Surname.
> > 
> > It must be a real name as per submitting-patches.rst
> 
> Hasn't this been relaxed last year by the d4563201f33a ("Documentation:
> simplify and clarify DCO contribution example language")?

Note, I do not imply that the existing variant in this patch is ideal, I also,
as Andi, prefer the proper spellings on the "name" parts.

> > The S-o-b is meaningless otherwise.

-- 
With Best Regards,
Andy Shevchenko

Re: [PATCH v2 1/2] ACPI: APD: Add clock frequency for HJMC01 I2C controller

[Rafael J. Wysocki]

On Thu, Oct 17, 2024 at 11:29 AM Andy Shevchenko
<andriy.shevchenko@linux.intel.com> wrote:
>
> On Thu, Oct 17, 2024 at 12:26:18PM +0300, Andy Shevchenko wrote:
> > On Wed, Oct 16, 2024 at 10:45:26PM +0200, Rafael J. Wysocki wrote:
> > > On Thu, Oct 3, 2024 at 12:13 AM Andi Shyti <andi.shyti@kernel.org> wrote:
> > > > On Thu, Sep 26, 2024 at 10:40:05AM GMT, hunter.yu wrote:
> > > > > I2C clock frequency for HJMC01 is 200M, define a new ACPI
> > > > > HID for it.
> > > > >
> > > > > Signed-off-by: hunter.yu <hunter.yu@hj-micro.com>
> > > >
> > > > Do you want your name to be hunter.yu or Hunter Yu? I prefer the
> > > > second and if you browse the git log, you can see that everyone
> > > > uses Name Surname.
> > >
> > > It must be a real name as per submitting-patches.rst
> >
> > Hasn't this been relaxed last year by the d4563201f33a ("Documentation:
> > simplify and clarify DCO contribution example language")?
>
> Note, I do not imply that the existing variant in this patch is ideal, I also,
> as Andi, prefer the proper spellings on the "name" parts.

Good.

Now, the way I understand the "known identity" part of the document is
that a misspelled name is not sufficient.

Re: [PATCH v2 1/2] ACPI: APD: Add clock frequency for HJMC01 I2C controller

[Hunter Yu]

On 2024/10/17 4:45, Rafael J. Wysocki wrote:
> On Thu, Oct 3, 2024 at 12:13 AM Andi Shyti <andi.shyti@kernel.org> wrote:
>>
>> On Thu, Sep 26, 2024 at 10:40:05AM GMT, hunter.yu wrote:
>>> I2C clock frequency for HJMC01 is 200M, define a new ACPI
>>> HID for it.
>>>
>>> Signed-off-by: hunter.yu <hunter.yu@hj-micro.com>
>>
>> Do you want your name to be hunter.yu or Hunter Yu? I prefer the
>> second and if you browse the git log, you can see that everyone
>> uses Name Surname.
> 
> It must be a real name as per submitting-patches.rst
> 
> The S-o-b is meaningless otherwise.
> 

Thanks for your help and advice. In our team, everyone have an alias 
name, which is the same as email address. I feel sorry for not noticing 
kernel community's requirement for real name.
My real name is Xiangyang Yu, I will use the following format to send v3 
patch later.

Signed-off-by: Xiangyang Yu <hunter.yu@hj-micro.com>

[PATCH v2 2/2] i2c: designware: Add a new ACPI HID for HJMC01 I2C controller

[hunter.yu]

Define a new ACPI HID for HJMC01

Signed-off-by: hunter.yu <hunter.yu@hj-micro.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 2d0c7348e491..701506e92380 100644
--- a/drivers/i2c/busses/i2c-designware-platdrv.c
+++ b/drivers/i2c/busses/i2c-designware-platdrv.c
@@ -354,6 +354,7 @@ static const struct acpi_device_id dw_i2c_acpi_match[] = {
 	{ "HISI02A1", 0 },
 	{ "HISI02A2", 0 },
 	{ "HISI02A3", 0 },
+	{ "HJMC3001", 0 },
 	{ "HYGO0010", ACCESS_INTR_MASK },
 	{ "INT33C2", 0 },
 	{ "INT33C3", 0 },
-- 
2.43.5

Re: [PATCH v2 2/2] i2c: designware: Add a new ACPI HID for HJMC01 I2C controller

[Jarkko Nikula]

On 9/26/24 5:40 AM, hunter.yu wrote:
> Define a new ACPI HID for HJMC01
> 
> Signed-off-by: hunter.yu <hunter.yu@hj-micro.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 2d0c7348e491..701506e92380 100644
> --- a/drivers/i2c/busses/i2c-designware-platdrv.c
> +++ b/drivers/i2c/busses/i2c-designware-platdrv.c
> @@ -354,6 +354,7 @@ static const struct acpi_device_id dw_i2c_acpi_match[] = {
>   	{ "HISI02A1", 0 },
>   	{ "HISI02A2", 0 },
>   	{ "HISI02A3", 0 },
> +	{ "HJMC3001", 0 },
>   	{ "HYGO0010", ACCESS_INTR_MASK },
>   	{ "INT33C2", 0 },
>   	{ "INT33C3", 0 },

Acked-by: Jarkko Nikula <jarkko.nikula@linux.intel.com>

Re: [PATCH v2 0/2] i2c: Add ACPI support for HJMC01

[Andy Shevchenko]

On Thu, Sep 26, 2024 at 10:40:04AM +0800, hunter.yu wrote:
> Hi, everyone:
> 
> HJMC01 is an ARM-based server processor that contain a
> Designware I2C controller. This series add ACPI support
> for the Designware i2c controller.

Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>

-- 
With Best Regards,
Andy Shevchenko

Re: [PATCH v2 0/2] i2c: Add ACPI support for HJMC01

[Andi Shyti]

Hi Hunter,

> hunter.yu (2):
>   ACPI: APD: Add clock frequency for HJMC01 I2C controller
>   i2c: designware: Add a new ACPI HID for HJMC01 I2C controller

Applied to i2c/i2c-host and I took the freedom to change
"hunter.yu" with "Hunter Yu".

Thanks,
Andi

Re: [PATCH v2 0/2] i2c: Add ACPI support for HJMC01

[Andi Shyti]

Hi,

On Thu, Oct 03, 2024 at 02:56:51PM +0200, Andi Shyti wrote:
> > hunter.yu (2):
> >   ACPI: APD: Add clock frequency for HJMC01 I2C controller
> >   i2c: designware: Add a new ACPI HID for HJMC01 I2C controller
> 
> Applied to i2c/i2c-host and I took the freedom to change
> "hunter.yu" with "Hunter Yu".

I'm sorry, I mistakenly included this patch in my branch and
removed it from the pull request.

Apologies, Rafael, for stepping on your toes. Passing the ball
to you now.

Andi