Re: New version of dyn-minor.patch

2004-09-11 Thread Simon 'corecode' Schubert
On 12.09.2004, at 01:58, Jon Smirl wrote: We know how to remove the DRM() macros and inter_module stuff by switching to a drm_core library model. DaveA has already coded up a prototype. We aren't switching because people are objecting to the change. I'm not sure what the status of the objections is

Re: New version of dyn-minor.patch

2004-09-11 Thread Jon Smirl
On Sat, 11 Sep 2004 11:54:49 -0400, Jon Smirl <[EMAIL PROTECTED]> wrote: > On Sat, 11 Sep 2004 13:53:41 +0100, Alan Cox <[EMAIL PROTECTED]> wrote: > > On Sad, 2004-09-11 at 00:36, Jon Smirl wrote: > > > inter_module can't be removed until we move to the drm_core design > > > with personality module

Re: New version of dyn-minor.patch

2004-09-11 Thread Jon Smirl
On Sat, 11 Sep 2004 13:53:41 +0100, Alan Cox <[EMAIL PROTECTED]> wrote: > On Sad, 2004-09-11 at 00:36, Jon Smirl wrote: > > inter_module can't be removed until we move to the drm_core design > > with personality modules > > Of course it can go. You just fix up the DRI to start using > try_module_g

Re: New version of dyn-minor.patch

2004-09-11 Thread Alan Cox
On Sad, 2004-09-11 at 00:36, Jon Smirl wrote: > inter_module can't be removed until we move to the drm_core design > with personality modules Of course it can go. You just fix up the DRI to start using try_module_get(). Actually when you have the video class driver layer it all comes for free anyw

Re: New version of dyn-minor.patch

2004-09-10 Thread Jon Smirl
inter_module can't be removed until we move to the drm_core design with personality modules. inter_module has always been in DRM and I believe DRM is the only current user. I'm all for switching to the drm_core design as soon as we can convince everyone to do it. The DRM(probe) macro is there for

Re: New version of dyn-minor.patch

2004-09-10 Thread Alan Cox
On Gwe, 2004-09-10 at 22:30, Jon Smirl wrote: > No changes in the code, it's just regenerated against current DRM CVS - drm_probe(pdev, &DRM(pciidlist[i])); + DRM(probe)(pdev, &DRM(pciidlist[i])); Seems to revert macro clean up work. +

New version of dyn-minor.patch

2004-09-10 Thread Jon Smirl
No changes in the code, it's just regenerated against current DRM CVS. -- Jon Smirl [EMAIL PROTECTED] = linux/drmP.h 1.11 vs edited = --- 1.11/linux/drmP.h Sun Sep 5 21:22:06 2004 +++ edited/linux/drmP.h Fri Sep 10 17:09:37 2004 @@ -56,6 +56,7 @@ #include /* For (un)lock_kernel */ #in