<div dir="ltr"><div>Hey Douglas,<br></div><div><br></div><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, 4 May 2021 at 16:39, Douglas Anderson <<a href="mailto:dianders@chromium.org">dianders@chromium.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">The ti_sn_gpio_unregister() is not just called from the remove path<br>
but also from the error handling of the init path. That means it can't<br>
have the __exit annotation.<br>
<br>
Fixes: bf73537f411b ("drm/bridge: ti-sn65dsi86: Break GPIO and MIPI-to-eDP bridge into sub-drivers")<br>
Reported-by: kernel test robot <<a href="mailto:lkp@intel.com" target="_blank">lkp@intel.com</a>><br>
Signed-off-by: Douglas Anderson <<a href="mailto:dianders@chromium.org" target="_blank">dianders@chromium.org</a>><br>
---<br>
<br>
 drivers/gpu/drm/bridge/ti-sn65dsi86.c | 2 +-<br>
 1 file changed, 1 insertion(+), 1 deletion(-)<br>
<br>
diff --git a/drivers/gpu/drm/bridge/ti-sn65dsi86.c b/drivers/gpu/drm/bridge/ti-sn65dsi86.c<br>
index db027528febd..bb0a0e1c6341 100644<br>
--- a/drivers/gpu/drm/bridge/ti-sn65dsi86.c<br>
+++ b/drivers/gpu/drm/bridge/ti-sn65dsi86.c<br>
@@ -1251,7 +1251,7 @@ static int __init ti_sn_gpio_register(void)<br>
        return auxiliary_driver_register(&ti_sn_gpio_driver);<br>
 }<br>
<br>
-static void __exit ti_sn_gpio_unregister(void)<br>
+static void ti_sn_gpio_unregister(void)<br>
 {<br>
        auxiliary_driver_unregister(&ti_sn_gpio_driver);<br>
 }<br>
-- <br>
2.31.1.527.g47e6f16901-goog<br><br></blockquote><div><br></div><div>Reviewed-by: Robert Foss <<a href="mailto:robert.foss@linaro.org">robert.foss@linaro.org</a>> <br></div></div></div>