From mboxrd@z Thu Jan 1 00:00:00 1970 From: Joe Perches Subject: Re: [PATCH] trivial treewide: Convert dev_set_uevent_suppress argument to bool Date: Fri, 02 Sep 2016 09:49:29 -0700 Message-ID: <1472834969.6758.52.camel@perches.com> References: <699402ce4c488995b4ddabff0f3b262851cf56ac.1472760613.git.joe@perches.com> <1472777516.4176.149.camel@perches.com> <1472830862.6758.7.camel@perches.com> <7405780b-3dcd-3c6a-9268-3fac6f3a7b3f@sandisk.com> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Return-path: In-Reply-To: <7405780b-3dcd-3c6a-9268-3fac6f3a7b3f@sandisk.com> Sender: linux-acpi-owner@vger.kernel.org List-Archive: List-Post: To: Bart Van Assche , Jens Axboe , "Rafael J. Wysocki" , Len Brown , Sebastian Ott , Peter Oberparleiter , Miklos Szeredi , Jiri Kosina Cc: Martin Schwidefsky , Heiko Carstens , "linux-block@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "linux-acpi@vger.kernel.org" , "linux-s390@vger.kernel.org" , "linux-fsdevel@vger.kernel.org" List-ID: On Fri, 2016-09-02 at 08:59 -0700, Bart Van Assche wrote: > How about the following: >=20 > dev_set_uevent_suppress() expects a boolean as second argument. Make=A0 > this clear by passing true/false instead of 1/0 as the second > argument. dev_set_uevent_suppress()=A0doesn't currently expect a boolean. The patch changes the function definition argument from int to bool and also changes all callers of the function to true/false from 1/0.