Re: [Intel-gfx] [PATCH] drm/dp: Do not prune the last mode on the connector

2017-09-27 Thread Manasi Navare
On Wed, Sep 27, 2017 at 05:31:56PM -0700, Keith Packard wrote: > Manasi Navare writes: > > > This patch fixes this problem by checking if the mode being pruned > > is the last mode on that connector and if so doesnt prune it. > > I think you want to stop pruning when

Re: [Intel-gfx] [PATCH] drm/dp: Do not prune the last mode on the connector

2017-09-27 Thread Keith Packard
Manasi Navare writes: > This patch fixes this problem by checking if the mode being pruned > is the last mode on that connector and if so doesnt prune it. I think you want to stop pruning when you've gotten to a single mode on the list, not at the last mode in the

[Intel-gfx] [PATCH] drm/dp: Do not prune the last mode on the connector

2017-09-27 Thread Manasi Navare
Currently the drm_mode_prune_invalid() function will prune all the modes if it finds that the mode-status is not MODE_OK. But if it ends up pruning all modes then there are no modes left for that connector which will eventually result into a black screen as userspace sees no modes from the kernel.