* [PATCH] staging: vc04_services: bcm2835-camera: Modify return statement.
@ 2019-04-29 7:36 Vatsala Narang
2019-04-30 13:00 ` Greg KH
0 siblings, 1 reply; 4+ messages in thread
From: Vatsala Narang @ 2019-04-29 7:36 UTC (permalink / raw)
To: gregkh
Cc: eric, stefan.wahren, f.fainelli, rjui, sbranden,
bcm-kernel-feedback-list, linux-rpi-kernel, linux-arm-kernel,
devel, linux-kernel, julia.lawall, Vatsala Narang
Modify return statement and remove the respective assignment.
Issue found by coccinelle.
Signed-off-by: Vatsala Narang <vatsalanarang@gmail.com>
---
drivers/staging/vc04_services/bcm2835-camera/bcm2835-camera.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/staging/vc04_services/bcm2835-camera/bcm2835-camera.c b/drivers/staging/vc04_services/bcm2835-camera/bcm2835-camera.c
index c9b6346111a5..cef6d5b758e8 100644
--- a/drivers/staging/vc04_services/bcm2835-camera/bcm2835-camera.c
+++ b/drivers/staging/vc04_services/bcm2835-camera/bcm2835-camera.c
@@ -1507,10 +1507,9 @@ static int set_camera_parameters(struct vchiq_mmal_instance *instance,
.use_stc_timestamp = MMAL_PARAM_TIMESTAMP_MODE_RAW_STC
};
- ret = vchiq_mmal_port_parameter_set(instance, &camera->control,
+ return vchiq_mmal_port_parameter_set(instance, &camera->control,
MMAL_PARAMETER_CAMERA_CONFIG,
&cam_config, sizeof(cam_config));
- return ret;
}
#define MAX_SUPPORTED_ENCODINGS 20
--
2.17.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] staging: vc04_services: bcm2835-camera: Modify return statement.
2019-04-29 7:36 [PATCH] staging: vc04_services: bcm2835-camera: Modify return statement Vatsala Narang
@ 2019-04-30 13:00 ` Greg KH
0 siblings, 0 replies; 4+ messages in thread
From: Greg KH @ 2019-04-30 13:00 UTC (permalink / raw)
To: Vatsala Narang
Cc: stefan.wahren, devel, f.fainelli, sbranden, julia.lawall, rjui,
linux-kernel, eric, bcm-kernel-feedback-list, linux-rpi-kernel,
linux-arm-kernel
On Mon, Apr 29, 2019 at 01:06:58PM +0530, Vatsala Narang wrote:
> Modify return statement and remove the respective assignment.
>
> Issue found by coccinelle.
>
> Signed-off-by: Vatsala Narang <vatsalanarang@gmail.com>
> ---
> drivers/staging/vc04_services/bcm2835-camera/bcm2835-camera.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
You sent two different patches that did different things with the same
exact subject: line :(
Please make them more unique.
thanks,
greg k-h
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] staging: vc04_services: bcm2835-camera: Modify return statement.
2019-04-27 16:07 Vatsala Narang
@ 2019-04-27 20:00 ` Stefan Wahren
0 siblings, 0 replies; 4+ messages in thread
From: Stefan Wahren @ 2019-04-27 20:00 UTC (permalink / raw)
To: Vatsala Narang, gregkh
Cc: eric, f.fainelli, rjui, sbranden, bcm-kernel-feedback-list,
linux-rpi-kernel, linux-arm-kernel, devel, linux-kernel,
julia.lawall
Am 27.04.19 um 18:07 schrieb Vatsala Narang:
> Modify return statement and remove the respective assignment.
>
> Issue found by Coccinelle.
>
> Signed-off-by: Vatsala Narang <vatsalanarang@gmail.com>
Acked-by: Stefan Wahren <stefan.wahren@i2se.com>
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH] staging: vc04_services: bcm2835-camera: Modify return statement.
@ 2019-04-27 16:07 Vatsala Narang
2019-04-27 20:00 ` Stefan Wahren
0 siblings, 1 reply; 4+ messages in thread
From: Vatsala Narang @ 2019-04-27 16:07 UTC (permalink / raw)
To: gregkh
Cc: eric, stefan.wahren, f.fainelli, rjui, sbranden,
bcm-kernel-feedback-list, linux-rpi-kernel, linux-arm-kernel,
devel, linux-kernel, julia.lawall, Vatsala Narang
Modify return statement and remove the respective assignment.
Issue found by Coccinelle.
Signed-off-by: Vatsala Narang <vatsalanarang@gmail.com>
---
drivers/staging/vc04_services/bcm2835-camera/controls.c | 8 ++------
1 file changed, 2 insertions(+), 6 deletions(-)
diff --git a/drivers/staging/vc04_services/bcm2835-camera/controls.c b/drivers/staging/vc04_services/bcm2835-camera/controls.c
index 2e0a422cdf3e..9841c30450ce 100644
--- a/drivers/staging/vc04_services/bcm2835-camera/controls.c
+++ b/drivers/staging/vc04_services/bcm2835-camera/controls.c
@@ -270,11 +270,9 @@ static int ctrl_set_rotate(struct bm2835_mmal_dev *dev,
if (ret < 0)
return ret;
- ret = vchiq_mmal_port_parameter_set(dev->instance, &camera->output[2],
+ return vchiq_mmal_port_parameter_set(dev->instance, &camera->output[2],
mmal_ctrl->mmal_id,
&u32_value, sizeof(u32_value));
-
- return ret;
}
static int ctrl_set_flip(struct bm2835_mmal_dev *dev,
@@ -313,11 +311,9 @@ static int ctrl_set_flip(struct bm2835_mmal_dev *dev,
if (ret < 0)
return ret;
- ret = vchiq_mmal_port_parameter_set(dev->instance, &camera->output[2],
+ return vchiq_mmal_port_parameter_set(dev->instance, &camera->output[2],
mmal_ctrl->mmal_id,
&u32_value, sizeof(u32_value));
-
- return ret;
}
static int ctrl_set_exposure(struct bm2835_mmal_dev *dev,
--
2.17.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-04-30 13:00 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-04-29 7:36 [PATCH] staging: vc04_services: bcm2835-camera: Modify return statement Vatsala Narang
2019-04-30 13:00 ` Greg KH
-- strict thread matches above, loose matches on Subject: below --
2019-04-27 16:07 Vatsala Narang
2019-04-27 20:00 ` Stefan Wahren
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).