On Thu, Dec 4, 2014 at 11:25 AM, Heiner Kallweit <hkallweit1@xxxxxxxxx> wrote: > Right, missed that. However the code was bogus before, the status = -EINVAL > didn't make sense as the value of variable status isn't used before the > return. > Therefore we can remove the "status = -EINVAL". Yes, I noticed that as well. > Shall I submit a patch or what would be your preferred approach? Yes, may be it would be better if you could submit a fix for this as separate patch, after Mark applies this one, so that we avoid conflicts. -- To unsubscribe from this list: send the line "unsubscribe linux-spi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html