Completely agree! Thanks for spotting that one.
Signed-off-by: Mike Isely <isely@xxxxxxxxx>
-Mike
On Thu, 25 Oct 2012, Alan Cox wrote:
> From: Alan Cox <alan@xxxxxxxxxxxxxxx>
>
> This should have break statements in it.
>
> Signed-off-by: Alan Cox <alan@xxxxxxxxxxxxxxx>
> ---
>
> drivers/media/usb/pvrusb2/pvrusb2-hdw.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/media/usb/pvrusb2/pvrusb2-hdw.c b/drivers/media/usb/pvrusb2/pvrusb2-hdw.c
> index fb828ba..299751a 100644
> --- a/drivers/media/usb/pvrusb2/pvrusb2-hdw.c
> +++ b/drivers/media/usb/pvrusb2/pvrusb2-hdw.c
> @@ -3563,9 +3563,9 @@ void pvr2_hdw_v4l_store_minor_number(struct pvr2_hdw *hdw,
> enum pvr2_v4l_type index,int v)
> {
> switch (index) {
> - case pvr2_v4l_type_video: hdw->v4l_minor_number_video = v;
> - case pvr2_v4l_type_vbi: hdw->v4l_minor_number_vbi = v;
> - case pvr2_v4l_type_radio: hdw->v4l_minor_number_radio = v;
> + case pvr2_v4l_type_video: hdw->v4l_minor_number_video = v;break;
> + case pvr2_v4l_type_vbi: hdw->v4l_minor_number_vbi = v;break;
> + case pvr2_v4l_type_radio: hdw->v4l_minor_number_radio = v;break;
> default: break;
> }
> }
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-media" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
--
Mike Isely
isely @ isely (dot) net
PGP: 03 54 43 4D 75 E5 CC 92 71 16 01 E2 B5 F5 C1 E8
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
