Thread (116 messages) 116 messages, 4 authors, 2012-06-22

[PATCH v5 11/14] ARM: OMAP2+: gpmc: handle connected peripherals

flat view
STALE5231d SUPERSEDED

From: Mohammed, Afzal <hidden>
Date: 2012-06-14 08:40:43
Also in: linux-omap

Revision v5 of 4 in this series.

Revisions (4)
  1. v5
  2. v5
  3. v5 current
  4. v6 [diff vs current]
Hi Jon,

On Wed, Jun 13, 2012 at 21:01:08, Hunter, Jon wrote:
On 06/11/2012 09:27 AM, Afzal Mohammed wrote:
quoted
+static __devinit int gpmc_setup_cs(struct gpmc_peripheral *g_per,
+				struct gpmc_cs_data *cs, struct resource *res) {
+	int num, ret;
+
+	num = gpmc_setup_cs_mem(cs, res);
+	if (IS_ERR_VALUE(num))
+		return num;
+
+	ret = gpmc_setup_cs_config_timing(g_per, cs);
+	if (IS_ERR_VALUE(ret))
+		return ret;
+
+	num += gpmc_setup_cs_irq(cs, res + num);

What happens if the above function returns an error?
That has been deliberately done as that implies user has not
specified irq flag & hence is not an error.

But it seems, I should print error information if user tries to use an
interrupt and if driver does not want/unable to use gpmc interrupt.

Regards
Afzal
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help