[Meego-kernel] [PATCH 27/28] GFX: Add TMD video mode support

Chris Leech christopher.leech at linux.intel.com
Fri Oct 8 12:43:44 PDT 2010


On 10/07/2010 03:33 PM, Patel, Hitesh K wrote:
> You can ignore this.  We already have another approach.
>
> Ignore the changes in mrst.c and mrst.h files.

Nobody's going to hand edit your patches to remove hunks.

If there are sections in this patch that shouldn't be, you need to fix 
them and resend the series.

- Chris

>
> Hitesh
>
> -----Original Message-----
> From: Arjan van de Ven [mailto:arjan at linux.intel.com]
> Sent: Thursday, October 07, 2010 3:31 PM
> To: Patel, Hitesh K
> Cc: meego-kernel at meego.com; Dou, Justin; Mai, Leonard
> Subject: Re: [Meego-kernel] [PATCH 27/28] GFX: Add TMD video mode support
>
>    @@ -46,6 +46,7 @@ enum {
>        MRST_PLATFORM_LANFORD = 1,
>        MRST_PLATFORM_SHCDK,
>        MRST_PLATFORM_AAVA_SC,
> +    MDFLD_PLATFORM_ES2,
>    };
>
>
> this is the wrong approach, and I'm not going to take this really...
>
>
> the code needs to look at the platform DMI data to find out what system
> it's running on, not expand this enum
> (which we're trying to remove)
>
> _______________________________________________
> Meego-kernel mailing list
> Meego-kernel at lists.meego.com
> http://lists.meego.com/listinfo/meego-kernel
>



More information about the Meego-kernel mailing list