[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <YIE4hh5NOJMT7QqK@kroah.com>
Date: Thu, 22 Apr 2021 10:49:10 +0200
From: Greg KH <gregkh@...uxfoundation.org>
To: Saurav Girepunje <saurav.girepunje@...il.com>
Cc: b-liu@...com, linux-usb@...r.kernel.org,
linux-kernel@...r.kernel.org, saurav.girepunje@...mail.com
Subject: Re: [PATCH] usb: musb: davinci: change the variable type
On Mon, Apr 19, 2021 at 12:06:18AM +0530, Saurav Girepunje wrote:
> vbus_state is define as bool but on davinci.c assigning a value
> '-1' to the bool variable.
Does it also test that value?
If so, shouldn't that logic error be fixed instead of working around it
by changing the variable type?
This feels wrong...
thanks,
greg k-h
Powered by blists - more mailing lists