linux-kernel - Re: [PATCH 3/6] gxfb: move MSR bit fields into gxfb.h

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20080310162844.6e8e9215@ephemeral>
Date:	Mon, 10 Mar 2008 16:28:44 -0400
From:	Andres Salomon <dilinger@...ued.net>
To:	Andrew Morton <akpm@...ux-foundation.org>
Cc:	adaplas@...il.com, linux-kernel@...r.kernel.org,
	linux-fbdev-devel@...ts.sourceforge.net, info-linux@...de.amd.com,
	jordan.crouse@....com
Subject: Re: [PATCH 3/6] gxfb: move MSR bit fields into gxfb.h

On Mon, 10 Mar 2008 13:14:32 -0700
Andrew Morton <akpm@...ux-foundation.org> wrote:

> On Sat, 8 Mar 2008 20:48:40 -0500
> Andres Salomon <dilinger@...ued.net> wrote:
> 
> > >From bee7242cf04c4a060b311b11985efb495f308355 Mon Sep 17 00:00:00 2001
> > From: Andres Salomon <dilinger@...ian.org>
> > Date: Mon, 25 Feb 2008 15:29:36 -0500
> > Subject: [PATCH] gxfb: move MSR bit fields into gxfb.h
> > 
> > This continues the gxfb header cleanups.  MSRs are defined in geode.h;
> > the specific bits we care about are defined in gxfb.h.
> > 
> > Signed-off-by: Andres Salomon <dilinger@...ian.org>
> > ---
> >  drivers/video/geode/display_gx.h |    3 ---
> >  drivers/video/geode/gxfb.h       |   16 ++++++++++++++++
> >  drivers/video/geode/gxfb_core.c  |    2 +-
> >  drivers/video/geode/video_gx.c   |    4 ++--
> >  drivers/video/geode/video_gx.h   |   14 --------------
> >  5 files changed, 19 insertions(+), 20 deletions(-)
> > 
> > diff --git a/drivers/video/geode/display_gx.h b/drivers/video/geode/display_gx.h
> > index 56e9d2e..ad556d3 100644
> > --- a/drivers/video/geode/display_gx.h
> > +++ b/drivers/video/geode/display_gx.h
> > @@ -16,7 +16,4 @@ int gx_line_delta(int xres, int bpp);
> >  
> >  extern struct geode_dc_ops gx_dc_ops;
> >  
> > -/* MSR that tells us if a TFT or CRT is attached */
> > -#define GLD_MSR_CONFIG_DM_FP 0x40
> > -
> >  #endif /* !__DISPLAY_GX1_H__ */
> 
> Something has gone wrong here.  The code in current mainline is
> 
> 
> 
> extern struct geode_dc_ops gx_dc_ops;
> 
> /* MSR that tells us if a TFT or CRT is attached */
> #define GLD_MSR_CONFIG   0xC0002001
> #define GLD_MSR_CONFIG_DM_FP 0x40
> 
> /* Display controller registers */
> 
> 
> whereas you're patching something else.
> 
> 
> In [1/6] you refer to "the MSR cleanup patch".  I don't know what that is.
> 

That would be this:

http://thread.gmane.org/gmane.linux.kernel/645387/focus=645636


> 
> Please clean up the changelogs - remove those email headers from the
> email-forwarding operation and remove the additional little commentary on
> top: if there's actually anything in there which is needed then it should
> be integrated into the changelog proper, thanks.
> 

For future requests, or should I resend these to you?
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/