Skip to content

CHECKPATCH: fpgadata.c cleanup

Matthias Fuchs u-boot 1 version New

Get series with b4
b4 am [email protected]

Versions newest first

  1. v2 Latest0 of 18 patches 0 repliesNew

Patches in v2 No applies recorded yet

None of the 18 patches the cover letter announces are in this archive.

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v2 yet.

Read the v2 thread

Cover letter [U-Boot] [PATCH 00/18 V2] CHECKPATCH: fpgadata.c cleanup

Hi Marek,

Quoted 30 lines · Marek Vasut
Quoted 16 lines
Quoted 16 lines

Hi Marek,

On Thu, 15 Dec 2011 14:26:41 +0100
Marek Vasut <[email protected]> wrote:
...

Quoted 9 lines
 18 files changed, 63874 insertions(+), 31942 deletions(-)

V2: Align to 8 bytes per line.
Cc: Anatolij Gustschin <[email protected]>
Cc: Stefano Babic <[email protected]>

Anatolij, Stefano, can you skim through these and give me your ACK so I can push them through staging ? Still, there is around three thousand files that don't conform to checkpatch style.

Quoted 8 lines
Quoted 4 lines

I'll look at these patches.

Yes, some video drivers files are on my checkpatch cleanup TODO list.

Well I can share the complete list of files that fail checkpatch. I ran checkpatch -f on the whole tree (.c and .h files) a few days ago :-) It ran for 5+ hours ;-)

http://marex.hackndev.com/fail-list.txt

This is after applying these two patchsets (logo and fpga) and partly cleaning up board/esd/)

Quoted 7 lines

btw. let's queue these (both logo and fpga) patches for -next please. They are not urgent and I'd like to see my stats for this release not tainted by this patchbomb.

Also, I'll share the partial cleanup of ESD boards if Matthias/Stefan is interested. I checked how arch/arm is also broken and me being mostly ARM guy, I'd like to continue cleaning up there.

I would do that clean up by myself for the esd files. As long as we do not have a strict timeline for this, I will see if I could start with that in January. I will also check if we could get rid of some boards that are not active anymore.

Matthias