net: wan: fsl_ucc_hdlc: fix ucc_hdlc_remove

If the driver is used in a non tdm mode priv->utdm is a NULL pointer.
Therefore we need to check this pointer first before checking si_regs.

Fixes: c19b6d246a35 ("drivers/net: support hdlc function for QE-UCC")
Signed-off-by: Holger Brunck <holger.brunck@hitachienergy.com>
Reviewed-by: Simon Horman <horms@kernel.org>
Signed-off-by: Jakub Kicinski <kuba@kernel.org>

diff --git a/drivers/net/wan/fsl_ucc_hdlc.c b/drivers/net/wan/fsl_ucc_hdlc.c
index 8155e92af14e23d3e83625de3d18ece56dbac006..15bfb78381d4cb749b4114558679fc5af86a34de 100644 (file)
--- a/drivers/net/wan/fsl_ucc_hdlc.c
+++ b/drivers/net/wan/fsl_ucc_hdlc.c
@@ -1250,12 +1250,12 @@ static void ucc_hdlc_remove(struct platform_device *pdev)
 
        uhdlc_memclean(priv);
 
-       if (priv->utdm->si_regs) {
+       if (priv->utdm && priv->utdm->si_regs) {
                iounmap(priv->utdm->si_regs);
                priv->utdm->si_regs = NULL;
        }
 
-       if (priv->utdm->siram) {
+       if (priv->utdm && priv->utdm->siram) {
                iounmap(priv->utdm->siram);
                priv->utdm->siram = NULL;
        }