Groups > linux.kernel > #1542053

Re: [PATCH linux v1 0/4] Seven segment display support

From	Greg KH <gregkh@linuxfoundation.org>
Newsgroups	linux.kernel
Subject	Re: [PATCH linux v1 0/4] Seven segment display support
Date	2016-12-14 18:00 +0100
Message-ID	<sOl4w-mI-37@gated-at.bofh.it> (permalink)
References	<sOcDL-3Kl-1@gated-at.bofh.it> <sOhap-6vv-5@gated-at.bofh.it> <sOhk5-6yG-15@gated-at.bofh.it> <sOhDs-6UF-5@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On Wed, Dec 14, 2016 at 02:12:41PM +0100, Neil Armstrong wrote:
> On 12/14/2016 01:56 PM, Greg KH wrote:
> > On Wed, Dec 14, 2016 at 01:45:30PM +0100, Thomas Petazzoni wrote:
> >> Hello,
> >>
> >> On Tue, 13 Dec 2016 23:55:00 -0800, Jaghathiswari Rankappagounder
> >> Natarajan wrote:
> >>
> >>> Documentation for the binding which provides an interface for adding clock,
> >>> data and clear signal GPIO lines to control seven segment display.
> >>>
> >>> The platform device driver provides an API for displaying on two 7-segment
> >>> displays, and implements the required bit-banging. The hardware assumed is
> >>> 74HC164 wired to two 7-segment displays.
> >>>
> >>> The character device driver implements the user-space API for letting a user
> >>> write to two 7-segment displays including any conversion methods necessary
> >>> to map the user input to two 7-segment displays.
> >>>
> >>> Adding clock, data and clear signal GPIO lines in the devicetree to control
> >>> seven segment display on zaius platform.
> >>>
> >>> The platform driver matches on the device tree node; the platform driver also
> >>> initializes the character device.
> >>>
> >>> Tested that the seven segment display works properly by writing to the
> >>> character device file on a EVB AST2500 board which also has 74HC164 wired
> >>> to two 7-segment displays.
> >>
> >> FWIW, I proposed a driver for seven segment displays back in 2013:
> >>
> >>    http://lists.infradead.org/pipermail/linux-arm-kernel/2013-January/139986.html
> >>
> >> And the feedback from Greg KH was: we don't need a driver for that, do
> >> it from userspace. See:
> >>
> >>    http://lists.infradead.org/pipermail/linux-arm-kernel/2013-January/139992.html
> >>
> >> So: good luck :-)
> > 
> > Did anyone ever write a library for this type of thing?
> > 
> > Again, I don't want to see one-off drivers for random devices like this
> > that should be able to all be controlled from userspace in a common
> > manner.  Much like we did for fingerprint readers a long long time
> > ago...
> > 
> > thanks,
> > 
> > greg k-h
> 
> Hi Greg,
> 
> Actually, it's more than a random interface, a lot of SoCs and boards actually have such displays
> and it's a pity to use UIO, sysfs gpio bitbanging and all sort of ugly stuff to only print a few
> characters a simple and clean driver could achieve.

Great, then let's make an API that all devices of this type could use,
and not just take individual drivers that all have a custom char or
sysfs interface which requires custom userspace code to be able to drive
all of the different devices in a common way (i.e. a library would have
to be written anyways...)

thanks,

greg k-h

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

Thread

[PATCH linux v1 0/4] Seven segment display support Jaghathiswari Rankappagounder Natarajan <jaghu@google.com> - 2016-12-14 09:00 +0100
  [PATCH linux v1 4/4] arm: dts: Add dt-binding to support seven segment display on zaius Jaghathiswari Rankappagounder Natarajan <jaghu@google.com> - 2016-12-14 09:00 +0100
    Re: [PATCH linux v1 4/4] arm: dts: Add dt-binding to support seven segment display on zaius Arnd Bergmann <arnd@arndb.de> - 2016-12-14 10:00 +0100
      Re: [PATCH linux v1 4/4] arm: dts: Add dt-binding to support seven segment display on zaius Arnd Bergmann <arnd@arndb.de> - 2016-12-14 10:10 +0100
        Re: [PATCH linux v1 4/4] arm: dts: Add dt-binding to support seven  segment display on zaius Russell King - ARM Linux <linux@armlinux.org.uk> - 2016-12-14 12:10 +0100
          Re: [PATCH linux v1 4/4] arm: dts: Add dt-binding to support seven  segment display on zaius Russell King - ARM Linux <linux@armlinux.org.uk> - 2016-12-14 12:50 +0100
            Re: [PATCH linux v1 4/4] arm: dts: Add dt-binding to support seven  segment display on zaius Linus Walleij <linus.walleij@linaro.org> - 2016-12-16 00:10 +0100
    Re: [PATCH linux v1 4/4] arm: dts: Add dt-binding to support seven  segment display on zaius Joel Stanley <joel@jms.id.au> - 2016-12-14 10:10 +0100
  [PATCH linux v1 2/4] drivers: misc: Character device driver for seven segment display Jaghathiswari Rankappagounder Natarajan <jaghu@google.com> - 2016-12-14 09:00 +0100
    Re: [PATCH linux v1 2/4] drivers: misc: Character device driver for  seven segment display Russell King - ARM Linux <linux@armlinux.org.uk> - 2016-12-14 13:40 +0100
  [PATCH linux v1 3/4] drivers: misc: Platform driver for seven segment display support Jaghathiswari Rankappagounder Natarajan <jaghu@google.com> - 2016-12-14 09:00 +0100
  [PATCH linux v1 1/4] Documentation: dt-bindings: Document bindings for seven segment display support Jaghathiswari Rankappagounder Natarajan <jaghu@google.com> - 2016-12-14 09:00 +0100
  Re: [PATCH linux v1 0/4] Seven segment display support Thomas Petazzoni <thomas.petazzoni@free-electrons.com> - 2016-12-14 13:50 +0100
    Re: [PATCH linux v1 0/4] Seven segment display support Greg KH <gregkh@linuxfoundation.org> - 2016-12-14 14:00 +0100
      Re: [PATCH linux v1 0/4] Seven segment display support Neil Armstrong <narmstrong@baylibre.com> - 2016-12-14 14:20 +0100
        Re: [PATCH linux v1 0/4] Seven segment display support Arnd Bergmann <arnd@arndb.de> - 2016-12-14 15:20 +0100
          Re: [PATCH linux v1 0/4] Seven segment display support David Daney <ddaney.cavm@gmail.com> - 2016-12-14 22:20 +0100
        Re: [PATCH linux v1 0/4] Seven segment display support Greg KH <gregkh@linuxfoundation.org> - 2016-12-14 18:00 +0100

csiph-web