* [U-Boot] [PATCH] NAND: MXS: include common.h first so cache.h is included in correct order
@ 2012-09-10 15:55 Tom Warren
2012-09-10 15:58 ` Simon Glass
2012-09-10 16:18 ` Marek Vasut
0 siblings, 2 replies; 7+ messages in thread
From: Tom Warren @ 2012-09-10 15:55 UTC (permalink / raw)
To: u-boot
With Simon Glass's include/nand.h alignment changes, some mxs builds
were generating errors. Fix is to ensure asm/cache.h is included before
linux/mtd/nand.h. Moving common.h to top of include list does that.
Signed-off-by: Tom Warren <twarren@nvidia.com>
---
drivers/mtd/nand/mxs_nand.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/mtd/nand/mxs_nand.c b/drivers/mtd/nand/mxs_nand.c
index bf9414f..4701be8 100644
--- a/drivers/mtd/nand/mxs_nand.c
+++ b/drivers/mtd/nand/mxs_nand.c
@@ -25,10 +25,10 @@
* 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
*/
+#include <common.h>
#include <linux/mtd/mtd.h>
#include <linux/mtd/nand.h>
#include <linux/types.h>
-#include <common.h>
#include <malloc.h>
#include <asm/errno.h>
#include <asm/io.h>
--
1.7.0.4
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH] NAND: MXS: include common.h first so cache.h is included in correct order
2012-09-10 15:55 [U-Boot] [PATCH] NAND: MXS: include common.h first so cache.h is included in correct order Tom Warren
@ 2012-09-10 15:58 ` Simon Glass
2012-09-10 16:18 ` Marek Vasut
1 sibling, 0 replies; 7+ messages in thread
From: Simon Glass @ 2012-09-10 15:58 UTC (permalink / raw)
To: u-boot
On Mon, Sep 10, 2012 at 8:55 AM, Tom Warren <twarren.nvidia@gmail.com> wrote:
> With Simon Glass's include/nand.h alignment changes, some mxs builds
> were generating errors. Fix is to ensure asm/cache.h is included before
> linux/mtd/nand.h. Moving common.h to top of include list does that.
>
> Signed-off-by: Tom Warren <twarren@nvidia.com>
Thanks Tom.
Acked-by: Simon Glass <sjg@chromium.org>
> ---
> drivers/mtd/nand/mxs_nand.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/mtd/nand/mxs_nand.c b/drivers/mtd/nand/mxs_nand.c
> index bf9414f..4701be8 100644
> --- a/drivers/mtd/nand/mxs_nand.c
> +++ b/drivers/mtd/nand/mxs_nand.c
> @@ -25,10 +25,10 @@
> * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
> */
>
> +#include <common.h>
> #include <linux/mtd/mtd.h>
> #include <linux/mtd/nand.h>
> #include <linux/types.h>
> -#include <common.h>
> #include <malloc.h>
> #include <asm/errno.h>
> #include <asm/io.h>
> --
> 1.7.0.4
>
^ permalink raw reply [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH] NAND: MXS: include common.h first so cache.h is included in correct order
2012-09-10 15:55 [U-Boot] [PATCH] NAND: MXS: include common.h first so cache.h is included in correct order Tom Warren
2012-09-10 15:58 ` Simon Glass
@ 2012-09-10 16:18 ` Marek Vasut
2012-09-10 20:20 ` Tom Warren
1 sibling, 1 reply; 7+ messages in thread
From: Marek Vasut @ 2012-09-10 16:18 UTC (permalink / raw)
To: u-boot
Dear Tom Warren,
> With Simon Glass's include/nand.h alignment changes, some mxs builds
> were generating errors. Fix is to ensure asm/cache.h is included before
> linux/mtd/nand.h. Moving common.h to top of include list does that.
>
> Signed-off-by: Tom Warren <twarren@nvidia.com>
Acked-by: Marek Vasut <marex@denx.de>
Ccing Stefano.
> ---
> drivers/mtd/nand/mxs_nand.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/mtd/nand/mxs_nand.c b/drivers/mtd/nand/mxs_nand.c
> index bf9414f..4701be8 100644
> --- a/drivers/mtd/nand/mxs_nand.c
> +++ b/drivers/mtd/nand/mxs_nand.c
> @@ -25,10 +25,10 @@
> * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
> */
>
> +#include <common.h>
> #include <linux/mtd/mtd.h>
> #include <linux/mtd/nand.h>
> #include <linux/types.h>
> -#include <common.h>
> #include <malloc.h>
> #include <asm/errno.h>
> #include <asm/io.h>
Best regards,
Marek Vasut
^ permalink raw reply [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH] NAND: MXS: include common.h first so cache.h is included in correct order
2012-09-10 16:18 ` Marek Vasut
@ 2012-09-10 20:20 ` Tom Warren
2012-09-11 4:32 ` stefano babic
0 siblings, 1 reply; 7+ messages in thread
From: Tom Warren @ 2012-09-10 20:20 UTC (permalink / raw)
To: u-boot
Marek/Simon,
> -----Original Message-----
> From: Marek Vasut [mailto:marex at denx.de]
> Sent: Monday, September 10, 2012 9:19 AM
> To: Tom Warren; sjg at chromium.org
> Cc: u-boot at lists.denx.de; scottwood at freescale.com; Tom Warren;
> sbabic at denx.de
> Subject: Re: [PATCH] NAND: MXS: include common.h first so cache.h is
> included in correct order
>
> Dear Tom Warren,
>
> > With Simon Glass's include/nand.h alignment changes, some mxs builds
> > were generating errors. Fix is to ensure asm/cache.h is included
> > before linux/mtd/nand.h. Moving common.h to top of include list does that.
> >
> > Signed-off-by: Tom Warren <twarren@nvidia.com>
>
> Acked-by: Marek Vasut <marex@denx.de>
Thanks. Applied to u-boot-tegra/next and pushed. I'll be moving /next to /master today or tomorrow and sending a new pull request to Albert to clear the decks for T30 support.
Tom
>
> Ccing Stefano.
>
> > ---
> > drivers/mtd/nand/mxs_nand.c | 2 +-
> > 1 files changed, 1 insertions(+), 1 deletions(-)
> >
> > diff --git a/drivers/mtd/nand/mxs_nand.c b/drivers/mtd/nand/mxs_nand.c
> > index bf9414f..4701be8 100644
> > --- a/drivers/mtd/nand/mxs_nand.c
> > +++ b/drivers/mtd/nand/mxs_nand.c
> > @@ -25,10 +25,10 @@
> > * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
> > */
> >
> > +#include <common.h>
> > #include <linux/mtd/mtd.h>
> > #include <linux/mtd/nand.h>
> > #include <linux/types.h>
> > -#include <common.h>
> > #include <malloc.h>
> > #include <asm/errno.h>
> > #include <asm/io.h>
>
> Best regards,
> Marek Vasut
--
nvpublic
^ permalink raw reply [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH] NAND: MXS: include common.h first so cache.h is included in correct order
2012-09-10 20:20 ` Tom Warren
@ 2012-09-11 4:32 ` stefano babic
2012-09-11 6:52 ` Marek Vasut
0 siblings, 1 reply; 7+ messages in thread
From: stefano babic @ 2012-09-11 4:32 UTC (permalink / raw)
To: u-boot
Am 10/09/2012 22:20, schrieb Tom Warren:
> Marek/Simon,
>
Hi Tom,
>> -----Original Message-----
>> From: Marek Vasut [mailto:marex at denx.de]
>> Sent: Monday, September 10, 2012 9:19 AM
>> To: Tom Warren; sjg at chromium.org
>> Cc: u-boot at lists.denx.de; scottwood at freescale.com; Tom Warren;
>> sbabic at denx.de
>> Subject: Re: [PATCH] NAND: MXS: include common.h first so cache.h is
>> included in correct order
>>
>> Dear Tom Warren,
>>
>>> With Simon Glass's include/nand.h alignment changes, some mxs builds
>>> were generating errors. Fix is to ensure asm/cache.h is included
>>> before linux/mtd/nand.h. Moving common.h to top of include list does that.
>>>
>>> Signed-off-by: Tom Warren <twarren@nvidia.com>
>>
>> Acked-by: Marek Vasut <marex@denx.de>
> Thanks. Applied to u-boot-tegra/next and pushed. I'll be moving /next to /master today or tomorrow and sending a new pull request to Albert to clear the decks for T30 support.
>
That's ok, but should this not be merged instead into the i.MX
repository ? That is also the reason why Marek CCed to me.
Regards,
Stefano
--
=====================================================================
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: office at denx.de
=====================================================================
^ permalink raw reply [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH] NAND: MXS: include common.h first so cache.h is included in correct order
2012-09-11 4:32 ` stefano babic
@ 2012-09-11 6:52 ` Marek Vasut
2012-09-11 7:44 ` Stefano Babic
0 siblings, 1 reply; 7+ messages in thread
From: Marek Vasut @ 2012-09-11 6:52 UTC (permalink / raw)
To: u-boot
Dear stefano babic,
> Am 10/09/2012 22:20, schrieb Tom Warren:
> > Marek/Simon,
>
> Hi Tom,
>
> >> -----Original Message-----
> >> From: Marek Vasut [mailto:marex at denx.de]
> >> Sent: Monday, September 10, 2012 9:19 AM
> >> To: Tom Warren; sjg at chromium.org
> >> Cc: u-boot at lists.denx.de; scottwood at freescale.com; Tom Warren;
> >> sbabic at denx.de
> >> Subject: Re: [PATCH] NAND: MXS: include common.h first so cache.h is
> >> included in correct order
> >>
> >> Dear Tom Warren,
> >>
> >>> With Simon Glass's include/nand.h alignment changes, some mxs builds
> >>> were generating errors. Fix is to ensure asm/cache.h is included
> >>> before linux/mtd/nand.h. Moving common.h to top of include list does
> >>> that.
> >>>
> >>> Signed-off-by: Tom Warren <twarren@nvidia.com>
> >>
> >> Acked-by: Marek Vasut <marex@denx.de>
> >
> > Thanks. Applied to u-boot-tegra/next and pushed. I'll be moving /next to
> > /master today or tomorrow and sending a new pull request to Albert to
> > clear the decks for T30 support.
>
> That's ok, but should this not be merged instead into the i.MX
> repository ? That is also the reason why Marek CCed to me.
There's some rework of MTD going on in the tegra tree, I'd say let it slide, it
won't cause conflict.
Best regards,
Marek Vasut
^ permalink raw reply [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH] NAND: MXS: include common.h first so cache.h is included in correct order
2012-09-11 6:52 ` Marek Vasut
@ 2012-09-11 7:44 ` Stefano Babic
0 siblings, 0 replies; 7+ messages in thread
From: Stefano Babic @ 2012-09-11 7:44 UTC (permalink / raw)
To: u-boot
On 11/09/2012 08:52, Marek Vasut wrote:
> There's some rework of MTD going on in the tegra tree, I'd say let it slide, it
> won't cause conflict.
Ok, that is fine with me ;-)
Regards,
Stefano
--
=====================================================================
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2012-09-11 7:44 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-09-10 15:55 [U-Boot] [PATCH] NAND: MXS: include common.h first so cache.h is included in correct order Tom Warren
2012-09-10 15:58 ` Simon Glass
2012-09-10 16:18 ` Marek Vasut
2012-09-10 20:20 ` Tom Warren
2012-09-11 4:32 ` stefano babic
2012-09-11 6:52 ` Marek Vasut
2012-09-11 7:44 ` Stefano Babic
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox