[PATCH v4 17/20] gpu: nova-core: compute layout of the FRTS region

[Danilo Krummrich]

On Wed, May 21, 2025 at 03:45:12PM +0900, Alexandre Courbot wrote:
> +impl Chipset {
> +    /// Returns the HAL corresponding to this chipset.
> +    pub(super) fn get_fb_fal(self) -> &'static dyn FbHal {

Please don't use the 'get' prefix here.

Also, I feel like it's a bit random to have this on Chipset. I think the
standalone function to create a HAL was fine. (Same for falcon/hal.rs, where I
missed this.)