Re: [PATCH 2/2] drm/nouveau: Remove nvkm_gsp_fwif.enable

[Timur Tabi <ttabi@nvidia.com>]

On Tue, 2025-06-24 at 15:01 -0400, M Henning wrote:
> We did no such error checking before this series (in fact, most of
> these options have almost no error checking). Are you saying you want
> to see this added in this patch series?

You have a good point, but I think your change, in effect, necessitates my request.  Previously, the
default was no GSP-RM unless needed.  Now it's yes GSP-RM, and the concept of "need" has been
removed.  So there's no indication any more that some GPUs need GSP-RM and some do not.

So to address that, I think it makes sense to add a warning if someone tries disable GSP-RM on a GPU
that is not supported in that configuration.

Now, whether or not we should ignore NvGspRm=0 on Ada+ is up for debate.  If I understand the code
correctly, today (and still with your patches), Ada+ would fail to boot.  I can't say whether or not
that's a good idea.  But I think a warning should be printed either way.