On 05/01/2015 12:59 PM, Guenter Roeck wrote:
You are saying that pretty much everyone in the kernel is, in your opinion, doing the Wrong Thing (tm) and you insist in doing it differently.
No, of course not, it's just that I've written code like this many times, and no one else ever complained about my error messages before.
As maintainer, I have seen lots of patches which remove this very same error checking as unnecessary. If we accept your code, we can be all but sure to see such a patch at some point, probably right after your patch was accepted and shows up in linux-next. So besides arguing about something we should not have to argue about in the first place, you are trying to create even more maintainer work going forward.
Ok, ok! I'll remove it. I just trying to understand the rationale. I don't agree with it, but I'll make the change you want.