Re: [PATCH v2 6/6] gpu: nova-core: enable GA100

["Gary Guo" <gary@garyguo.net>]

On Wed Apr 15, 2026 at 5:43 PM BST, Timur Tabi wrote:
> On Wed, 2026-04-15 at 14:51 +0100, Gary Guo wrote:
>> > GA100 is a compute-only variant of GA102 that boots GSP-RM like a
>> > Turing, although it also has its own unique requirements.
>> > 
>> > Now that all the pieces are in place, we can enable GA100 support.
>> > Although architecturally like an Ampere, GA100 uses the same GSP-RM
>> > firmware files as Turing, and therefore must boot it like Turing does.
>> > However, as a compute-only part, GA100 has no display engine.
>> 
>> This should go into the code as a comment.
>
> We don't describe any other GPU in the code.  Where exactly do you think this comment should go?

Something like

    GA100 | // GA100 boots like Turing so use Turing HAL
    TU102 | TU104 | TU106 | TU116 | TU117 => {

? Otherwise it just looks like broken code for those unaware of the details,
given it goes and use TU HAL despite being GA.

Best,
Gary