* [PATCH] drm: Fix drm.h uapi header for GNU/kFreeBSD @ 2019-01-15 15:04 James Clarke 2019-01-15 15:17 ` Daniel Vetter 2019-05-09 17:42 ` Eric Anholt 0 siblings, 2 replies; 10+ messages in thread From: James Clarke @ 2019-01-15 15:04 UTC (permalink / raw) To: dri-devel; +Cc: James Clarke Like GNU/Linux, GNU/kFreeBSD's sys/types.h does not define the uintX_t types, which differs from the BSDs' headers. Thus we should include stdint.h to ensure we have all the required integer types. Signed-off-by: James Clarke <jrtc27@jrtc27.com> --- include/uapi/drm/drm.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/uapi/drm/drm.h b/include/uapi/drm/drm.h index 300f33663..caf4d9b38 100644 --- a/include/uapi/drm/drm.h +++ b/include/uapi/drm/drm.h @@ -50,6 +50,7 @@ typedef unsigned int drm_handle_t; #else /* One of the BSDs */ +#include <stdint.h> #include <sys/ioccom.h> #include <sys/types.h> typedef int8_t __s8; -- 2.17.1 _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH] drm: Fix drm.h uapi header for GNU/kFreeBSD 2019-01-15 15:04 [PATCH] drm: Fix drm.h uapi header for GNU/kFreeBSD James Clarke @ 2019-01-15 15:17 ` Daniel Vetter 2019-01-15 18:41 ` Eric Anholt 2019-05-09 17:42 ` Eric Anholt 1 sibling, 1 reply; 10+ messages in thread From: Daniel Vetter @ 2019-01-15 15:17 UTC (permalink / raw) To: James Clarke; +Cc: dri-devel On Tue, Jan 15, 2019 at 03:04:18PM +0000, James Clarke wrote: > Like GNU/Linux, GNU/kFreeBSD's sys/types.h does not define the uintX_t > types, which differs from the BSDs' headers. Thus we should include > stdint.h to ensure we have all the required integer types. > > Signed-off-by: James Clarke <jrtc27@jrtc27.com> Would be good to get an ack from some other *bsd that this is still all fine. lgtm otherwise. -Daniel > --- > include/uapi/drm/drm.h | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/include/uapi/drm/drm.h b/include/uapi/drm/drm.h > index 300f33663..caf4d9b38 100644 > --- a/include/uapi/drm/drm.h > +++ b/include/uapi/drm/drm.h > @@ -50,6 +50,7 @@ typedef unsigned int drm_handle_t; > > #else /* One of the BSDs */ > > +#include <stdint.h> > #include <sys/ioccom.h> > #include <sys/types.h> > typedef int8_t __s8; > -- > 2.17.1 > > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] drm: Fix drm.h uapi header for GNU/kFreeBSD 2019-01-15 15:17 ` Daniel Vetter @ 2019-01-15 18:41 ` Eric Anholt 2019-05-04 20:43 ` James Clarke 0 siblings, 1 reply; 10+ messages in thread From: Eric Anholt @ 2019-01-15 18:41 UTC (permalink / raw) To: Daniel Vetter, James Clarke; +Cc: dri-devel [-- Attachment #1.1: Type: text/plain, Size: 646 bytes --] Daniel Vetter <daniel@ffwll.ch> writes: > On Tue, Jan 15, 2019 at 03:04:18PM +0000, James Clarke wrote: >> Like GNU/Linux, GNU/kFreeBSD's sys/types.h does not define the uintX_t >> types, which differs from the BSDs' headers. Thus we should include >> stdint.h to ensure we have all the required integer types. >> >> Signed-off-by: James Clarke <jrtc27@jrtc27.com> > > Would be good to get an ack from some other *bsd that this is still all > fine. lgtm otherwise. > -Daniel I think there was some need for inttypes.h instead of stdint like a decade ago when I was working on BSDs, but that was already almost irrelevant then. [-- Attachment #1.2: signature.asc --] [-- Type: application/pgp-signature, Size: 832 bytes --] [-- Attachment #2: Type: text/plain, Size: 160 bytes --] _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] drm: Fix drm.h uapi header for GNU/kFreeBSD 2019-01-15 18:41 ` Eric Anholt @ 2019-05-04 20:43 ` James Clarke 2019-05-06 8:16 ` Daniel Vetter 0 siblings, 1 reply; 10+ messages in thread From: James Clarke @ 2019-05-04 20:43 UTC (permalink / raw) To: Eric Anholt; +Cc: dri-devel On 15 Jan 2019, at 18:41, Eric Anholt <eric@anholt.net> wrote: > > Daniel Vetter <daniel@ffwll.ch> writes: > >> On Tue, Jan 15, 2019 at 03:04:18PM +0000, James Clarke wrote: >>> Like GNU/Linux, GNU/kFreeBSD's sys/types.h does not define the uintX_t >>> types, which differs from the BSDs' headers. Thus we should include >>> stdint.h to ensure we have all the required integer types. >>> >>> Signed-off-by: James Clarke <jrtc27@jrtc27.com> >> >> Would be good to get an ack from some other *bsd that this is still all >> fine. lgtm otherwise. >> -Daniel > > I think there was some need for inttypes.h instead of stdint like a > decade ago when I was working on BSDs, but that was already almost > irrelevant then. Hi, just following up on this; is there still the need for an ACK? Regards, James _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] drm: Fix drm.h uapi header for GNU/kFreeBSD 2019-05-04 20:43 ` James Clarke @ 2019-05-06 8:16 ` Daniel Vetter 2019-05-06 23:10 ` James Clarke 0 siblings, 1 reply; 10+ messages in thread From: Daniel Vetter @ 2019-05-06 8:16 UTC (permalink / raw) To: James Clarke; +Cc: dri-devel On Sat, May 04, 2019 at 09:43:14PM +0100, James Clarke wrote: > On 15 Jan 2019, at 18:41, Eric Anholt <eric@anholt.net> wrote: > > > > Daniel Vetter <daniel@ffwll.ch> writes: > > > >> On Tue, Jan 15, 2019 at 03:04:18PM +0000, James Clarke wrote: > >>> Like GNU/Linux, GNU/kFreeBSD's sys/types.h does not define the uintX_t > >>> types, which differs from the BSDs' headers. Thus we should include > >>> stdint.h to ensure we have all the required integer types. > >>> > >>> Signed-off-by: James Clarke <jrtc27@jrtc27.com> > >> > >> Would be good to get an ack from some other *bsd that this is still all > >> fine. lgtm otherwise. > >> -Daniel > > > > I think there was some need for inttypes.h instead of stdint like a > > decade ago when I was working on BSDs, but that was already almost > > irrelevant then. > > Hi, just following up on this; is there still the need for an ACK? Yeah and ack without a "decades ago" qualifier would be nice. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] drm: Fix drm.h uapi header for GNU/kFreeBSD 2019-05-06 8:16 ` Daniel Vetter @ 2019-05-06 23:10 ` James Clarke 2019-05-07 8:26 ` Daniel Vetter 0 siblings, 1 reply; 10+ messages in thread From: James Clarke @ 2019-05-06 23:10 UTC (permalink / raw) To: Daniel Vetter; +Cc: dri-devel On 6 May 2019, at 09:16, Daniel Vetter <daniel@ffwll.ch> wrote: > On Sat, May 04, 2019 at 09:43:14PM +0100, James Clarke wrote: >> On 15 Jan 2019, at 18:41, Eric Anholt <eric@anholt.net> wrote: >>> >>> Daniel Vetter <daniel@ffwll.ch> writes: >>> >>>> On Tue, Jan 15, 2019 at 03:04:18PM +0000, James Clarke wrote: >>>>> Like GNU/Linux, GNU/kFreeBSD's sys/types.h does not define the uintX_t >>>>> types, which differs from the BSDs' headers. Thus we should include >>>>> stdint.h to ensure we have all the required integer types. >>>>> >>>>> Signed-off-by: James Clarke <jrtc27@jrtc27.com> >>>> >>>> Would be good to get an ack from some other *bsd that this is still all >>>> fine. lgtm otherwise. >>>> -Daniel >>> >>> I think there was some need for inttypes.h instead of stdint like a >>> decade ago when I was working on BSDs, but that was already almost >>> irrelevant then. >> >> Hi, just following up on this; is there still the need for an ACK? > > Yeah and ack without a "decades ago" qualifier would be nice. > -Daniel I have the following from Brooks Davis, FreeBSD contributor and core team member: -----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline With respect to the thread: https://lists.freedesktop.org/archives/dri-devel/2019-May/216784.html I can't see any reason why you'd want to include inttypes.h instead of stdint.h unless you want the intmax_t using types/functions it defines. Your patch is certainly safe on FreeBSD. At a glance it looks like the same is true (with somewhat different implementation) on OpenBSD. - -- Brooks -----BEGIN PGP SIGNATURE----- iQEcBAEBAgAGBQJc0LwmAAoJEKzQXbSebgfArrIIAItDHcDG5d9QcMnbQibk+d6w EW1OnvijeVZ6PCNcbEEldZEHXsRFOe7zE8sYKSPegDnYve1y8jVyh2YyJtDI7eWL hD2dSY3DGOKatEfd3CxoG2vpN8zv3giDS35hXIqMRF1iD0/jYixTYLcfZgQ7U6Ml 34gxe3q6v30ZoWK5SHCDCac4z0OYjfKTpwZnyNkag/DW5CqqOTQOao6AJOLcosqJ orIUZVl0nvk0dIGzK09ef+BWOyVA5h6f+V3F3x/cK3E2qiTRxVLhhLVaEgdBDp5C hPXH33q/GK54m5jWVaZ5BtpKSA1Et6HjF48+wyJCADQh//DxIWGEAa1fmvKryQw= =oJBq -----END PGP SIGNATURE----- Hopefully the signature survives; I converted it from PGP/MIME and it seems to verify locally. Regards, James _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] drm: Fix drm.h uapi header for GNU/kFreeBSD 2019-05-06 23:10 ` James Clarke @ 2019-05-07 8:26 ` Daniel Vetter 0 siblings, 0 replies; 10+ messages in thread From: Daniel Vetter @ 2019-05-07 8:26 UTC (permalink / raw) To: James Clarke; +Cc: dri-devel On Tue, May 07, 2019 at 12:10:52AM +0100, James Clarke wrote: > On 6 May 2019, at 09:16, Daniel Vetter <daniel@ffwll.ch> wrote: > > On Sat, May 04, 2019 at 09:43:14PM +0100, James Clarke wrote: > >> On 15 Jan 2019, at 18:41, Eric Anholt <eric@anholt.net> wrote: > >>> > >>> Daniel Vetter <daniel@ffwll.ch> writes: > >>> > >>>> On Tue, Jan 15, 2019 at 03:04:18PM +0000, James Clarke wrote: > >>>>> Like GNU/Linux, GNU/kFreeBSD's sys/types.h does not define the uintX_t > >>>>> types, which differs from the BSDs' headers. Thus we should include > >>>>> stdint.h to ensure we have all the required integer types. > >>>>> > >>>>> Signed-off-by: James Clarke <jrtc27@jrtc27.com> > >>>> > >>>> Would be good to get an ack from some other *bsd that this is still all > >>>> fine. lgtm otherwise. > >>>> -Daniel > >>> > >>> I think there was some need for inttypes.h instead of stdint like a > >>> decade ago when I was working on BSDs, but that was already almost > >>> irrelevant then. > >> > >> Hi, just following up on this; is there still the need for an ACK? > > > > Yeah and ack without a "decades ago" qualifier would be nice. > > -Daniel > > I have the following from Brooks Davis, FreeBSD contributor and core team > member: > > -----BEGIN PGP SIGNED MESSAGE----- > Hash: SHA1 > > Content-Type: text/plain; charset=us-ascii > Content-Disposition: inline > > With respect to the thread: > https://lists.freedesktop.org/archives/dri-devel/2019-May/216784.html > > I can't see any reason why you'd want to include inttypes.h instead of > stdint.h unless you want the intmax_t using types/functions it defines. > Your patch is certainly safe on FreeBSD. > > At a glance it looks like the same is true (with somewhat different > implementation) on OpenBSD. > > - -- Brooks > > -----BEGIN PGP SIGNATURE----- > > iQEcBAEBAgAGBQJc0LwmAAoJEKzQXbSebgfArrIIAItDHcDG5d9QcMnbQibk+d6w > EW1OnvijeVZ6PCNcbEEldZEHXsRFOe7zE8sYKSPegDnYve1y8jVyh2YyJtDI7eWL > hD2dSY3DGOKatEfd3CxoG2vpN8zv3giDS35hXIqMRF1iD0/jYixTYLcfZgQ7U6Ml > 34gxe3q6v30ZoWK5SHCDCac4z0OYjfKTpwZnyNkag/DW5CqqOTQOao6AJOLcosqJ > orIUZVl0nvk0dIGzK09ef+BWOyVA5h6f+V3F3x/cK3E2qiTRxVLhhLVaEgdBDp5C > hPXH33q/GK54m5jWVaZ5BtpKSA1Et6HjF48+wyJCADQh//DxIWGEAa1fmvKryQw= > =oJBq > -----END PGP SIGNATURE----- > > Hopefully the signature survives; I converted it from PGP/MIME and it seems to > verify locally. It's not about the signature, we don't care, it's about the email addresses, so in case there's ever a change in the future we'll have people to contact. Can you pls resend that patch with those acks (including mail address) added? Needs permission of the people involved ofc (which is usually implied for public mailing lists, but this seems like a private mail). Thanks, Daniel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] drm: Fix drm.h uapi header for GNU/kFreeBSD 2019-01-15 15:04 [PATCH] drm: Fix drm.h uapi header for GNU/kFreeBSD James Clarke 2019-01-15 15:17 ` Daniel Vetter @ 2019-05-09 17:42 ` Eric Anholt 2019-05-16 16:37 ` Eric Anholt 1 sibling, 1 reply; 10+ messages in thread From: Eric Anholt @ 2019-05-09 17:42 UTC (permalink / raw) To: dri-devel; +Cc: James Clarke [-- Attachment #1.1: Type: text/plain, Size: 342 bytes --] James Clarke <jrtc27@jrtc27.com> writes: > Like GNU/Linux, GNU/kFreeBSD's sys/types.h does not define the uintX_t > types, which differs from the BSDs' headers. Thus we should include > stdint.h to ensure we have all the required integer types. > > Signed-off-by: James Clarke <jrtc27@jrtc27.com> Reviewed-by: Eric Anholt <eric@anholt.net> [-- Attachment #1.2: signature.asc --] [-- Type: application/pgp-signature, Size: 832 bytes --] [-- Attachment #2: Type: text/plain, Size: 159 bytes --] _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] drm: Fix drm.h uapi header for GNU/kFreeBSD 2019-05-09 17:42 ` Eric Anholt @ 2019-05-16 16:37 ` Eric Anholt 2019-05-16 16:58 ` Eric Engestrom 0 siblings, 1 reply; 10+ messages in thread From: Eric Anholt @ 2019-05-16 16:37 UTC (permalink / raw) To: dri-devel; +Cc: James Clarke [-- Attachment #1.1: Type: text/plain, Size: 457 bytes --] Eric Anholt <eric@anholt.net> writes: > [ Unknown signature status ] > James Clarke <jrtc27@jrtc27.com> writes: > >> Like GNU/Linux, GNU/kFreeBSD's sys/types.h does not define the uintX_t >> types, which differs from the BSDs' headers. Thus we should include >> stdint.h to ensure we have all the required integer types. >> >> Signed-off-by: James Clarke <jrtc27@jrtc27.com> > > Reviewed-by: Eric Anholt <eric@anholt.net> And pushed to drm-misc-next now. [-- Attachment #1.2: signature.asc --] [-- Type: application/pgp-signature, Size: 832 bytes --] [-- Attachment #2: Type: text/plain, Size: 159 bytes --] _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] drm: Fix drm.h uapi header for GNU/kFreeBSD 2019-05-16 16:37 ` Eric Anholt @ 2019-05-16 16:58 ` Eric Engestrom 0 siblings, 0 replies; 10+ messages in thread From: Eric Engestrom @ 2019-05-16 16:58 UTC (permalink / raw) To: Eric Anholt; +Cc: James Clarke, dri-devel On Thursday, 2019-05-16 09:37:40 -0700, Eric Anholt wrote: > Eric Anholt <eric@anholt.net> writes: > > > [ Unknown signature status ] > > James Clarke <jrtc27@jrtc27.com> writes: > > > >> Like GNU/Linux, GNU/kFreeBSD's sys/types.h does not define the uintX_t > >> types, which differs from the BSDs' headers. Thus we should include > >> stdint.h to ensure we have all the required integer types. > >> > >> Signed-off-by: James Clarke <jrtc27@jrtc27.com> > > > > Reviewed-by: Eric Anholt <eric@anholt.net> > > And pushed to drm-misc-next now. Thanks Eric! James, that means you can now sync libdrm to get the fix there as well :) See include/drm/README for details _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2019-05-16 16:58 UTC | newest] Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2019-01-15 15:04 [PATCH] drm: Fix drm.h uapi header for GNU/kFreeBSD James Clarke 2019-01-15 15:17 ` Daniel Vetter 2019-01-15 18:41 ` Eric Anholt 2019-05-04 20:43 ` James Clarke 2019-05-06 8:16 ` Daniel Vetter 2019-05-06 23:10 ` James Clarke 2019-05-07 8:26 ` Daniel Vetter 2019-05-09 17:42 ` Eric Anholt 2019-05-16 16:37 ` Eric Anholt 2019-05-16 16:58 ` Eric Engestrom
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.