From: Lee Jones <lee.jones@linaro.org>
To: Lee Jones <lee.jones@linaro.org>, sameo@linux.intel.com
Cc: linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org,
STEricsson_nomadik_linux@list.st.com,
linus.walleij@stericsson.com, arnd@arndb.de
Subject: Re: [PATCH 1/1] mfd: ab8500: Clean-up the last two ab8500 related DT match tables
Date: Mon, 09 Jul 2012 22:48:40 +0200 [thread overview]
Message-ID: <4FFB43A8.2070307@linaro.org> (raw)
In-Reply-To: <1341578598-9307-1-git-send-email-lee.jones@linaro.org>
Hi Sam,
I've just seen that you have a new for-next branch, but this patch is
missing in it.
Just making sure you didn't miss it:
On 06/07/12 14:43, Lee Jones wrote:
> A patch was recently submitted to stop all ab8500 devices from
> being registered through Device Tree. Instead, only the db8500
> will be DT:ed and the rest will continue to be registered via
> the MFD API, as they always were.
>
> Two patches have recently been applied which enable Device Tree
> probing; one for rtc-ab8500 and the other for ponkey-ab8500.
> These two need to be removed to prevent double-probing these
> devices with Device Tree is enabled.
>
> Cc: Samuel Ortiz <sameo@linux.intel.com>
> Signed-off-by: Lee Jones <lee.jones@linaro.org>
> ---
> drivers/input/misc/ab8500-ponkey.c | 8 --------
> drivers/rtc/rtc-ab8500.c | 6 ------
> 2 files changed, 14 deletions(-)
>
> diff --git a/drivers/input/misc/ab8500-ponkey.c b/drivers/input/misc/ab8500-ponkey.c
> index 84ec691..5ceb23e 100644
> --- a/drivers/input/misc/ab8500-ponkey.c
> +++ b/drivers/input/misc/ab8500-ponkey.c
> @@ -132,18 +132,10 @@ static int __devexit ab8500_ponkey_remove(struct platform_device *pdev)
> return 0;
> }
>
> -#ifdef CONFIG_OF
> -static const struct of_device_id ab8500_ponkey_match[] = {
> - { .compatible = "stericsson,ab8500-ponkey", },
> - {}
> -};
> -#endif
> -
> static struct platform_driver ab8500_ponkey_driver = {
> .driver = {
> .name = "ab8500-poweron-key",
> .owner = THIS_MODULE,
> - .of_match_table = of_match_ptr(ab8500_ponkey_match),
> },
> .probe = ab8500_ponkey_probe,
> .remove = __devexit_p(ab8500_ponkey_remove),
> diff --git a/drivers/rtc/rtc-ab8500.c b/drivers/rtc/rtc-ab8500.c
> index bf3c2f6..2e5970f 100644
> --- a/drivers/rtc/rtc-ab8500.c
> +++ b/drivers/rtc/rtc-ab8500.c
> @@ -462,16 +462,10 @@ static int __devexit ab8500_rtc_remove(struct platform_device *pdev)
> return 0;
> }
>
> -static const struct of_device_id ab8500_rtc_match[] = {
> - { .compatible = "stericsson,ab8500-rtc", },
> - {}
> -};
> -
> static struct platform_driver ab8500_rtc_driver = {
> .driver = {
> .name = "ab8500-rtc",
> .owner = THIS_MODULE,
> - .of_match_table = ab8500_rtc_match,
> },
> .probe = ab8500_rtc_probe,
> .remove = __devexit_p(ab8500_rtc_remove),
>
Kind regards,
Lee
--
Lee Jones
Linaro ST-Ericsson Landing Team Lead
M: +44 77 88 633 515
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
WARNING: multiple messages have this Message-ID (diff)
From: lee.jones@linaro.org (Lee Jones)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 1/1] mfd: ab8500: Clean-up the last two ab8500 related DT match tables
Date: Mon, 09 Jul 2012 22:48:40 +0200 [thread overview]
Message-ID: <4FFB43A8.2070307@linaro.org> (raw)
In-Reply-To: <1341578598-9307-1-git-send-email-lee.jones@linaro.org>
Hi Sam,
I've just seen that you have a new for-next branch, but this patch is
missing in it.
Just making sure you didn't miss it:
On 06/07/12 14:43, Lee Jones wrote:
> A patch was recently submitted to stop all ab8500 devices from
> being registered through Device Tree. Instead, only the db8500
> will be DT:ed and the rest will continue to be registered via
> the MFD API, as they always were.
>
> Two patches have recently been applied which enable Device Tree
> probing; one for rtc-ab8500 and the other for ponkey-ab8500.
> These two need to be removed to prevent double-probing these
> devices with Device Tree is enabled.
>
> Cc: Samuel Ortiz <sameo@linux.intel.com>
> Signed-off-by: Lee Jones <lee.jones@linaro.org>
> ---
> drivers/input/misc/ab8500-ponkey.c | 8 --------
> drivers/rtc/rtc-ab8500.c | 6 ------
> 2 files changed, 14 deletions(-)
>
> diff --git a/drivers/input/misc/ab8500-ponkey.c b/drivers/input/misc/ab8500-ponkey.c
> index 84ec691..5ceb23e 100644
> --- a/drivers/input/misc/ab8500-ponkey.c
> +++ b/drivers/input/misc/ab8500-ponkey.c
> @@ -132,18 +132,10 @@ static int __devexit ab8500_ponkey_remove(struct platform_device *pdev)
> return 0;
> }
>
> -#ifdef CONFIG_OF
> -static const struct of_device_id ab8500_ponkey_match[] = {
> - { .compatible = "stericsson,ab8500-ponkey", },
> - {}
> -};
> -#endif
> -
> static struct platform_driver ab8500_ponkey_driver = {
> .driver = {
> .name = "ab8500-poweron-key",
> .owner = THIS_MODULE,
> - .of_match_table = of_match_ptr(ab8500_ponkey_match),
> },
> .probe = ab8500_ponkey_probe,
> .remove = __devexit_p(ab8500_ponkey_remove),
> diff --git a/drivers/rtc/rtc-ab8500.c b/drivers/rtc/rtc-ab8500.c
> index bf3c2f6..2e5970f 100644
> --- a/drivers/rtc/rtc-ab8500.c
> +++ b/drivers/rtc/rtc-ab8500.c
> @@ -462,16 +462,10 @@ static int __devexit ab8500_rtc_remove(struct platform_device *pdev)
> return 0;
> }
>
> -static const struct of_device_id ab8500_rtc_match[] = {
> - { .compatible = "stericsson,ab8500-rtc", },
> - {}
> -};
> -
> static struct platform_driver ab8500_rtc_driver = {
> .driver = {
> .name = "ab8500-rtc",
> .owner = THIS_MODULE,
> - .of_match_table = ab8500_rtc_match,
> },
> .probe = ab8500_rtc_probe,
> .remove = __devexit_p(ab8500_rtc_remove),
>
Kind regards,
Lee
--
Lee Jones
Linaro ST-Ericsson Landing Team Lead
M: +44 77 88 633 515
Linaro.org ? Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
next prev parent reply other threads:[~2012-07-09 20:48 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-07-06 12:43 [PATCH 1/1] mfd: ab8500: Clean-up the last two ab8500 related DT match tables Lee Jones
2012-07-06 12:43 ` Lee Jones
2012-07-09 20:48 ` Lee Jones [this message]
2012-07-09 20:48 ` Lee Jones
2012-07-10 9:13 ` Samuel Ortiz
2012-07-10 9:13 ` Samuel Ortiz
2012-07-10 15:28 ` Lee Jones
2012-07-10 15:28 ` Lee Jones
2012-07-10 16:55 ` Samuel Ortiz
2012-07-10 16:55 ` Samuel Ortiz
2012-07-10 18:19 ` Lee Jones
2012-07-10 18:19 ` Lee Jones
2012-07-16 12:38 ` Samuel Ortiz
2012-07-16 12:38 ` Samuel Ortiz
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=4FFB43A8.2070307@linaro.org \
--to=lee.jones@linaro.org \
--cc=STEricsson_nomadik_linux@list.st.com \
--cc=arnd@arndb.de \
--cc=linus.walleij@stericsson.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=sameo@linux.intel.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.