[U-Boot] [PATCH v4 2/3] mxs: Fix iomux.h to not break build during assembly stage

Otavio Salvador otavio at ossystems.com.br
Mon Jan 28 18:09:21 CET 2013


On Mon, Jan 28, 2013 at 2:35 PM, Marek Vasut <marex at denx.de> wrote:
> Dear Otavio Salvador,
>
>> On Mon, Jan 28, 2013 at 2:16 PM, Marek Vasut <marex at denx.de> wrote:
>> > Dear Otavio Salvador,
>> >
>> >> On Mon, Jan 28, 2013 at 1:57 PM, Marek Vasut <marex at denx.de> wrote:
>> >> > Dear Otavio Salvador,
>> >> >
>> >> > Missing commit message.
>> >>
>> >> I think short description is clear enough. What you'd like me to put
>> >> in long one?
>> >
>> > Reasoning why this patch is necessary.
>>
>> Well, it seems quite clear it fix a build failure when it is used in
>> assembly stage; it is clear enough for me, in the shortlog
>
> I don't understand why did you also add the header below. That's not clear.
>
>> >> >> Signed-off-by: Otavio Salvador <otavio at ossystems.com.br>
>> >> >> ---
>> >> >> Changes in v4:
>> >> >> - New patch
>> >> >>
>> >> >> Changes in v3: None
>> >> >> Changes in v2: None
>> >> >>
>> >> >>  arch/arm/include/asm/arch-mxs/iomux.h | 5 +++++
>> >> >>  1 file changed, 5 insertions(+)
>> >> >>
>> >> >> diff --git a/arch/arm/include/asm/arch-mxs/iomux.h
>> >> >> b/arch/arm/include/asm/arch-mxs/iomux.h index 7abdf58..4288715 100644
>> >> >> --- a/arch/arm/include/asm/arch-mxs/iomux.h
>> >> >> +++ b/arch/arm/include/asm/arch-mxs/iomux.h
>> >> >> @@ -21,6 +21,10 @@
>> >> >>
>> >> >>  #ifndef __MACH_MXS_IOMUX_H__
>> >> >>  #define __MACH_MXS_IOMUX_H__
>> >> >>
>> >> >> +#ifndef __ASSEMBLY__
>> >> >> +
>> >> >> +#include <asm/types.h>
>> >> >> +
>> >> >
>> >> > Is this needed?
>> >>
>> >> Yes; or I would not have added it right? :)
>> >
>> > Why is it needed?
>
> Here, I fail to understand what this has to do with fixing the inclusion of this
> in assembly files.

I sent v5; hope it is fine now.

--
Otavio Salvador                             O.S. Systems
E-mail: otavio at ossystems.com.br  http://www.ossystems.com.br
Mobile: +55 53 9981-7854              http://projetos.ossystems.com.br


More information about the U-Boot mailing list