Michel Hermier <[email protected]> writes: > For my culture, why did you not changed the type to uint32_t like in > my original patch ? As I told you before, those changes went over the 80 column limit so I just left them out. I hope you don't mind, the modified patch should do the same thing as the original one for all practical purposes.
> Also looking at include/drm/nouveau_drm.h, I see some inconsistant > type for channel in ioctls structures ? > Right, explicitly sized integers are usually the preferred choice in the IOCTL API. > 2010/12/25 Francisco Jerez <[email protected]>: >> Michel Hermier <[email protected]> writes: >> >>> Second try ^^ I'm not used at all at preparing patch for kernel >> >> Thanks. I've pushed it with a couple of minor fixes. >>
pgpb87sbrYx7P.pgp
Description: PGP signature
_______________________________________________ Nouveau mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/nouveau
