On 04/10/16 12:25, Daniel Vetter wrote:
On Tue, Oct 4, 2016 at 12:10 PM, Jon Hunter jonathanh@nvidia.com wrote:
Looks like crtc is a errno in the above case. I see this function is called by looping through all the crtc and we never check to see if they are valid. Should we?
Tegra is still using the load/unload hooks. That didn't mesh well with Tomeu's patches (and Tomeu's patches have been thrown out meanwhile because of that). Still would be neat if tegra could be demidlayered and loose it's load/unload hooks. See the kerneldoc in drm_drv.c (especially the DOC: section).
Adding Thierry and Alex as this is more their domain and CC'ing linux-tegra.
Cheers Jon