On Mon, Jul 20, 2015 at 05:59:29PM +0530, Sudip Mukherjee wrote: > We were dereferencing dev first and then checking if it is NULL. Lets > check for NULL first and then dereference. The code is bonkers. Testing for a lack of a correctly constructed debugfs seq_file inside the debugfs seq_file callback is inane. -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx