Re: [PATCH v1 1/2] auxdisplay: Move panel.c to drivers/auxdisplay folder

From	Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH v1 1/2] auxdisplay: Move panel.c to drivers/auxdisplay folder
Date	2017-03-30 13:30 +0200
Message-ID	<tqGr8-5AE-13@gated-at.bofh.it> (permalink)
References	(2 earlier) <toyxI-2SC-13@gated-at.bofh.it> <tozk6-3uM-35@gated-at.bofh.it> <tpy2C-51D-9@gated-at.bofh.it> <tpych-55m-9@gated-at.bofh.it> <tpz8l-5M6-1@gated-at.bofh.it>
Organization	Intel Finland Oy

Show all headers | View raw

On Mon, 2017-03-27 at 11:01 +0200, Willy TARREAU wrote:
> On Mon, Mar 27, 2017 at 10:26:07AM +0200, Geert Uytterhoeven wrote:
> > On Mon, Mar 27, 2017 at 10:11 AM, Willy TARREAU <wtarreau@haproxy.co
> > m> wrote:
> > > On Fri, Mar 24, 2017 at 04:19:43PM +0100, Geert Uytterhoeven
> > > wrote:
> > > > On Fri, Mar 24, 2017 at 3:29 PM, Andy Shevchenko
> > > > <andriy.shevchenko@linux.intel.com> wrote:
> > > > > On Fri, 2017-03-24 at 15:19 +0100, Geert Uytterhoeven wrote:
> > > > > > On Fri, Mar 24, 2017 at 3:06 PM, Andy Shevchenko
> > > > > > <andriy.shevchenko@linux.intel.com> wrote:

> In fact here the output signals should be seen as a shared bus with
> multiple
> chip select signals. Note that in some designs it's even possible that
> pressing
> multiple buttons will cause crap to be sent to the LCD by short-
> circuiting
> the lines (if no diodes are used) but it might be acceptable for many
> designs,
> especially the DIY field where the principle is "don't do it".

So, summarizing the comments are we okay to proceed or there is a clear
objection?

P.S. Note, that there is at least one driver under drivers/auxdisplay
which supports key buttons.

-- 
Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Intel Finland Oy

Back to linux.kernel | Previous | Next — Next in thread | Find similar | Unroll thread

Thread

Re: [PATCH v1 1/2] auxdisplay: Move panel.c to drivers/auxdisplay  folder Andy Shevchenko <andriy.shevchenko@linux.intel.com> - 2017-03-30 13:30 +0200
  Re: [PATCH v1 1/2] auxdisplay: Move panel.c to drivers/auxdisplay folder Willy TARREAU <wtarreau@haproxy.com> - 2017-03-30 15:00 +0200
    Re: [PATCH v1 1/2] auxdisplay: Move panel.c to drivers/auxdisplay  folder Andy Shevchenko <andriy.shevchenko@linux.intel.com> - 2017-04-04 18:30 +0200

csiph-web