Messages in this thread | | | Date | Wed, 27 Jun 2007 17:00:49 +0530 | From | "Trilok Soni" <> | Subject | Re: [PATCH 02/17] OMAP: Add TI OMAP1610 accelerator entry. |
| |
On 6/26/07, Sam Ravnborg <sam@ravnborg.org> wrote: > Your patch ordering is bad. > Patch 1 uses the symbol: FB_ACCEL_OMAP1610 > but it is defined by patch 2. > > Please reorder patches so the module compiles for each > and every patch being applied. > > In this way you do not break git bisect for example. >
Yes, in this case I will merge 1st and 2nd patch.
-- --Trilok Soni - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |