Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
MIME-Version: 1.0
In-Reply-To: <20180129065053.GC20230@1wt.eu>
References: <1516979955-7466-1-git-send-email-baijiaju1990@gmail.com> <20180129065053.GC20230@1wt.eu>
From:   Miguel Ojeda <miguel.ojeda.sandonis@gmail.com>
Date:   Mon, 12 Feb 2018 13:00:21 +0100
Message-ID: <CANiq72=QsiiUgaQAh2Jp9Mh2Fmrkn=oNwNcQPT3mHX0THcqyiw@mail.gmail.com>
Subject: Re: [PATCH] auxdisplay: charlcd: delete mdelay in long_sleep
To:     Willy Tarreau <w@1wt.eu>
Cc:     Jia-Ju Bai <baijiaju1990@gmail.com>,
        linux-kernel <linux-kernel@vger.kernel.org>
Content-Type: text/plain; charset="UTF-8"
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Mon, Jan 29, 2018 at 7:50 AM, Willy Tarreau <w@1wt.eu> wrote:
> Hi,
>
> On Fri, Jan 26, 2018 at 11:19:15PM +0800, Jia-Ju Bai wrote:
>> The function long_sleep() calls mdelay() when in an interrupt handler.
>> But only charlcd_clear_display() and charlcd_init_display calls
>> long_sleep(), and my tool finds that the two functions
>> are never called in an interrupt handler.
>> Thus mdelay() and in_interrupt() are not necessary.
>>
>> This is found by a static analysis tool named DCNS written by myself.
>
> Looks good. This code is extremely old (started in 2.2) so I'm not
> surprised at all that after many changes such parts are not used
> anymore.
>
> Acked-by: Willy Tarreau <w@1wt.eu>

Thanks for the patch and the ack, putting it in the queue.

Miguel

>
> Willy