6 Mar
2009
6 Mar
'09
1:28 a.m.
ok, i will resend the patch.
- switch (w->id) {
- case snd_soc_dapm_mixer:
- default:
Your chanegs here appear to only flip to an if/else from a switch statement? Please mention things like this in your commit logs (or split them into separate patches) - it makes your changes easier to review.
I think that a switch statement is used unnecessarily because this part has only to check whether w->id is named_ctl. ok, i will send them into splited patches. Thanks.
I found your commit at for-2.6.30 branch of your repository. Thanks for your fixes.
--
- Joonyoung Shim