Re: [PATCH] staging: sm7xx: Use kernel framebuffer mode setting

[Greg KH <gregkh@suse.de>]

On Sat, May 07, 2011 at 12:26:49AM +0200, Javier Muñoz wrote:
> On 05/06/2011 11:22 PM, Greg KH wrote:
> >On Fri, May 06, 2011 at 07:24:56PM +0200, Javier M. Mellid wrote:
> >>This patch implements dynamic framebuffer mode setting.
> >>
> >>Previous code works with mode setting in a hard code way. Previous hard
> >>code configuration is used as default configuration if dynamic mode
> >>setting or boot mode setting (via sm712vga_setup) is not used.
> >>
> >>Tested with SM712 supporting 1024x600x16 as default hardware resolution.
> >>
> >>Changes:
> >>
> >>- Implement fb_check_var and fb_set_par callbacks
> >>- Remove __maybe_unused decorator in function being used (sm712vga_setup)
> >>- Minor cleanup on initialization structs related with mode settings
> >>- Updated author copyright
> >>- Updated TODO file
> >>
> >>Signed-off-by: Javier M. Mellid<jmunhoz@igalia.com>
> >>---
> >>  drivers/staging/sm7xx/TODO     |    1 -
> >
> >This applies with fuzz, this leads me to believe that you didn't test
> >this on the latest linux-next tree, right?
> >
> >What did you make this patch against?
> 
> Sorry, I didn't know. I made this patch against
> linux-2.6 (bfd412db9e7b0d8f7b9c09d12d07aa2ac785f1d0) as I did with
> previous patches.
> 
> Testing was about creating, applying, compiling and running patch
> with kernel built from linux-2.6
> 
> Do you need a new patch against linux-next's master? I can redo this patch.

Yes, please always make patches against linux-next as it includes any
other pending patches that are in my tree.

thanks,

greg k-h