Linux-RTC Archive on lore.kernel.org
 help / color / Atom feed
* [PATCH v5] rtc: rx6110: add ACPI bindings to I2C
@ 2021-03-17  7:52 Claudius Heine
  2021-03-17  9:28 ` Andy Shevchenko
  2021-03-23 21:26 ` Alexandre Belloni
  0 siblings, 2 replies; 5+ messages in thread
From: Claudius Heine @ 2021-03-17  7:52 UTC (permalink / raw)
  To: johannes hahn, Alessandro Zummo, Alexandre Belloni,
	open list:REAL TIME CLOCK (RTC) SUBSYSTEM, open list
  Cc: werner zeh, henning schild, Andy Shevchenko, martin mantel,
	Claudius Heine, Andy Shevchenko, kernel test robot

From: Johannes Hahn <johannes-hahn@siemens.com>

This allows the RX6110 driver to be automatically assigned to the right
device on the I2C bus.

Signed-off-by: Johannes Hahn <johannes-hahn@siemens.com>
Co-developed-by: Claudius Heine <ch@denx.de>
Signed-off-by: Claudius Heine <ch@denx.de>
Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Reported-by: kernel test robot <lkp@intel.com>
---
 drivers/rtc/rtc-rx6110.c | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/drivers/rtc/rtc-rx6110.c b/drivers/rtc/rtc-rx6110.c
index 79161d4c6ce4..f4d425002f7f 100644
--- a/drivers/rtc/rtc-rx6110.c
+++ b/drivers/rtc/rtc-rx6110.c
@@ -447,6 +447,12 @@ static int rx6110_i2c_probe(struct i2c_client *client,
 	return rx6110_probe(rx6110, &client->dev);
 }
 
+static const struct acpi_device_id rx6110_i2c_acpi_match[] = {
+	{ "SECC6110" },
+	{ }
+};
+MODULE_DEVICE_TABLE(acpi, rx6110_i2c_acpi_match);
+
 static const struct i2c_device_id rx6110_i2c_id[] = {
 	{ "rx6110", 0 },
 	{ }
@@ -456,6 +462,7 @@ MODULE_DEVICE_TABLE(i2c, rx6110_i2c_id);
 static struct i2c_driver rx6110_i2c_driver = {
 	.driver = {
 		.name = RX6110_DRIVER_NAME,
+		.acpi_match_table = rx6110_i2c_acpi_match,
 	},
 	.probe		= rx6110_i2c_probe,
 	.id_table	= rx6110_i2c_id,

base-commit: 1df27313f50a57497c1faeb6a6ae4ca939c85a7d
-- 
2.30.1


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

* Re: [PATCH v5] rtc: rx6110: add ACPI bindings to I2C
  2021-03-17  7:52 [PATCH v5] rtc: rx6110: add ACPI bindings to I2C Claudius Heine
@ 2021-03-17  9:28 ` Andy Shevchenko
  2021-03-17  9:32   ` Claudius Heine
  2021-03-23 21:26 ` Alexandre Belloni
  1 sibling, 1 reply; 5+ messages in thread
From: Andy Shevchenko @ 2021-03-17  9:28 UTC (permalink / raw)
  To: Claudius Heine
  Cc: johannes hahn, Alessandro Zummo, Alexandre Belloni,
	open list:REAL TIME CLOCK (RTC) SUBSYSTEM, open list, werner zeh,
	henning schild, Andy Shevchenko, martin mantel, Andy Shevchenko,
	kernel test robot

On Wed, Mar 17, 2021 at 9:56 AM Claudius Heine <ch@denx.de> wrote:
>
> From: Johannes Hahn <johannes-hahn@siemens.com>
>
> This allows the RX6110 driver to be automatically assigned to the right
> device on the I2C bus.

Thanks for an update!

> Signed-off-by: Johannes Hahn <johannes-hahn@siemens.com>
> Co-developed-by: Claudius Heine <ch@denx.de>
> Signed-off-by: Claudius Heine <ch@denx.de>
> Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>

> Reported-by: kernel test robot <lkp@intel.com>

This is usually for patches that do fix found problems, here it's a
completely new item and the report was done in the middle of the
development. That said, you may give credit to LKP by just mentioning
it in the comments section (after the cutter '---' line). I'll leave
this to Alexandre and Alessandro to decide if you need a resend or
they may remove it when applying. (In my opinion resend is not needed
right now)


> ---
>  drivers/rtc/rtc-rx6110.c | 7 +++++++
>  1 file changed, 7 insertions(+)
>
> diff --git a/drivers/rtc/rtc-rx6110.c b/drivers/rtc/rtc-rx6110.c
> index 79161d4c6ce4..f4d425002f7f 100644
> --- a/drivers/rtc/rtc-rx6110.c
> +++ b/drivers/rtc/rtc-rx6110.c
> @@ -447,6 +447,12 @@ static int rx6110_i2c_probe(struct i2c_client *client,
>         return rx6110_probe(rx6110, &client->dev);
>  }
>
> +static const struct acpi_device_id rx6110_i2c_acpi_match[] = {
> +       { "SECC6110" },
> +       { }
> +};
> +MODULE_DEVICE_TABLE(acpi, rx6110_i2c_acpi_match);
> +
>  static const struct i2c_device_id rx6110_i2c_id[] = {
>         { "rx6110", 0 },
>         { }
> @@ -456,6 +462,7 @@ MODULE_DEVICE_TABLE(i2c, rx6110_i2c_id);
>  static struct i2c_driver rx6110_i2c_driver = {
>         .driver = {
>                 .name = RX6110_DRIVER_NAME,
> +               .acpi_match_table = rx6110_i2c_acpi_match,
>         },
>         .probe          = rx6110_i2c_probe,
>         .id_table       = rx6110_i2c_id,
>
> base-commit: 1df27313f50a57497c1faeb6a6ae4ca939c85a7d
> --
> 2.30.1
>


-- 
With Best Regards,
Andy Shevchenko

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

* Re: [PATCH v5] rtc: rx6110: add ACPI bindings to I2C
  2021-03-17  9:28 ` Andy Shevchenko
@ 2021-03-17  9:32   ` Claudius Heine
  2021-03-23 21:29     ` Alexandre Belloni
  0 siblings, 1 reply; 5+ messages in thread
From: Claudius Heine @ 2021-03-17  9:32 UTC (permalink / raw)
  To: Andy Shevchenko
  Cc: johannes hahn, Alessandro Zummo, Alexandre Belloni,
	open list:REAL TIME CLOCK (RTC) SUBSYSTEM, open list, werner zeh,
	henning schild, Andy Shevchenko, martin mantel, Andy Shevchenko,
	kernel test robot

Hi Andy,

On 2021-03-17 10:28, Andy Shevchenko wrote:
> On Wed, Mar 17, 2021 at 9:56 AM Claudius Heine <ch@denx.de> wrote:
>>
>> From: Johannes Hahn <johannes-hahn@siemens.com>
>>
>> This allows the RX6110 driver to be automatically assigned to the right
>> device on the I2C bus.
> 
> Thanks for an update!
> 
>> Signed-off-by: Johannes Hahn <johannes-hahn@siemens.com>
>> Co-developed-by: Claudius Heine <ch@denx.de>
>> Signed-off-by: Claudius Heine <ch@denx.de>
>> Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> 
>> Reported-by: kernel test robot <lkp@intel.com>
> 
> This is usually for patches that do fix found problems, here it's a
> completely new item and the report was done in the middle of the
> development. That said, you may give credit to LKP by just mentioning
> it in the comments section (after the cutter '---' line). I'll leave
> this to Alexandre and Alessandro to decide if you need a resend or
> they may remove it when applying. (In my opinion resend is not needed
> right now)
Ok. Thanks a lot for your reviews and patience!

regards,
Claudius

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

* Re: [PATCH v5] rtc: rx6110: add ACPI bindings to I2C
  2021-03-17  7:52 [PATCH v5] rtc: rx6110: add ACPI bindings to I2C Claudius Heine
  2021-03-17  9:28 ` Andy Shevchenko
@ 2021-03-23 21:26 ` Alexandre Belloni
  1 sibling, 0 replies; 5+ messages in thread
From: Alexandre Belloni @ 2021-03-23 21:26 UTC (permalink / raw)
  To: Claudius Heine, Alessandro Zummo, open list, johannes hahn,
	open list:REAL TIME CLOCK RTC SUBSYSTEM
  Cc: Alexandre Belloni, werner zeh, henning schild, Andy Shevchenko,
	Andy Shevchenko, martin mantel, kernel test robot

On Wed, 17 Mar 2021 08:52:27 +0100, Claudius Heine wrote:
> This allows the RX6110 driver to be automatically assigned to the right
> device on the I2C bus.

Applied, thanks!

[1/1] rtc: rx6110: add ACPI bindings to I2C
      commit: 8d69f62fddf6c1a8c7745120c4d6aab9322b001a

Best regards,
-- 
Alexandre Belloni <alexandre.belloni@bootlin.com>

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

* Re: [PATCH v5] rtc: rx6110: add ACPI bindings to I2C
  2021-03-17  9:32   ` Claudius Heine
@ 2021-03-23 21:29     ` Alexandre Belloni
  0 siblings, 0 replies; 5+ messages in thread
From: Alexandre Belloni @ 2021-03-23 21:29 UTC (permalink / raw)
  To: Claudius Heine
  Cc: Andy Shevchenko, johannes hahn, Alessandro Zummo,
	open list:REAL TIME CLOCK (RTC) SUBSYSTEM, open list, werner zeh,
	henning schild, Andy Shevchenko, martin mantel, Andy Shevchenko,
	kernel test robot

On 17/03/2021 10:32:39+0100, Claudius Heine wrote:
> Hi Andy,
> 
> On 2021-03-17 10:28, Andy Shevchenko wrote:
> > On Wed, Mar 17, 2021 at 9:56 AM Claudius Heine <ch@denx.de> wrote:
> > > 
> > > From: Johannes Hahn <johannes-hahn@siemens.com>
> > > 
> > > This allows the RX6110 driver to be automatically assigned to the right
> > > device on the I2C bus.
> > 
> > Thanks for an update!
> > 
> > > Signed-off-by: Johannes Hahn <johannes-hahn@siemens.com>
> > > Co-developed-by: Claudius Heine <ch@denx.de>
> > > Signed-off-by: Claudius Heine <ch@denx.de>
> > > Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> > 
> > > Reported-by: kernel test robot <lkp@intel.com>
> > 
> > This is usually for patches that do fix found problems, here it's a
> > completely new item and the report was done in the middle of the
> > development. That said, you may give credit to LKP by just mentioning
> > it in the comments section (after the cutter '---' line). I'll leave
> > this to Alexandre and Alessandro to decide if you need a resend or
> > they may remove it when applying. (In my opinion resend is not needed
> > right now)
> Ok. Thanks a lot for your reviews and patience!
> 

I removed it when applying. Thanks for the work and the reviews!


-- 
Alexandre Belloni, co-owner and COO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

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

end of thread, back to index

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-03-17  7:52 [PATCH v5] rtc: rx6110: add ACPI bindings to I2C Claudius Heine
2021-03-17  9:28 ` Andy Shevchenko
2021-03-17  9:32   ` Claudius Heine
2021-03-23 21:29     ` Alexandre Belloni
2021-03-23 21:26 ` Alexandre Belloni

Linux-RTC Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/linux-rtc/0 linux-rtc/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-rtc linux-rtc/ https://lore.kernel.org/linux-rtc \
		linux-rtc@vger.kernel.org
	public-inbox-index linux-rtc

Example config snippet for mirrors

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


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