[PATCH v5 3/4] drm/bridge: Add ti-tfp410 DVI transmitter driver

[Jyri Sarha]

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