diff options
author | Arnd Bergmann <arnd@arndb.de> | 2012-04-30 16:26:01 +0000 |
---|---|---|
committer | Arnd Bergmann <arnd@arndb.de> | 2012-10-07 10:33:09 +0200 |
commit | 5f07809e93b4c05516d85a3f2770b1a77115eb70 (patch) | |
tree | e64dc5192285bd085ce05d371ca913ba4b69bedc /arch/arm/mach-rpc | |
parent | 5063557ac0e83a52182fd0d49e83cc770e52ccd1 (diff) |
ARM: rpc: check device_register return code in ecard_probe
device_register is marked __must_check, so we better propagate the error
value by returning it from ecard_probe.
Without this patch, building rpc_defconfig results in:
arch/arm/mach-rpc/ecard.c: In function 'ecard_probe':
arch/arm/mach-rpc/ecard.c:963:17: warning: ignoring return value of 'device_register', declared with attribute warn_unused_result [-Wunused-result]
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Cc: Russell King <rmk+kernel@arm.linux.org.uk>
Diffstat (limited to 'arch/arm/mach-rpc')
-rw-r--r-- | arch/arm/mach-rpc/ecard.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/arch/arm/mach-rpc/ecard.c b/arch/arm/mach-rpc/ecard.c index b91bc87b3dc..fcb1d59f7ae 100644 --- a/arch/arm/mach-rpc/ecard.c +++ b/arch/arm/mach-rpc/ecard.c @@ -960,7 +960,9 @@ static int __init ecard_probe(int slot, unsigned irq, card_type_t type) *ecp = ec; slot_to_expcard[slot] = ec; - device_register(&ec->dev); + rc = device_register(&ec->dev); + if (rc) + goto nodev; return 0; |