[PATCH v3] soc: fsl: qe: Add check for ioremap

2022-10-10 Thread Jiasheng Jiang
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

[PATCH v2] soc: fsl: qe: Add check for ioremap

2022-10-10 Thread Jiasheng Jiang
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

[PATCH] soc: fsl: qe: Add check for ioremap

2022-10-07 Thread Jiasheng Jiang
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/

[PATCH] soc: fsl: qe: return platform_driver_register

2022-09-13 Thread Jiasheng Jiang
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

[PATCH] soc: fsl: qe: Deal with the return value of platform_driver_register

2022-05-26 Thread Jiasheng Jiang
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

[PATCH] soc: fsl: qe: Check of ioremap return value

2021-12-29 Thread Jiasheng Jiang
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:

[PATCH] PCI/AER: potential dereference of null pointer

2021-12-09 Thread Jiasheng Jiang
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