Re: [PATCH] checkpatch: check for missing \n at the end of logging message

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Joe Perches <joe@xxxxxxxxxxx>, apw@xxxxxxxxxxxxx, Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
Subject: Re: [PATCH] checkpatch: check for missing \n at the end of logging message
From: Marion & Christophe JAILLET <christophe.jaillet@xxxxxxxxxx>
Date: Sat, 11 Apr 2020 09:13:49 +0200
Cc: linux-kernel@xxxxxxxxxxxxxxx, kernel-janitors@xxxxxxxxxxxxxxx
In-reply-to: <26b49cf5-453b-9a81-b045-fdc99c42fcf8@wanadoo.fr>
References: <20200407204908.10420-1-christophe.jaillet@wanadoo.fr> <8617a6b94c0644bce1fd4ca77309d67a612e6300.camel@perches.com> <6e52383e-100d-b016-32c2-6fb54938b6fe@wanadoo.fr> <c9fd4bc75812fed4799c2fb87b452b809a7e9a7a.camel@perches.com> <db2730700ab3eb7008413a1e7bba94ca7c49a031.camel@perches.com> <26b49cf5-453b-9a81-b045-fdc99c42fcf8@wanadoo.fr>
User-agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:68.0) Gecko/20100101 Thunderbird/68.7.0


Le 11/04/2020 à 09:12, Christophe JAILLET a écrit :

Hi,

    ./scripts/checkpatch.pl -f drivers/soc/kendryte/k210-sysctl.c
is missing line 189, even if it looks like a construction correctlyspotted in some other files:
    if (foo) {
        dev_err(...);
        ...
    };

CJ

Oops, my mistake, this one is correctly spotted :)

CJ

References:
- [PATCH] checkpatch: check for missing \n at the end of logging message
  - From: Christophe JAILLET
- Re: [PATCH] checkpatch: check for missing \n at the end of logging message
  - From: Joe Perches
- Re: [PATCH] checkpatch: check for missing \n at the end of logging message
  - From: Christophe JAILLET
- Re: [PATCH] checkpatch: check for missing \n at the end of logging message
  - From: Joe Perches
- Re: [PATCH] checkpatch: check for missing \n at the end of logging message
  - From: Joe Perches
- Re: [PATCH] checkpatch: check for missing \n at the end of logging message
  - From: Christophe JAILLET

Prev by Date: Re: [PATCH] checkpatch: check for missing \n at the end of logging message
Next by Date: [PATCH] soc: qcom: ipa: Add a missing '\n' in a log message
Previous by thread: Re: [PATCH] checkpatch: check for missing \n at the end of logging message
Next by thread: Re: [PATCH] checkpatch: check for missing \n at the end of logging message
Index(es):
- Date
- Thread

[Index of Archives] [Kernel Development] [Kernel Announce] [Kernel Newbies] [Linux Networking Development] [Share Photos] [IDE] [Security] [Git] [Netfilter] [Yosemite News] [MIPS Linux] [ARM Linux] [Device Mapper]