On Wed, 20 Mar 2024, José Roberto de Souza <jose.souza@xxxxxxxxx> wrote:
> It is misleading, if the intention was to also print something
> in case it succeed it should have a different string.
>
> Cc: Alan Previn <alan.previn.teres.alexis@xxxxxxxxx>
> Signed-off-by: José Roberto de Souza <jose.souza@xxxxxxxxx>
Fixes: 698e19da2914 ("drm/i915: Skip pxp init if gt is wedged")
Cc: <stable@xxxxxxxxxxxxxxx> # v6.8+
Reviewed-by: Jani Nikula <jani.nikula@xxxxxxxxx>
> ---
> drivers/gpu/drm/i915/i915_driver.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/i915/i915_driver.c b/drivers/gpu/drm/i915/i915_driver.c
> index 9ee902d5b72c4..4b9233c07a22c 100644
> --- a/drivers/gpu/drm/i915/i915_driver.c
> +++ b/drivers/gpu/drm/i915/i915_driver.c
> @@ -800,7 +800,7 @@ int i915_driver_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
> goto out_cleanup_modeset2;
>
> ret = intel_pxp_init(i915);
> - if (ret != -ENODEV)
> + if (ret && ret != -ENODEV)
> drm_dbg(&i915->drm, "pxp init failed with %d\n", ret);
>
> ret = intel_display_driver_probe(i915);
--
Jani Nikula, Intel
