Re: [PATCH] [media] rc: ir-spi: add support for IR LEDs connected with SPI

From	Andi Shyti <andi@etezian.org>
Newsgroups	linux.kernel
Subject	Re: [PATCH] [media] rc: ir-spi: add support for IR LEDs connected with SPI
Date	2016-07-05 17:10 +0200
Message-ID	<rRA93-4vb-5@gated-at.bofh.it> (permalink)
References	<rQ29s-4m1-43@gated-at.bofh.it> <rRzwm-3YJ-31@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

Hi Rob,

> > The ir-spi is a simple device driver which supports the
> > connection between an IR LED and the MOSI line of an SPI device.
> 
> Please split the binding from the driver.

OK!

> > +Device tree bindings for IR LED connected through SPI bus which is used as
> > +remote controller.
> 
> Do said devices have part numbers? Seems kind of generic and I've never 
> seen such device.

No, it doesn't, this is a simple irled driven by the SPI MOSI
line that works as a remote controller.

It looked strange to me as well when I heard of it the first
time, but it will be present in the upcoming tm2/tm2e patchset 
(i.e. Samsung Note 4 and Note 4 edge).

> > +Optional properties:
> > +	- irled,switch: specifies the gpio switch which enables the irled
> 
> Just "switch-gpios"

OK!

> > +                controller-data {
> 
> This is part of the controller binding? Omit that from the example.

OK!

Thanks,
Andi

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

Thread

[PATCH] [media] rc: ir-spi: add support for IR LEDs connected with SPI Andi Shyti <andi.shyti@samsung.com> - 2016-07-01 10:40 +0200
  Re: [PATCH] [media] rc: ir-spi: add support for IR LEDs connected  with SPI Sean Young <sean@mess.org> - 2016-07-01 11:50 +0200
    Re: [PATCH] [media] rc: ir-spi: add support for IR LEDs connected with  SPI Andi Shyti <andi.shyti@samsung.com> - 2016-07-01 14:40 +0200
      Re: [PATCH] [media] rc: ir-spi: add support for IR LEDs connected  with SPI Sean Young <sean@mess.org> - 2016-07-01 15:30 +0200
        Re: [PATCH] [media] rc: ir-spi: add support for IR LEDs connected  with SPI Andi Shyti <andi@etezian.org> - 2016-07-01 16:10 +0200
  Re: [PATCH] [media] rc: ir-spi: add support for IR LEDs connected  with SPI kbuild test robot <lkp@intel.com> - 2016-07-02 07:00 +0200
  Re: [PATCH] [media] rc: ir-spi: add support for IR LEDs connected  with SPI Rob Herring <robh@kernel.org> - 2016-07-05 16:30 +0200
    Re: [PATCH] [media] rc: ir-spi: add support for IR LEDs connected  with SPI Andi Shyti <andi@etezian.org> - 2016-07-05 17:10 +0200

csiph-web