On Wed, Oct 25, 2017 at 07:18:28PM +0300, Ville Syrjälä wrote: > It also occurs to me that we might want to combine this fb dumper with > the atomic state dumper. That one already prints out stuf about > framebuffers in drm_atomic_plane_print_state(). Not sure if there > would be a nice way to keep the extra indentation with shared code > though. I guess one could always pass on some kind of prefix string to > thee fb dump function. Yeah dumping the full details with the same function introduced in this patch here might be useful there too. Since we just pass around seq_file it should be all cool (well probably should pass around drm_printer if we don't do that already). -Daniel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
