On Tue, Oct 11, 2022 at 04:49:48PM +0300, Jani Nikula wrote:
> Conform to device specific logging.
> 
> Signed-off-by: Jani Nikula <[email protected]>
> ---
>  drivers/gpu/drm/drm_edid_load.c | 10 ++++++----
>  1 file changed, 6 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/gpu/drm/drm_edid_load.c b/drivers/gpu/drm/drm_edid_load.c
> index 882caaa6e663..dd472c66cb3c 100644
> --- a/drivers/gpu/drm/drm_edid_load.c
> +++ b/drivers/gpu/drm/drm_edid_load.c
> @@ -177,16 +177,18 @@ static const struct drm_edid *edid_load(struct 
> drm_connector *connector, const c
>  
>               pdev = platform_device_register_simple(connector->name, -1, 
> NULL, 0);
>               if (IS_ERR(pdev)) {
> -                     DRM_ERROR("Failed to register EDID firmware platform 
> device "
> -                               "for connector \"%s\"\n", connector->name);
> +                     drm_err(connector->dev,
> +                             "Failed to register EDID firmware platform 
> device for connector \"%s\"\n",
> +                             connector->name);

Can go for the full [CONNECTOR:...] thing?

>                       return ERR_CAST(pdev);
>               }
>  
>               err = request_firmware(&fw, name, &pdev->dev);
>               platform_device_unregister(pdev);
>               if (err) {
> -                     DRM_ERROR("Requesting EDID firmware \"%s\" failed 
> (err=%d)\n",
> -                               name, err);
> +                     drm_err(connector->dev,
> +                             "Requesting EDID firmware \"%s\" failed 
> (err=%d)\n",
> +                             name, err);
>                       return ERR_PTR(err);
>               }
>  
> -- 
> 2.34.1

-- 
Ville Syrjälä
Intel

Reply via email to