[PATCH 2/2] atm: iphase: fix misleading indention

2015-10-10 Thread Tillmann Heidsieck
Fix a smatch warning: drivers/atm/iphase.c:1178 rx_pkt() warn: curly braces intended? The code is correct, the indention is misleading. In case the allocation of skb fails, we want to skip to the end. Signed-off-by: Tillmann Heidsieck --- drivers/atm/iphase.c | 2 +- 1 file changed, 1

[PATCH 0/2] atm: iphase: Fix misleading indention and return -ENOMEM on error

2015-10-10 Thread Tillmann Heidsieck
not. Regards, Tillmann Heidsieck Tillmann Heidsieck (2): atm: iphase: return -ENOMEM instead of -1 in case of failed kmalloc() atm: iphase: fix misleading indention drivers/atm/iphase.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) -- 2.6.1 -- To unsubscribe from this list: s

[PATCH 1/2] atm: iphase: return -ENOMEM instead of -1 in case of failed kmalloc()

2015-10-10 Thread Tillmann Heidsieck
Smatch complains about returning hard coded error codes, silence this warning. drivers/atm/iphase.c:115 ia_enque_rtn_q() warn: returning -1 instead of -ENOMEM is sloppy Signed-off-by: Tillmann Heidsieck --- drivers/atm/iphase.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff