On Wed, 13 Apr 2011 09:03:00 +0200 Wim Van Sebroeck <wim@xxxxxxxxx> wrote: > Hi Nat, > > > + (void) request_muxed_region(REG, 2, NAME); > > Why do we need to typecast this? Can't we just use > + request_muxed_region(REG, 2, NAME); We really ought to use if () in theory the request can fail if someone has hogged the resource and not muxed it. I'm not clear what the right thing to do in that case is - given it should never happen I guess log an error and bail out but that's a rather bigger change and perhaps should be a follow up patch ? -- To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
