All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 00/28] media: don't print error when allocating urb fails
@ 2016-08-11 21:03 Wolfram Sang
  2016-08-11 21:03 ` [PATCH 01/28] media: dvb-frontends: rtl2832_sdr: " Wolfram Sang
                   ` (28 more replies)
  0 siblings, 29 replies; 30+ messages in thread
From: Wolfram Sang @ 2016-08-11 21:03 UTC (permalink / raw)
  To: linux-usb; +Cc: Wolfram Sang, linux-media

This per-subsystem series is part of a tree wide cleanup. usb_alloc_urb() uses
kmalloc which already prints enough information on failure. So, let's simply
remove those "allocation failed" messages from drivers like we did already for
other -ENOMEM cases. gkh acked this approach when we talked about it at LCJ in
Tokyo a few weeks ago.


Wolfram Sang (28):
  media: dvb-frontends: rtl2832_sdr: don't print error when allocating
    urb fails
  media: radio: si470x: radio-si470x-usb: don't print error when
    allocating urb fails
  media: rc: imon: don't print error when allocating urb fails
  media: rc: redrat3: don't print error when allocating urb fails
  media: usb: airspy: airspy: don't print error when allocating urb
    fails
  media: usb: as102: as102_usb_drv: don't print error when allocating
    urb fails
  media: usb: au0828: au0828-video: don't print error when allocating
    urb fails
  media: usb: cpia2: cpia2_usb: don't print error when allocating urb
    fails
  media: usb: cx231xx: cx231xx-audio: don't print error when allocating
    urb fails
  media: usb: cx231xx: cx231xx-core: don't print error when allocating
    urb fails
  media: usb: cx231xx: cx231xx-vbi: don't print error when allocating
    urb fails
  media: usb: dvb-usb: dib0700_core: don't print error when allocating
    urb fails
  media: usb: em28xx: em28xx-audio: don't print error when allocating
    urb fails
  media: usb: em28xx: em28xx-core: don't print error when allocating urb
    fails
  media: usb: gspca: benq: don't print error when allocating urb fails
  media: usb: gspca: gspca: don't print error when allocating urb fails
  media: usb: gspca: konica: don't print error when allocating urb fails
  media: usb: hackrf: hackrf: don't print error when allocating urb
    fails
  media: usb: hdpvr: hdpvr-video: don't print error when allocating urb
    fails
  media: usb: msi2500: msi2500: don't print error when allocating urb
    fails
  media: usb: pwc: pwc-if: don't print error when allocating urb fails
  media: usb: s2255: s2255drv: don't print error when allocating urb
    fails
  media: usb: stk1160: stk1160-video: don't print error when allocating
    urb fails
  media: usb: stkwebcam: stk-webcam: don't print error when allocating
    urb fails
  media: usb: tm6000: tm6000-dvb: don't print error when allocating urb
    fails
  media: usb: tm6000: tm6000-video: don't print error when allocating
    urb fails
  media: usb: usbvision: usbvision-core: don't print error when
    allocating urb fails
  media: usb: zr364xx: zr364xx: don't print error when allocating urb
    fails

 drivers/media/dvb-frontends/rtl2832_sdr.c     |  1 -
 drivers/media/radio/si470x/radio-si470x-usb.c |  1 -
 drivers/media/rc/imon.c                       | 13 +++----------
 drivers/media/rc/redrat3.c                    |  4 +---
 drivers/media/usb/airspy/airspy.c             |  1 -
 drivers/media/usb/as102/as102_usb_drv.c       |  2 --
 drivers/media/usb/au0828/au0828-video.c       |  1 -
 drivers/media/usb/cpia2/cpia2_usb.c           |  1 -
 drivers/media/usb/cx231xx/cx231xx-audio.c     |  2 --
 drivers/media/usb/cx231xx/cx231xx-core.c      |  4 ----
 drivers/media/usb/cx231xx/cx231xx-vbi.c       |  2 --
 drivers/media/usb/dvb-usb/dib0700_core.c      |  4 +---
 drivers/media/usb/em28xx/em28xx-audio.c       |  1 -
 drivers/media/usb/em28xx/em28xx-core.c        |  1 -
 drivers/media/usb/gspca/benq.c                |  4 +---
 drivers/media/usb/gspca/gspca.c               |  4 +---
 drivers/media/usb/gspca/konica.c              |  4 +---
 drivers/media/usb/hackrf/hackrf.c             |  1 -
 drivers/media/usb/hdpvr/hdpvr-video.c         |  4 +---
 drivers/media/usb/msi2500/msi2500.c           |  1 -
 drivers/media/usb/pwc/pwc-if.c                |  1 -
 drivers/media/usb/s2255/s2255drv.c            |  9 ++-------
 drivers/media/usb/stk1160/stk1160-video.c     |  4 +---
 drivers/media/usb/stkwebcam/stk-webcam.c      |  4 +---
 drivers/media/usb/tm6000/tm6000-dvb.c         |  4 +---
 drivers/media/usb/tm6000/tm6000-video.c       |  1 -
 drivers/media/usb/usbvision/usbvision-core.c  |  5 +----
 drivers/media/usb/zr364xx/zr364xx.c           |  4 +---
 28 files changed, 16 insertions(+), 72 deletions(-)

-- 
2.8.1


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

* [PATCH 01/28] media: dvb-frontends: rtl2832_sdr: don't print error when allocating urb fails
  2016-08-11 21:03 [PATCH 00/28] media: don't print error when allocating urb fails Wolfram Sang
@ 2016-08-11 21:03 ` Wolfram Sang
  2016-08-11 21:03 ` [PATCH 02/28] media: radio: si470x: radio-si470x-usb: " Wolfram Sang
                   ` (27 subsequent siblings)
  28 siblings, 0 replies; 30+ messages in thread
From: Wolfram Sang @ 2016-08-11 21:03 UTC (permalink / raw)
  To: linux-usb
  Cc: Wolfram Sang, Antti Palosaari, Mauro Carvalho Chehab, linux-media

kmalloc will print enough information in case of failure.

Signed-off-by: Wolfram Sang <wsa-dev@sang-engineering.com>
---
 drivers/media/dvb-frontends/rtl2832_sdr.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/media/dvb-frontends/rtl2832_sdr.c b/drivers/media/dvb-frontends/rtl2832_sdr.c
index 6e22af36b637b7..e038e886731b13 100644
--- a/drivers/media/dvb-frontends/rtl2832_sdr.c
+++ b/drivers/media/dvb-frontends/rtl2832_sdr.c
@@ -392,7 +392,6 @@ static int rtl2832_sdr_alloc_urbs(struct rtl2832_sdr_dev *dev)
 		dev_dbg(&pdev->dev, "alloc urb=%d\n", i);
 		dev->urb_list[i] = usb_alloc_urb(0, GFP_ATOMIC);
 		if (!dev->urb_list[i]) {
-			dev_dbg(&pdev->dev, "failed\n");
 			for (j = 0; j < i; j++)
 				usb_free_urb(dev->urb_list[j]);
 			return -ENOMEM;
-- 
2.8.1


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

* [PATCH 02/28] media: radio: si470x: radio-si470x-usb: don't print error when allocating urb fails
  2016-08-11 21:03 [PATCH 00/28] media: don't print error when allocating urb fails Wolfram Sang
  2016-08-11 21:03 ` [PATCH 01/28] media: dvb-frontends: rtl2832_sdr: " Wolfram Sang
@ 2016-08-11 21:03 ` Wolfram Sang
  2016-08-11 21:03 ` [PATCH 03/28] media: rc: imon: " Wolfram Sang
                   ` (26 subsequent siblings)
  28 siblings, 0 replies; 30+ messages in thread
From: Wolfram Sang @ 2016-08-11 21:03 UTC (permalink / raw)
  To: linux-usb; +Cc: Wolfram Sang, Hans Verkuil, Mauro Carvalho Chehab, linux-media

kmalloc will print enough information in case of failure.

Signed-off-by: Wolfram Sang <wsa-dev@sang-engineering.com>
---
 drivers/media/radio/si470x/radio-si470x-usb.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/media/radio/si470x/radio-si470x-usb.c b/drivers/media/radio/si470x/radio-si470x-usb.c
index 091d793f658361..4b132c29f2900c 100644
--- a/drivers/media/radio/si470x/radio-si470x-usb.c
+++ b/drivers/media/radio/si470x/radio-si470x-usb.c
@@ -627,7 +627,6 @@ static int si470x_usb_driver_probe(struct usb_interface *intf,
 
 	radio->int_in_urb = usb_alloc_urb(0, GFP_KERNEL);
 	if (!radio->int_in_urb) {
-		dev_info(&intf->dev, "could not allocate int_in_urb");
 		retval = -ENOMEM;
 		goto err_intbuffer;
 	}
-- 
2.8.1


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

* [PATCH 03/28] media: rc: imon: don't print error when allocating urb fails
  2016-08-11 21:03 [PATCH 00/28] media: don't print error when allocating urb fails Wolfram Sang
  2016-08-11 21:03 ` [PATCH 01/28] media: dvb-frontends: rtl2832_sdr: " Wolfram Sang
  2016-08-11 21:03 ` [PATCH 02/28] media: radio: si470x: radio-si470x-usb: " Wolfram Sang
@ 2016-08-11 21:03 ` Wolfram Sang
  2016-08-11 21:03 ` [PATCH 04/28] media: rc: redrat3: " Wolfram Sang
                   ` (25 subsequent siblings)
  28 siblings, 0 replies; 30+ messages in thread
From: Wolfram Sang @ 2016-08-11 21:03 UTC (permalink / raw)
  To: linux-usb; +Cc: Wolfram Sang, Mauro Carvalho Chehab, linux-media

kmalloc will print enough information in case of failure.

Signed-off-by: Wolfram Sang <wsa-dev@sang-engineering.com>
---
 drivers/media/rc/imon.c | 13 +++----------
 1 file changed, 3 insertions(+), 10 deletions(-)

diff --git a/drivers/media/rc/imon.c b/drivers/media/rc/imon.c
index 65f80b8b9f7ab9..86cc70fe25348f 100644
--- a/drivers/media/rc/imon.c
+++ b/drivers/media/rc/imon.c
@@ -2211,16 +2211,11 @@ static struct imon_context *imon_init_intf0(struct usb_interface *intf,
 		goto exit;
 	}
 	rx_urb = usb_alloc_urb(0, GFP_KERNEL);
-	if (!rx_urb) {
-		dev_err(dev, "%s: usb_alloc_urb failed for IR urb", __func__);
+	if (!rx_urb)
 		goto rx_urb_alloc_failed;
-	}
 	tx_urb = usb_alloc_urb(0, GFP_KERNEL);
-	if (!tx_urb) {
-		dev_err(dev, "%s: usb_alloc_urb failed for display urb",
-			__func__);
+	if (!tx_urb)
 		goto tx_urb_alloc_failed;
-	}
 
 	mutex_init(&ictx->lock);
 	spin_lock_init(&ictx->kc_lock);
@@ -2305,10 +2300,8 @@ static struct imon_context *imon_init_intf1(struct usb_interface *intf,
 	int ret = -ENOMEM;
 
 	rx_urb = usb_alloc_urb(0, GFP_KERNEL);
-	if (!rx_urb) {
-		pr_err("usb_alloc_urb failed for IR urb\n");
+	if (!rx_urb)
 		goto rx_urb_alloc_failed;
-	}
 
 	mutex_lock(&ictx->lock);
 
-- 
2.8.1


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

* [PATCH 04/28] media: rc: redrat3: don't print error when allocating urb fails
  2016-08-11 21:03 [PATCH 00/28] media: don't print error when allocating urb fails Wolfram Sang
                   ` (2 preceding siblings ...)
  2016-08-11 21:03 ` [PATCH 03/28] media: rc: imon: " Wolfram Sang
@ 2016-08-11 21:03 ` Wolfram Sang
  2016-08-11 21:03 ` [PATCH 05/28] media: usb: airspy: airspy: " Wolfram Sang
                   ` (24 subsequent siblings)
  28 siblings, 0 replies; 30+ messages in thread
From: Wolfram Sang @ 2016-08-11 21:03 UTC (permalink / raw)
  To: linux-usb; +Cc: Wolfram Sang, Mauro Carvalho Chehab, linux-media

kmalloc will print enough information in case of failure.

Signed-off-by: Wolfram Sang <wsa-dev@sang-engineering.com>
---
 drivers/media/rc/redrat3.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/media/rc/redrat3.c b/drivers/media/rc/redrat3.c
index 399f44d89a2986..ec8016d9b00907 100644
--- a/drivers/media/rc/redrat3.c
+++ b/drivers/media/rc/redrat3.c
@@ -970,10 +970,8 @@ static int redrat3_dev_probe(struct usb_interface *intf,
 
 	/* set up bulk-in endpoint */
 	rr3->read_urb = usb_alloc_urb(0, GFP_KERNEL);
-	if (!rr3->read_urb) {
-		dev_err(dev, "Read urb allocation failure\n");
+	if (!rr3->read_urb)
 		goto error;
-	}
 
 	rr3->ep_in = ep_in;
 	rr3->bulk_in_buf = usb_alloc_coherent(udev,
-- 
2.8.1


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

* [PATCH 05/28] media: usb: airspy: airspy: don't print error when allocating urb fails
  2016-08-11 21:03 [PATCH 00/28] media: don't print error when allocating urb fails Wolfram Sang
                   ` (3 preceding siblings ...)
  2016-08-11 21:03 ` [PATCH 04/28] media: rc: redrat3: " Wolfram Sang
@ 2016-08-11 21:03 ` Wolfram Sang
  2016-08-11 21:03 ` [PATCH 06/28] media: usb: as102: as102_usb_drv: " Wolfram Sang
                   ` (23 subsequent siblings)
  28 siblings, 0 replies; 30+ messages in thread
From: Wolfram Sang @ 2016-08-11 21:03 UTC (permalink / raw)
  To: linux-usb
  Cc: Wolfram Sang, Antti Palosaari, Mauro Carvalho Chehab, linux-media

kmalloc will print enough information in case of failure.

Signed-off-by: Wolfram Sang <wsa-dev@sang-engineering.com>
---
 drivers/media/usb/airspy/airspy.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/media/usb/airspy/airspy.c b/drivers/media/usb/airspy/airspy.c
index fe031b06935fbb..3c556ee306cd33 100644
--- a/drivers/media/usb/airspy/airspy.c
+++ b/drivers/media/usb/airspy/airspy.c
@@ -426,7 +426,6 @@ static int airspy_alloc_urbs(struct airspy *s)
 		dev_dbg(s->dev, "alloc urb=%d\n", i);
 		s->urb_list[i] = usb_alloc_urb(0, GFP_ATOMIC);
 		if (!s->urb_list[i]) {
-			dev_dbg(s->dev, "failed\n");
 			for (j = 0; j < i; j++)
 				usb_free_urb(s->urb_list[j]);
 			return -ENOMEM;
-- 
2.8.1


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

* [PATCH 06/28] media: usb: as102: as102_usb_drv: don't print error when allocating urb fails
  2016-08-11 21:03 [PATCH 00/28] media: don't print error when allocating urb fails Wolfram Sang
                   ` (4 preceding siblings ...)
  2016-08-11 21:03 ` [PATCH 05/28] media: usb: airspy: airspy: " Wolfram Sang
@ 2016-08-11 21:03 ` Wolfram Sang
  2016-08-11 21:03 ` [PATCH 07/28] media: usb: au0828: au0828-video: " Wolfram Sang
                   ` (22 subsequent siblings)
  28 siblings, 0 replies; 30+ messages in thread
From: Wolfram Sang @ 2016-08-11 21:03 UTC (permalink / raw)
  To: linux-usb; +Cc: Wolfram Sang, Mauro Carvalho Chehab, linux-media

kmalloc will print enough information in case of failure.

Signed-off-by: Wolfram Sang <wsa-dev@sang-engineering.com>
---
 drivers/media/usb/as102/as102_usb_drv.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/drivers/media/usb/as102/as102_usb_drv.c b/drivers/media/usb/as102/as102_usb_drv.c
index 0e8030c071b8e7..68c3a80ce349b7 100644
--- a/drivers/media/usb/as102/as102_usb_drv.c
+++ b/drivers/media/usb/as102/as102_usb_drv.c
@@ -270,8 +270,6 @@ static int as102_alloc_usb_stream_buffer(struct as102_dev_t *dev)
 
 		urb = usb_alloc_urb(0, GFP_ATOMIC);
 		if (urb == NULL) {
-			dev_dbg(&dev->bus_adap.usb_dev->dev,
-				"%s: usb_alloc_urb failed\n", __func__);
 			as102_free_usb_stream_buffer(dev);
 			return -ENOMEM;
 		}
-- 
2.8.1


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

* [PATCH 07/28] media: usb: au0828: au0828-video: don't print error when allocating urb fails
  2016-08-11 21:03 [PATCH 00/28] media: don't print error when allocating urb fails Wolfram Sang
                   ` (5 preceding siblings ...)
  2016-08-11 21:03 ` [PATCH 06/28] media: usb: as102: as102_usb_drv: " Wolfram Sang
@ 2016-08-11 21:03 ` Wolfram Sang
  2016-08-11 21:03 ` [PATCH 08/28] media: usb: cpia2: cpia2_usb: " Wolfram Sang
                   ` (21 subsequent siblings)
  28 siblings, 0 replies; 30+ messages in thread
From: Wolfram Sang @ 2016-08-11 21:03 UTC (permalink / raw)
  To: linux-usb; +Cc: Wolfram Sang, Mauro Carvalho Chehab, linux-media

kmalloc will print enough information in case of failure.

Signed-off-by: Wolfram Sang <wsa-dev@sang-engineering.com>
---
 drivers/media/usb/au0828/au0828-video.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/media/usb/au0828/au0828-video.c b/drivers/media/usb/au0828/au0828-video.c
index 82b02698586835..13b8387082f277 100644
--- a/drivers/media/usb/au0828/au0828-video.c
+++ b/drivers/media/usb/au0828/au0828-video.c
@@ -245,7 +245,6 @@ static int au0828_init_isoc(struct au0828_dev *dev, int max_packets,
 	for (i = 0; i < dev->isoc_ctl.num_bufs; i++) {
 		urb = usb_alloc_urb(max_packets, GFP_KERNEL);
 		if (!urb) {
-			au0828_isocdbg("cannot alloc isoc_ctl.urb %i\n", i);
 			au0828_uninit_isoc(dev);
 			return -ENOMEM;
 		}
-- 
2.8.1


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

* [PATCH 08/28] media: usb: cpia2: cpia2_usb: don't print error when allocating urb fails
  2016-08-11 21:03 [PATCH 00/28] media: don't print error when allocating urb fails Wolfram Sang
                   ` (6 preceding siblings ...)
  2016-08-11 21:03 ` [PATCH 07/28] media: usb: au0828: au0828-video: " Wolfram Sang
@ 2016-08-11 21:03 ` Wolfram Sang
  2016-08-11 21:03 ` [PATCH 09/28] media: usb: cx231xx: cx231xx-audio: " Wolfram Sang
                   ` (20 subsequent siblings)
  28 siblings, 0 replies; 30+ messages in thread
From: Wolfram Sang @ 2016-08-11 21:03 UTC (permalink / raw)
  To: linux-usb; +Cc: Wolfram Sang, Mauro Carvalho Chehab, linux-media

kmalloc will print enough information in case of failure.

Signed-off-by: Wolfram Sang <wsa-dev@sang-engineering.com>
---
 drivers/media/usb/cpia2/cpia2_usb.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/media/usb/cpia2/cpia2_usb.c b/drivers/media/usb/cpia2/cpia2_usb.c
index c1aa1ab2ece9ff..13620cdf05996f 100644
--- a/drivers/media/usb/cpia2/cpia2_usb.c
+++ b/drivers/media/usb/cpia2/cpia2_usb.c
@@ -662,7 +662,6 @@ static int submit_urbs(struct camera_data *cam)
 		}
 		urb = usb_alloc_urb(FRAMES_PER_DESC, GFP_KERNEL);
 		if (!urb) {
-			ERR("%s: usb_alloc_urb error!\n", __func__);
 			for (j = 0; j < i; j++)
 				usb_free_urb(cam->sbuf[j].urb);
 			return -ENOMEM;
-- 
2.8.1


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

* [PATCH 09/28] media: usb: cx231xx: cx231xx-audio: don't print error when allocating urb fails
  2016-08-11 21:03 [PATCH 00/28] media: don't print error when allocating urb fails Wolfram Sang
                   ` (7 preceding siblings ...)
  2016-08-11 21:03 ` [PATCH 08/28] media: usb: cpia2: cpia2_usb: " Wolfram Sang
@ 2016-08-11 21:03 ` Wolfram Sang
  2016-08-11 21:03 ` [PATCH 10/28] media: usb: cx231xx: cx231xx-core: " Wolfram Sang
                   ` (19 subsequent siblings)
  28 siblings, 0 replies; 30+ messages in thread
From: Wolfram Sang @ 2016-08-11 21:03 UTC (permalink / raw)
  To: linux-usb; +Cc: Wolfram Sang, Mauro Carvalho Chehab, linux-media

kmalloc will print enough information in case of failure.

Signed-off-by: Wolfram Sang <wsa-dev@sang-engineering.com>
---
 drivers/media/usb/cx231xx/cx231xx-audio.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/drivers/media/usb/cx231xx/cx231xx-audio.c b/drivers/media/usb/cx231xx/cx231xx-audio.c
index a6a9508418f8ee..4cd5fa91612f62 100644
--- a/drivers/media/usb/cx231xx/cx231xx-audio.c
+++ b/drivers/media/usb/cx231xx/cx231xx-audio.c
@@ -293,7 +293,6 @@ static int cx231xx_init_audio_isoc(struct cx231xx *dev)
 		memset(dev->adev.transfer_buffer[i], 0x80, sb_size);
 		urb = usb_alloc_urb(CX231XX_ISO_NUM_AUDIO_PACKETS, GFP_ATOMIC);
 		if (!urb) {
-			dev_err(dev->dev, "usb_alloc_urb failed!\n");
 			for (j = 0; j < i; j++) {
 				usb_free_urb(dev->adev.urb[j]);
 				kfree(dev->adev.transfer_buffer[j]);
@@ -355,7 +354,6 @@ static int cx231xx_init_audio_bulk(struct cx231xx *dev)
 		memset(dev->adev.transfer_buffer[i], 0x80, sb_size);
 		urb = usb_alloc_urb(CX231XX_NUM_AUDIO_PACKETS, GFP_ATOMIC);
 		if (!urb) {
-			dev_err(dev->dev, "usb_alloc_urb failed!\n");
 			for (j = 0; j < i; j++) {
 				usb_free_urb(dev->adev.urb[j]);
 				kfree(dev->adev.transfer_buffer[j]);
-- 
2.8.1


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

* [PATCH 10/28] media: usb: cx231xx: cx231xx-core: don't print error when allocating urb fails
  2016-08-11 21:03 [PATCH 00/28] media: don't print error when allocating urb fails Wolfram Sang
                   ` (8 preceding siblings ...)
  2016-08-11 21:03 ` [PATCH 09/28] media: usb: cx231xx: cx231xx-audio: " Wolfram Sang
@ 2016-08-11 21:03 ` Wolfram Sang
  2016-08-11 21:03 ` [PATCH 11/28] media: usb: cx231xx: cx231xx-vbi: " Wolfram Sang
                   ` (18 subsequent siblings)
  28 siblings, 0 replies; 30+ messages in thread
From: Wolfram Sang @ 2016-08-11 21:03 UTC (permalink / raw)
  To: linux-usb; +Cc: Wolfram Sang, Mauro Carvalho Chehab, linux-media

kmalloc will print enough information in case of failure.

Signed-off-by: Wolfram Sang <wsa-dev@sang-engineering.com>
---
 drivers/media/usb/cx231xx/cx231xx-core.c | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/drivers/media/usb/cx231xx/cx231xx-core.c b/drivers/media/usb/cx231xx/cx231xx-core.c
index 630f4fc5155f12..8ec05cb306d89a 100644
--- a/drivers/media/usb/cx231xx/cx231xx-core.c
+++ b/drivers/media/usb/cx231xx/cx231xx-core.c
@@ -1035,8 +1035,6 @@ int cx231xx_init_isoc(struct cx231xx *dev, int max_packets,
 	for (i = 0; i < dev->video_mode.isoc_ctl.num_bufs; i++) {
 		urb = usb_alloc_urb(max_packets, GFP_KERNEL);
 		if (!urb) {
-			dev_err(dev->dev,
-				"cannot alloc isoc_ctl.urb %i\n", i);
 			cx231xx_uninit_isoc(dev);
 			return -ENOMEM;
 		}
@@ -1172,8 +1170,6 @@ int cx231xx_init_bulk(struct cx231xx *dev, int max_packets,
 	for (i = 0; i < dev->video_mode.bulk_ctl.num_bufs; i++) {
 		urb = usb_alloc_urb(0, GFP_KERNEL);
 		if (!urb) {
-			dev_err(dev->dev,
-				"cannot alloc bulk_ctl.urb %i\n", i);
 			cx231xx_uninit_bulk(dev);
 			return -ENOMEM;
 		}
-- 
2.8.1


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

* [PATCH 11/28] media: usb: cx231xx: cx231xx-vbi: don't print error when allocating urb fails
  2016-08-11 21:03 [PATCH 00/28] media: don't print error when allocating urb fails Wolfram Sang
                   ` (9 preceding siblings ...)
  2016-08-11 21:03 ` [PATCH 10/28] media: usb: cx231xx: cx231xx-core: " Wolfram Sang
@ 2016-08-11 21:03 ` Wolfram Sang
  2016-08-11 21:03 ` [PATCH 12/28] media: usb: dvb-usb: dib0700_core: " Wolfram Sang
                   ` (17 subsequent siblings)
  28 siblings, 0 replies; 30+ messages in thread
From: Wolfram Sang @ 2016-08-11 21:03 UTC (permalink / raw)
  To: linux-usb; +Cc: Wolfram Sang, Mauro Carvalho Chehab, linux-media

kmalloc will print enough information in case of failure.

Signed-off-by: Wolfram Sang <wsa-dev@sang-engineering.com>
---
 drivers/media/usb/cx231xx/cx231xx-vbi.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/drivers/media/usb/cx231xx/cx231xx-vbi.c b/drivers/media/usb/cx231xx/cx231xx-vbi.c
index 15bb573b78ac82..76e901920f6f9b 100644
--- a/drivers/media/usb/cx231xx/cx231xx-vbi.c
+++ b/drivers/media/usb/cx231xx/cx231xx-vbi.c
@@ -442,8 +442,6 @@ int cx231xx_init_vbi_isoc(struct cx231xx *dev, int max_packets,
 
 		urb = usb_alloc_urb(0, GFP_KERNEL);
 		if (!urb) {
-			dev_err(dev->dev,
-				"cannot alloc bulk_ctl.urb %i\n", i);
 			cx231xx_uninit_vbi_isoc(dev);
 			return -ENOMEM;
 		}
-- 
2.8.1


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

* [PATCH 12/28] media: usb: dvb-usb: dib0700_core: don't print error when allocating urb fails
  2016-08-11 21:03 [PATCH 00/28] media: don't print error when allocating urb fails Wolfram Sang
                   ` (10 preceding siblings ...)
  2016-08-11 21:03 ` [PATCH 11/28] media: usb: cx231xx: cx231xx-vbi: " Wolfram Sang
@ 2016-08-11 21:03 ` Wolfram Sang
  2016-08-11 21:03 ` [PATCH 13/28] media: usb: em28xx: em28xx-audio: " Wolfram Sang
                   ` (16 subsequent siblings)
  28 siblings, 0 replies; 30+ messages in thread
From: Wolfram Sang @ 2016-08-11 21:03 UTC (permalink / raw)
  To: linux-usb; +Cc: Wolfram Sang, Mauro Carvalho Chehab, linux-media

kmalloc will print enough information in case of failure.

Signed-off-by: Wolfram Sang <wsa-dev@sang-engineering.com>
---
 drivers/media/usb/dvb-usb/dib0700_core.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/media/usb/dvb-usb/dib0700_core.c b/drivers/media/usb/dvb-usb/dib0700_core.c
index bf890c3d9cda02..26797979ebce4b 100644
--- a/drivers/media/usb/dvb-usb/dib0700_core.c
+++ b/drivers/media/usb/dvb-usb/dib0700_core.c
@@ -783,10 +783,8 @@ int dib0700_rc_setup(struct dvb_usb_device *d, struct usb_interface *intf)
 	/* Starting in firmware 1.20, the RC info is provided on a bulk pipe */
 
 	purb = usb_alloc_urb(0, GFP_KERNEL);
-	if (purb == NULL) {
-		err("rc usb alloc urb failed");
+	if (purb == NULL)
 		return -ENOMEM;
-	}
 
 	purb->transfer_buffer = kzalloc(RC_MSG_SIZE_V1_20, GFP_KERNEL);
 	if (purb->transfer_buffer == NULL) {
-- 
2.8.1


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

* [PATCH 13/28] media: usb: em28xx: em28xx-audio: don't print error when allocating urb fails
  2016-08-11 21:03 [PATCH 00/28] media: don't print error when allocating urb fails Wolfram Sang
                   ` (11 preceding siblings ...)
  2016-08-11 21:03 ` [PATCH 12/28] media: usb: dvb-usb: dib0700_core: " Wolfram Sang
@ 2016-08-11 21:03 ` Wolfram Sang
  2016-08-11 21:03 ` [PATCH 14/28] media: usb: em28xx: em28xx-core: " Wolfram Sang
                   ` (15 subsequent siblings)
  28 siblings, 0 replies; 30+ messages in thread
From: Wolfram Sang @ 2016-08-11 21:03 UTC (permalink / raw)
  To: linux-usb; +Cc: Wolfram Sang, Mauro Carvalho Chehab, linux-media

kmalloc will print enough information in case of failure.

Signed-off-by: Wolfram Sang <wsa-dev@sang-engineering.com>
---
 drivers/media/usb/em28xx/em28xx-audio.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/media/usb/em28xx/em28xx-audio.c b/drivers/media/usb/em28xx/em28xx-audio.c
index 49a5f9532bd8d8..78f3687772bfe3 100644
--- a/drivers/media/usb/em28xx/em28xx-audio.c
+++ b/drivers/media/usb/em28xx/em28xx-audio.c
@@ -850,7 +850,6 @@ static int em28xx_audio_urb_init(struct em28xx *dev)
 
 		urb = usb_alloc_urb(npackets, GFP_ATOMIC);
 		if (!urb) {
-			em28xx_errdev("usb_alloc_urb failed!\n");
 			em28xx_audio_free_urb(dev);
 			return -ENOMEM;
 		}
-- 
2.8.1


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

* [PATCH 14/28] media: usb: em28xx: em28xx-core: don't print error when allocating urb fails
  2016-08-11 21:03 [PATCH 00/28] media: don't print error when allocating urb fails Wolfram Sang
                   ` (12 preceding siblings ...)
  2016-08-11 21:03 ` [PATCH 13/28] media: usb: em28xx: em28xx-audio: " Wolfram Sang
@ 2016-08-11 21:03 ` Wolfram Sang
  2016-08-11 21:03 ` [PATCH 15/28] media: usb: gspca: benq: " Wolfram Sang
                   ` (14 subsequent siblings)
  28 siblings, 0 replies; 30+ messages in thread
From: Wolfram Sang @ 2016-08-11 21:03 UTC (permalink / raw)
  To: linux-usb; +Cc: Wolfram Sang, Mauro Carvalho Chehab, linux-media

kmalloc will print enough information in case of failure.

Signed-off-by: Wolfram Sang <wsa-dev@sang-engineering.com>
---
 drivers/media/usb/em28xx/em28xx-core.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/media/usb/em28xx/em28xx-core.c b/drivers/media/usb/em28xx/em28xx-core.c
index 37456079f490d9..eebd5d7088d009 100644
--- a/drivers/media/usb/em28xx/em28xx-core.c
+++ b/drivers/media/usb/em28xx/em28xx-core.c
@@ -934,7 +934,6 @@ int em28xx_alloc_urbs(struct em28xx *dev, enum em28xx_mode mode, int xfer_bulk,
 	for (i = 0; i < usb_bufs->num_bufs; i++) {
 		urb = usb_alloc_urb(usb_bufs->num_packets, GFP_KERNEL);
 		if (!urb) {
-			em28xx_err("cannot alloc usb_ctl.urb %i\n", i);
 			em28xx_uninit_usb_xfer(dev, mode);
 			return -ENOMEM;
 		}
-- 
2.8.1


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

* [PATCH 15/28] media: usb: gspca: benq: don't print error when allocating urb fails
  2016-08-11 21:03 [PATCH 00/28] media: don't print error when allocating urb fails Wolfram Sang
                   ` (13 preceding siblings ...)
  2016-08-11 21:03 ` [PATCH 14/28] media: usb: em28xx: em28xx-core: " Wolfram Sang
@ 2016-08-11 21:03 ` Wolfram Sang
  2016-08-11 21:03 ` [PATCH 16/28] media: usb: gspca: gspca: " Wolfram Sang
                   ` (13 subsequent siblings)
  28 siblings, 0 replies; 30+ messages in thread
From: Wolfram Sang @ 2016-08-11 21:03 UTC (permalink / raw)
  To: linux-usb; +Cc: Wolfram Sang, Hans Verkuil, Mauro Carvalho Chehab, linux-media

kmalloc will print enough information in case of failure.

Signed-off-by: Wolfram Sang <wsa-dev@sang-engineering.com>
---
 drivers/media/usb/gspca/benq.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/media/usb/gspca/benq.c b/drivers/media/usb/gspca/benq.c
index 790baed339639d..5fa67b78ad49b1 100644
--- a/drivers/media/usb/gspca/benq.c
+++ b/drivers/media/usb/gspca/benq.c
@@ -95,10 +95,8 @@ static int sd_start(struct gspca_dev *gspca_dev)
 #define SD_NPKT 32
 	for (n = 0; n < 4; n++) {
 		urb = usb_alloc_urb(SD_NPKT, GFP_KERNEL);
-		if (!urb) {
-			pr_err("usb_alloc_urb failed\n");
+		if (!urb)
 			return -ENOMEM;
-		}
 		gspca_dev->urb[n] = urb;
 		urb->transfer_buffer = usb_alloc_coherent(gspca_dev->dev,
 						SD_PKT_SZ * SD_NPKT,
-- 
2.8.1


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

* [PATCH 16/28] media: usb: gspca: gspca: don't print error when allocating urb fails
  2016-08-11 21:03 [PATCH 00/28] media: don't print error when allocating urb fails Wolfram Sang
                   ` (14 preceding siblings ...)
  2016-08-11 21:03 ` [PATCH 15/28] media: usb: gspca: benq: " Wolfram Sang
@ 2016-08-11 21:03 ` Wolfram Sang
  2016-08-11 21:03 ` [PATCH 17/28] media: usb: gspca: konica: " Wolfram Sang
                   ` (12 subsequent siblings)
  28 siblings, 0 replies; 30+ messages in thread
From: Wolfram Sang @ 2016-08-11 21:03 UTC (permalink / raw)
  To: linux-usb; +Cc: Wolfram Sang, Hans Verkuil, Mauro Carvalho Chehab, linux-media

kmalloc will print enough information in case of failure.

Signed-off-by: Wolfram Sang <wsa-dev@sang-engineering.com>
---
 drivers/media/usb/gspca/gspca.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/media/usb/gspca/gspca.c b/drivers/media/usb/gspca/gspca.c
index b17bd7ebcb47f7..af2395a76d8bf9 100644
--- a/drivers/media/usb/gspca/gspca.c
+++ b/drivers/media/usb/gspca/gspca.c
@@ -795,10 +795,8 @@ static int create_urbs(struct gspca_dev *gspca_dev,
 
 	for (n = 0; n < nurbs; n++) {
 		urb = usb_alloc_urb(npkt, GFP_KERNEL);
-		if (!urb) {
-			pr_err("usb_alloc_urb failed\n");
+		if (!urb)
 			return -ENOMEM;
-		}
 		gspca_dev->urb[n] = urb;
 		urb->transfer_buffer = usb_alloc_coherent(gspca_dev->dev,
 						bsize,
-- 
2.8.1


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

* [PATCH 17/28] media: usb: gspca: konica: don't print error when allocating urb fails
  2016-08-11 21:03 [PATCH 00/28] media: don't print error when allocating urb fails Wolfram Sang
                   ` (15 preceding siblings ...)
  2016-08-11 21:03 ` [PATCH 16/28] media: usb: gspca: gspca: " Wolfram Sang
@ 2016-08-11 21:03 ` Wolfram Sang
  2016-08-11 21:03 ` [PATCH 18/28] media: usb: hackrf: hackrf: " Wolfram Sang
                   ` (11 subsequent siblings)
  28 siblings, 0 replies; 30+ messages in thread
From: Wolfram Sang @ 2016-08-11 21:03 UTC (permalink / raw)
  To: linux-usb; +Cc: Wolfram Sang, Hans Verkuil, Mauro Carvalho Chehab, linux-media

kmalloc will print enough information in case of failure.

Signed-off-by: Wolfram Sang <wsa-dev@sang-engineering.com>
---
 drivers/media/usb/gspca/konica.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/media/usb/gspca/konica.c b/drivers/media/usb/gspca/konica.c
index 0712b1bc90b4dd..40aaaa9c5f30eb 100644
--- a/drivers/media/usb/gspca/konica.c
+++ b/drivers/media/usb/gspca/konica.c
@@ -208,10 +208,8 @@ static int sd_start(struct gspca_dev *gspca_dev)
 		packet_size =
 			le16_to_cpu(alt->endpoint[i].desc.wMaxPacketSize);
 		urb = usb_alloc_urb(SD_NPKT, GFP_KERNEL);
-		if (!urb) {
-			pr_err("usb_alloc_urb failed\n");
+		if (!urb)
 			return -ENOMEM;
-		}
 		gspca_dev->urb[n] = urb;
 		urb->transfer_buffer = usb_alloc_coherent(gspca_dev->dev,
 						packet_size * SD_NPKT,
-- 
2.8.1


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

* [PATCH 18/28] media: usb: hackrf: hackrf: don't print error when allocating urb fails
  2016-08-11 21:03 [PATCH 00/28] media: don't print error when allocating urb fails Wolfram Sang
                   ` (16 preceding siblings ...)
  2016-08-11 21:03 ` [PATCH 17/28] media: usb: gspca: konica: " Wolfram Sang
@ 2016-08-11 21:03 ` Wolfram Sang
  2016-08-11 21:03 ` [PATCH 19/28] media: usb: hdpvr: hdpvr-video: " Wolfram Sang
                   ` (10 subsequent siblings)
  28 siblings, 0 replies; 30+ messages in thread
From: Wolfram Sang @ 2016-08-11 21:03 UTC (permalink / raw)
  To: linux-usb
  Cc: Wolfram Sang, Antti Palosaari, Mauro Carvalho Chehab, linux-media

kmalloc will print enough information in case of failure.

Signed-off-by: Wolfram Sang <wsa-dev@sang-engineering.com>
---
 drivers/media/usb/hackrf/hackrf.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/media/usb/hackrf/hackrf.c b/drivers/media/usb/hackrf/hackrf.c
index b1e229a44192d3..c2c8d12e949868 100644
--- a/drivers/media/usb/hackrf/hackrf.c
+++ b/drivers/media/usb/hackrf/hackrf.c
@@ -691,7 +691,6 @@ static int hackrf_alloc_urbs(struct hackrf_dev *dev, bool rcv)
 		dev_dbg(dev->dev, "alloc urb=%d\n", i);
 		dev->urb_list[i] = usb_alloc_urb(0, GFP_ATOMIC);
 		if (!dev->urb_list[i]) {
-			dev_dbg(dev->dev, "failed\n");
 			for (j = 0; j < i; j++)
 				usb_free_urb(dev->urb_list[j]);
 			return -ENOMEM;
-- 
2.8.1


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

* [PATCH 19/28] media: usb: hdpvr: hdpvr-video: don't print error when allocating urb fails
  2016-08-11 21:03 [PATCH 00/28] media: don't print error when allocating urb fails Wolfram Sang
                   ` (17 preceding siblings ...)
  2016-08-11 21:03 ` [PATCH 18/28] media: usb: hackrf: hackrf: " Wolfram Sang
@ 2016-08-11 21:03 ` Wolfram Sang
  2016-08-11 21:03 ` [PATCH 20/28] media: usb: msi2500: msi2500: " Wolfram Sang
                   ` (9 subsequent siblings)
  28 siblings, 0 replies; 30+ messages in thread
From: Wolfram Sang @ 2016-08-11 21:03 UTC (permalink / raw)
  To: linux-usb; +Cc: Wolfram Sang, Hans Verkuil, Mauro Carvalho Chehab, linux-media

kmalloc will print enough information in case of failure.

Signed-off-by: Wolfram Sang <wsa-dev@sang-engineering.com>
---
 drivers/media/usb/hdpvr/hdpvr-video.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/media/usb/hdpvr/hdpvr-video.c b/drivers/media/usb/hdpvr/hdpvr-video.c
index 2a3a8b470555b9..6d43d75493ea0e 100644
--- a/drivers/media/usb/hdpvr/hdpvr-video.c
+++ b/drivers/media/usb/hdpvr/hdpvr-video.c
@@ -155,10 +155,8 @@ int hdpvr_alloc_buffers(struct hdpvr_device *dev, uint count)
 		buf->dev = dev;
 
 		urb = usb_alloc_urb(0, GFP_KERNEL);
-		if (!urb) {
-			v4l2_err(&dev->v4l2_dev, "cannot allocate urb\n");
+		if (!urb)
 			goto exit_urb;
-		}
 		buf->urb = urb;
 
 		mem = usb_alloc_coherent(dev->udev, dev->bulk_in_size, GFP_KERNEL,
-- 
2.8.1


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

* [PATCH 20/28] media: usb: msi2500: msi2500: don't print error when allocating urb fails
  2016-08-11 21:03 [PATCH 00/28] media: don't print error when allocating urb fails Wolfram Sang
                   ` (18 preceding siblings ...)
  2016-08-11 21:03 ` [PATCH 19/28] media: usb: hdpvr: hdpvr-video: " Wolfram Sang
@ 2016-08-11 21:03 ` Wolfram Sang
  2016-08-11 21:03 ` [PATCH 21/28] media: usb: pwc: pwc-if: " Wolfram Sang
                   ` (8 subsequent siblings)
  28 siblings, 0 replies; 30+ messages in thread
From: Wolfram Sang @ 2016-08-11 21:03 UTC (permalink / raw)
  To: linux-usb
  Cc: Wolfram Sang, Antti Palosaari, Mauro Carvalho Chehab, linux-media

kmalloc will print enough information in case of failure.

Signed-off-by: Wolfram Sang <wsa-dev@sang-engineering.com>
---
 drivers/media/usb/msi2500/msi2500.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/media/usb/msi2500/msi2500.c b/drivers/media/usb/msi2500/msi2500.c
index e7f167d44c61ce..367eb7e2a31dbc 100644
--- a/drivers/media/usb/msi2500/msi2500.c
+++ b/drivers/media/usb/msi2500/msi2500.c
@@ -509,7 +509,6 @@ static int msi2500_isoc_init(struct msi2500_dev *dev)
 	for (i = 0; i < MAX_ISO_BUFS; i++) {
 		urb = usb_alloc_urb(ISO_FRAMES_PER_DESC, GFP_KERNEL);
 		if (urb == NULL) {
-			dev_err(dev->dev, "Failed to allocate urb %d\n", i);
 			msi2500_isoc_cleanup(dev);
 			return -ENOMEM;
 		}
-- 
2.8.1


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

* [PATCH 21/28] media: usb: pwc: pwc-if: don't print error when allocating urb fails
  2016-08-11 21:03 [PATCH 00/28] media: don't print error when allocating urb fails Wolfram Sang
                   ` (19 preceding siblings ...)
  2016-08-11 21:03 ` [PATCH 20/28] media: usb: msi2500: msi2500: " Wolfram Sang
@ 2016-08-11 21:03 ` Wolfram Sang
  2016-08-11 21:03 ` [PATCH 22/28] media: usb: s2255: s2255drv: " Wolfram Sang
                   ` (7 subsequent siblings)
  28 siblings, 0 replies; 30+ messages in thread
From: Wolfram Sang @ 2016-08-11 21:03 UTC (permalink / raw)
  To: linux-usb; +Cc: Wolfram Sang, Hans Verkuil, Mauro Carvalho Chehab, linux-media

kmalloc will print enough information in case of failure.

Signed-off-by: Wolfram Sang <wsa-dev@sang-engineering.com>
---
 drivers/media/usb/pwc/pwc-if.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/media/usb/pwc/pwc-if.c b/drivers/media/usb/pwc/pwc-if.c
index b51b27a3fd6108..c4454c928776f7 100644
--- a/drivers/media/usb/pwc/pwc-if.c
+++ b/drivers/media/usb/pwc/pwc-if.c
@@ -410,7 +410,6 @@ retry:
 	for (i = 0; i < MAX_ISO_BUFS; i++) {
 		urb = usb_alloc_urb(ISO_FRAMES_PER_DESC, GFP_KERNEL);
 		if (urb == NULL) {
-			PWC_ERROR("Failed to allocate urb %d\n", i);
 			pwc_isoc_cleanup(pdev);
 			return -ENOMEM;
 		}
-- 
2.8.1


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

* [PATCH 22/28] media: usb: s2255: s2255drv: don't print error when allocating urb fails
  2016-08-11 21:03 [PATCH 00/28] media: don't print error when allocating urb fails Wolfram Sang
                   ` (20 preceding siblings ...)
  2016-08-11 21:03 ` [PATCH 21/28] media: usb: pwc: pwc-if: " Wolfram Sang
@ 2016-08-11 21:03 ` Wolfram Sang
  2016-08-11 21:03 ` [PATCH 23/28] media: usb: stk1160: stk1160-video: " Wolfram Sang
                   ` (6 subsequent siblings)
  28 siblings, 0 replies; 30+ messages in thread
From: Wolfram Sang @ 2016-08-11 21:03 UTC (permalink / raw)
  To: linux-usb; +Cc: Wolfram Sang, Mauro Carvalho Chehab, linux-media

kmalloc will print enough information in case of failure.

Signed-off-by: Wolfram Sang <wsa-dev@sang-engineering.com>
---
 drivers/media/usb/s2255/s2255drv.c | 9 ++-------
 1 file changed, 2 insertions(+), 7 deletions(-)

diff --git a/drivers/media/usb/s2255/s2255drv.c b/drivers/media/usb/s2255/s2255drv.c
index 43ba71a7d02b8f..9458eb0ef66f43 100644
--- a/drivers/media/usb/s2255/s2255drv.c
+++ b/drivers/media/usb/s2255/s2255drv.c
@@ -2113,11 +2113,8 @@ static int s2255_start_readpipe(struct s2255_dev *dev)
 	pipe_info->state = 1;
 	pipe_info->err_count = 0;
 	pipe_info->stream_urb = usb_alloc_urb(0, GFP_KERNEL);
-	if (!pipe_info->stream_urb) {
-		dev_err(&dev->udev->dev,
-			"ReadStream: Unable to alloc URB\n");
+	if (!pipe_info->stream_urb)
 		return -ENOMEM;
-	}
 	/* transfer buffer allocated in board_init */
 	usb_fill_bulk_urb(pipe_info->stream_urb, dev->udev,
 			  pipe,
@@ -2290,10 +2287,8 @@ static int s2255_probe(struct usb_interface *interface,
 	}
 
 	dev->fw_data->fw_urb = usb_alloc_urb(0, GFP_KERNEL);
-	if (!dev->fw_data->fw_urb) {
-		dev_err(&interface->dev, "out of memory!\n");
+	if (!dev->fw_data->fw_urb)
 		goto errorFWURB;
-	}
 
 	dev->fw_data->pfw_data = kzalloc(CHUNK_SIZE, GFP_KERNEL);
 	if (!dev->fw_data->pfw_data) {
-- 
2.8.1


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

* [PATCH 23/28] media: usb: stk1160: stk1160-video: don't print error when allocating urb fails
  2016-08-11 21:03 [PATCH 00/28] media: don't print error when allocating urb fails Wolfram Sang
                   ` (21 preceding siblings ...)
  2016-08-11 21:03 ` [PATCH 22/28] media: usb: s2255: s2255drv: " Wolfram Sang
@ 2016-08-11 21:03 ` Wolfram Sang
  2016-08-11 21:04 ` [PATCH 24/28] media: usb: stkwebcam: stk-webcam: " Wolfram Sang
                   ` (5 subsequent siblings)
  28 siblings, 0 replies; 30+ messages in thread
From: Wolfram Sang @ 2016-08-11 21:03 UTC (permalink / raw)
  To: linux-usb
  Cc: Wolfram Sang, Ezequiel Garcia, Mauro Carvalho Chehab, linux-media

kmalloc will print enough information in case of failure.

Signed-off-by: Wolfram Sang <wsa-dev@sang-engineering.com>
---
 drivers/media/usb/stk1160/stk1160-video.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/media/usb/stk1160/stk1160-video.c b/drivers/media/usb/stk1160/stk1160-video.c
index 6ecb0b48423f3d..ce8ebbe395a620 100644
--- a/drivers/media/usb/stk1160/stk1160-video.c
+++ b/drivers/media/usb/stk1160/stk1160-video.c
@@ -457,10 +457,8 @@ int stk1160_alloc_isoc(struct stk1160 *dev)
 	for (i = 0; i < num_bufs; i++) {
 
 		urb = usb_alloc_urb(max_packets, GFP_KERNEL);
-		if (!urb) {
-			stk1160_err("cannot alloc urb[%d]\n", i);
+		if (!urb)
 			goto free_i_bufs;
-		}
 		dev->isoc_ctl.urb[i] = urb;
 
 #ifndef CONFIG_DMA_NONCOHERENT
-- 
2.8.1


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

* [PATCH 24/28] media: usb: stkwebcam: stk-webcam: don't print error when allocating urb fails
  2016-08-11 21:03 [PATCH 00/28] media: don't print error when allocating urb fails Wolfram Sang
                   ` (22 preceding siblings ...)
  2016-08-11 21:03 ` [PATCH 23/28] media: usb: stk1160: stk1160-video: " Wolfram Sang
@ 2016-08-11 21:04 ` Wolfram Sang
  2016-08-11 21:04 ` [PATCH 25/28] media: usb: tm6000: tm6000-dvb: " Wolfram Sang
                   ` (4 subsequent siblings)
  28 siblings, 0 replies; 30+ messages in thread
From: Wolfram Sang @ 2016-08-11 21:04 UTC (permalink / raw)
  To: linux-usb; +Cc: Wolfram Sang, Mauro Carvalho Chehab, linux-media

kmalloc will print enough information in case of failure.

Signed-off-by: Wolfram Sang <wsa-dev@sang-engineering.com>
---
 drivers/media/usb/stkwebcam/stk-webcam.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/media/usb/stkwebcam/stk-webcam.c b/drivers/media/usb/stkwebcam/stk-webcam.c
index c21c4c004f9778..db200c9d796d36 100644
--- a/drivers/media/usb/stkwebcam/stk-webcam.c
+++ b/drivers/media/usb/stkwebcam/stk-webcam.c
@@ -452,10 +452,8 @@ static int stk_prepare_iso(struct stk_camera *dev)
 			STK_ERROR("isobuf data already allocated\n");
 		if (dev->isobufs[i].urb == NULL) {
 			urb = usb_alloc_urb(ISO_FRAMES_PER_DESC, GFP_KERNEL);
-			if (urb == NULL) {
-				STK_ERROR("Failed to allocate URB %d\n", i);
+			if (urb == NULL)
 				goto isobufs_out;
-			}
 			dev->isobufs[i].urb = urb;
 		} else {
 			STK_ERROR("Killing URB\n");
-- 
2.8.1


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

* [PATCH 25/28] media: usb: tm6000: tm6000-dvb: don't print error when allocating urb fails
  2016-08-11 21:03 [PATCH 00/28] media: don't print error when allocating urb fails Wolfram Sang
                   ` (23 preceding siblings ...)
  2016-08-11 21:04 ` [PATCH 24/28] media: usb: stkwebcam: stk-webcam: " Wolfram Sang
@ 2016-08-11 21:04 ` Wolfram Sang
  2016-08-11 21:04 ` [PATCH 26/28] media: usb: tm6000: tm6000-video: " Wolfram Sang
                   ` (3 subsequent siblings)
  28 siblings, 0 replies; 30+ messages in thread
From: Wolfram Sang @ 2016-08-11 21:04 UTC (permalink / raw)
  To: linux-usb; +Cc: Wolfram Sang, Mauro Carvalho Chehab, linux-media

kmalloc will print enough information in case of failure.

Signed-off-by: Wolfram Sang <wsa-dev@sang-engineering.com>
---
 drivers/media/usb/tm6000/tm6000-dvb.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/media/usb/tm6000/tm6000-dvb.c b/drivers/media/usb/tm6000/tm6000-dvb.c
index 095f5db1a790f9..0426b210383b7c 100644
--- a/drivers/media/usb/tm6000/tm6000-dvb.c
+++ b/drivers/media/usb/tm6000/tm6000-dvb.c
@@ -129,10 +129,8 @@ static int tm6000_start_stream(struct tm6000_core *dev)
 	}
 
 	dvb->bulk_urb = usb_alloc_urb(0, GFP_KERNEL);
-	if (dvb->bulk_urb == NULL) {
-		printk(KERN_ERR "tm6000: couldn't allocate urb\n");
+	if (dvb->bulk_urb == NULL)
 		return -ENOMEM;
-	}
 
 	pipe = usb_rcvbulkpipe(dev->udev, dev->bulk_in.endp->desc.bEndpointAddress
 							  & USB_ENDPOINT_NUMBER_MASK);
-- 
2.8.1


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

* [PATCH 26/28] media: usb: tm6000: tm6000-video: don't print error when allocating urb fails
  2016-08-11 21:03 [PATCH 00/28] media: don't print error when allocating urb fails Wolfram Sang
                   ` (24 preceding siblings ...)
  2016-08-11 21:04 ` [PATCH 25/28] media: usb: tm6000: tm6000-dvb: " Wolfram Sang
@ 2016-08-11 21:04 ` Wolfram Sang
  2016-08-11 21:04 ` [PATCH 27/28] media: usb: usbvision: usbvision-core: " Wolfram Sang
                   ` (2 subsequent siblings)
  28 siblings, 0 replies; 30+ messages in thread
From: Wolfram Sang @ 2016-08-11 21:04 UTC (permalink / raw)
  To: linux-usb; +Cc: Wolfram Sang, Mauro Carvalho Chehab, linux-media

kmalloc will print enough information in case of failure.

Signed-off-by: Wolfram Sang <wsa-dev@sang-engineering.com>
---
 drivers/media/usb/tm6000/tm6000-video.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/media/usb/tm6000/tm6000-video.c b/drivers/media/usb/tm6000/tm6000-video.c
index fa5e8bda2ae4a7..dee7e7d3d47d71 100644
--- a/drivers/media/usb/tm6000/tm6000-video.c
+++ b/drivers/media/usb/tm6000/tm6000-video.c
@@ -635,7 +635,6 @@ static int tm6000_prepare_isoc(struct tm6000_core *dev)
 	for (i = 0; i < dev->isoc_ctl.num_bufs; i++) {
 		urb = usb_alloc_urb(max_packets, GFP_KERNEL);
 		if (!urb) {
-			tm6000_err("cannot alloc isoc_ctl.urb %i\n", i);
 			tm6000_uninit_isoc(dev);
 			usb_free_urb(urb);
 			return -ENOMEM;
-- 
2.8.1


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

* [PATCH 27/28] media: usb: usbvision: usbvision-core: don't print error when allocating urb fails
  2016-08-11 21:03 [PATCH 00/28] media: don't print error when allocating urb fails Wolfram Sang
                   ` (25 preceding siblings ...)
  2016-08-11 21:04 ` [PATCH 26/28] media: usb: tm6000: tm6000-video: " Wolfram Sang
@ 2016-08-11 21:04 ` Wolfram Sang
  2016-08-11 21:04 ` [PATCH 28/28] media: usb: zr364xx: zr364xx: " Wolfram Sang
  2016-08-30 17:14 ` [PATCH 00/28] media: " Greg KH
  28 siblings, 0 replies; 30+ messages in thread
From: Wolfram Sang @ 2016-08-11 21:04 UTC (permalink / raw)
  To: linux-usb; +Cc: Wolfram Sang, Hans Verkuil, Mauro Carvalho Chehab, linux-media

kmalloc will print enough information in case of failure.

Signed-off-by: Wolfram Sang <wsa-dev@sang-engineering.com>
---
 drivers/media/usb/usbvision/usbvision-core.c | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/drivers/media/usb/usbvision/usbvision-core.c b/drivers/media/usb/usbvision/usbvision-core.c
index 52ac4391582c49..c23bf73a68ea97 100644
--- a/drivers/media/usb/usbvision/usbvision-core.c
+++ b/drivers/media/usb/usbvision/usbvision-core.c
@@ -2303,11 +2303,8 @@ int usbvision_init_isoc(struct usb_usbvision *usbvision)
 		struct urb *urb;
 
 		urb = usb_alloc_urb(USBVISION_URB_FRAMES, GFP_KERNEL);
-		if (urb == NULL) {
-			dev_err(&usbvision->dev->dev,
-				"%s: usb_alloc_urb() failed\n", __func__);
+		if (urb == NULL)
 			return -ENOMEM;
-		}
 		usbvision->sbuf[buf_idx].urb = urb;
 		usbvision->sbuf[buf_idx].data =
 			usb_alloc_coherent(usbvision->dev,
-- 
2.8.1


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

* [PATCH 28/28] media: usb: zr364xx: zr364xx: don't print error when allocating urb fails
  2016-08-11 21:03 [PATCH 00/28] media: don't print error when allocating urb fails Wolfram Sang
                   ` (26 preceding siblings ...)
  2016-08-11 21:04 ` [PATCH 27/28] media: usb: usbvision: usbvision-core: " Wolfram Sang
@ 2016-08-11 21:04 ` Wolfram Sang
  2016-08-30 17:14 ` [PATCH 00/28] media: " Greg KH
  28 siblings, 0 replies; 30+ messages in thread
From: Wolfram Sang @ 2016-08-11 21:04 UTC (permalink / raw)
  To: linux-usb
  Cc: Wolfram Sang, Antoine Jacquet, Mauro Carvalho Chehab, linux-media

kmalloc will print enough information in case of failure.

Signed-off-by: Wolfram Sang <wsa-dev@sang-engineering.com>
---
 drivers/media/usb/zr364xx/zr364xx.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/media/usb/zr364xx/zr364xx.c b/drivers/media/usb/zr364xx/zr364xx.c
index 7433ba5c4bad8b..cc128db85723c9 100644
--- a/drivers/media/usb/zr364xx/zr364xx.c
+++ b/drivers/media/usb/zr364xx/zr364xx.c
@@ -1045,10 +1045,8 @@ static int zr364xx_start_readpipe(struct zr364xx_camera *cam)
 	pipe_info->state = 1;
 	pipe_info->err_count = 0;
 	pipe_info->stream_urb = usb_alloc_urb(0, GFP_KERNEL);
-	if (!pipe_info->stream_urb) {
-		dev_err(&cam->udev->dev, "ReadStream: Unable to alloc URB\n");
+	if (!pipe_info->stream_urb)
 		return -ENOMEM;
-	}
 	/* transfer buffer allocated in board_init */
 	usb_fill_bulk_urb(pipe_info->stream_urb, cam->udev,
 			  pipe,
-- 
2.8.1


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

* Re: [PATCH 00/28] media: don't print error when allocating urb fails
  2016-08-11 21:03 [PATCH 00/28] media: don't print error when allocating urb fails Wolfram Sang
                   ` (27 preceding siblings ...)
  2016-08-11 21:04 ` [PATCH 28/28] media: usb: zr364xx: zr364xx: " Wolfram Sang
@ 2016-08-30 17:14 ` Greg KH
  28 siblings, 0 replies; 30+ messages in thread
From: Greg KH @ 2016-08-30 17:14 UTC (permalink / raw)
  To: Wolfram Sang; +Cc: linux-usb, linux-media

On Thu, Aug 11, 2016 at 11:03:36PM +0200, Wolfram Sang wrote:
> This per-subsystem series is part of a tree wide cleanup. usb_alloc_urb() uses
> kmalloc which already prints enough information on failure. So, let's simply
> remove those "allocation failed" messages from drivers like we did already for
> other -ENOMEM cases. gkh acked this approach when we talked about it at LCJ in
> Tokyo a few weeks ago.

I've taken all of these through the usb tree given the delay in response
from the media developers :)

thanks,

greg k-h

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

end of thread, other threads:[~2016-08-30 17:15 UTC | newest]

Thread overview: 30+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-08-11 21:03 [PATCH 00/28] media: don't print error when allocating urb fails Wolfram Sang
2016-08-11 21:03 ` [PATCH 01/28] media: dvb-frontends: rtl2832_sdr: " Wolfram Sang
2016-08-11 21:03 ` [PATCH 02/28] media: radio: si470x: radio-si470x-usb: " Wolfram Sang
2016-08-11 21:03 ` [PATCH 03/28] media: rc: imon: " Wolfram Sang
2016-08-11 21:03 ` [PATCH 04/28] media: rc: redrat3: " Wolfram Sang
2016-08-11 21:03 ` [PATCH 05/28] media: usb: airspy: airspy: " Wolfram Sang
2016-08-11 21:03 ` [PATCH 06/28] media: usb: as102: as102_usb_drv: " Wolfram Sang
2016-08-11 21:03 ` [PATCH 07/28] media: usb: au0828: au0828-video: " Wolfram Sang
2016-08-11 21:03 ` [PATCH 08/28] media: usb: cpia2: cpia2_usb: " Wolfram Sang
2016-08-11 21:03 ` [PATCH 09/28] media: usb: cx231xx: cx231xx-audio: " Wolfram Sang
2016-08-11 21:03 ` [PATCH 10/28] media: usb: cx231xx: cx231xx-core: " Wolfram Sang
2016-08-11 21:03 ` [PATCH 11/28] media: usb: cx231xx: cx231xx-vbi: " Wolfram Sang
2016-08-11 21:03 ` [PATCH 12/28] media: usb: dvb-usb: dib0700_core: " Wolfram Sang
2016-08-11 21:03 ` [PATCH 13/28] media: usb: em28xx: em28xx-audio: " Wolfram Sang
2016-08-11 21:03 ` [PATCH 14/28] media: usb: em28xx: em28xx-core: " Wolfram Sang
2016-08-11 21:03 ` [PATCH 15/28] media: usb: gspca: benq: " Wolfram Sang
2016-08-11 21:03 ` [PATCH 16/28] media: usb: gspca: gspca: " Wolfram Sang
2016-08-11 21:03 ` [PATCH 17/28] media: usb: gspca: konica: " Wolfram Sang
2016-08-11 21:03 ` [PATCH 18/28] media: usb: hackrf: hackrf: " Wolfram Sang
2016-08-11 21:03 ` [PATCH 19/28] media: usb: hdpvr: hdpvr-video: " Wolfram Sang
2016-08-11 21:03 ` [PATCH 20/28] media: usb: msi2500: msi2500: " Wolfram Sang
2016-08-11 21:03 ` [PATCH 21/28] media: usb: pwc: pwc-if: " Wolfram Sang
2016-08-11 21:03 ` [PATCH 22/28] media: usb: s2255: s2255drv: " Wolfram Sang
2016-08-11 21:03 ` [PATCH 23/28] media: usb: stk1160: stk1160-video: " Wolfram Sang
2016-08-11 21:04 ` [PATCH 24/28] media: usb: stkwebcam: stk-webcam: " Wolfram Sang
2016-08-11 21:04 ` [PATCH 25/28] media: usb: tm6000: tm6000-dvb: " Wolfram Sang
2016-08-11 21:04 ` [PATCH 26/28] media: usb: tm6000: tm6000-video: " Wolfram Sang
2016-08-11 21:04 ` [PATCH 27/28] media: usb: usbvision: usbvision-core: " Wolfram Sang
2016-08-11 21:04 ` [PATCH 28/28] media: usb: zr364xx: zr364xx: " Wolfram Sang
2016-08-30 17:14 ` [PATCH 00/28] media: " Greg KH

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.