From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andy Shevchenko Subject: Re: [PATCH] serial: 8250: Fix THRE flag usage for CAP_MINI Date: Tue, 27 Jun 2017 12:15:36 +0300 Message-ID: <1498554936.22624.180.camel@linux.intel.com> References: <1498485581-72097-1-git-send-email-phil@raspberrypi.org> <627a530c-d78c-5338-5ddf-0b3f6a09c37e@i2se.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-rpi-kernel" Errors-To: linux-rpi-kernel-bounces+glkr-linux-rpi-kernel=m.gmane.org-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org To: Phil Elwell , Stefan Wahren , Greg Kroah-Hartman , Peter Hurley , Yegor Yefremov , Jan Kiszka , linux-serial-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-rpi-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org List-Id: linux-serial@vger.kernel.org On Mon, 2017-06-26 at 16:15 +0100, Phil Elwell wrote: > > But this looks like a bug / quirk than a capability? > > > > It would be better to name this define more self-explaining. > > > > Btw can't see the definition of UART_CAP_MINI? > > The capability was added in d087e7a991f1f61ee2c07db1be7c5cc2aa373f5d, > which > is in linux-next. Given that the "capability" already exists and the > quirk > is likely to be unique to BCM2835 MINI UART, I don't think we should > create > a new quirk. > > Besides, the "HFIFO" capability looks a lot like quirk to me. To me either, which raises a question "Should it be fixed accordingly?" -- Andy Shevchenko Intel Finland Oy