search for: nvkm_disp_func

...7 +359,7 @@ nvkm_disp_oneinit(struct nvkm_engine *engine) if (ret) return ret; - if (disp->func->oneinit) { + if (disp->func && disp->func->oneinit) { ret = disp->func->oneinit(disp); if (ret) return ret; @@ -461,8 +461,10 @@ nvkm_disp_new_(const struct nvkm_disp_func *func, struct nvkm_device *device, spin_lock_init(&disp->client.lock); ret = nvkm_engine_ctor(&nvkm_disp, device, type, inst, true, &disp->engine); - if (ret) + if (ret) { + disp->func = NULL; return ret; + } if (func->super) { disp->super.wq = create_sin...

...t; - if (disp->func->oneinit) { > + if (disp->func && disp->func->oneinit) { > ret = disp->func->oneinit(disp); > if (ret) > return ret; > @@ -461,8 +461,10 @@ nvkm_disp_new_(const struct nvkm_disp_func *func, struct nvkm_device *device, > spin_lock_init(&disp->client.lock); > > ret = nvkm_engine_ctor(&nvkm_disp, device, type, inst, true, &disp->engine); > - if (ret) > + if (ret) { > + disp->func = NULL; >...

...; + * > > + * This structure is for <some abstraction here>. It has <some assumptions > > + * about its usage here>. > > + */ > > struct nvkm_disp { > > + /** @func: Chipset-specific vtable for manipulating the display. */ > > const struct nvkm_disp_func *func; > > + > > + /** @engine: The engine for this display. */ > > struct nvkm_engine engine; > > > > + /** @head: list of heads attached to this display. */ > > struct list_head head; > > + > > + /** @ior: List of IO resourc...

...structure is for <some abstraction here>. It has <some assumptions > > > + * about its usage here>. > > > + */ > > > struct nvkm_disp { > > > + /** @func: Chipset-specific vtable for manipulating the display. */ > > > const struct nvkm_disp_func *func; > > > + > > > + /** @engine: The engine for this display. */ > > > struct nvkm_engine engine; > > > > > > + /** @head: list of heads attached to this display. */ > > > struct list_head head; > > > + > &gt...

...here>. It has <some assumptions > >>>> + * about its usage here>. > >>>> + */ > >>>> struct nvkm_disp { > >>>> + /** @func: Chipset-specific vtable for manipulating the display. */ > >>>> const struct nvkm_disp_func *func; > >>>> + > >>>> + /** @engine: The engine for this display. */ > >>>> struct nvkm_engine engine; > >>>> > >>>> + /** @head: list of heads attached to this display. */ > >>>> stru...

...for <some abstraction here>. It has <some assumptions >>>> + * about its usage here>. >>>> + */ >>>> struct nvkm_disp { >>>> + /** @func: Chipset-specific vtable for manipulating the display. */ >>>> const struct nvkm_disp_func *func; >>>> + >>>> + /** @engine: The engine for this display. */ >>>> struct nvkm_engine engine; >>>> >>>> + /** @head: list of heads attached to this display. */ >>>> struct list_head head; >>&gt...

...nt.h> +/** + * struct nvkm_disp - Represents a display engine. + * + * This structure is for <some abstraction here>. It has <some assumptions + * about its usage here>. + */ struct nvkm_disp { + /** @func: Chipset-specific vtable for manipulating the display. */ const struct nvkm_disp_func *func; + + /** @engine: The engine for this display. */ struct nvkm_engine engine; + /** @head: list of heads attached to this display. */ struct list_head head; + + /** @ior: List of IO resources for this display. */ struct list_head ior; + + /** @outp: List of outputs for this...

...presents a display engine. > + * > + * This structure is for <some abstraction here>. It has <some assumptions > + * about its usage here>. > + */ > struct nvkm_disp { > + /** @func: Chipset-specific vtable for manipulating the display. */ > const struct nvkm_disp_func *func; > + > + /** @engine: The engine for this display. */ > struct nvkm_engine engine; > > + /** @head: list of heads attached to this display. */ > struct list_head head; > + > + /** @ior: List of IO resources for this display. */ > str...

This is a collection of nouveau debug prints, memory leak, a very annoying race condition causing system hangs with prime scenarios, and a fix from Lyude to get the panel on my laptop working. I'd like to get these into 6.7, Dave.