[U-Boot] [PATCH v15 1/9] nds32: add header files support for nds32

Wolfgang Denk wd at denx.de
Fri Oct 7 08:24:18 CEST 2011


Dear =?UTF-8?B?6aas5YWL5rOh?=,

In message <CACCg+XM0r0z0Xm1C9zu9Zc4bJ+sX5b2Gv3w_v5eBjOVX9AsTbA at mail.gmail.com> you wrote:
> 
> I believe you must be exhausted for reviewing these patches again and again.

Not yet - actually I'm playing with doing such things in a mostly
automated way.

> However, we've discussed before, to support a new architecture, there
> are some definitions
> and codes which the checkpatch rules cannot be adapted.

First, we can adapt chackpatch rules.  It just needs to be done.

> We've also discussed that Linux documents said checkpatch is just a
> reference rule,

I'dlike to see checkpatch customized for our needs.  The tools to do
that are in place now.

> you still need to have human examination.

Indeed.

> I guess you just forget that because the patch "works" is too heavy
> and patch check
> automation didn't help on special case much.

I manually inspected all output before actually sending any messages
out.

> When we discuss about those was about NDS32 patch v7, and now is patch v15.
> We've to a lot of fix, includes relocation.
>
> So, could you please tell me now, should I resend the patch v16 or you
> still have my v15
> patches and kindly tell me which part need to be fixed? Thanks!

There is a large number of WARNING: Use of volatile is usually wrong;
I think you bit macros and I/O accessors should be cleaned up. I don;t
want to see volatile there.

WARNING: line over 80 characters needs to be fixed, obviously.


> The 8 errors is due to the new assembly march.h added.
> Only 2 new cases introduced.
>
> ERROR: space prohibited before open square bracket '['
> #1032: FILE: arch/nds32/include/asm/macro.h:66:
> +       lwi     $r5, [$r4]
> This is to load a 32bit from the address x, while the value of
> address x is stored in $r4, and load it into $r5
>
> ERROR: spaces required around that ':' (ctx:VxE)
> #1055: FILE: arch/nds32/include/asm/macro.h:89:
> +1:
>   ^

I agree that we can ignore these (or rather try and add exception
rules).


Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
"The number  of  Unix  installations  has  grown  to  10,  with  more
expected."    - The Unix Programmer's Manual, 2nd Edition, June, 1972


More information about the U-Boot mailing list