| Message ID | 20191015223433.18784-1-jbi.octave@gmail.com (mailing list archive) |
|---|---|
| State | Not Applicable |
| Headers | show |
| Series | staging: meson: add space to fix check warning | expand |
On 10/16/19 12:34 AM, Jules Irenge wrote: > Fix "alignment should mactch open parenthesis" check. > Issue detected by checkpatch tool > > Signed-off-by: Jules Irenge <jbi.octave@gmail.com> > --- > drivers/staging/media/meson/vdec/codec_mpeg12.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/staging/media/meson/vdec/codec_mpeg12.c b/drivers/staging/media/meson/vdec/codec_mpeg12.c > index 48869cc3d973..21e93a13356c 100644 > --- a/drivers/staging/media/meson/vdec/codec_mpeg12.c > +++ b/drivers/staging/media/meson/vdec/codec_mpeg12.c > @@ -81,7 +81,7 @@ static int codec_mpeg12_start(struct amvdec_session *sess) > } > > ret = amvdec_set_canvases(sess, (u32[]){ AV_SCRATCH_0, 0 }, > - (u32[]){ 8, 0 }); This is clearly intended to be aligned with the (u32[]) prefix for readability. So this is one checkpatch issue where I prefer to keep the original. Regards, Hans > + (u32[]){ 8, 0 }); > if (ret) > goto free_workspace; > >
diff --git a/drivers/staging/media/meson/vdec/codec_mpeg12.c b/drivers/staging/media/meson/vdec/codec_mpeg12.c index 48869cc3d973..21e93a13356c 100644 --- a/drivers/staging/media/meson/vdec/codec_mpeg12.c +++ b/drivers/staging/media/meson/vdec/codec_mpeg12.c @@ -81,7 +81,7 @@ static int codec_mpeg12_start(struct amvdec_session *sess) } ret = amvdec_set_canvases(sess, (u32[]){ AV_SCRATCH_0, 0 }, - (u32[]){ 8, 0 }); + (u32[]){ 8, 0 }); if (ret) goto free_workspace;
Fix "alignment should mactch open parenthesis" check. Issue detected by checkpatch tool Signed-off-by: Jules Irenge <jbi.octave@gmail.com> --- drivers/staging/media/meson/vdec/codec_mpeg12.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)