Re: [PATCH 1/2] pinctrl: move strict option to pinmux_ops

2015-05-07 Thread Sonic Zhang
Hi Walleij, Acked-by: Sonic Zhang Sonic On Thu, May 7, 2015 at 5:53 PM, Linus Walleij wrote: > While the pinmux_ops are ideally just a vtable for pin mux > calls, the "strict" setting belongs so intuitively with the > pin multiplexing that we should move it h

Re: [PATCH 1/2] pinctrl: move strict option to pinmux_ops

2015-05-07 Thread Sonic Zhang
Hi Walleij, Acked-by: Sonic Zhang sonic.zh...@analog.com Sonic On Thu, May 7, 2015 at 5:53 PM, Linus Walleij linus.wall...@linaro.org wrote: While the pinmux_ops are ideally just a vtable for pin mux calls, the strict setting belongs so intuitively with the pin multiplexing that we should

Re: [PATCH] serial: bfin: ctsrts: enfore Kconfig naming convention

2015-04-13 Thread Sonic Zhang
Acked-by: Sonic Zhang On Sun, Apr 12, 2015 at 11:54 PM, Valentin Rothberg wrote: > The CONFIG_ prefix is reserved for Kconfig options in Make and CPP > syntax; static analysis tools rely on this convention. This patch > enforces this behavior for SERIAL_BFIN_{HARD_}CTSRTS. >

Re: [PATCH] serial: bfin: ctsrts: enfore Kconfig naming convention

2015-04-13 Thread Sonic Zhang
Acked-by: Sonic Zhang sonic.zh...@analog.com On Sun, Apr 12, 2015 at 11:54 PM, Valentin Rothberg valentinrothb...@gmail.com wrote: The CONFIG_ prefix is reserved for Kconfig options in Make and CPP syntax; static analysis tools rely on this convention. This patch enforces this behavior

Re: [PATCH] pinmux: allow exlusive pin allocation among GPIO and peripheral funtions via flag strict in struct pinctrl_desc

2015-03-19 Thread Sonic Zhang
Hi Linus, On Wed, Mar 18, 2015 at 6:21 PM, Linus Walleij wrote: > On Thu, Mar 12, 2015 at 10:56 AM, Sonic Zhang wrote: > >> From: Sonic Zhang >> >> The blackfin pinmux and gpio controller doesn't allow user to set up 1 pin >> for both GPIO and peripheral function.

Re: [PATCH] pinmux: allow exlusive pin allocation among GPIO and peripheral funtions via flag strict in struct pinctrl_desc

2015-03-19 Thread Sonic Zhang
Hi Linus, On Wed, Mar 18, 2015 at 6:21 PM, Linus Walleij linus.wall...@linaro.org wrote: On Thu, Mar 12, 2015 at 10:56 AM, Sonic Zhang sonic@gmail.com wrote: From: Sonic Zhang sonic.zh...@analog.com The blackfin pinmux and gpio controller doesn't allow user to set up 1 pin for both GPIO

Re: [RFT/PATCH] Input: bfin_rotary - introduce open and close methods

2015-02-12 Thread Sonic Zhang
Acked-by: Sonic Zhang On Thu, Feb 12, 2015 at 3:54 PM, Dmitry Torokhov wrote: > Hi Sonic, > > On Fri, Feb 06, 2015 at 05:37:29PM +0800, Sonic Zhang wrote: >> Hi Dmitry, >> >> After apply your patch manually to my blackfin kernel tree, I got >> kernel panic

Re: [RFT/PATCH] Input: bfin_rotary - introduce open and close methods

2015-02-12 Thread Sonic Zhang
Acked-by: Sonic Zhang sonic.zh...@analog.com On Thu, Feb 12, 2015 at 3:54 PM, Dmitry Torokhov dmitry.torok...@gmail.com wrote: Hi Sonic, On Fri, Feb 06, 2015 at 05:37:29PM +0800, Sonic Zhang wrote: Hi Dmitry, After apply your patch manually to my blackfin kernel tree, I got kernel panic

Re: [PATCH] Input: bfin_rotary - mark suspend and resume code as __maybe_unused

2015-02-09 Thread Sonic Zhang
Hi Dmitry, Since the line I mentioned was removed in v3.19-rc1, I am fine with this patch. Acked-by: Sonic Zhang On Fri, Feb 6, 2015 at 2:09 PM, Dmitry Torokhov wrote: > Instead of using #ifdef to guard potentially unused suspend and resume code > let's mark them as __maybe_unu

Re: [PATCH] Input: bfin_rotary - mark suspend and resume code as __maybe_unused

2015-02-09 Thread Sonic Zhang
Hi Dmitry, Since the line I mentioned was removed in v3.19-rc1, I am fine with this patch. Acked-by: Sonic Zhang sonic.zh...@analog.com On Fri, Feb 6, 2015 at 2:09 PM, Dmitry Torokhov dmitry.torok...@gmail.com wrote: Instead of using #ifdef to guard potentially unused suspend and resume code

Re: [RFT/PATCH] Input: bfin_rotary - introduce open and close methods

2015-02-06 Thread Sonic Zhang
f 2806 Return addresses in stack: address : <0x00100950> { _bus_for_each_dev + 0x5c } address : <0x106e> { _do_one_initcall + 0x6e } address : <0x00100100> { _device_add + 0x3b4 } address : <0x0001bcde> { _parse_args + 0x12e } address : <0

Re: [RFT/PATCH] Input: bfin_rotary - introduce open and close methods

2015-02-06 Thread Sonic Zhang
Hi Dmitry, On Fri, Feb 6, 2015 at 2:56 PM, Dmitry Torokhov wrote: > Introduce open and close methods for the input device to postpone enabling > the device until it is needed. > > Signed-off-by: Dmitry Torokhov > --- > > Hi Sonic, > > Could you please tell me if the driver still works with this

Re: [PATCH] Input: bfin_rotary - mark suspend and resume code as __maybe_unused

2015-02-06 Thread Sonic Zhang
Hi Dmitry, On Fri, Feb 6, 2015 at 2:09 PM, Dmitry Torokhov wrote: > Instead of using #ifdef to guard potentially unused suspend and resume code > let's mark them as __maybe_unused so they still get discarded if they are > not used but we do not get warning. This allows for better compile

Re: [PATCH] Input: bfin_rotary - mark suspend and resume code as __maybe_unused

2015-02-06 Thread Sonic Zhang
Hi Dmitry, On Fri, Feb 6, 2015 at 2:09 PM, Dmitry Torokhov dmitry.torok...@gmail.com wrote: Instead of using #ifdef to guard potentially unused suspend and resume code let's mark them as __maybe_unused so they still get discarded if they are not used but we do not get warning. This allows for

Re: [RFT/PATCH] Input: bfin_rotary - introduce open and close methods

2015-02-06 Thread Sonic Zhang
Hi Dmitry, On Fri, Feb 6, 2015 at 2:56 PM, Dmitry Torokhov dmitry.torok...@gmail.com wrote: Introduce open and close methods for the input device to postpone enabling the device until it is needed. Signed-off-by: Dmitry Torokhov dmitry.torok...@gmail.com --- Hi Sonic, Could you please

Re: [RFT/PATCH] Input: bfin_rotary - introduce open and close methods

2015-02-06 Thread Sonic Zhang
: 0x00100100 { _device_add + 0x3b4 } address : 0x0001bcde { _parse_args + 0x12e } address : 0xffa00014 { _ret_from_fork + 0x14 } Modules linked in: Kernel panic - not syncing: Kernel exception ---[ end Kernel panic - not syncing: Kernel exception Regards, Sonic Zhang -- To unsubscribe from

[PATCH] blackfin: Remove non-existed Kconfig symbol VIDEO_MT9M114

2014-02-14 Thread Sonic Zhang
From: Sonic Zhang Bind different groups of one pinmux function to respective state names. Add all states at once, no need to exclude each other by Kconfig symbol. Signed-off-by: Sonic Zhang --- arch/blackfin/mach-bf548/boards/ezkit.c | 6 -- arch/blackfin/mach-bf609/boards/ezkit.c | 17

[PATCH] pinctrl: pinctrl-adi: combine multiple groups of one function together

2014-02-14 Thread Sonic Zhang
From: Sonic Zhang The data pins of some peripheral are different if connecting to different devices in one pinmux function. In the PPI case, data pins can be used in 8, 16 and 24 pin groups individually. Add these groups into one ppi function. Signed-off-by: Sonic Zhang --- drivers/pinctrl

[PATCH] pinctrl: pinctrl-adi: combine multiple groups of one function together

2014-02-14 Thread Sonic Zhang
From: Sonic Zhang sonic.zh...@analog.com The data pins of some peripheral are different if connecting to different devices in one pinmux function. In the PPI case, data pins can be used in 8, 16 and 24 pin groups individually. Add these groups into one ppi function. Signed-off-by: Sonic Zhang

[PATCH] blackfin: Remove non-existed Kconfig symbol VIDEO_MT9M114

2014-02-14 Thread Sonic Zhang
From: Sonic Zhang sonic.zh...@analog.com Bind different groups of one pinmux function to respective state names. Add all states at once, no need to exclude each other by Kconfig symbol. Signed-off-by: Sonic Zhang sonic.zh...@analog.com --- arch/blackfin/mach-bf548/boards/ezkit.c | 6

Re: [PATCH 06/17] i2c: i2c-bfin-twi: deprecate class based instantiation

2014-02-11 Thread Sonic Zhang
Acked-by: Sonic Zhang On Mon, Feb 10, 2014 at 6:04 PM, Wolfram Sang wrote: > Warn users that class based instantiation is going away soon in favour > of more robust probing and faster bootup times. > > Signed-off-by: Wolfram Sang > Cc: Sonic Zhang > --- > > This patch

Re: [PATCH 06/17] i2c: i2c-bfin-twi: deprecate class based instantiation

2014-02-11 Thread Sonic Zhang
Acked-by: Sonic Zhang sonic.zh...@analog.com On Mon, Feb 10, 2014 at 6:04 PM, Wolfram Sang w...@the-dreams.de wrote: Warn users that class based instantiation is going away soon in favour of more robust probing and faster bootup times. Signed-off-by: Wolfram Sang w...@the-dreams.de Cc: Sonic

[PATCH] MAINTAINERS: ADI Linux development mailing lists change to the new server.

2014-01-27 Thread Sonic Zhang
From: Sonic Zhang Update Blackfin arch branch maintainer's email as well. Signed-off-by: Sonic Zhang --- MAINTAINERS | 42 -- 1 file changed, 20 insertions(+), 22 deletions(-) diff --git a/MAINTAINERS b/MAINTAINERS index 7cacc88..bd49e70 100644

[PATCH 2/3 v2] pinctrl-adi2: change irq_base from usigned int to int

2014-01-27 Thread Sonic Zhang
From: Sonic Zhang Negative irq_base means there is no fixed Linux irq mappings are created from the platform data. The driver calls irq_create_mapping to allocate a virtual Linux irq. Signed-off-by: Sonic Zhang --- v2-changes: - reword the patch description drivers/pinctrl/pinctrl-adi2.c

[PATCH 3/3] pinctrl-adi2: fix coding style issue

2014-01-27 Thread Sonic Zhang
From: Sonic Zhang Signed-off-by: Sonic Zhang --- drivers/pinctrl/pinctrl-adi2.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/pinctrl/pinctrl-adi2.c b/drivers/pinctrl/pinctrl-adi2.c index 9fb53c9..72450a1 100644 --- a/drivers/pinctrl/pinctrl-adi2.c +++ b/drivers

[PATCH 1/3] pinctrl: Move pint PM storage structure out of blackfin architecture.

2014-01-27 Thread Sonic Zhang
From: Sonic Zhang It is better to keep this structure in the pinctrl-adi2 driver. Signed-off-by: Sonic Zhang --- arch/blackfin/include/asm/irq.h | 9 - drivers/pinctrl/pinctrl-adi2.c | 15 ++- 2 files changed, 14 insertions(+), 10 deletions(-) diff --git a/arch/blackfin

[PATCH 2/3] pinctrl-adi2: change irq_base from usigned int to int

2014-01-27 Thread Sonic Zhang
From: Sonic Zhang Negative irq_base means this gpio port doens't support interrupts. Signed-off-by: Sonic Zhang --- drivers/pinctrl/pinctrl-adi2.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/pinctrl/pinctrl-adi2.c b/drivers/pinctrl/pinctrl-adi2.c index e8120fa

[PATCH 2/3] pinctrl-adi2: change irq_base from usigned int to int

2014-01-27 Thread Sonic Zhang
From: Sonic Zhang sonic.zh...@analog.com Negative irq_base means this gpio port doens't support interrupts. Signed-off-by: Sonic Zhang sonic.zh...@analog.com --- drivers/pinctrl/pinctrl-adi2.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/pinctrl/pinctrl-adi2.c b

[PATCH 1/3] pinctrl: Move pint PM storage structure out of blackfin architecture.

2014-01-27 Thread Sonic Zhang
From: Sonic Zhang sonic.zh...@analog.com It is better to keep this structure in the pinctrl-adi2 driver. Signed-off-by: Sonic Zhang sonic.zh...@analog.com --- arch/blackfin/include/asm/irq.h | 9 - drivers/pinctrl/pinctrl-adi2.c | 15 ++- 2 files changed, 14 insertions

[PATCH 3/3] pinctrl-adi2: fix coding style issue

2014-01-27 Thread Sonic Zhang
From: Sonic Zhang sonic.zh...@analog.com Signed-off-by: Sonic Zhang sonic.zh...@analog.com --- drivers/pinctrl/pinctrl-adi2.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/pinctrl/pinctrl-adi2.c b/drivers/pinctrl/pinctrl-adi2.c index 9fb53c9..72450a1 100644

[PATCH 2/3 v2] pinctrl-adi2: change irq_base from usigned int to int

2014-01-27 Thread Sonic Zhang
From: Sonic Zhang sonic.zh...@analog.com Negative irq_base means there is no fixed Linux irq mappings are created from the platform data. The driver calls irq_create_mapping to allocate a virtual Linux irq. Signed-off-by: Sonic Zhang sonic.zh...@analog.com --- v2-changes: - reword the patch

[PATCH] MAINTAINERS: ADI Linux development mailing lists change to the new server.

2014-01-27 Thread Sonic Zhang
From: Sonic Zhang sonic.zh...@analog.com Update Blackfin arch branch maintainer's email as well. Signed-off-by: Sonic Zhang sonic.zh...@analog.com --- MAINTAINERS | 42 -- 1 file changed, 20 insertions(+), 22 deletions(-) diff --git a/MAINTAINERS b

[PATCH] MAINTAINERS: ADI Linux development mailing lists change to the new server.

2014-01-25 Thread Sonic Zhang
From: Sonic Zhang Update Blackfin arch branch maintainer's email as well. Signed-off-by: Sonic Zhang --- MAINTAINERS | 42 -- 1 file changed, 20 insertions(+), 22 deletions(-) diff --git a/MAINTAINERS b/MAINTAINERS index 7cacc88..bd49e70 100644

[PATCH] MAINTAINERS: ADI Linux development mailing lists change to the new server.

2014-01-25 Thread Sonic Zhang
From: Sonic Zhang sonic.zh...@analog.com Update Blackfin arch branch maintainer's email as well. Signed-off-by: Sonic Zhang sonic.zh...@analog.com --- MAINTAINERS | 42 -- 1 file changed, 20 insertions(+), 22 deletions(-) diff --git a/MAINTAINERS b

Re: [PATCH] pinctrl: pinctrl-adi2: GPIO output value should be set after the GPIO interrupt is disabled.

2013-10-08 Thread Sonic Zhang
Hi Axel, On Tue, Oct 8, 2013 at 3:48 PM, Axel Lin wrote: > 2013/10/8 Sonic Zhang : >> From: Sonic Zhang >> >> Use BIT macro as well. >> >> Signed-off-by: Sonic Zhang >> --- >> drivers/pinctrl/pinctrl-adi2.c | 14 -- >> 1 file

[PATCH] pinctrl: pinctrl-adi2: GPIO output value should be set after the GPIO interrupt is disabled.

2013-10-08 Thread Sonic Zhang
From: Sonic Zhang Use BIT macro as well. Signed-off-by: Sonic Zhang --- drivers/pinctrl/pinctrl-adi2.c | 14 -- 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/drivers/pinctrl/pinctrl-adi2.c b/drivers/pinctrl/pinctrl-adi2.c index a74e6f4..7a39562 100644 --- a/drivers

[PATCH v2] pinctrl: pinctrl-adi2: Remove nested lock+irqsave that resue flags.

2013-10-08 Thread Sonic Zhang
From: Sonic Zhang Also avoid use NULL pointer in error message. v2-changes: - use port pinter only after checking Signed-off-by: Sonic Zhang --- drivers/pinctrl/pinctrl-adi2.c | 40 ++-- 1 file changed, 22 insertions(+), 18 deletions(-) diff --git

[PATCH v2] pinctrl: pinctrl-adi2: Remove nested lock+irqsave that resue flags.

2013-10-08 Thread Sonic Zhang
From: Sonic Zhang sonic.zh...@analog.com Also avoid use NULL pointer in error message. v2-changes: - use port pinter only after checking Signed-off-by: Sonic Zhang sonic.zh...@analog.com --- drivers/pinctrl/pinctrl-adi2.c | 40 ++-- 1 file changed, 22

[PATCH] pinctrl: pinctrl-adi2: GPIO output value should be set after the GPIO interrupt is disabled.

2013-10-08 Thread Sonic Zhang
From: Sonic Zhang sonic.zh...@analog.com Use BIT macro as well. Signed-off-by: Sonic Zhang sonic.zh...@analog.com --- drivers/pinctrl/pinctrl-adi2.c | 14 -- 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/drivers/pinctrl/pinctrl-adi2.c b/drivers/pinctrl/pinctrl-adi2.c

Re: [PATCH] pinctrl: pinctrl-adi2: GPIO output value should be set after the GPIO interrupt is disabled.

2013-10-08 Thread Sonic Zhang
Hi Axel, On Tue, Oct 8, 2013 at 3:48 PM, Axel Lin axel@ingics.com wrote: 2013/10/8 Sonic Zhang sonic@gmail.com: From: Sonic Zhang sonic.zh...@analog.com Use BIT macro as well. Signed-off-by: Sonic Zhang sonic.zh...@analog.com --- drivers/pinctrl/pinctrl-adi2.c | 14

Re: [PATCH 03/27] mmc: bfin_sdh: Move away from using deprecated APIs

2013-10-07 Thread Sonic Zhang
Acked-by: Sonic Zhang On Thu, Sep 26, 2013 at 10:54 PM, Ulf Hansson wrote: > Suspend and resume of cards are being handled from the protocol layer > and consequently the mmc_suspend|resume_host APIs are deprecated. > > This means we can simplify the suspend|resume callbacks

Re: [PATCH] serial: bfin_uart: remove deprecated IRQF_DISABLED

2013-10-07 Thread Sonic Zhang
Acked-by: Sonic Zhang On Sun, Oct 6, 2013 at 2:27 PM, Michael Opdenacker wrote: > This patch proposes to remove the use of the IRQF_DISABLED flag > > It's a NOOP since 2.6.35 and it will be removed one day. > > Signed-off-by: Michael Opdenacker > --- > drivers/tty/s

Re: [PATCH 2/2] pinctrl: adi2: Fix dead lock in adi_gpio_direction_output

2013-10-07 Thread Sonic Zhang
Hi Linus, On Fri, Sep 27, 2013 at 8:20 PM, Linus Walleij wrote: > On Tue, Sep 24, 2013 at 9:47 AM, Axel Lin wrote: > >> Current code hold port->lock spinlock and then try to grab the lock again >> in adi_gpio_set_value(). Fix it. >> >> Signed-off-by: Axel Lin > > Patch applied, thanks Axel. >

Re: [PATCH 2/2] pinctrl: adi2: Fix dead lock in adi_gpio_direction_output

2013-10-07 Thread Sonic Zhang
Hi Linus, On Fri, Sep 27, 2013 at 8:20 PM, Linus Walleij linus.wall...@linaro.org wrote: On Tue, Sep 24, 2013 at 9:47 AM, Axel Lin axel@ingics.com wrote: Current code hold port-lock spinlock and then try to grab the lock again in adi_gpio_set_value(). Fix it. Signed-off-by: Axel Lin

Re: [PATCH] serial: bfin_uart: remove deprecated IRQF_DISABLED

2013-10-07 Thread Sonic Zhang
Acked-by: Sonic Zhang sonc.zh...@analog.com On Sun, Oct 6, 2013 at 2:27 PM, Michael Opdenacker michael.opdenac...@free-electrons.com wrote: This patch proposes to remove the use of the IRQF_DISABLED flag It's a NOOP since 2.6.35 and it will be removed one day. Signed-off-by: Michael

Re: [PATCH 03/27] mmc: bfin_sdh: Move away from using deprecated APIs

2013-10-07 Thread Sonic Zhang
Acked-by: Sonic Zhang sonic.zh...@analog.com On Thu, Sep 26, 2013 at 10:54 PM, Ulf Hansson ulf.hans...@linaro.org wrote: Suspend and resume of cards are being handled from the protocol layer and consequently the mmc_suspend|resume_host APIs are deprecated. This means we can simplify

[PATCH] pinctrl: pinctrl-adi2: Remove nested lock+irqsave that resue flags.

2013-09-23 Thread Sonic Zhang
From: Sonic Zhang Also avoid use NULL pointer in error message. Signed-off-by: Sonic Zhang --- drivers/pinctrl/pinctrl-adi2.c | 32 1 file changed, 16 insertions(+), 16 deletions(-) diff --git a/drivers/pinctrl/pinctrl-adi2.c b/drivers/pinctrl/pinctrl-adi2.c

Re: [PATCH 2/3 v4] blackfin: gpio: Remove none gpio lib code.

2013-09-23 Thread Sonic Zhang
Hi Linus, On Mon, Sep 23, 2013 at 3:48 PM, Linus Walleij wrote: > On Tue, Sep 3, 2013 at 10:29 AM, Sonic Zhang wrote: > >> From: Sonic Zhang >> >> - Remove non gpio lib code from blackfin architecture. >> - Limit the lagecy blackfin gpio driver to bf5xx proce

Re: [PATCH 2/3 v4] blackfin: gpio: Remove none gpio lib code.

2013-09-23 Thread Sonic Zhang
Hi Linus, On Mon, Sep 23, 2013 at 3:48 PM, Linus Walleij linus.wall...@linaro.org wrote: On Tue, Sep 3, 2013 at 10:29 AM, Sonic Zhang sonic@gmail.com wrote: From: Sonic Zhang sonic.zh...@analog.com - Remove non gpio lib code from blackfin architecture. - Limit the lagecy blackfin gpio

[PATCH] pinctrl: pinctrl-adi2: Remove nested lock+irqsave that resue flags.

2013-09-23 Thread Sonic Zhang
From: Sonic Zhang sonic.zh...@analog.com Also avoid use NULL pointer in error message. Signed-off-by: Sonic Zhang sonic.zh...@analog.com --- drivers/pinctrl/pinctrl-adi2.c | 32 1 file changed, 16 insertions(+), 16 deletions(-) diff --git a/drivers/pinctrl

[PATCH] pinctrl: pinctrl-adi2-bf60x: remove useless and duplicated GPIO definition for PPI2.

2013-09-22 Thread Sonic Zhang
From: Sonic Zhang Signed-off-by: Sonic Zhang --- drivers/pinctrl/pinctrl-adi2-bf60x.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/pinctrl/pinctrl-adi2-bf60x.c b/drivers/pinctrl/pinctrl-adi2-bf60x.c index e90cb80..bf57aea 100644 --- a/drivers/pinctrl/pinctrl

[PATCH] pinctrl: pinctrl-adi2: Add dependency to arch BLACKFIN in Kconfig.

2013-09-22 Thread Sonic Zhang
From: Sonic Zhang Signed-off-by: Sonic Zhang --- drivers/pinctrl/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/pinctrl/Kconfig b/drivers/pinctrl/Kconfig index 08b8337..9bb6efb 100644 --- a/drivers/pinctrl/Kconfig +++ b/drivers/pinctrl/Kconfig @@ -51,6 +51,7 @@ config

[PATCH] pinctrl: pinctrl-adi2: Add dependency to arch BLACKFIN in Kconfig.

2013-09-22 Thread Sonic Zhang
From: Sonic Zhang sonic.zh...@analog.com Signed-off-by: Sonic Zhang sonic.zh...@analog.com --- drivers/pinctrl/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/pinctrl/Kconfig b/drivers/pinctrl/Kconfig index 08b8337..9bb6efb 100644 --- a/drivers/pinctrl/Kconfig +++ b/drivers

[PATCH] pinctrl: pinctrl-adi2-bf60x: remove useless and duplicated GPIO definition for PPI2.

2013-09-22 Thread Sonic Zhang
From: Sonic Zhang sonic.zh...@analog.com Signed-off-by: Sonic Zhang sonic.zh...@analog.com --- drivers/pinctrl/pinctrl-adi2-bf60x.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/pinctrl/pinctrl-adi2-bf60x.c b/drivers/pinctrl/pinctrl-adi2-bf60x.c index e90cb80

Re: [uclinux-dist-devel] [GIT PULL] Blackfin updates for 3.12

2013-09-15 Thread Sonic Zhang
Hi Linus, On Fri, Sep 13, 2013 at 10:29 PM, Linus Torvalds wrote: > On Thu, Sep 12, 2013 at 10:12 PM, Steven Miao wrote: >> arch/blackfin/mach-bf609/Kconfig | 1655 >> > > WTF? This is getting crazy. The blackfin board Kconfig files are being > used as a

Re: [uclinux-dist-devel] [GIT PULL] Blackfin updates for 3.12

2013-09-15 Thread Sonic Zhang
Hi Linus, On Fri, Sep 13, 2013 at 10:29 PM, Linus Torvalds torva...@linux-foundation.org wrote: On Thu, Sep 12, 2013 at 10:12 PM, Steven Miao real...@kernel.org wrote: arch/blackfin/mach-bf609/Kconfig | 1655 WTF? This is getting crazy. The blackfin

[PATCH 3/3 v4] blackfin: pinctrl-adi2: Enable PINCTRL framework for BF54x and BF60x.

2013-09-03 Thread Sonic Zhang
From: Sonic Zhang - Remove unused macro MAX_RESOURCES - Override blackfin legacy peripheral pinmux request and free APIs by devm_pinctrl_get_select_default() to init the peripheral portmux setting. v3-chagnes: - Move pinctrl soc data out of blackfin arch folder. Signed-off-by: Sonic Zhang

[PATCH 2/3 v4] blackfin: gpio: Remove none gpio lib code.

2013-09-03 Thread Sonic Zhang
From: Sonic Zhang - Remove non gpio lib code from blackfin architecture. - Limit the lagecy blackfin gpio driver to bf5xx processors only. - Remove unused definition of the pint power functions. Signed-off-by: Sonic Zhang Acked-by: Linus Walleij --- arch/blackfin/Kconfig| 7

[PATCH 2/3 v4] blackfin: gpio: Remove none gpio lib code.

2013-09-03 Thread Sonic Zhang
From: Sonic Zhang sonic.zh...@analog.com - Remove non gpio lib code from blackfin architecture. - Limit the lagecy blackfin gpio driver to bf5xx processors only. - Remove unused definition of the pint power functions. Signed-off-by: Sonic Zhang sonic.zh...@analog.com Acked-by: Linus Walleij

[PATCH 3/3 v4] blackfin: pinctrl-adi2: Enable PINCTRL framework for BF54x and BF60x.

2013-09-03 Thread Sonic Zhang
From: Sonic Zhang sonic.zh...@analog.com - Remove unused macro MAX_RESOURCES - Override blackfin legacy peripheral pinmux request and free APIs by devm_pinctrl_get_select_default() to init the peripheral portmux setting. v3-chagnes: - Move pinctrl soc data out of blackfin arch folder. Signed

Re: [PATCH 1/3 v3] pinctrl: ADI PIN control driver for the GPIO controller on bf54x and bf60x.

2013-08-29 Thread Sonic Zhang
Hi Linus, On Thu, Aug 29, 2013 at 4:19 PM, Linus Walleij wrote: > On Thu, Aug 22, 2013 at 10:48 PM, Stephen Warren > wrote: >> On 08/22/2013 01:07 AM, Sonic Zhang wrote: >>> >>> There are 6 to 9 GPIO HW blocks in one Blackfin SoC. Function >>> pin

Re: [PATCH 1/3 v3] pinctrl: ADI PIN control driver for the GPIO controller on bf54x and bf60x.

2013-08-29 Thread Sonic Zhang
Hi Linus, On Thu, Aug 29, 2013 at 4:12 PM, Linus Walleij wrote: > On Thu, Aug 22, 2013 at 9:07 AM, Sonic Zhang wrote: > >> There are 6 to 9 GPIO HW blocks in one Blackfin SoC. Function >> pinmux_enable_setting() in current pinctrl framework assumes the >> function mux s

Re: [PATCH 1/3 v3] pinctrl: ADI PIN control driver for the GPIO controller on bf54x and bf60x.

2013-08-29 Thread Sonic Zhang
Hi Stephen, On Wed, Aug 28, 2013 at 10:23 PM, Stephen Warren wrote: > On 08/27/2013 09:56 PM, Sonic Zhang wrote: >> Hi Stephen, >> >> On Wed, Aug 28, 2013 at 5:39 AM, Stephen Warren >> wrote: >>> On 08/27/2013 03:30 AM, Sonic Zhang wrote: >>>> Hi

Re: [PATCH 1/3 v3] pinctrl: ADI PIN control driver for the GPIO controller on bf54x and bf60x.

2013-08-29 Thread Sonic Zhang
Hi Linus, On Thu, Aug 29, 2013 at 4:02 PM, Linus Walleij wrote: > On Wed, Aug 21, 2013 at 8:30 AM, Sonic Zhang wrote: > >> From: Sonic Zhang >> >> The new ADI GPIO2 controller was introduced since the BF548 and BF60x >> processors. It differs a lot from the o

Re: [PATCH 1/3 v3] pinctrl: ADI PIN control driver for the GPIO controller on bf54x and bf60x.

2013-08-29 Thread Sonic Zhang
Hi Linus, On Thu, Aug 29, 2013 at 4:02 PM, Linus Walleij linus.wall...@linaro.org wrote: On Wed, Aug 21, 2013 at 8:30 AM, Sonic Zhang sonic@gmail.com wrote: From: Sonic Zhang sonic.zh...@analog.com The new ADI GPIO2 controller was introduced since the BF548 and BF60x processors

Re: [PATCH 1/3 v3] pinctrl: ADI PIN control driver for the GPIO controller on bf54x and bf60x.

2013-08-29 Thread Sonic Zhang
Hi Stephen, On Wed, Aug 28, 2013 at 10:23 PM, Stephen Warren swar...@wwwdotorg.org wrote: On 08/27/2013 09:56 PM, Sonic Zhang wrote: Hi Stephen, On Wed, Aug 28, 2013 at 5:39 AM, Stephen Warren swar...@wwwdotorg.org wrote: On 08/27/2013 03:30 AM, Sonic Zhang wrote: Hi Stephen, On Fri

Re: [PATCH 1/3 v3] pinctrl: ADI PIN control driver for the GPIO controller on bf54x and bf60x.

2013-08-29 Thread Sonic Zhang
Hi Linus, On Thu, Aug 29, 2013 at 4:12 PM, Linus Walleij linus.wall...@linaro.org wrote: On Thu, Aug 22, 2013 at 9:07 AM, Sonic Zhang sonic@gmail.com wrote: There are 6 to 9 GPIO HW blocks in one Blackfin SoC. Function pinmux_enable_setting() in current pinctrl framework assumes

Re: [PATCH 1/3 v3] pinctrl: ADI PIN control driver for the GPIO controller on bf54x and bf60x.

2013-08-29 Thread Sonic Zhang
Hi Linus, On Thu, Aug 29, 2013 at 4:19 PM, Linus Walleij linus.wall...@linaro.org wrote: On Thu, Aug 22, 2013 at 10:48 PM, Stephen Warren swar...@wwwdotorg.org wrote: On 08/22/2013 01:07 AM, Sonic Zhang wrote: There are 6 to 9 GPIO HW blocks in one Blackfin SoC. Function

Re: [PATCH 1/3 v3] pinctrl: ADI PIN control driver for the GPIO controller on bf54x and bf60x.

2013-08-27 Thread Sonic Zhang
Hi Stephen, On Wed, Aug 28, 2013 at 5:39 AM, Stephen Warren wrote: > On 08/27/2013 03:30 AM, Sonic Zhang wrote: >> Hi Stephen, >> >> On Fri, Aug 23, 2013 at 4:48 AM, Stephen Warren >> wrote: >>> On 08/22/2013 01:07 AM, Sonic Zhang wrote: >>>> Hi

Re: [PATCH 1/3 v3] pinctrl: ADI PIN control driver for the GPIO controller on bf54x and bf60x.

2013-08-27 Thread Sonic Zhang
Hi Stephen, On Fri, Aug 23, 2013 at 4:48 AM, Stephen Warren wrote: > On 08/22/2013 01:07 AM, Sonic Zhang wrote: >> Hi Stephen, >> >> On Thu, Aug 22, 2013 at 2:45 AM, Stephen Warren >> wrote: >>> On 08/21/2013 12:30 AM, Sonic Zhang wrote: >>>>

Re: [PATCH 1/3 v3] pinctrl: ADI PIN control driver for the GPIO controller on bf54x and bf60x.

2013-08-27 Thread Sonic Zhang
Hi Stephen, On Fri, Aug 23, 2013 at 4:48 AM, Stephen Warren swar...@wwwdotorg.org wrote: On 08/22/2013 01:07 AM, Sonic Zhang wrote: Hi Stephen, On Thu, Aug 22, 2013 at 2:45 AM, Stephen Warren swar...@wwwdotorg.org wrote: On 08/21/2013 12:30 AM, Sonic Zhang wrote: From: Sonic Zhang

Re: [PATCH 1/3 v3] pinctrl: ADI PIN control driver for the GPIO controller on bf54x and bf60x.

2013-08-27 Thread Sonic Zhang
Hi Stephen, On Wed, Aug 28, 2013 at 5:39 AM, Stephen Warren swar...@wwwdotorg.org wrote: On 08/27/2013 03:30 AM, Sonic Zhang wrote: Hi Stephen, On Fri, Aug 23, 2013 at 4:48 AM, Stephen Warren swar...@wwwdotorg.org wrote: On 08/22/2013 01:07 AM, Sonic Zhang wrote: Hi Stephen, On Thu, Aug

Re: [PATCH 1/3 v3] pinctrl: ADI PIN control driver for the GPIO controller on bf54x and bf60x.

2013-08-22 Thread Sonic Zhang
Hi Stephen, On Thu, Aug 22, 2013 at 2:45 AM, Stephen Warren wrote: > On 08/21/2013 12:30 AM, Sonic Zhang wrote: >> From: Sonic Zhang >> >> The new ADI GPIO2 controller was introduced since the BF548 and BF60x >> processors. It differs a lot from the old one on BF5x

Re: [PATCH 1/3 v3] pinctrl: ADI PIN control driver for the GPIO controller on bf54x and bf60x.

2013-08-22 Thread Sonic Zhang
Hi Stephen, On Thu, Aug 22, 2013 at 2:45 AM, Stephen Warren swar...@wwwdotorg.org wrote: On 08/21/2013 12:30 AM, Sonic Zhang wrote: From: Sonic Zhang sonic.zh...@analog.com The new ADI GPIO2 controller was introduced since the BF548 and BF60x processors. It differs a lot from the old one

[PATCH 2/3 v3] blackfin: gpio: Remove none gpio lib code.

2013-08-21 Thread Sonic Zhang
From: Sonic Zhang - Remove non gpio lib code from blackfin architecture. - Limit the lagecy blackfin gpio driver to bf5xx processors only. - Remove unused definition of the pint power functions. Signed-off-by: Sonic Zhang Acked-by: Linus Walleij --- arch/blackfin/Kconfig| 7

[PATCH 3/3 v3] blackfin: pinctrl-adi2: Enable PINCTRL framework for BF54x and BF60x.

2013-08-21 Thread Sonic Zhang
From: Sonic Zhang - Remove unused macro MAX_RESOURCES - Override blackfin legacy peripheral pinmux request and free APIs by devm_pinctrl_get_select_default() to init the peripheral portmux setting. v2-chagnes: - Move pinctrl soc data out of blackfin arch folder. Signed-off-by: Sonic Zhang

[PATCH 2/3 v3] blackfin: gpio: Remove none gpio lib code.

2013-08-21 Thread Sonic Zhang
From: Sonic Zhang - Remove non gpio lib code from blackfin architecture. - Limit the lagecy blackfin gpio driver to bf5xx processors only. - Remove unused definition of the pint power functions. Signed-off-by: Sonic Zhang --- arch/blackfin/Kconfig| 7 ++ arch/blackfin/include

[PATCH 2/3 v3] blackfin: gpio: Remove none gpio lib code.

2013-08-21 Thread Sonic Zhang
From: Sonic Zhang sonic.zh...@analog.com - Remove non gpio lib code from blackfin architecture. - Limit the lagecy blackfin gpio driver to bf5xx processors only. - Remove unused definition of the pint power functions. Signed-off-by: Sonic Zhang sonic.zh...@analog.com --- arch/blackfin/Kconfig

[PATCH 3/3 v3] blackfin: pinctrl-adi2: Enable PINCTRL framework for BF54x and BF60x.

2013-08-21 Thread Sonic Zhang
From: Sonic Zhang sonic.zh...@analog.com - Remove unused macro MAX_RESOURCES - Override blackfin legacy peripheral pinmux request and free APIs by devm_pinctrl_get_select_default() to init the peripheral portmux setting. v2-chagnes: - Move pinctrl soc data out of blackfin arch folder. Signed

[PATCH 2/3 v3] blackfin: gpio: Remove none gpio lib code.

2013-08-21 Thread Sonic Zhang
From: Sonic Zhang sonic.zh...@analog.com - Remove non gpio lib code from blackfin architecture. - Limit the lagecy blackfin gpio driver to bf5xx processors only. - Remove unused definition of the pint power functions. Signed-off-by: Sonic Zhang sonic.zh...@analog.com Acked-by: Linus Walleij

[PATCH v2] MAINTAINERS: ADI Linux development mailing lists change to the new server.

2013-08-15 Thread Sonic Zhang
From: Sonic Zhang Update Blackfin arch branch maintainer's email as well. Signed-off-by: Sonic Zhang --- MAINTAINERS | 42 -- 1 file changed, 20 insertions(+), 22 deletions(-) diff --git a/MAINTAINERS b/MAINTAINERS index bf61e04..e5229d6 100644

Re: [PATCH] MAINTAINERS: ADI Linux development mailing lists change to the new server.

2013-08-15 Thread Sonic Zhang
Hi Geert, Thanks. Sonic On Thu, Aug 15, 2013 at 3:53 PM, Geert Uytterhoeven wrote: > On Wed, Aug 14, 2013 at 8:20 AM, Sonic Zhang wrote: >> ANALOG DEVICES INC ASOC DRIVERS >> -L: uclinux-dist-de...@blackfin.uclinux.org >> +L: adi-buidroot-de...@lists.sourceforge.

Re: [PATCH] MAINTAINERS: ADI Linux development mailing lists change to the new server.

2013-08-15 Thread Sonic Zhang
Hi Geert, Thanks. Sonic On Thu, Aug 15, 2013 at 3:53 PM, Geert Uytterhoeven ge...@linux-m68k.org wrote: On Wed, Aug 14, 2013 at 8:20 AM, Sonic Zhang sonic@gmail.com wrote: ANALOG DEVICES INC ASOC DRIVERS -L: uclinux-dist-de...@blackfin.uclinux.org +L: adi-buidroot-de

[PATCH v2] MAINTAINERS: ADI Linux development mailing lists change to the new server.

2013-08-15 Thread Sonic Zhang
From: Sonic Zhang sonic.zh...@analog.com Update Blackfin arch branch maintainer's email as well. Signed-off-by: Sonic Zhang sonic.zh...@analog.com --- MAINTAINERS | 42 -- 1 file changed, 20 insertions(+), 22 deletions(-) diff --git a/MAINTAINERS b

[PATCH] MAINTAINERS: ADI Linux development mailing lists change to the new server.

2013-08-14 Thread Sonic Zhang
From: Sonic Zhang Update Blackfin arch branch maintainer's email as well. Signed-off-by: Sonic Zhang --- MAINTAINERS | 42 -- 1 file changed, 20 insertions(+), 22 deletions(-) diff --git a/MAINTAINERS b/MAINTAINERS index bf61e04..9578c2c 100644

[PATCH] MAINTAINERS: ADI Linux development mailing lists change to the new server.

2013-08-14 Thread Sonic Zhang
From: Sonic Zhang sonic.zh...@analog.com Update Blackfin arch branch maintainer's email as well. Signed-off-by: Sonic Zhang sonic.zh...@analog.com --- MAINTAINERS | 42 -- 1 file changed, 20 insertions(+), 22 deletions(-) diff --git a/MAINTAINERS b

[PATCH v4] pinctrl: pinmux: Don't free pins requested by other devices in pinmux_disable_setting.

2013-08-13 Thread Sonic Zhang
From: Sonic Zhang One peripheral may share part of its pins with the 2nd peripheral and the other pins with the 3rd. If it requests all pins when part of them has already be requested and owned by the 2nd peripheral, this request fails and pinmux_disable_setting() is called

[PATCH v4] pinctrl: pinmux: Don't free pins requested by other devices in pinmux_disable_setting.

2013-08-13 Thread Sonic Zhang
From: Sonic Zhang sonic.zh...@analog.com One peripheral may share part of its pins with the 2nd peripheral and the other pins with the 3rd. If it requests all pins when part of them has already be requested and owned by the 2nd peripheral, this request fails and pinmux_disable_setting() is called

[PATCH 3/3 v2] blackfin: pinctrl-adi2: Add pin control device groups and function data.

2013-08-12 Thread Sonic Zhang
From: Sonic Zhang Select PINCTRL_ADI2 for bf54x and bf60x by default. Signed-off-by: Sonic Zhang --- arch/blackfin/Kconfig | 7 + arch/blackfin/include/asm/portmux.h | 14 +- arch/blackfin/mach-bf548/include/mach/portmux.h | 595

[PATCH 2/3 v2] blackfin: gpio: Remove none gpio lib code.

2013-08-12 Thread Sonic Zhang
From: Sonic Zhang - Remove non gpio lib code from blackfin architecture. - Limit the lagecy blackfin gpio driver to bf5xx processors only. - Remove unused definition of the pint power functions. Signed-off-by: Sonic Zhang --- arch/blackfin/Kconfig| 7 ++ arch/blackfin/include

[PATCH 1/3 v2] pinctrl: ADI PIN control driver for the GPIO controller on bf54x and bf60x.

2013-08-12 Thread Sonic Zhang
From: Sonic Zhang The new ADI GPIO2 controller was introduced since the BF548 and BF60x processors. It differs a lot from the old one on BF5xx processors. So, create a pinctrl driver under pinctrl framework. - Define gpio ports and gpio interrupt controllers as individual platform devices

[PATCH v3] pinctrl: pinmux: Don't free pins requested by other devices in pinmux_disable_setting.

2013-08-12 Thread Sonic Zhang
From: Sonic Zhang One peripheral may share part of its pins with the 2nd peripheral and the other pins with the 3rd. If it requests all pins when part of them has already be requested and owned by the 2nd peripheral, this request fails and pinmux_disable_setting() is called

[PATCH v2] pinctrl: pinmux: Don't free pins requested by other devices in pinmux_disable_setting.

2013-08-12 Thread Sonic Zhang
From: Sonic Zhang One peripheral may share part of its pins with the 2nd peripheral and the other pins with the 3rd. If it requests all pins when part of them has already be requested and owned by the 2nd peripheral, this request fails and pinmux_disable_setting() is called

[PATCH v2] pinctrl: pinmux: Don't free pins requested by other devices in pinmux_disable_setting.

2013-08-12 Thread Sonic Zhang
From: Sonic Zhang sonic.zh...@analog.com One peripheral may share part of its pins with the 2nd peripheral and the other pins with the 3rd. If it requests all pins when part of them has already be requested and owned by the 2nd peripheral, this request fails and pinmux_disable_setting() is called

[PATCH v3] pinctrl: pinmux: Don't free pins requested by other devices in pinmux_disable_setting.

2013-08-12 Thread Sonic Zhang
From: Sonic Zhang sonic.zh...@analog.com One peripheral may share part of its pins with the 2nd peripheral and the other pins with the 3rd. If it requests all pins when part of them has already be requested and owned by the 2nd peripheral, this request fails and pinmux_disable_setting() is called

[PATCH 1/3 v2] pinctrl: ADI PIN control driver for the GPIO controller on bf54x and bf60x.

2013-08-12 Thread Sonic Zhang
From: Sonic Zhang sonic.zh...@analog.com The new ADI GPIO2 controller was introduced since the BF548 and BF60x processors. It differs a lot from the old one on BF5xx processors. So, create a pinctrl driver under pinctrl framework. - Define gpio ports and gpio interrupt controllers as individual

[PATCH 2/3 v2] blackfin: gpio: Remove none gpio lib code.

2013-08-12 Thread Sonic Zhang
From: Sonic Zhang sonic.zh...@analog.com - Remove non gpio lib code from blackfin architecture. - Limit the lagecy blackfin gpio driver to bf5xx processors only. - Remove unused definition of the pint power functions. Signed-off-by: Sonic Zhang sonic.zh...@analog.com --- arch/blackfin/Kconfig

[PATCH 3/3 v2] blackfin: pinctrl-adi2: Add pin control device groups and function data.

2013-08-12 Thread Sonic Zhang
From: Sonic Zhang sonic.zh...@analog.com Select PINCTRL_ADI2 for bf54x and bf60x by default. Signed-off-by: Sonic Zhang sonic.zh...@analog.com --- arch/blackfin/Kconfig | 7 + arch/blackfin/include/asm/portmux.h | 14 +- arch/blackfin/mach-bf548/include

Re: [PATCH] pinctrl: pinmux: Don't free pins requested by other devices

2013-08-07 Thread Sonic Zhang
Hi Linus, On Thu, Aug 8, 2013 at 12:23 AM, Linus Walleij wrote: > On Wed, Jul 17, 2013 at 7:31 AM, Sonic Zhang wrote: > > I'd like Stephen and Axel to have a look at this as well... > >> From: Sonic Zhang >> >> in pinmux_disable_setting after current device fai

Re: [PATCH] pinctrl: pinmux: Don't free pins requested by other devices

2013-08-07 Thread Sonic Zhang
Hi Stephen, On Thu, Aug 8, 2013 at 1:09 AM, Stephen Warren wrote: > On 08/07/2013 10:23 AM, Linus Walleij wrote: >> On Wed, Jul 17, 2013 at 7:31 AM, Sonic Zhang wrote: >> >> I'd like Stephen and Axel to have a look at this as well... >> >>> From: Sonic Zhan

  1   2   >