| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Presubmit complains when uploading this file as it has no copyright
header. Add one to quiet down the presubmit.
BUG=none
BRANCH=none
TEST=repo upload
Signed-off-by: Jack Rosenthal <jrosenth@chromium.org>
Change-Id: I5df2c89ee3de6d3289d45747b35d27f429822c8a
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2622508
Commit-Queue: Paul Fagerburg <pfagerburg@chromium.org>
Reviewed-by: Paul Fagerburg <pfagerburg@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Zephyr commits occasionally use a function called "printk", but this
is not the same printk used in the Linux kernel. So we need to
disable this kernel level check.
BUG=none
BRANCH=none
TEST=presubmit with commit that uses printk
Signed-off-by: Jack Rosenthal <jrosenth@chromium.org>
Change-Id: Ida9eb1a543e3ca243c7db12b33938f0cd2524c44
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2622506
Commit-Queue: Paul Fagerburg <pfagerburg@chromium.org>
Reviewed-by: Paul Fagerburg <pfagerburg@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The Zephyr integration includes KConfig support. Reduce the minimum
length description.
BUG=none
BRANCH=none
TEST=Run 'pre-upload.py` against KConfig change with 1 line description.
Signed-off-by: Keith Short <keithshort@chromium.org>
Change-Id: I4f22e828e420d7e80c58d10eb767a6e72a058463
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2582278
Reviewed-by: Jack Rosenthal <jrosenth@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Linux has its own crazy set of %p pointer extensions. We're moving
the EC into using its own set of pointer extensions so we can turn on
compile-time printf format checking, but they're not necessarily the
same as Linux. Ignore those warnings so people doing perfectly legit
things with our printf don't get pestered.
BUG=chromium:984041
TEST=make -j buildall
BRANCH=None
Change-Id: Ife713d8c8bd2dbb904b2cf6610e278cd3cd4af6d
Signed-off-by: Evan Green <evgreen@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/1728441
Reviewed-by: Jack Rosenthal <jrosenth@chromium.org>
Reviewed-by: Daisuke Nojiri <dnojiri@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Recent update of checkpatch.pl made it ignore C99 style comment in the
new patches. This change will reinstate the check.
BRANCH=none
BUG=none
TEST=attempt to repo upload patches with C99 style comments fail at
pre-upload check phase.
Change-Id: I606e2618cc39abcf2dc0b051aefea722a2359f3b
Signed-off-by: Vadim Bendebury <vbendeb@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/1409495
Reviewed-by: Aseda Aboagye <aaboagye@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Let's not force the linux kernel description format on cherry picked
patchs, especially since this check sometimes gets false positive, and
since this formwat is not used in the ec codebase.
BRANCH=none
BUG=none
TEST=no more annoying error reports on cherry-picked patches
Change-Id: I903bb87c48998932bc9308245d158f212a57fdd1
Signed-off-by: Vadim Bendebury <vbendeb@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/362074
Reviewed-by: Aseda Aboagye <aaboagye@chromium.org>
Reviewed-by: Bill Richardson <wfrichar@chromium.org>
|
|
The checkpatch complaints about both lines longer than 80 characters
AND character strings split between lines. Damn if you do, damn if you
don't. With this addition split character strings are not a problem
any more.
BRANCH=none
BUG=none
TEST=split character strings are not reported as a violation by the
pre-upload script.
Change-Id: I8b5919f086f8c19fe4e6e3a4b99a816111882f0d
Signed-off-by: Vadim Bendebury <vbendeb@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/340535
Reviewed-by: Randall Spangler <rspangler@chromium.org>
Reviewed-by: Shawn N <shawnn@chromium.org>
|