[Nouveau] [PATCH 2/3] fb/gk20a: use regular gf100's functions

Karol Herbst karolherbst at gmail.com
Thu Oct 27 11:52:06 UTC 2016


looks good

Reviewed-By: Karol Herbst <karolherbst at gmail.com>

2016-10-27 9:55 GMT+02:00 Alexandre Courbot <acourbot at nvidia.com>:
> gk20a's FB is not special compared to other Kepler chips, besides the
> fact it does not have VRAM. Use the regular gf100 hooks instead of the
> incomplete versions we rewrote.
>
> Signed-off-by: Alexandre Courbot <acourbot at nvidia.com>
> ---
>  drm/nouveau/nvkm/subdev/fb/gk20a.c | 17 +++++------------
>  1 file changed, 5 insertions(+), 12 deletions(-)
>
> diff --git a/drm/nouveau/nvkm/subdev/fb/gk20a.c b/drm/nouveau/nvkm/subdev/fb/gk20a.c
> index 3f516f5adc30..5d34d6136616 100644
> --- a/drm/nouveau/nvkm/subdev/fb/gk20a.c
> +++ b/drm/nouveau/nvkm/subdev/fb/gk20a.c
> @@ -1,5 +1,5 @@
>  /*
> - * Copyright (c) 2014, NVIDIA CORPORATION. All rights reserved.
> + * Copyright (c) 2014-2016, NVIDIA CORPORATION. All rights reserved.
>   *
>   * Permission is hereby granted, free of charge, to any person obtaining a
>   * copy of this software and associated documentation files (the "Software"),
> @@ -22,21 +22,14 @@
>  #include "priv.h"
>  #include "gf100.h"
>
> -#include <core/memory.h>
> -
> -static void
> -gk20a_fb_init(struct nvkm_fb *fb)
> -{
> -       struct nvkm_device *device = fb->subdev.device;
> -       nvkm_wr32(device, 0x100cc8, nvkm_memory_addr(fb->mmu_wr) >> 8);
> -       nvkm_wr32(device, 0x100ccc, nvkm_memory_addr(fb->mmu_rd) >> 8);
> -}
> -
> +/* GK20A's FB is similar to GF100's, but without the ability to allocate VRAM */
>  static const struct nvkm_fb_func
>  gk20a_fb = {
> +       .dtor = gf100_fb_dtor,
>         .oneinit = gf100_fb_oneinit,
> -       .init = gk20a_fb_init,
> +       .init = gf100_fb_init,
>         .init_page = gf100_fb_init_page,
> +       .intr = gf100_fb_intr,
>         .memtype_valid = gf100_fb_memtype_valid,
>  };
>
> --
> 2.10.0
>
> _______________________________________________
> Nouveau mailing list
> Nouveau at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/nouveau


More information about the Nouveau mailing list