ned-off-by: Jiasheng Jiang
---
Changelog:
v2 -> v3:
1. Add return statement of qe_reset when CONFIG_QUICC_ENGINE is not selected.
v1 -> v2:
1. Change the position of the check for ioremap.
2. Simplify the check for qe_reset.
3. Remove the 'extern' keyword.
---
drivers/soc/fsl/qe/qe.c
ned-off-by: Jiasheng Jiang
---
Changelog:
v1 -> v2:
1. Change the position of the check for ioremap.
2. Simplify the check for qe_reset.
3. Remove the 'extern' keyword.
---
drivers/soc/fsl/qe/qe.c | 12
include/soc/fsl/qe/qe.h | 4 ++--
2 files changed, 10 insertions(+), 6 del
As ioremap can return NULL pointer, it should
be better to check the return value return error
if fails.
Moreover, the return value of qe_reset should be
checked by cascade.
Fixes: 68f047e3d62e ("fsl/qe: add rx_sync and tx_sync for TDM mode")
Signed-off-by: Jiasheng Jiang
---
drivers/
As platform_driver_register() can return error numbers,
it should be better to return platform_driver_register()
instead of return 0.
Fixes: be7ecbd240b2 ("soc: fsl: qe: convert QE interrupt controller to
platform_device")
Signed-off-by: Jiasheng Jiang
---
drivers/soc/fsl/qe/qe_ic.c
As platform_driver_register() could fail, it should be better
to deal with the return value in order to maintain the code
consisitency.
Fixes: be7ecbd240b2 ("soc: fsl: qe: convert QE interrupt controller to
platform_device")
Signed-off-by: Jiasheng Jiang
---
drivers/soc/fsl/qe/q
x27;t check the return value of
the par_io_init().
Actually, par_io_init() needs to check to handle the potential error.
I will submit another patch to fix that.
Anyway, par_io_init() itsely should be fixed.
Fixes: 7aa1aa6ecec2 ("QE: Move QE from arch/powerpc to drivers/soc")
Signed-off-by:
he return value of kzalloc() needs to be checked.
To avoid use of null pointer in case of the failure of alloc.
Fixes: db89ccbe52c7 ("PCI/AER: Define aer_stats structure for AER capable
devices")
Signed-off-by: Jiasheng Jiang
---
drivers/pci/pcie/aer.c | 2 ++
1 file changed, 2