On Sat, 19 Jan 2013 07:51:35 -0600
Timur Tabi <timur@xxxxxxxx> wrote:
> Anatolij Gustschin wrote:
> > Disabling the interrupts in fsl_diu_release() must happen only if all
> > other AOIs are closed. Otherwise closing an overlay plane will disable
> > the interrupts even if the primary frame buffer plane is opened. Add
> > an appropriate check in the release function.
>
> I thought the release function is only called when the driver is unloaded.
> Wouldn't the framebuffers all already be closed by then?
when driver is unloaded the .remove() function is called, which is
fsl_diu_remove().
> > +static inline void fsl_diu_enable_interrupts(struct fsl_diu_data *data)
> > +{
> > + u32 int_mask = INT_UNDRUN; /* enable underrun detection */
> > +
> > + if (IS_ENABLED(CONFIG_NOT_COHERENT_CACHE))
> > + int_mask |= INT_VSYNC; /* enable vertical sync */
>
> Why did you turn this into a run-time check?
actually it is not a run-time check since this code will be optimized
away at compile in cases where CONFIG_NOT_COHERENT_CACHE is not selected
in the kernel config.
Thanks,
Anatolij
--
To unsubscribe from this list: send the line "unsubscribe linux-fbdev" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
