Ping ? 2012/11/30, Stefan Hajnoczi <stefa...@gmail.com>: > Changes are fine. > > In the future, please split changes into individual patches. For example: > Patch 1 - Rename 'rv' to 'ret' > Patch 2 - Replace decprecated simple_strtoul() with kstrtoint() in > pcm_set_impulse_period() > Patch 3 - Realign comment in line6_init_pcm() switch statement > > Keeping patches focussed on doing just one thing makes it easier to > revert, bisect, and review them. > > Reviewed-by: Stefan Hajnoczi <stefa...@gmail.com> >
Should I split patches as suggested by Stefan ? Laurent. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
