Re: [PATCH v3 5/7] gpu: nova-core: transition gen_bootloader to TLV images

[Timur Tabi <ttabi@nvidia.com>]

On Mon, 2026-07-06 at 15:31 +0900, Alexandre Courbot wrote:
> > +        dev_info!(
> > +            dev,
> > +            "loaded generic bootloader firmware v{}\n",
> > +            tlv.get_string(b"VERS")?
> > +        );
> 
> s/dev_info/dev_dbg.
> 
> Also, since `dev_dbg` can be compiled out, the check performed on the
> `VERS` tag existing is not guaranteed to be performed. We should
> probably extract it out of the `dev_dbg` statement. This is true across
> the series; which makes me think it might also benefit from a helper to
> factor out the request_t.v/Tlv::new/dev_dbg common sequence.

We don't care about the VERS tag if we're not going to print it out, so we probably should not
extract it otherwise.

I know it's a common pattern, but I couldn't find a way to cleanly combine request_tlv() with
Tlv::new().