Question about switch() of soc_mbus_config_compatible()

[Kuninori Morimoto]

Hi Mauro, Guennadi

I would like to ask you about switch() of
linux/drivers/media/platform/soc_camera/soc_mediabus.c :: 
soc_mbus_config_compatible

unsigned int soc_mbus_config_compatible(const struct v4l2_mbus_config *cfg,
                                        unsigned int flags)
{
        ...

        switch (cfg->type) {
        case V4L2_MBUS_PARALLEL:
                hsync = common_flags & (V4L2_MBUS_HSYNC_ACTIVE_HIGH |
                                        V4L2_MBUS_HSYNC_ACTIVE_LOW);
                vsync = common_flags & (V4L2_MBUS_VSYNC_ACTIVE_HIGH |
=>                                      V4L2_MBUS_VSYNC_ACTIVE_LOW);
        case V4L2_MBUS_BT656:
                pclk = common_flags & (V4L2_MBUS_PCLK_SAMPLE_RISING |
                                       V4L2_MBUS_PCLK_SAMPLE_FALLING);
                data = common_flags & (V4L2_MBUS_DATA_ACTIVE_HIGH |
                                       V4L2_MBUS_DATA_ACTIVE_LOW);
                mode = common_flags & (V4L2_MBUS_MASTER | V4L2_MBUS_SLAVE);
                return (!hsync || !vsync || !pclk || !data || !mode) ?
                        0 : common_flags;
        ...
}

Here, there is no break, no return, no /* FALL THROUGH */
It is very confusable, but what is this intention ?

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