From: Dmitry Torokhov <dmitry.torokhov@gmail.com>
To: Sonic Zhang <sonic.adi@gmail.com>
Cc: linux-input@vger.kernel.org,
Michael Hennerich <michael.hennerich@analog.com>,
adi-buildroot-devel@lists.sourceforge.net,
Sonic Zhang <sonic.zhang@analog.com>
Subject: Re: [PATCH v2 2/3] bfin_rotary: Move peripheral pinmux definition into platform data
Date: Tue, 3 Feb 2015 17:22:30 -0800 [thread overview]
Message-ID: <20150204012230.GC23942@dtor-ws> (raw)
In-Reply-To: <1421225654-27872-2-git-send-email-sonic.adi@gmail.com>
On Wed, Jan 14, 2015 at 04:54:13PM +0800, Sonic Zhang wrote:
> From: Sonic Zhang <sonic.zhang@analog.com>
>
Please tell me why this is needed. Also, should not this be converted to
pinctrl infrastructure or I misunderstand what it does?
Thanks.
> Signed-off-by: Sonic Zhang <sonic.zhang@analog.com>
> ---
> arch/blackfin/mach-bf527/boards/ad7160eval.c | 8 ++++++++
> arch/blackfin/mach-bf527/boards/ezkit.c | 8 ++++++++
> arch/blackfin/mach-bf548/boards/ezkit.c | 8 ++++++++
> arch/blackfin/mach-bf609/boards/ezkit.c | 8 ++++++++
> drivers/input/misc/bfin_rotary.c | 12 +++---------
> include/linux/platform_data/bfin_rotary.h | 1 +
> 6 files changed, 36 insertions(+), 9 deletions(-)
>
> diff --git a/arch/blackfin/mach-bf527/boards/ad7160eval.c b/arch/blackfin/mach-bf527/boards/ad7160eval.c
> index beb011b..029a050 100644
> --- a/arch/blackfin/mach-bf527/boards/ad7160eval.c
> +++ b/arch/blackfin/mach-bf527/boards/ad7160eval.c
> @@ -668,6 +668,13 @@ static struct platform_device bfin_sport1_uart_device = {
> #if IS_ENABLED(CONFIG_INPUT_BFIN_ROTARY)
> #include <linux/platform_data/bfin_rotary.h>
>
> +static const u16 per_cnt[] = {
> + P_CNT_CUD,
> + P_CNT_CDG,
> + P_CNT_CZM,
> + 0
> +};
> +
> static struct bfin_rotary_platform_data bfin_rotary_data = {
> /*.rotary_up_key = KEY_UP,*/
> /*.rotary_down_key = KEY_DOWN,*/
> @@ -676,6 +683,7 @@ static struct bfin_rotary_platform_data bfin_rotary_data = {
> .debounce = 10, /* 0..17 */
> .mode = ROT_QUAD_ENC | ROT_DEBE,
> .pm_wakeup = 1,
> + .pin_list = per_cnt,
> };
>
> static struct resource bfin_rotary_resources[] = {
> diff --git a/arch/blackfin/mach-bf527/boards/ezkit.c b/arch/blackfin/mach-bf527/boards/ezkit.c
> index 728cda4..cc80c5b 100644
> --- a/arch/blackfin/mach-bf527/boards/ezkit.c
> +++ b/arch/blackfin/mach-bf527/boards/ezkit.c
> @@ -1094,6 +1094,13 @@ static struct platform_device bfin_device_gpiokeys = {
> #if IS_ENABLED(CONFIG_INPUT_BFIN_ROTARY)
> #include <linux/platform_data/bfin_rotary.h>
>
> +static const u16 per_cnt[] = {
> + P_CNT_CUD,
> + P_CNT_CDG,
> + P_CNT_CZM,
> + 0
> +};
> +
> static struct bfin_rotary_platform_data bfin_rotary_data = {
> /*.rotary_up_key = KEY_UP,*/
> /*.rotary_down_key = KEY_DOWN,*/
> @@ -1102,6 +1109,7 @@ static struct bfin_rotary_platform_data bfin_rotary_data = {
> .debounce = 10, /* 0..17 */
> .mode = ROT_QUAD_ENC | ROT_DEBE,
> .pm_wakeup = 1,
> + .pin_list = per_cnt,
> };
>
> static struct resource bfin_rotary_resources[] = {
> diff --git a/arch/blackfin/mach-bf548/boards/ezkit.c b/arch/blackfin/mach-bf548/boards/ezkit.c
> index 8f70f83..0b68781 100644
> --- a/arch/blackfin/mach-bf548/boards/ezkit.c
> +++ b/arch/blackfin/mach-bf548/boards/ezkit.c
> @@ -161,6 +161,13 @@ static struct platform_device bf54x_kpad_device = {
> #if IS_ENABLED(CONFIG_INPUT_BFIN_ROTARY)
> #include <linux/platform_data/bfin_rotary.h>
>
> +static const u16 per_cnt[] = {
> + P_CNT_CUD,
> + P_CNT_CDG,
> + P_CNT_CZM,
> + 0
> +};
> +
> static struct bfin_rotary_platform_data bfin_rotary_data = {
> /*.rotary_up_key = KEY_UP,*/
> /*.rotary_down_key = KEY_DOWN,*/
> @@ -169,6 +176,7 @@ static struct bfin_rotary_platform_data bfin_rotary_data = {
> .debounce = 10, /* 0..17 */
> .mode = ROT_QUAD_ENC | ROT_DEBE,
> .pm_wakeup = 1,
> + .pin_list = per_cnt,
> };
>
> static struct resource bfin_rotary_resources[] = {
> diff --git a/arch/blackfin/mach-bf609/boards/ezkit.c b/arch/blackfin/mach-bf609/boards/ezkit.c
> index f9dc64d..4ac72c6 100644
> --- a/arch/blackfin/mach-bf609/boards/ezkit.c
> +++ b/arch/blackfin/mach-bf609/boards/ezkit.c
> @@ -77,6 +77,13 @@ static struct platform_device bfin_isp1760_device = {
> #if IS_ENABLED(CONFIG_INPUT_BFIN_ROTARY)
> #include <linux/platform_data/bfin_rotary.h>
>
> +static const u16 per_cnt[] = {
> + P_CNT_CUD,
> + P_CNT_CDG,
> + P_CNT_CZM,
> + 0
> +};
> +
> static struct bfin_rotary_platform_data bfin_rotary_data = {
> /*.rotary_up_key = KEY_UP,*/
> /*.rotary_down_key = KEY_DOWN,*/
> @@ -84,6 +91,7 @@ static struct bfin_rotary_platform_data bfin_rotary_data = {
> .rotary_button_key = KEY_ENTER,
> .debounce = 10, /* 0..17 */
> .mode = ROT_QUAD_ENC | ROT_DEBE,
> + .pin_list = per_cnt,
> };
>
> static struct resource bfin_rotary_resources[] = {
> diff --git a/drivers/input/misc/bfin_rotary.c b/drivers/input/misc/bfin_rotary.c
> index 8f6c7cd..dd62fc9 100644
> --- a/drivers/input/misc/bfin_rotary.c
> +++ b/drivers/input/misc/bfin_rotary.c
> @@ -16,13 +16,6 @@
>
> #include <asm/portmux.h>
>
> -static const u16 per_cnt[] = {
> - P_CNT_CUD,
> - P_CNT_CDG,
> - P_CNT_CZM,
> - 0
> -};
> -
> struct bfin_rot {
> struct input_dev *input;
> int irq;
> @@ -102,7 +95,7 @@ static int bfin_rotary_probe(struct platform_device *pdev)
> return -EINVAL;
> }
>
> - error = peripheral_request_list(per_cnt, dev_name(&pdev->dev));
> + error = peripheral_request_list(pdata->pin_list, dev_name(&pdev->dev));
> if (error) {
> dev_err(&pdev->dev, "requesting peripherals failed\n");
> return error;
> @@ -197,6 +190,7 @@ out1:
>
> static int bfin_rotary_remove(struct platform_device *pdev)
> {
> + struct bfin_rotary_platform_data *pdata = dev_get_platdata(&pdev->dev);
> struct bfin_rot *rotary = platform_get_drvdata(pdev);
>
> bfin_write_CNT_CONFIG(0);
> @@ -204,7 +198,7 @@ static int bfin_rotary_remove(struct platform_device *pdev)
>
> free_irq(rotary->irq, pdev);
> input_unregister_device(rotary->input);
> - peripheral_free_list(per_cnt);
> + peripheral_free_list(pdata->pin_list);
>
> kfree(rotary);
>
> diff --git a/include/linux/platform_data/bfin_rotary.h b/include/linux/platform_data/bfin_rotary.h
> index 8895a75..9882937 100644
> --- a/include/linux/platform_data/bfin_rotary.h
> +++ b/include/linux/platform_data/bfin_rotary.h
> @@ -40,6 +40,7 @@ struct bfin_rotary_platform_data {
> unsigned short debounce; /* 0..17 */
> unsigned short mode;
> unsigned short pm_wakeup;
> + unsigned short *pin_list;
> };
>
> /* CNT_CONFIG bitmasks */
> --
> 1.7.9.5
>
--
Dmitry
next prev parent reply other threads:[~2015-02-04 1:22 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-01-14 8:54 [PATCH v2 1/3] bfin_rotary: move rotary platform header definition to linux platform_data head path Sonic Zhang
2015-01-14 8:54 ` [PATCH v2 2/3] bfin_rotary: Move peripheral pinmux definition into platform data Sonic Zhang
2015-02-04 1:22 ` Dmitry Torokhov [this message]
2015-02-04 6:28 ` Sonic Zhang
2015-01-14 8:54 ` [PATCH v2 3/3] bfin_rotary: replace bfin specific MMR function with generic IO function Sonic Zhang
2015-01-22 6:26 ` Sonic Zhang
2015-01-29 9:35 ` Sonic Zhang
2015-02-04 1:24 ` Dmitry Torokhov
2015-02-04 6:31 ` Sonic Zhang
2015-02-04 7:23 ` Sonic Zhang
2015-01-22 6:25 ` [PATCH v2 1/3] bfin_rotary: move rotary platform header definition to linux platform_data head path Sonic Zhang
2015-01-29 9:35 ` Sonic Zhang
2015-02-02 6:52 ` Sonic Zhang
2015-02-04 1:20 ` Dmitry Torokhov
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=20150204012230.GC23942@dtor-ws \
--to=dmitry.torokhov@gmail.com \
--cc=adi-buildroot-devel@lists.sourceforge.net \
--cc=linux-input@vger.kernel.org \
--cc=michael.hennerich@analog.com \
--cc=sonic.adi@gmail.com \
--cc=sonic.zhang@analog.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;
as well as URLs for NNTP newsgroup(s).