On 11/29/16 22:26, Laurent Pinchart wrote: > Hi Jyri, > > On Tuesday 29 Nov 2016 22:18:25 Jyri Sarha wrote: >> > The module unload should not be allowed while the bridge is attached. So >> > still need to add these: >> > >> > On 11/25/16 11:02, Jyri Sarha wrote: >>> > > + >>> > > +static int tfp410_attach(struct drm_bridge *bridge) >>> > > +{ >>> > > + struct tfp410 *dvi = drm_bridge_to_tfp410(bridge); >>> > > + int ret; >>> > > + >> > >> > + if (!try_module_get(THIS_MODULE)) { >> > + dev_err(dvi->dev, "Module unloading\n"); >> > + return -ENODEV; >> > + } >> > + > Shouldn't this be done in core code ? > Hmmm, probably. I'll send a patch shortly. Cheers, Jyri _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel