[PATCH 3/3] drm/bridge: Return NULL instead of plain 0 in drm_dp_hpd_bridge_register() stub
Bryan O'Donoghue
bryan.odonoghue at linaro.org
Tue Dec 5 22:04:31 UTC 2023
On 05/12/2023 21:13, Nathan Chancellor wrote:
> sparse complains:
>
> drivers/usb/typec/tcpm/qcom/qcom_pmic_typec.c: note: in included file:
> include/drm/bridge/aux-bridge.h:29:16: sparse: sparse: Using plain integer as NULL pointer
>
> Return NULL to clear up the warning.
>
> Reported-by: kernel test robot <lkp at intel.com>
> Closes: https://lore.kernel.org/oe-kbuild-all/202312060025.BdeqZrWx-lkp@intel.com/
> Fixes: e560518a6c2e ("drm/bridge: implement generic DP HPD bridge")
> Signed-off-by: Nathan Chancellor <nathan at kernel.org>
> ---
> include/drm/bridge/aux-bridge.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/include/drm/bridge/aux-bridge.h b/include/drm/bridge/aux-bridge.h
> index 66249ff0858e..c4c423e97f06 100644
> --- a/include/drm/bridge/aux-bridge.h
> +++ b/include/drm/bridge/aux-bridge.h
> @@ -26,7 +26,7 @@ void drm_aux_hpd_bridge_notify(struct device *dev, enum drm_connector_status sta
> static inline struct device *drm_dp_hpd_bridge_register(struct device *parent,
> struct device_node *np)
> {
> - return 0;
> + return NULL;
> }
>
> static inline void drm_aux_hpd_bridge_notify(struct device *dev, enum drm_connector_status status)
>
Nice
Reviewed-by: Bryan O'Donoghue <bryan.odonoghue at linaro.org>
More information about the dri-devel
mailing list