Re: [PATCH xf86-video-nouveau] wfb: Fix missing init function decls behind FB_ACCESS_WRAPPER

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]<

 



On Wed, Jan 23, 2019 at 12:41 AM Ilia Mirkin <imirkin@xxxxxxxxxxxx> wrote:
This causes nouveau_drv.so to not be loadable due to a missing
wfbPictureInit. I'm not 100% sure what causes it -- it happens at
runtime in the middle of probe, and I don't have wfb enabled. Either
way, I'm going to revert this until we get a better idea of what's
going on.

OK to revert, this patch must be exposing something else that is going on with wrappedfb support on some setups.


On Sun, Jan 20, 2019 at 7:20 PM Rhys Kidd <rhyskidd@xxxxxxxxx> wrote:
>
> Fixes warning with xserver 1.20 and gcc 8.2:
>
> nv_driver.c:1443:9: warning: implicit declaration of function ‘wfbScreenInit’; did you mean ‘fbScreenInit’? [-Wimplicit-function-declaration]
>    ret = wfbScreenInit(pScreen, FBStart, pScrn->virtualX,
>          ^~~~~~~~~~~~~
>          fbScreenInit
>
> (See xserver 706e6d9cd074da606016ed4ecff51e9c2a822087)
>
> Signed-off-by: Rhys Kidd <rhyskidd@xxxxxxxxx>
> ---
>  src/nv_include.h | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/src/nv_include.h b/src/nv_include.h
> index de55570..6fdeeab 100644
> --- a/src/nv_include.h
> +++ b/src/nv_include.h
> @@ -38,6 +38,7 @@
>  #include "dixstruct.h"
>  #include "scrnintstr.h"
>
> +#define FB_ACCESS_WRAPPER
>  #include "fb.h"
>
>  #include "xf86cmap.h"
> --
> 2.19.1
>
> _______________________________________________
> Nouveau mailing list
> Nouveau@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/nouveau
_______________________________________________
Nouveau mailing list
Nouveau@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/nouveau

[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux