
2 Jun
2009
2 Jun
'09
1:58 a.m.
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.