On 2025-04-29 15:47, Alex Deucher wrote: > This is an optional TA which is only available on > certain embedded systems. Mark it as optional to avoid > user confusion. This mirrors what we already do for > other optional TAs. > > Closes: https://gitlab.freedesktop.org/drm/amd/-/issues/4181 > Signed-off-by: Alex Deucher <alexander.deuc...@amd.com> > --- > drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c > b/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c > index d54bb13772622..1c843b888475e 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c > @@ -2214,7 +2214,8 @@ static int psp_securedisplay_initialize(struct > psp_context *psp) > > if (!psp->securedisplay_context.context.bin_desc.size_bytes || > !psp->securedisplay_context.context.bin_desc.start_addr) { > - dev_info(psp->adev->dev, "SECUREDISPLAY: securedisplay ta ucode > is not available\n"); > + dev_info(psp->adev->dev, > + "SECUREDISPLAY: optional securedisplay ta ucode is not > available\n"); > return 0; > } >
Does it need to be dev_info, or could it be dev_dbg? -- Earthling Michel Dänzer \ GNOME / Xwayland / Mesa developer https://redhat.com \ Libre software enthusiast
