Messages in this thread | | | Subject | Re: [PATCH v1 1/2] auxdisplay: Move panel.c to drivers/auxdisplay folder | From | Andy Shevchenko <> | Date | Thu, 30 Mar 2017 14:27:06 +0300 |
| |
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
| |