* Error handling in drivers/pci/host/pci-dra7xx.c
@ 2016-07-13 4:47 Christophe JAILLET
2016-07-13 4:59 ` Christophe JAILLET
0 siblings, 1 reply; 3+ messages in thread
From: Christophe JAILLET @ 2016-07-13 4:47 UTC (permalink / raw)
To: kernel-janitors
Hi,
in file 'drivers/pci/host/pci-dra7xx.c', error handling in
'dra7xx_pcie_init_irq_domain()' seams to be broken.
The pattern used is:
if (!var) {
...
dev_err(...);
return PTR_ERR(var);
}
so we always return '0', which means success.
Returning PTR_ERR(-EINVAL) or PTR_ERR(-ENODEV) could be, IMHO, better.
Best regards,
CJ
^ permalink raw reply [flat|nested] 3+ messages in thread* Error handling in drivers/pci/host/pci-dra7xx.c
@ 2016-07-13 4:59 ` Christophe JAILLET
0 siblings, 0 replies; 3+ messages in thread
From: Christophe JAILLET @ 2016-07-13 4:59 UTC (permalink / raw)
To: linux-omap, linux-pci
(resent because of broken cc)
Hi,
in file 'drivers/pci/host/pci-dra7xx.c', error handling in
'dra7xx_pcie_init_irq_domain()' seams to be broken.
The pattern used is:
if (!var) {
...
dev_err(...);
return PTR_ERR(var);
}
so we always return '0', which means success.
Returning PTR_ERR(-EINVAL) or PTR_ERR(-ENODEV) could be, IMHO, better.
Best regards,
CJ
^ permalink raw reply [flat|nested] 3+ messages in thread* Error handling in drivers/pci/host/pci-dra7xx.c
@ 2016-07-13 4:59 ` Christophe JAILLET
0 siblings, 0 replies; 3+ messages in thread
From: Christophe JAILLET @ 2016-07-13 4:59 UTC (permalink / raw)
To: linux-omap, linux-pci
(resent because of broken cc)
Hi,
in file 'drivers/pci/host/pci-dra7xx.c', error handling in
'dra7xx_pcie_init_irq_domain()' seams to be broken.
The pattern used is:
if (!var) {
...
dev_err(...);
return PTR_ERR(var);
}
so we always return '0', which means success.
Returning PTR_ERR(-EINVAL) or PTR_ERR(-ENODEV) could be, IMHO, better.
Best regards,
CJ
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2016-07-13 4:59 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-07-13 4:47 Error handling in drivers/pci/host/pci-dra7xx.c Christophe JAILLET
2016-07-13 4:59 ` Christophe JAILLET
2016-07-13 4:59 ` Christophe JAILLET
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.