Linux RTC
 help / color / mirror / Atom feed
From: Alexandre Belloni <alexandre.belloni@free-electrons.com>
To: Dirk Behme <dirk.behme@de.bosch.com>
Cc: rtc-linux@googlegroups.com, devicetree@vger.kernel.org,
	Knut Wohlrab <Knut.Wohlrab@de.bosch.com>,
	Oleksij Rempel <fixed-term.Oleksij.Rempel@de.bosch.com>
Subject: Re: [rtc-linux] [PATCH v2 3/4] rtc: rv8803: add device tree compatible names
Date: Fri, 24 Jun 2016 09:00:21 +0200	[thread overview]
Message-ID: <20160624070021.GH5809@piout.net> (raw)
In-Reply-To: <1466577742-18824-4-git-send-email-dirk.behme@de.bosch.com>

On 22/06/2016 at 08:42:21 +0200, Dirk Behme wrote :
> From: Knut Wohlrab <Knut.Wohlrab@de.bosch.com>
> 
> Add device tree compatible names for usage in a compatible
> list of alternative RTC devices.
> 

This patch is unnecessary, the i2c subsystem is making sure the driver
is probed without it.

> Signed-off-by: Knut Wohlrab <Knut.Wohlrab@de.bosch.com>
> Signed-off-by: Oleksij Rempel <fixed-term.Oleksij.Rempel@de.bosch.com>
> Cc: devicetree@vger.kernel.org
> ---
>  drivers/rtc/rtc-rv8803.c | 11 +++++++++++
>  1 file changed, 11 insertions(+)
> 
> diff --git a/drivers/rtc/rtc-rv8803.c b/drivers/rtc/rtc-rv8803.c
> index f623038..e122b51 100644
> --- a/drivers/rtc/rtc-rv8803.c
> +++ b/drivers/rtc/rtc-rv8803.c
> @@ -18,6 +18,7 @@
>  #include <linux/kernel.h>
>  #include <linux/module.h>
>  #include <linux/rtc.h>
> +#include <linux/of.h>
>  
>  #define RV8803_SEC			0x00
>  #define RV8803_MIN			0x01
> @@ -520,9 +521,19 @@ static const struct i2c_device_id rv8803_id[] = {
>  };
>  MODULE_DEVICE_TABLE(i2c, rv8803_id);
>  
> +#ifdef CONFIG_OF
> +static const struct of_device_id rv8803_of_match[] = {
> +	{ .compatible = "microcrystal,rv8803", },
> +	{ .compatible = "epson,rx8900", },
> +	{},
> +};
> +MODULE_DEVICE_TABLE(of, rv8803_of_match);
> +#endif
> +
>  static struct i2c_driver rv8803_driver = {
>  	.driver = {
>  		.name = "rtc-rv8803",
> +		.of_match_table = of_match_ptr(rv8803_of_match),
>  	},
>  	.probe		= rv8803_probe,
>  	.remove		= rv8803_remove,
> -- 
> 1.9.1
> 
> -- 
> You received this message because you are subscribed to "rtc-linux".
> Membership options at http://groups.google.com/group/rtc-linux .
> Please read http://groups.google.com/group/rtc-linux/web/checklist
> before submitting a driver.
> --- 
> You received this message because you are subscribed to the Google Groups "rtc-linux" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to rtc-linux+unsubscribe@googlegroups.com.
> For more options, visit https://groups.google.com/d/optout.

-- 
Alexandre Belloni, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

-- 
You received this message because you are subscribed to "rtc-linux".
Membership options at http://groups.google.com/group/rtc-linux .
Please read http://groups.google.com/group/rtc-linux/web/checklist
before submitting a driver.
--- 
You received this message because you are subscribed to the Google Groups "rtc-linux" group.
To unsubscribe from this group and stop receiving emails from it, send an email to rtc-linux+unsubscribe@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

  reply	other threads:[~2016-06-24  7:00 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-06-22  6:42 [rtc-linux] [PATCH v2 0/4] add device tree support for rtc-rv8803 Dirk Behme
2016-06-22  6:42 ` [rtc-linux] [PATCH v2 1/4] devicetree: Add Micro Crystal AG vendor id Dirk Behme
2016-06-24  7:00   ` Alexandre Belloni
2016-06-24 15:43   ` [rtc-linux] " Rob Herring
2016-06-22  6:42 ` [rtc-linux] [PATCH v2 2/4] dt/bindings: Add bindings for Micro Crystal rv8803 Dirk Behme
2016-06-24  7:48   ` Alexandre Belloni
2016-06-24 19:14     ` Andrey Smirnov
2016-06-29  4:34       ` fixed-term.Oleksij.Rempel
2016-06-29  8:03         ` Alexandre Belloni
2016-06-24 15:44   ` [rtc-linux] " Rob Herring
2016-06-22  6:42 ` [rtc-linux] [PATCH v2 3/4] rtc: rv8803: add device tree compatible names Dirk Behme
2016-06-24  7:00   ` Alexandre Belloni [this message]
2016-06-22  6:42 ` [rtc-linux] [PATCH v2 4/4] rtc: rv8803: Be able to set VDETOFF and SWOFF via devcie tree Dirk Behme
2016-06-24  7:44   ` Alexandre Belloni
2016-06-29  4:35     ` fixed-term.Oleksij.Rempel
2016-06-29  8:02       ` Alexandre Belloni

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20160624070021.GH5809@piout.net \
    --to=alexandre.belloni@free-electrons.com \
    --cc=Knut.Wohlrab@de.bosch.com \
    --cc=devicetree@vger.kernel.org \
    --cc=dirk.behme@de.bosch.com \
    --cc=fixed-term.Oleksij.Rempel@de.bosch.com \
    --cc=rtc-linux@googlegroups.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox