On Sat, Dec 14, 2013 at 11:44:06AM +0100, Daniel Vetter wrote: > On Fri, Dec 13, 2013 at 04:43:50PM -0800, Jesse Barnes wrote: > > This is just fixing up existing code to use the new field name, so no > > functional change. I see what you mean about splitting out the field > > change, but now that would be a pain :/ > > Yeah, the switch from struct to pointer for ifbdev->fb would be neat as a > separate patch, but also real pain to split out now. I'm inclined to say that we actually need the split-out conversion to a pointer. I might have lost my traces somewhere in this maze, but I think with the current patches we can have a NULL deref on ifbdev->fb in intelfb_create. Not sure, but a split-out patch would definitely help ;-) -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
