Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1632192

Re: [PATCH] checkpatch: add --typedefsfile

From Jerome Forissier <jerome.forissier@linaro.org>
Newsgroups linux.kernel
Subject Re: [PATCH] checkpatch: add --typedefsfile
Date 2017-04-27 17:50 +0200
Message-ID <tATQ6-2Ak-3@gated-at.bofh.it> (permalink)
References <tymvf-2j8-9@gated-at.bofh.it> <tynAZ-2Vh-5@gated-at.bofh.it> <tyAoy-2um-11@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On 04/21/2017 08:31 AM, Jerome Forissier wrote:
> On 04/20/2017 06:49 PM, Joe Perches wrote:
>> On Thu, 2017-04-20 at 17:39 +0200, Jerome Forissier wrote:
>>> When using checkpatch on out-of-tree code, it may occur that some
>>> project-specific types are used, which will cause spurious warnings.
>>> Add the --typedefsfile option as a way to extend the known types and
>>> deal with this issue.
>>
>> I'm not opposed to the addition.
>> What out-of-tree project is this for?
> 
> OP-TEE [1]. We run a Travis job on all pull requests [2], and checkpatch
> is part of that. The typical false warning we get on a regular basis is
> with some pointers to functions returning TEE_Result [3], which is a
> typedef from the GlobalPlatform APIs. We consider it is acceptable to
> use GP types in the OP-TEE core implementation, that's why this patch
> would be helpful for us.
> 
> [1] https://github.com/OP-TEE/optee_os
> [2] https://travis-ci.org/OP-TEE/optee_os/builds
> [3] https://travis-ci.org/OP-TEE/optee_os/builds/193355335#L1733 

Ping?

Back to linux.kernel | Previous | NextNext in thread | Find similar | Unroll thread


Thread

Re: [PATCH] checkpatch: add --typedefsfile Jerome Forissier <jerome.forissier@linaro.org> - 2017-04-27 17:50 +0200
  Re: [PATCH] checkpatch: add --typedefsfile Joe Perches <joe@perches.com> - 2017-04-27 18:10 +0200

csiph-web