Re: drm: Branch 'master'

2010-08-27 Thread Julien Cristau
On Fri, Aug 27, 2010 at 11:26:59 +0200, Zdenek Kabelac wrote:

 Why keeping different name for C++ and C ?
 
Because changing the name for C means breaking API for existing users,
which is not a nice thing to do.

Cheers,
Julien

--
Sell apps to millions through the Intel(R) Atom(Tm) Developer Program
Be part of this innovative community and reach millions of netbook users 
worldwide. Take advantage of special opportunities to increase revenue and 
speed time-to-market. Join now, and jumpstart your future.
http://p.sf.net/sfu/intel-atom-d2d
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: [PATCH] tests/ programs need -lrt too since they use clock_gettime.

2010-04-10 Thread Julien Cristau
On Fri, Apr  9, 2010 at 16:46:46 +0300, Török Edwin wrote:

 When using the binutils gold linker, even if libdrm.so is already linked 
 against
 -lrt you still need to link your app against -lrt if your app needs symbols 
 from
 it.
 
 Signed-off-by: Török Edwin edwinto...@gmail.com
 ---
  tests/Makefile.am |2 +-
  1 files changed, 1 insertions(+), 1 deletions(-)
 
 diff --git a/tests/Makefile.am b/tests/Makefile.am
 index 3e74705..81006ab 100644
 --- a/tests/Makefile.am
 +++ b/tests/Makefile.am
 @@ -4,7 +4,7 @@ AM_CPPFLAGS = \
   -I $(top_srcdir)/include/drm \
   -I $(top_srcdir)
  
 -LDADD = $(top_builddir)/libdrm.la
 +LDADD = $(top_builddir)/libdrm.la @CLOCK_LIB@
  
  noinst_PROGRAMS = \
   dristat \

The tests don't use librt afaict, the uses of clock_gettime() are in
libdrm and libdrm_intel.  So your explanation doesn't seem to make
sense...

Cheers,
Julien

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: [Intel-gfx] [PATCH 1/2] libdrm: Move intel_atomic.h to libdrm core for sharing.

2010-03-11 Thread Julien Cristau
 On Wed, Mar 10, 2010 at 11:20 AM, Pauli Nieminen suok...@gmail.com wrote:
  intel_atomic.h includes very usefull atomic operations for
  lock free parrallel access of variables. Moving these to
  core libdrm for code sharing with radeon.
 
  Signed-off-by: Pauli Nieminen suok...@gmail.com
  ---
   Makefile.am          |    2 +-
   configure.ac         |    2 +-
   intel/intel_atomic.h |   56 +-
   xf86atomic.h         |   93 
  ++
   4 files changed, 96 insertions(+), 57 deletions(-)
   create mode 100644 xf86atomic.h
 
  diff --git a/Makefile.am b/Makefile.am
  index ee3ccc7..295121f 100644
  --- a/Makefile.am
  +++ b/Makefile.am
  @@ -59,7 +59,7 @@ libdrm_la_SOURCES =                           \
         libdrm_lists.h
 
   libdrmincludedir = ${includedir}
  -libdrminclude_HEADERS = xf86drm.h xf86drmMode.h
  +libdrminclude_HEADERS = xf86drm.h xf86drmMode.h xf86atomic.h
 

Should this really get installed?  I'm not sure this should be public
libdrm interface.

Cheers,
Julien

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: [PATCH 3/3] Install headers to $(includedir)/libdrm

2010-03-10 Thread Julien Cristau
On Fri, Feb 26, 2010 at 19:07:24 +0100, Julien Cristau wrote:

 Avoids conflicts with kernel headers.
 
 Signed-off-by: Julien Cristau jcris...@debian.org
 ---
 This was suggested by Eric so distros can let the kernel install drm
 headers, but provide updated headers from libdrm so we can build new
 drivers regardless of the kernel version.
 
any comments on this?

Cheers,
Julien

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: [PATCH] DRM: fix headers_check warnings

2010-03-02 Thread Julien Cristau
On Sat, Feb 27, 2010 at 14:57:36 +0530, Jaswinder Singh Rajput wrote:

 
 Fixed following headers_check warnings:
   CHECK   include/drm (14 files)
  include/drm/drm_mode.h:84: found __[us]{8,16,32,64} type without #include 
 linux/types.h
  include/drm/i915_drm.h:119: found __[us]{8,16,32,64} type without #include 
 linux/types.h
  include/drm/mga_drm.h:260: found __[us]{8,16,32,64} type without #include 
 linux/types.h
  include/drm/radeon_drm.h:758: found __[us]{8,16,32,64} type without #include 
 linux/types.h
  include/drm/via_drm.h:117: found __[us]{8,16,32,64} type without #include 
 linux/types.h
 
These files are shared with BSD, afaik, and they get the linux/types.h
include from drm.h.

Cheers,
Julien

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[PATCH 1/3] libdrm_intel.pc: don't include ${includedir}/drm

2010-02-27 Thread Julien Cristau
intel_bufmgr.h is installed in ${includedir} directly, and the other
headers are taken care of by libdrm.pc's Cflags.

Signed-off-by: Julien Cristau jcris...@debian.org
---
 intel/libdrm_intel.pc.in |2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/intel/libdrm_intel.pc.in b/intel/libdrm_intel.pc.in
index b93f1c6..e8080d9 100644
--- a/intel/libdrm_intel.pc.in
+++ b/intel/libdrm_intel.pc.in
@@ -8,4 +8,4 @@ Description: Userspace interface to kernel DRM services
 Version: @PACKAGE_VERSION@
 Requires: libdrm
 Libs: -L${libdir} -ldrm_intel
-Cflags: -I${includedir} -I${includedir}/drm
+Cflags: -I${includedir}
-- 
1.6.6.1


--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: [PATCH 2/3] libdrm_nouveau requires libdrm

2010-02-27 Thread Julien Cristau
On Fri, Feb 26, 2010 at 13:38:59 -0500, Kristian Høgsberg wrote:

 On Fri, Feb 26, 2010 at 1:07 PM, Julien Cristau jcris...@debian.org wrote:
  nouveau_drmif.h includes xf86drm.h.
 
 If it's a source level dependency it should be a regular Requires: in
 the .pc.  Requires.private: is only for private libraries you link to
 but don't expose their API in your header files.  The idea is (AFAIK)
 that you can ask pkg-config to list all libraries to link to for
 static linking, in which case pkg-config needs a way to find all
 needed libraries.  It even looks like Requires.private has been
 obsoleted by Libs.private in recent pkg-config.
 
That's not what the pkg-config maintainer says...

http://err.no/personal/blog/2008/Mar/25

Using Requires.private lets us get the necessary cflags, but avoids
linking to the library if not using --static.

Cheers,
Julien

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[PATCH 2/3] libdrm_nouveau requires libdrm

2010-02-26 Thread Julien Cristau
nouveau_drmif.h includes xf86drm.h.

Signed-off-by: Julien Cristau jcris...@debian.org
---
 nouveau/libdrm_nouveau.pc.in |1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/nouveau/libdrm_nouveau.pc.in b/nouveau/libdrm_nouveau.pc.in
index 7ef49e5..6ae6287 100644
--- a/nouveau/libdrm_nouveau.pc.in
+++ b/nouveau/libdrm_nouveau.pc.in
@@ -8,3 +8,4 @@ Description: Userspace interface to nouveau kernel DRM services
 Version: 0.6
 Libs: -L${libdir} -ldrm_nouveau
 Cflags: -I${includedir} -I${includedir}/drm -I${includedir}/nouveau
+Requires.private: libdrm
-- 
1.6.6.1


--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[PATCH 3/3] Install headers to $(includedir)/libdrm

2010-02-26 Thread Julien Cristau
Avoids conflicts with kernel headers.

Signed-off-by: Julien Cristau jcris...@debian.org
---
This was suggested by Eric so distros can let the kernel install drm
headers, but provide updated headers from libdrm so we can build new
drivers regardless of the kernel version.

 include/drm/Makefile.am  |2 +-
 libdrm.pc.in |2 +-
 nouveau/Makefile.am  |2 +-
 nouveau/libdrm_nouveau.pc.in |2 +-
 radeon/Makefile.am   |2 +-
 radeon/libdrm_radeon.pc.in   |2 +-
 6 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/include/drm/Makefile.am b/include/drm/Makefile.am
index 9046955..43695bd 100644
--- a/include/drm/Makefile.am
+++ b/include/drm/Makefile.am
@@ -22,7 +22,7 @@
 # however, r300 and via need their reg headers installed in order to build.
 # better solutions are welcome.
 
-klibdrmincludedir = ${includedir}/drm
+klibdrmincludedir = ${includedir}/libdrm
 klibdrminclude_HEADERS = \
drm.h \
drm_mode.h \
diff --git a/libdrm.pc.in b/libdrm.pc.in
index 6e00597..b46e2a6 100644
--- a/libdrm.pc.in
+++ b/libdrm.pc.in
@@ -7,4 +7,4 @@ Name: libdrm
 Description: Userspace interface to kernel DRM services
 Version: @PACKAGE_VERSION@
 Libs: -L${libdir} -ldrm
-Cflags: -I${includedir} -I${includedir}/drm
+Cflags: -I${includedir} -I${includedir}/libdrm
diff --git a/nouveau/Makefile.am b/nouveau/Makefile.am
index 5d759c5..b6d214a 100644
--- a/nouveau/Makefile.am
+++ b/nouveau/Makefile.am
@@ -34,7 +34,7 @@ libdrm_nouveaucommoninclude_HEADERS = \
nouveau_reloc.h
 
 
-libdrm_nouveauincludedir = ${includedir}/drm
+libdrm_nouveauincludedir = ${includedir}/libdrm
 libdrm_nouveauinclude_HEADERS = \
nouveau_drmif.h
 
diff --git a/nouveau/libdrm_nouveau.pc.in b/nouveau/libdrm_nouveau.pc.in
index 6ae6287..c78a28a 100644
--- a/nouveau/libdrm_nouveau.pc.in
+++ b/nouveau/libdrm_nouveau.pc.in
@@ -7,5 +7,5 @@ Name: libdrm_nouveau
 Description: Userspace interface to nouveau kernel DRM services
 Version: 0.6
 Libs: -L${libdir} -ldrm_nouveau
-Cflags: -I${includedir} -I${includedir}/drm -I${includedir}/nouveau
+Cflags: -I${includedir} -I${includedir}/libdrm -I${includedir}/nouveau
 Requires.private: libdrm
diff --git a/radeon/Makefile.am b/radeon/Makefile.am
index 29af1df..de78232 100644
--- a/radeon/Makefile.am
+++ b/radeon/Makefile.am
@@ -41,7 +41,7 @@ libdrm_radeon_la_SOURCES = \
radeon_bo.c \
radeon_cs.c
 
-libdrm_radeonincludedir = ${includedir}/drm
+libdrm_radeonincludedir = ${includedir}/libdrm
 libdrm_radeoninclude_HEADERS = \
radeon_bo.h \
radeon_cs.h \
diff --git a/radeon/libdrm_radeon.pc.in b/radeon/libdrm_radeon.pc.in
index ab703e1..68ef0ab 100644
--- a/radeon/libdrm_radeon.pc.in
+++ b/radeon/libdrm_radeon.pc.in
@@ -7,4 +7,4 @@ Name: libdrm_radeon
 Description: Userspace interface to kernel DRM services for radeon
 Version: @PACKAGE_VERSION@
 Libs: -L${libdir} -ldrm_radeon
-Cflags: -I${includedir} -I${includedir}/drm
+Cflags: -I${includedir} -I${includedir}/libdrm
-- 
1.6.6.1


--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: [PATCH] DRM / i915: Fix resume regression on MSI Wind U100 w/o KMS

2010-01-11 Thread Julien Cristau
On Mon, Jan 11, 2010 at 22:04:36 +0100, Rafael J. Wysocki wrote:

 Hmm, are you trying to say radeon is better at that?
 
 My experience is quite the opposite to be honest.
 
radeon kms is in staging, doesn't pretend to be stable and force all
users to the experimental paths.  So yes, I would say radeon is better
at that.

Cheers,
Julien

--
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev 
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: drivers/gpu/drm/radeon: move a dereference below a NULL test

2009-12-25 Thread Julien Cristau
On Thu, Dec 24, 2009 at 23:44:30 +1100, Darren Jenkins wrote:

 If a NULL value is possible, the dereference should only occur after the
 NULL test.
 
 Signed-off-by: Darren Jenkins darrenrjenk...@gmail.com
 diff --git a/drivers/gpu/drm/radeon/radeon_irq.c
 b/drivers/gpu/drm/radeon/radeon_irq.c
 index b79ecc4..2f349a3 100644
 --- a/drivers/gpu/drm/radeon/radeon_irq.c
 +++ b/drivers/gpu/drm/radeon/radeon_irq.c
 @@ -289,16 +289,16 @@ int radeon_irq_emit(struct drm_device *dev, void
 *data, struct drm_file *file_pr

Looks like your patches are whitespace-damaged.  You'll need to fix your
MUA...

Cheers,
Julien

--
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev 
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: [PATCH 3/5] drm/edid: Add DMT modes to the pool if the monitor is GTF-capable

2009-11-25 Thread Julien Cristau
On Tue, Nov 24, 2009 at 17:36:19 -0500, Adam Jackson wrote:

 diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
 index e655916..ee84ffe 100644
 --- a/drivers/gpu/drm/drm_edid.c
 +++ b/drivers/gpu/drm/drm_edid.c
 @@ -481,16 +481,17 @@ static struct drm_display_mode drm_dmt_modes[] = {
  3048, 3536, 0, 1600, 1603, 1609, 1682, 0,
  DRM_MODE_FLAG_NHSYNC | DRM_MODE_FLAG_PVSYNC) },
  };
 +static const int drm_num_dmt_modes =
 + sizeof(drm_dmt_modes) / sizeof(struct drm_display_mode);
  
ARRAY_SIZE(drm_dmt_modes)?

Cheers,
Julien

--
Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day 
trial. Simplify your report design, integration and deployment - and focus on 
what you do best, core application coding. Discover what's new with
Crystal Reports now.  http://p.sf.net/sfu/bobj-july
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: RFC: libdrm repo

2009-11-18 Thread Julien Cristau
On Tue, Nov 17, 2009 at 18:54:40 +0100, Stephane Marchesin wrote:

 Yes, but the positive side is that distros using a standard/old (about
 a year) kernel don't need to crawl the old libdrm repo and find the
 right version (in your case they have to do this ° backport stuff) ...
 I think that plus the fact that it makes development and merging
 simpler is just a reason to do it.
 
Why would they have to do that?  Newer libdrms should stay compatible
with older kernels...

Cheers,
Julien

--
Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day 
trial. Simplify your report design, integration and deployment - and focus on 
what you do best, core application coding. Discover what's new with
Crystal Reports now.  http://p.sf.net/sfu/bobj-july
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: RFC: libdrm repo

2009-11-08 Thread Julien Cristau
On Fri, Nov  6, 2009 at 22:23:46 +0100, Jerome Glisse wrote:

 I think Joe user will install the kernel-header package of its
 distribution, and libdrm should detect at configure time kernel
 header version and people should take care to only enable new
 libdrm stuff when libdrm find the proper header. This imply that
 we are studious coder and don't forget about that when adding
 new features needing new headers :)
 
 For instance if it detects header with no KMS stuff it should loudly
 print at end of configure that it disable KMS support.
 
That would lead to a hell of ifdefs in libdrm.  I think keeping copies
of the latest kernel drm headers in the libdrm repo and tarballs would
be best.

Cheers,
Julien

--
Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day 
trial. Simplify your report design, integration and deployment - and focus on 
what you do best, core application coding. Discover what's new with
Crystal Reports now.  http://p.sf.net/sfu/bobj-july
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[PATCH] Only install libdrm_intel.pc if we build libdrm_intel

2009-10-31 Thread Julien Cristau
---
 Makefile.am |4 ++--
 configure.ac|4 ++--
 libdrm/intel/Makefile.am|2 ++
 libdrm/intel/libdrm_intel.pc.in |   10 ++
 libdrm_intel.pc.in  |   10 --
 5 files changed, 16 insertions(+), 14 deletions(-)
 create mode 100644 libdrm/intel/libdrm_intel.pc.in
 delete mode 100644 libdrm_intel.pc.in

diff --git a/Makefile.am b/Makefile.am
index e105edd..5b1ae60 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -25,6 +25,6 @@ AUTOMAKE_OPTIONS = foreign
 SUBDIRS = libdrm shared-core tests
 
 pkgconfigdir = @pkgconfigdir@
-pkgconfig_DATA = libdrm.pc libdrm_intel.pc
+pkgconfig_DATA = libdrm.pc
 
-EXTRA_DIST = libdrm.pc.in libdrm_intel.pc.in
+EXTRA_DIST = libdrm.pc.in
diff --git a/configure.ac b/configure.ac
index 7efc4d4..410a359 100644
--- a/configure.ac
+++ b/configure.ac
@@ -194,6 +194,7 @@ AC_OUTPUT([
Makefile
libdrm/Makefile
libdrm/intel/Makefile
+   libdrm/intel/libdrm_intel.pc
libdrm/radeon/Makefile
libdrm/radeon/libdrm_radeon.pc
libdrm/nouveau/Makefile
@@ -202,5 +203,4 @@ AC_OUTPUT([
tests/Makefile
tests/modeprint/Makefile
tests/modetest/Makefile
-   libdrm.pc
-   libdrm_intel.pc])
+   libdrm.pc])
diff --git a/libdrm/intel/Makefile.am b/libdrm/intel/Makefile.am
index 28f8952..0faa10d 100644
--- a/libdrm/intel/Makefile.am
+++ b/libdrm/intel/Makefile.am
@@ -46,3 +46,5 @@ libdrm_intel_la_SOURCES = \
 
 libdrm_intelincludedir = ${includedir}
 libdrm_intelinclude_HEADERS = intel_bufmgr.h
+
+pkgconfig_DATA = libdrm_intel.pc
diff --git a/libdrm/intel/libdrm_intel.pc.in b/libdrm/intel/libdrm_intel.pc.in
new file mode 100644
index 000..ea71cc3
--- /dev/null
+++ b/libdrm/intel/libdrm_intel.pc.in
@@ -0,0 +1,10 @@
+pref...@prefix@
+exec_pref...@exec_prefix@
+libd...@libdir@
+included...@includedir@
+
+Name: libdrm
+Description: Userspace interface to kernel DRM services
+Version: @PACKAGE_VERSION@
+Libs: -L${libdir} -ldrm -ldrm_intel
+Cflags: -I${includedir} -I${includedir}/drm
diff --git a/libdrm_intel.pc.in b/libdrm_intel.pc.in
deleted file mode 100644
index ea71cc3..000
--- a/libdrm_intel.pc.in
+++ /dev/null
@@ -1,10 +0,0 @@
-pref...@prefix@
-exec_pref...@exec_prefix@
-libd...@libdir@
-included...@includedir@
-
-Name: libdrm
-Description: Userspace interface to kernel DRM services
-Version: @PACKAGE_VERSION@
-Libs: -L${libdir} -ldrm -ldrm_intel
-Cflags: -I${includedir} -I${includedir}/drm
-- 
1.6.5


--
Come build with us! The BlackBerry(R) Developer Conference in SF, CA
is the only developer event you need to attend this year. Jumpstart your
developing skills, take BlackBerry mobile applications to market and stay 
ahead of the curve. Join us from November 9 - 12, 2009. Register now!
http://p.sf.net/sfu/devconference
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: libdrm: Fixing compilers warnings

2009-07-21 Thread Julien Cristau
On Mon, Jul 20, 2009 at 19:17:01 +0300, Pauli Nieminen wrote:

 From b7e77b71d1f2d8ff6741e534911e09f10e3f3d4e Mon Sep 17 00:00:00 2001
 From: Pauli Nieminen suok...@gmail.com
 Date: Mon, 20 Jul 2009 14:39:57 +0300
 Subject: [PATCH 01/15] libdrm: Add function attribute for debug
 functions to let gcc check parameter correctness.
 
 ---
  libdrm/xf86drm.c |5 -
  1 files changed, 4 insertions(+), 1 deletions(-)
 
 diff --git a/libdrm/xf86drm.c b/libdrm/xf86drm.c
 index 26dd812..4214b91 100644
 --- a/libdrm/xf86drm.c
 +++ b/libdrm/xf86drm.c
 @@ -107,13 +107,16 @@ void drmSetServerInfo(drmServerInfoPtr info)
   * \internal
   * This function is a wrapper around vfprintf().
   */
 +static int drmDebugPrint(const char *format, va_list ap)
 __attribute__((format(printf,1,0)));
 
why add a forward declaration instead of adding the attribute to the
function definition?  also your patches are whitespace damaged.

  static int drmDebugPrint(const char *format, va_list ap)
  {
  return vfprintf(stderr, format, ap);
  }
 
 -static int (*drm_debug_print)(const char *format, va_list ap) = 
 drmDebugPrint;
 +static int (*drm_debug_print)(const char *format, va_list ap)
 __attribute__((format(printf,1,0))) = drmDebugPrint;
 +
 +void drmMsg(const char *format, ...) __attribute__((format(printf,1,2)));
 
  void
  drmMsg(const char *format, ...)

Cheers,
Julien

--
Enter the BlackBerry Developer Challenge  
This is your chance to win up to $100,000 in prizes! For a limited time, 
vendors submitting new applications to BlackBerry App World(TM) will have
the opportunity to enter the BlackBerry Developer Challenge. See full prize  
details at: http://p.sf.net/sfu/Challenge
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: [Intel-gfx] [PATCH] Make xorg.conf NoAccel/DRI options work under KMS

2009-07-10 Thread Julien Cristau
On Thu, Jul  9, 2009 at 09:36:21 -0700, Jesse Barnes wrote:

 I wonder if fbdev could fill the get something on the screen when the
 driver is busted hole.  Is it good enough to rely on for that?
 
IIRC that won't currently work well, because
probe_devices_from_device_sections will claim the pci device for the
intel driver, at which point fbdev's xf86ClaimFbSlot will fail and it
will be disabled before intel has a chance to run its init stuff (so
before we know if it's busted).  So you need to either uninstall intel,
or know the magic xorg.conf incantation to use fbdev.

Cheers,
Julien

--
Enter the BlackBerry Developer Challenge  
This is your chance to win up to $100,000 in prizes! For a limited time, 
vendors submitting new applications to BlackBerry App World(TM) will have
the opportunity to enter the BlackBerry Developer Challenge. See full prize  
details at: http://p.sf.net/sfu/Challenge
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: [Intel-gfx] [PATCH] Make xorg.conf NoAccel/DRI options work under KMS

2009-07-08 Thread Julien Cristau
On Wed, Jul  8, 2009 at 12:04:44 -0700, Jesse Barnes wrote:

 On Wed,  8 Jul 2009 11:50:38 -0700
 Keith Packard kei...@keithp.com wrote:
 
  KMS mode does not call I830AccelMethodInit as that does a ton of
  random device initialization, but that means that the NoAccel and DRI
  options were ignored. Split out the option parsing from
  I830AccelMethodInit and have both KMS and UMS invoke it.
  
  Signed-off-by: Keith Packard kei...@keithp.com
  ---
 
 Ugg more configurations... Assuming this is for fallback when something
 bad happens or for debugging couldn't users just use the fbdev driver
 instead?  I'd rather KMS just be a single everything works case...
 
DRI2 init can fail for various reasons, and it's easier to test if you
don't have to hack the driver.  No particular opinion on kms without
uxa, though.

Cheers,
Julien

--
Enter the BlackBerry Developer Challenge  
This is your chance to win up to $100,000 in prizes! For a limited time, 
vendors submitting new applications to BlackBerry App World(TM) will have
the opportunity to enter the BlackBerry Developer Challenge. See full prize  
details at: http://p.sf.net/sfu/Challenge
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: [PATCH 4/6] libdrm: Request for _XOPEN_SOURCE to get full posix headers including extensions (XSH: mknod)

2009-07-06 Thread Julien Cristau
On Mon, Jul  6, 2009 at 09:46:40 -0700, Ian Romanick wrote:

 Here's an alterate patch the puts _XOPEN_SOURCE and _GNU_SOURCE in config.h
 via configure.ac.  Comments?
 
maybe use AC_USE_SYSTEM_EXTENSIONS instead?

Cheers,
Julien

--
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: [PATCH 2/6] libdrm: Add missing string.h include

2009-07-05 Thread Julien Cristau
On Sat, Jul  4, 2009 at 02:18:50 +0300, Pauli Nieminen wrote:

 ---
  libdrm/xf86drm.h |1 +
  1 files changed, 1 insertions(+), 0 deletions(-)
 
 diff --git a/libdrm/xf86drm.h b/libdrm/xf86drm.h
 index c1d173c..a7f9763 100644
 --- a/libdrm/xf86drm.h
 +++ b/libdrm/xf86drm.h
 @@ -37,6 +37,7 @@
  #include stdarg.h
  #include sys/types.h
  #include stdint.h
 +#include strings.h
  #include drm.h
  
   /* Defaults, if nothing set in xf86config */

commit message says string.h, patch has strings.h.  Which is it?

Cheers,
Julien

--
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: [PATCH] radeon_cp: use request_firmware

2008-12-22 Thread Julien Cristau
On Sun, Dec 21, 2008 at 20:58:28 +, Dave Airlie wrote:

 
  
  Is available at
  http://git.infradead.org/users/jaswinder/firm-jsr-2.6.git?a=commit;h=3a911a216742e4ab998f3281409d46a62f252716
  
  
  Please let me know, should I need to resend this patch for :
  1. git kernel.org:/pub/scm/linux/kernel/git/airlied/drm-2.6.git
  OR
  2. git kernel.org:/pub/scm/linux/kernel/git/torvalds/linux-2.6.git
 
 How much testing has this seen on what range of cards and on what 
 architectures? I'd like some success stories from people using this patch.
 Is Debian shipping it?
 
Not at this time.  AFAIK Ben Hutchings has tested that patch, although I
don't know on what hardware, and if he got other testers.  Ben?

 Its do think firmware patches are stupid (I do), but I'm not against 
 applying them if they are proven working.
 
Cheers,
Julien

--
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: intel text mode restore problem

2008-12-14 Thread Julien Cristau
On Fri, 2008-12-12 at 20:14 +0100, Norbert Preining wrote:
 Where can I get this intel_reg_dumper from? Is there a tree to check
 out, or a ready made binary from x86_64 (Debian/unstable)?
 
you can install the xserver-xorg-video-intel-dbg package, or build the
reg dumper from the xf86-video-intel source tree.

Cheers,
Julien

--
SF.Net email is Sponsored by MIX09, March 18-20, 2009 in Las Vegas, Nevada.
The future of the web can't happen without you.  Join us at MIX09 to help
pave the way to the Next Web now. Learn more and register at
http://ad.doubleclick.net/clk;208669438;13503038;i?http://2009.visitmix.com/
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: [Mesa3d-dev] Mesa 7.1 rc 4

2008-08-17 Thread Julien Cristau
On Sat, Aug 16, 2008 at 10:20:12 -0600, Brian Paul wrote:

 Yet another Mesa 7.1 release candidate is at http://www.mesa3d.org/beta/
 
 This includes the latest GLSL fixes/features plus other assorted fixes 
 from the past 2-3 weeks.
 
This also includes the drm-gem merge, which means that it now has a
dependency on an unreleased libdrm for the intel dri drivers.

Cheers,
Julien

-
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK  win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100url=/
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel