Re: [PATCH 2/2] sata_mv: More error handling for phy_power_off() in mv_platform_remove()

Tejun Heo
2015-02-03T12:57:57Z

On Tue, Feb 03, 2015 at 07:48:14AM -0500, Tejun Heo wrote:
> That's too anal to be helpful and too spurious to become even remotely

Let me clarify the "too anal" part a bit. Let's say people push this
like crazy and we successfully mark all places to either check the
return value or have (void) in front. It's not difficult to realize
that that would be the exact same situation as right now just with
more visual clutter. That's the problem when these things become
obsessive - they become indisciminate and thus meaningless. Annotate
the exceptionals, not the other way around.

Thanks.

--
tejun