<feed xmlns='http://www.w3.org/2005/Atom'>
<title>bcm63xx/u-boot/tools/patman/checkpatch.py, branch master</title>
<subtitle>Broadcom-s U-Boot</subtitle>
<id>https://git-03.infra.openwrt.org/project/bcm63xx/u-boot/atom?h=master</id>
<link rel='self' href='https://git-03.infra.openwrt.org/project/bcm63xx/u-boot/atom?h=master'/>
<link rel='alternate' type='text/html' href='https://git-03.infra.openwrt.org/project/bcm63xx/u-boot/'/>
<updated>2018-05-07T13:34:12Z</updated>
<entry>
<title>SPDX: Convert all of our single license tags to Linux Kernel style</title>
<updated>2018-05-07T13:34:12Z</updated>
<author>
<name>Tom Rini</name>
</author>
<published>2018-05-06T21:58:06Z</published>
<link rel='alternate' type='text/html' href='https://git-03.infra.openwrt.org/project/bcm63xx/u-boot/commit/?id=83d290c56fab2d38cd1ab4c4cc7099559c1d5046'/>
<id>urn:sha1:83d290c56fab2d38cd1ab4c4cc7099559c1d5046</id>
<content type='text'>
When U-Boot started using SPDX tags we were among the early adopters and
there weren't a lot of other examples to borrow from.  So we picked the
area of the file that usually had a full license text and replaced it
with an appropriate SPDX-License-Identifier: entry.  Since then, the
Linux Kernel has adopted SPDX tags and they place it as the very first
line in a file (except where shebangs are used, then it's second line)
and with slightly different comment styles than us.

In part due to community overlap, in part due to better tag visibility
and in part for other minor reasons, switch over to that style.

This commit changes all instances where we have a single declared
license in the tag as both the before and after are identical in tag
contents.  There's also a few places where I found we did not have a tag
and have introduced one.

Signed-off-by: Tom Rini &lt;trini@konsulko.com&gt;
</content>
</entry>
<entry>
<title>patman: Format checkpatch messages for IDE throwback</title>
<updated>2017-02-08T13:07:35Z</updated>
<author>
<name>Simon Glass</name>
</author>
<published>2017-01-17T23:52:23Z</published>
<link rel='alternate' type='text/html' href='https://git-03.infra.openwrt.org/project/bcm63xx/u-boot/commit/?id=8aa41363ebf46b9ff63b311bd2aa793c67a49def'/>
<id>urn:sha1:8aa41363ebf46b9ff63b311bd2aa793c67a49def</id>
<content type='text'>
It is convenient to be able to deal with checkpatch warnings in the same
way as build warnings. Tools such as emacs and kate can quickly locate
the source file and line automatically.

To achieve this, adjust the format to match the C compiler, and output to
stderr.

Signed-off-by: Simon Glass &lt;sjg@chromium.org&gt;
</content>
</entry>
<entry>
<title>patman: Make print statements python 3.x safe</title>
<updated>2016-10-09T15:30:32Z</updated>
<author>
<name>Paul Burton</name>
</author>
<published>2016-09-27T15:03:50Z</published>
<link rel='alternate' type='text/html' href='https://git-03.infra.openwrt.org/project/bcm63xx/u-boot/commit/?id=a920a17b2f418535870788ae81234dc6b8aa6661'/>
<id>urn:sha1:a920a17b2f418535870788ae81234dc6b8aa6661</id>
<content type='text'>
In python 3.x, print must be used as a function call. Convert all print
statements to the function call style, importing from __future__ where
we print with no trailing newline or print to a file object.

Signed-off-by: Paul Burton &lt;paul.burton@imgtec.com&gt;
Acked-by: Simon Glass &lt;sjg@chromium.org&gt;
</content>
</entry>
<entry>
<title>patman: Adjust command.Output() to raise an error by default</title>
<updated>2016-09-19T03:04:38Z</updated>
<author>
<name>Simon Glass</name>
</author>
<published>2016-07-26T00:59:00Z</published>
<link rel='alternate' type='text/html' href='https://git-03.infra.openwrt.org/project/bcm63xx/u-boot/commit/?id=785f1548a9e74cf4796c96e6f32ed67b25f79b81'/>
<id>urn:sha1:785f1548a9e74cf4796c96e6f32ed67b25f79b81</id>
<content type='text'>
It is more useful to have this method raise an error when something goes
wrong. Make this the default and adjust the few callers that don't want to
use it this way.

Signed-off-by: Simon Glass &lt;sjg@chromium.org&gt;
</content>
</entry>
<entry>
<title>tools, scripts: refactor error-out statements of Python scripts</title>
<updated>2014-08-21T16:01:11Z</updated>
<author>
<name>Masahiro Yamada</name>
</author>
<published>2014-08-15T15:59:26Z</published>
<link rel='alternate' type='text/html' href='https://git-03.infra.openwrt.org/project/bcm63xx/u-boot/commit/?id=31e2141d5ad76b55e4e34ee7241adc1c3d9ef099'/>
<id>urn:sha1:31e2141d5ad76b55e4e34ee7241adc1c3d9ef099</id>
<content type='text'>
In Python, sys.exit() function can also take an object other
than an integer.

If an integer is given to the argument, Python exits with the return
code of it.  If a non-integer argument is given, Python outputs it
to stderr and exits with the return code of 1.

That means,

    print &gt;&gt; sys.stderr, "Blah Blah"
    sys.exit(1)

is equivalent to

    sys.exit("Blah Blah")

The latter is a useful shorthand.

Note:
Some error messages in Buildman and Patman were output to stdout.
But they should go to stderr.  They are also fixed by this commit.
This is a nice side effect.

Signed-off-by: Masahiro Yamada &lt;yamada.m@jp.panasonic.com&gt;
Acked-by: Simon Glass &lt;sjg@chromium.org&gt;
</content>
</entry>
<entry>
<title>Add GPL-2.0+ SPDX-License-Identifier to source files</title>
<updated>2013-07-24T13:44:38Z</updated>
<author>
<name>Wolfgang Denk</name>
</author>
<published>2013-07-08T07:37:19Z</published>
<link rel='alternate' type='text/html' href='https://git-03.infra.openwrt.org/project/bcm63xx/u-boot/commit/?id=1a4596601fd395f3afb8f82f3f840c5e00bdd57a'/>
<id>urn:sha1:1a4596601fd395f3afb8f82f3f840c5e00bdd57a</id>
<content type='text'>
Signed-off-by: Wolfgang Denk &lt;wd@denx.de&gt;
[trini: Fixup common/cmd_io.c]
Signed-off-by: Tom Rini &lt;trini@ti.com&gt;
</content>
</entry>
<entry>
<title>patman: Fix up checkpatch parsing to deal with 'CHECK' lines</title>
<updated>2013-04-04T21:04:34Z</updated>
<author>
<name>Simon Glass</name>
</author>
<published>2013-03-26T13:09:39Z</published>
<link rel='alternate' type='text/html' href='https://git-03.infra.openwrt.org/project/bcm63xx/u-boot/commit/?id=d29fe6e2d2cbcbfd1bfaaf886818d84edde0fc0d'/>
<id>urn:sha1:d29fe6e2d2cbcbfd1bfaaf886818d84edde0fc0d</id>
<content type='text'>
checkpatch has a new type of warning, a 'CHECK'. At present patman fails
with these, which makes it less than useful.

Add support for checks, making it backwards compatible with the old
checkpatch.

At the same time, clean up formatting of the CheckPatches() output,
fix erroneous "internal error" if multiple patches have warnings and
be more robust to new types of problems.

Signed-off-by: Simon Glass &lt;sjg@chromium.org&gt;
Reviewed-by: Doug Anderson &lt;dianders@chromium.org&gt;
</content>
</entry>
<entry>
<title>patman: Allow use outside of u-boot tree</title>
<updated>2013-01-31T23:23:40Z</updated>
<author>
<name>Vadim Bendebury</name>
</author>
<published>2013-01-09T16:00:10Z</published>
<link rel='alternate' type='text/html' href='https://git-03.infra.openwrt.org/project/bcm63xx/u-boot/commit/?id=99adf6eda7bed1beb3fa3e18951342f67b108db7'/>
<id>urn:sha1:99adf6eda7bed1beb3fa3e18951342f67b108db7</id>
<content type='text'>
To make it usable in git trees not providing a patch checker
implementation, add a command line option, allowing to suppress patch
check. While we are at it, sort debug options alphabetically.

Also, do not raise an exception if checkpatch.pl is not found - just
print an error message suggesting to use the new option, and return
nonzero status.

   . unit test passes:
    $ ./patman  -t
    &lt;unittest.result.TestResult run=7 errors=0 failures=0&gt;
   . successfully used patman in the autotest tree to generate a patch
     email (with --no-check option)
   . successfully used patman in the u-boot tree to generate a patch
     email
   . `patman --help' now shows command line options ordered
     alphabetically

Signed-off-by: Vadim Bendebury &lt;vbendeb@chromium.org&gt;
Acked-by: Doug Anderson &lt;dianders@chromium.org&gt;
Acked-by: Simon Glass &lt;sjg@chromium.org&gt;
</content>
</entry>
<entry>
<title>patman: Look for checkpatch in the scripts directory</title>
<updated>2013-01-31T23:23:39Z</updated>
<author>
<name>Doug Anderson</name>
</author>
<published>2012-11-26T15:23:23Z</published>
<link rel='alternate' type='text/html' href='https://git-03.infra.openwrt.org/project/bcm63xx/u-boot/commit/?id=d96ef37df78eac5c636866d8f226991a5d96e6bb'/>
<id>urn:sha1:d96ef37df78eac5c636866d8f226991a5d96e6bb</id>
<content type='text'>
The Linux kernel stores checkpatch.pl in the scripts directory.  Add
that to the search path to make things more automatic for kernel
development.

Signed-off-by: Doug Anderson &lt;dianders@chromium.org&gt;
Acked-by: Simon Glass &lt;sjg@chromium.org&gt;
</content>
</entry>
<entry>
<title>patman: Handle checkpatch.pl not providing file/line info</title>
<updated>2012-10-15T18:54:04Z</updated>
<author>
<name>Simon Glass</name>
</author>
<published>2012-09-27T15:33:46Z</published>
<link rel='alternate' type='text/html' href='https://git-03.infra.openwrt.org/project/bcm63xx/u-boot/commit/?id=afb9bf55d093287458f4c2d3d2e42d90a7e91c95'/>
<id>urn:sha1:afb9bf55d093287458f4c2d3d2e42d90a7e91c95</id>
<content type='text'>
Sometimes we don't get a valid filename or line number from checkpatch.pl,
for example if the patch is in a bad format. Deal with this by using a
default value, rather than a stack trace.

Signed-off-by: Simon Glass &lt;sjg@chromium.org&gt;
</content>
</entry>
</feed>
