If 'platform_get_irq()' fails, it is spurious to call 'of_reserved_mem_device_release()' in the error handling path, because 'of_reserved_mem_device_init() has not been called yet.

Moreover, a previous 'kmb_initialize_clocks()' is unbalanced by a corresponding 'kmb_display_clk_disable()' call, has already done in the remove function.

It is likely that 'kmb_display_clk_disable()' is expected in the error handling path, instead of 'of_reserved_mem_device_release()'.

Also, it is spurious to return directly if 'of_reserved_mem_device_init()' fails. Goto the error handling path instead to free some resources.

Fixes: 7f7b96a8a0a1 ("drm/kmb: Add support for KeemBay Display") Signed-off-by: Christophe JAILLET christophe.jaillet@wanadoo.fr --- v2: Keep label name Fix the commit message where a wrong function name was used --- drivers/gpu/drm/kmb/kmb_drv.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/kmb/kmb_drv.c b/drivers/gpu/drm/kmb/kmb_drv.c index f64e06e1067d..fa28e42da460 100644 --- a/drivers/gpu/drm/kmb/kmb_drv.c +++ b/drivers/gpu/drm/kmb/kmb_drv.c @@ -144,7 +144,7 @@ static int kmb_hw_init(struct drm_device *drm, unsigned long flags) /* Get the optional framebuffer memory resource */ ret = of_reserved_mem_device_init(drm->dev); if (ret && ret != -ENODEV) - return ret; + goto setup_fail;

spin_lock_init(&kmb->irq_lock);

@@ -153,7 +153,7 @@ static int kmb_hw_init(struct drm_device *drm, unsigned long flags) return 0;

setup_fail: - of_reserved_mem_device_release(drm->dev); + kmb_display_clk_disable(kmb);

return ret; }