linux-kernel - Re: [PATCH v5] rtc: rx6110: add ACPI bindings to I2C

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Wed, 17 Mar 2021 11:28:37 +0200
From:   Andy Shevchenko <andy.shevchenko@...il.com>
To:     Claudius Heine <ch@...x.de>
Cc:     johannes hahn <johannes-hahn@...mens.com>,
        Alessandro Zummo <a.zummo@...ertech.it>,
        Alexandre Belloni <alexandre.belloni@...tlin.com>,
        "open list:REAL TIME CLOCK (RTC) SUBSYSTEM" 
        <linux-rtc@...r.kernel.org>,
        open list <linux-kernel@...r.kernel.org>,
        werner zeh <werner.zeh@...mens.com>,
        henning schild <henning.schild@...mens.com>,
        Andy Shevchenko <andriy.shevchenko@...el.com>,
        martin mantel <martin.mantel@...mens.com>,
        Andy Shevchenko <andriy.shevchenko@...ux.intel.com>,
        kernel test robot <lkp@...el.com>
Subject: Re: [PATCH v5] rtc: rx6110: add ACPI bindings to I2C

On Wed, Mar 17, 2021 at 9:56 AM Claudius Heine <ch@...x.de> wrote:
>
> From: Johannes Hahn <johannes-hahn@...mens.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@...mens.com>
> Co-developed-by: Claudius Heine <ch@...x.de>
> Signed-off-by: Claudius Heine <ch@...x.de>
> Reviewed-by: Andy Shevchenko <andriy.shevchenko@...ux.intel.com>

> Reported-by: kernel test robot <lkp@...el.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