On Mon, Jun 01, 2009 at 10:30:58PM +0200, Marek Vasut wrote: > - if (!(machine_is_palmtx() || machine_is_palmt5() || > - machine_is_palmld() || machine_is_palmte2())) > - return -ENODEV; > - > - if (pdev->dev.platform_data) These machine_is_() checks don't get re-added elsehwere (and it looks like you generated the diff against a different verison of the file to the one that's in ALSA git). It'd be better to keep them here and just add the { to the platform_data check. Otherwise this is fine. _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel