Re: [PATCH] drm/bridge: ti-sn65dsi86: Use dev_err_probe() to avoid polluting the log

2022-07-22 Thread Javier Martinez Canillas
On 7/22/22 15:35, Doug Anderson wrote: > Hi, > > On Fri, Jul 22, 2022 at 12:48 AM Javier Martinez Canillas > wrote: >> >> If devm_drm_of_get_bridge() can't find the connected bridge, it returns an >> ERR_PTR(-EPROBE_DEFER) to indicate that the probe should be deferred. >> >> But this path also pr

Re: [PATCH] drm/bridge: ti-sn65dsi86: Use dev_err_probe() to avoid polluting the log

2022-07-22 Thread Doug Anderson
Hi, On Fri, Jul 22, 2022 at 12:48 AM Javier Martinez Canillas wrote: > > If devm_drm_of_get_bridge() can't find the connected bridge, it returns an > ERR_PTR(-EPROBE_DEFER) to indicate that the probe should be deferred. > > But this path also prints an error message, which pollutes the kernel log

Re: [PATCH] drm/bridge: ti-sn65dsi86: Use dev_err_probe() to avoid polluting the log

2022-07-22 Thread Andrzej Hajda
On 22.07.2022 09:47, Javier Martinez Canillas wrote: If devm_drm_of_get_bridge() can't find the connected bridge, it returns an ERR_PTR(-EPROBE_DEFER) to indicate that the probe should be deferred. But this path also prints an error message, which pollutes the kernel log since is printed on e

Re: [PATCH] drm/bridge: ti-sn65dsi86: Use dev_err_probe() to avoid polluting the log

2022-07-22 Thread Neil Armstrong
On 22/07/2022 09:47, Javier Martinez Canillas wrote: If devm_drm_of_get_bridge() can't find the connected bridge, it returns an ERR_PTR(-EPROBE_DEFER) to indicate that the probe should be deferred. But this path also prints an error message, which pollutes the kernel log since is printed on ever

[PATCH] drm/bridge: ti-sn65dsi86: Use dev_err_probe() to avoid polluting the log

2022-07-22 Thread Javier Martinez Canillas
If devm_drm_of_get_bridge() can't find the connected bridge, it returns an ERR_PTR(-EPROBE_DEFER) to indicate that the probe should be deferred. But this path also prints an error message, which pollutes the kernel log since is printed on every probe deferral, i.e: $ dmesg | grep "failed to cre