On Tue, Mar 10, 2015 at 09:17:54PM +0100, Greg Kroah-Hartman wrote: > On Tue, Mar 10, 2015 at 09:11:00PM +0100, Greg Kroah-Hartman wrote: > > On Tue, Mar 10, 2015 at 10:46:55PM +0530, Sudip Mukherjee wrote: > > > we were getting build warnings about assignment of incompatible <snip> > > > > That's a nice cleanup, but it's not even the correct cleanup. > > Oops, sorry, it is the correct cleanup, I was looking at the '*' > placement, not the const part. > > but the patch needs to be redone, I'll just go fix up the build warnings > for now... thanks Greg for your patience and time to redo this patch. I saw you broke it into two - three different patches. But just one doubt, was your compiler showing any warning for the patch - "Staging: sm750fb: provide error path for hw_sm750le_setBLANK()" , commit id - e74ac550298ec4635cc32e99f966568a808fd370 . my compiler didnot show any warning for that. mine is gcc 4.7.3, is it time to update? thanks again, and sorry that you had to do fix this. I should have made the patches in a more proper way. regards sudip > > greg k-h _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel