[PATCH] drm/sun4i: Clear encoder->bridge if a bridge is not found
Maxime Ripard
maxime.ripard at free-electrons.com
Wed Aug 31 16:43:00 UTC 2016
On Wed, Aug 31, 2016 at 06:27:08PM +0200, Daniel Vetter wrote:
> > > >> + if (IS_ERR(encoder->bridge))
> > > >> + encoder->bridge = NULL;
> > > >> +
> > > >
> > > > And that could be the else condition of the if statement below.
> > >
> > > That would be a bit confusing, changing it after calling drm_encoder_init.
> > > The code says it ok to do though.
> >
> > The magic really happens only after the encoder has been attached to
> > something, so it's really safe.
>
> s/attached/registered using drm_dev_register(). Which should
> happen _way_ later for all drivers which have gotten rid of their ->load
> callback and implemented the recommend driver load sequence.
My bad, thanks!
Maxime
--
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160831/39904c8d/attachment.sig>
More information about the dri-devel
mailing list