Re: [Intel-gfx] [PATCH] [TRIVIAL] Fix declaration of intel_gmbus_{is_forced_bit/is_port_falid} in i915 driver.

2013-05-07 Thread Jan-Simon Möller
Ok, so let me resend a version with static inline .

Best,
JS


On Saturday 04 May 2013 10:55:50 PaX Team wrote:
 On 3 May 2013 at 15:03, Jani Nikula wrote:
   This fixes a compilation issue with clang. An initial version of this
   patch
   was developed by PaX Team pageexec at freemail.hu.
   This is respin of this patch.
   
   Signed-off-by: Jan-Simon Möller dl...@gmx.de
   CC: pagee...@freemail.hu
   CC: daniel.vet...@ffwll.ch
   CC: airl...@linux.ie
   CC: intel-...@lists.freedesktop.org
   CC: dri-devel@lists.freedesktop.org
   CC: linux-ker...@vger.kernel.org
   
   Picked up for -fixes, thanks for the patch.
  
  Please drop it.
  
  The patch removes the inline keyword, creating dozens of copies of the
 
  functions, and consequently loads of warnings:
 in my original patch they were both static inline, not sure where the
 inline got lost...
-- 

Dipl.-Ing.
Jan-Simon Möller

jansimon.moel...@gmx.de
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


[Intel-gfx] [PATCH] [TRIVIAL] Fix declaration of intel_gmbus_{is_forced_bit/is_port_falid} in i915 driver.

2013-05-06 Thread Jan-Simon Möller
Ok, so let me resend a version with "static inline" .

Best,
JS


On Saturday 04 May 2013 10:55:50 PaX Team wrote:
> On 3 May 2013 at 15:03, Jani Nikula wrote:
> > >> This fixes a compilation issue with clang. An initial version of this
> > >> patch
> > >> was developed by PaX Team .
> > >> This is respin of this patch.
> > >> 
> > >> Signed-off-by: Jan-Simon M?ller 
> > >> CC: pageexec at freemail.hu
> > >> CC: daniel.vetter at ffwll.ch
> > >> CC: airlied at linux.ie
> > >> CC: intel-gfx at lists.freedesktop.org
> > >> CC: dri-devel at lists.freedesktop.org
> > >> CC: linux-kernel at vger.kernel.org
> > > 
> > > Picked up for -fixes, thanks for the patch.
> > 
> > Please drop it.
> > 
> > The patch removes the inline keyword, creating dozens of copies of the
> 
> > functions, and consequently loads of warnings:
> in my original patch they were both static inline, not sure where the
> inline got lost...
-- 

Dipl.-Ing.
Jan-Simon M?ller

jansimon.moeller at gmx.de