Re: [PATCH 02/14] ALSA: portman2x4: return proper error values from attach

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello.

On 4/8/2015 2:20 PM, Sudip Mukherjee wrote:

now that we are monitoring the return value from attach, make the

So you've first changed the method prototype and follow up with the changes to the actual implementations? That's backward. I'm afraid such changes can't be done piecemeal.

required changes to return proper value from its attach function.

Signed-off-by: Sudip Mukherjee <sudip@xxxxxxxxxxxxxxx>
---
  sound/drivers/portman2x4.c | 15 ++++++++++-----
  1 file changed, 10 insertions(+), 5 deletions(-)

diff --git a/sound/drivers/portman2x4.c b/sound/drivers/portman2x4.c
index 464385a..866adbb 100644
--- a/sound/drivers/portman2x4.c
+++ b/sound/drivers/portman2x4.c
@@ -672,32 +672,37 @@ static int snd_portman_probe_port(struct parport *p)
  	return res ? -EIO : 0;
  }

-static void snd_portman_attach(struct parport *p)
+static int snd_portman_attach(struct parport *p)
  {
  	struct platform_device *device;
+	int ret;

  	device = platform_device_alloc(PLATFORM_DRIVER, device_count);
  	if (!device)
-		return;
+		return -ENOMEM;

  	/* Temporary assignment to forward the parport */
  	platform_set_drvdata(device, p);

-	if (platform_device_add(device) < 0) {
+	ret = platform_device_add(device);
+

   I don't think empty line is needed here.

+	if (ret < 0) {
  		platform_device_put(device);
-		return;
+		return ret;
  	}

[...]

WBR, Sergei

--
To unsubscribe from this list: send the line "unsubscribe linux-spi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Kernel]     [Linux ARM (vger)]     [Linux ARM MSM]     [Linux Omap]     [Linux Arm]     [Linux Tegra]     [Fedora ARM]     [Linux for Samsung SOC]     [eCos]     [Linux Fastboot]     [Gcc Help]     [Git]     [DCCP]     [IETF Announce]     [Security]     [Linux MIPS]     [Yosemite Campsites]

  Powered by Linux