linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 1/3] pinctrl: Move pint PM storage structure out of blackfin architecture.
@ 2014-01-27 10:23 Sonic Zhang
  2014-01-27 10:23 ` [PATCH 2/3] pinctrl-adi2: change irq_base from usigned int to int Sonic Zhang
                   ` (2 more replies)
  0 siblings, 3 replies; 6+ messages in thread
From: Sonic Zhang @ 2014-01-27 10:23 UTC (permalink / raw)
  To: Linus Walleij, Steven Miao; +Cc: LKML, adi-buildroot-devel, Sonic Zhang

From: Sonic Zhang <sonic.zhang@analog.com>

It is better to keep this structure in the pinctrl-adi2 driver.

Signed-off-by: Sonic Zhang <sonic.zhang@analog.com>
---
 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/include/asm/irq.h b/arch/blackfin/include/asm/irq.h
index 2fd04f1..89de539 100644
--- a/arch/blackfin/include/asm/irq.h
+++ b/arch/blackfin/include/asm/irq.h
@@ -20,15 +20,6 @@
 /* SYS_IRQS and NR_IRQS are defined in <mach-bf5xx/irq.h> */
 #include <mach/irq.h>
 
-/*
- * pm save bfin pint registers
- */
-struct adi_pm_pint_save {
-	u32 assign;
-	u32 edge_set;
-	u32 invert_set;
-};
-
 #if ANOMALY_05000244 && defined(CONFIG_BFIN_ICACHE)
 # define NOP_PAD_ANOMALY_05000244 "nop; nop;"
 #else
diff --git a/drivers/pinctrl/pinctrl-adi2.c b/drivers/pinctrl/pinctrl-adi2.c
index 7a39562..e8120fa 100644
--- a/drivers/pinctrl/pinctrl-adi2.c
+++ b/drivers/pinctrl/pinctrl-adi2.c
@@ -89,6 +89,19 @@ struct gpio_port_saved {
 	u32 mux;
 };
 
+/*
+ * struct gpio_pint_saved - PINT registers saved in PM operations
+ *
+ * @assign: ASSIGN register
+ * @edge_set: EDGE_SET register
+ * @invert_set: INVERT_SET register
+ */
+struct gpio_pint_saved {
+	u32 assign;
+	u32 edge_set;
+	u32 invert_set;
+};
+
 /**
  * struct gpio_pint - Pin interrupt controller device. Multiple ADI GPIO
  * banks can be mapped into one Pin interrupt controller.
@@ -114,7 +127,7 @@ struct gpio_pint {
 	int irq;
 	struct irq_domain *domain[2];
 	struct gpio_pint_regs *regs;
-	struct adi_pm_pint_save saved_data;
+	struct gpio_pint_saved saved_data;
 	int map_count;
 	spinlock_t lock;
 
-- 
1.8.2.3



^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [PATCH 2/3] pinctrl-adi2: change irq_base from usigned int to int
  2014-01-27 10:23 [PATCH 1/3] pinctrl: Move pint PM storage structure out of blackfin architecture Sonic Zhang
@ 2014-01-27 10:23 ` Sonic Zhang
  2014-01-31 11:36   ` Linus Walleij
  2014-01-27 10:23 ` [PATCH 3/3] pinctrl-adi2: fix coding style issue Sonic Zhang
  2014-01-31 11:35 ` [PATCH 1/3] pinctrl: Move pint PM storage structure out of blackfin architecture Linus Walleij
  2 siblings, 1 reply; 6+ messages in thread
From: Sonic Zhang @ 2014-01-27 10:23 UTC (permalink / raw)
  To: Linus Walleij, Steven Miao; +Cc: LKML, adi-buildroot-devel, Sonic Zhang

From: Sonic Zhang <sonic.zhang@analog.com>

Negative irq_base means this gpio port doens't support interrupts.

Signed-off-by: Sonic Zhang <sonic.zhang@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 e8120fa..9fb53c9 100644
--- a/drivers/pinctrl/pinctrl-adi2.c
+++ b/drivers/pinctrl/pinctrl-adi2.c
@@ -173,7 +173,7 @@ struct adi_pinctrl {
 struct gpio_port {
 	struct list_head node;
 	void __iomem *base;
-	unsigned int irq_base;
+	int irq_base;
 	unsigned int width;
 	struct gpio_port_t *regs;
 	struct gpio_port_saved saved_data;
-- 
1.8.2.3



^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [PATCH 3/3] pinctrl-adi2: fix coding style issue
  2014-01-27 10:23 [PATCH 1/3] pinctrl: Move pint PM storage structure out of blackfin architecture Sonic Zhang
  2014-01-27 10:23 ` [PATCH 2/3] pinctrl-adi2: change irq_base from usigned int to int Sonic Zhang
@ 2014-01-27 10:23 ` Sonic Zhang
  2014-01-31 11:37   ` Linus Walleij
  2014-01-31 11:35 ` [PATCH 1/3] pinctrl: Move pint PM storage structure out of blackfin architecture Linus Walleij
  2 siblings, 1 reply; 6+ messages in thread
From: Sonic Zhang @ 2014-01-27 10:23 UTC (permalink / raw)
  To: Linus Walleij, Steven Miao; +Cc: LKML, adi-buildroot-devel, Sonic Zhang

From: Sonic Zhang <sonic.zhang@analog.com>

Signed-off-by: Sonic Zhang <sonic.zhang@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
--- a/drivers/pinctrl/pinctrl-adi2.c
+++ b/drivers/pinctrl/pinctrl-adi2.c
@@ -641,7 +641,7 @@ static int adi_pinmux_enable(struct pinctrl_dev *pctldev, unsigned selector,
 		spin_lock_irqsave(&port->lock, flags);
 
 		portmux_setup(port, pin_to_offset(range, pin),
-				 P_FUNCT2MUX(*mux));
+				P_FUNCT2MUX(*mux));
 		port_setup(port, pin_to_offset(range, pin), false);
 		mux++;
 
-- 
1.8.2.3



^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH 1/3] pinctrl: Move pint PM storage structure out of blackfin architecture.
  2014-01-27 10:23 [PATCH 1/3] pinctrl: Move pint PM storage structure out of blackfin architecture Sonic Zhang
  2014-01-27 10:23 ` [PATCH 2/3] pinctrl-adi2: change irq_base from usigned int to int Sonic Zhang
  2014-01-27 10:23 ` [PATCH 3/3] pinctrl-adi2: fix coding style issue Sonic Zhang
@ 2014-01-31 11:35 ` Linus Walleij
  2 siblings, 0 replies; 6+ messages in thread
From: Linus Walleij @ 2014-01-31 11:35 UTC (permalink / raw)
  To: Sonic Zhang; +Cc: Steven Miao, LKML, adi-buildroot-devel, Sonic Zhang

On Mon, Jan 27, 2014 at 11:23 AM, Sonic Zhang <sonic.adi@gmail.com> wrote:

> From: Sonic Zhang <sonic.zhang@analog.com>
>
> It is better to keep this structure in the pinctrl-adi2 driver.
>
> Signed-off-by: Sonic Zhang <sonic.zhang@analog.com>

Seems like a good idea, patch applied.
(Steven, complain if any problem.)

Yours,
Linus Walleij

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 2/3] pinctrl-adi2: change irq_base from usigned int to int
  2014-01-27 10:23 ` [PATCH 2/3] pinctrl-adi2: change irq_base from usigned int to int Sonic Zhang
@ 2014-01-31 11:36   ` Linus Walleij
  0 siblings, 0 replies; 6+ messages in thread
From: Linus Walleij @ 2014-01-31 11:36 UTC (permalink / raw)
  To: Sonic Zhang; +Cc: Steven Miao, LKML, adi-buildroot-devel, Sonic Zhang

On Mon, Jan 27, 2014 at 11:23 AM, Sonic Zhang <sonic.adi@gmail.com> wrote:

> From: Sonic Zhang <sonic.zhang@analog.com>
>
> Negative irq_base means this gpio port doens't support interrupts.
>
> Signed-off-by: Sonic Zhang <sonic.zhang@analog.com>

Patch applied.

Yours,
Linus Walleij

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 3/3] pinctrl-adi2: fix coding style issue
  2014-01-27 10:23 ` [PATCH 3/3] pinctrl-adi2: fix coding style issue Sonic Zhang
@ 2014-01-31 11:37   ` Linus Walleij
  0 siblings, 0 replies; 6+ messages in thread
From: Linus Walleij @ 2014-01-31 11:37 UTC (permalink / raw)
  To: Sonic Zhang; +Cc: Steven Miao, LKML, adi-buildroot-devel, Sonic Zhang

On Mon, Jan 27, 2014 at 11:23 AM, Sonic Zhang <sonic.adi@gmail.com> wrote:

> From: Sonic Zhang <sonic.zhang@analog.com>
>
> Signed-off-by: Sonic Zhang <sonic.zhang@analog.com>

Patch applied.

Yours,
Linus Walleij

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2014-01-31 11:37 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-01-27 10:23 [PATCH 1/3] pinctrl: Move pint PM storage structure out of blackfin architecture Sonic Zhang
2014-01-27 10:23 ` [PATCH 2/3] pinctrl-adi2: change irq_base from usigned int to int Sonic Zhang
2014-01-31 11:36   ` Linus Walleij
2014-01-27 10:23 ` [PATCH 3/3] pinctrl-adi2: fix coding style issue Sonic Zhang
2014-01-31 11:37   ` Linus Walleij
2014-01-31 11:35 ` [PATCH 1/3] pinctrl: Move pint PM storage structure out of blackfin architecture Linus Walleij

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).