Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1494014 > unrolled thread

[PATCH 1/1] Add devices to spidev device tree compatibility list

Back to article view | Back to linux.kernel

Contents

  [PATCH 1/1] Add devices to spidev device tree compatibility list Fabien Lahoudere <fabien.lahoudere@collabora.co.uk> - 2016-09-30 12:20 +0200
    Re: [PATCH 1/1] Add devices to spidev device tree compatibility list Mark Brown <broonie@kernel.org> - 2016-09-30 18:10 +0200
      [PATCH v2 1/1] Add device to spidev device tree compatibility list Fabien Lahoudere <fabien.lahoudere@collabora.co.uk> - 2016-10-06 16:50 +0200
        Re: [PATCH v2 1/1] Add device to spidev device tree compatibility  list Mark Brown <broonie@kernel.org> - 2016-10-06 18:40 +0200
    Re: [PATCH 1/1] Add devices to spidev device tree compatibility list Geert Uytterhoeven <geert@linux-m68k.org> - 2016-09-30 19:40 +0200
      Re: [PATCH 1/1] Add devices to spidev device tree compatibility list Fabien Lahoudere <fabien.lahoudere@collabora.co.uk> - 2016-10-06 10:10 +0200

#1494014 — [PATCH 1/1] Add devices to spidev device tree compatibility list

Entries are needed in the spidev ID list to configure configure it from a
device tree. Add entries for the following devices:
- "fsl,mk20fn1m0vmd12" : a cortex m4 based chip
- "exar,xra1403" : a GPIO expander

Signed-off-by: Fabien Lahoudere <fabien.lahoudere@collabora.co.uk>
---
 drivers/spi/spidev.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/spi/spidev.c b/drivers/spi/spidev.c
index 2e05046..821937a 100644
--- a/drivers/spi/spidev.c
+++ b/drivers/spi/spidev.c
@@ -696,6 +696,8 @@ static struct class *spidev_class;
 static const struct of_device_id spidev_dt_ids[] = {
 	{ .compatible = "rohm,dh2228fv" },
 	{ .compatible = "lineartechnology,ltc2488" },
+	{ .compatible = "fsl,MK20FN1M0VMD12" },
+	{ .compatible = "exar,xra1403" },
 	{},
 };
 MODULE_DEVICE_TABLE(of, spidev_dt_ids);
-- 
2.1.4

[toc] | [next] | [standalone]

#1494180

[Multipart message — attachments visible in raw view] — view raw

On Fri, Sep 30, 2016 at 12:15:54PM +0200, Fabien Lahoudere wrote:

> - "fsl,mk20fn1m0vmd12" : a cortex m4 based chip

> +	{ .compatible = "fsl,MK20FN1M0VMD12" },

This compatible is in uppercase which is not the normal style for DT
bindings at all.  I'd also guess that this is a binding for a specific
application running on this device rather than the chip itself.

[toc] | [prev] | [next] | [standalone]

#1496686 — [PATCH v2 1/1] Add device to spidev device tree compatibility list

Entries are needed in the spidev ID list to configure configure it from a
device tree. Add entry for the following device:
- "ge,achc" :  GE Healthcare USB Management Controller

The USB Management Controller does not expose USB to the host, but acts as
an offload engine, communicating with specific USB based data acquisition
devices which are connected to it, extracting the required data and
providing it to the host via other methods. SPI is used as an out-of-band
configuration channel.

Signed-off-by: Fabien Lahoudere <fabien.lahoudere@collabora.co.uk>
---
 drivers/spi/spidev.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/spi/spidev.c b/drivers/spi/spidev.c
index 2e05046..9e2e099 100644
--- a/drivers/spi/spidev.c
+++ b/drivers/spi/spidev.c
@@ -696,6 +696,7 @@ static struct class *spidev_class;
 static const struct of_device_id spidev_dt_ids[] = {
 	{ .compatible = "rohm,dh2228fv" },
 	{ .compatible = "lineartechnology,ltc2488" },
+	{ .compatible = "ge,achc" },
 	{},
 };
 MODULE_DEVICE_TABLE(of, spidev_dt_ids);
-- 
2.1.4

[toc] | [prev] | [next] | [standalone]

#1496756 — Re: [PATCH v2 1/1] Add device to spidev device tree compatibility list

[Multipart message — attachments visible in raw view] — view raw

On Thu, Oct 06, 2016 at 04:44:27PM +0200, Fabien Lahoudere wrote:
> Entries are needed in the spidev ID list to configure configure it from a
> device tree. Add entry for the following device:
> - "ge,achc" :  GE Healthcare USB Management Controller

Please use subject lines matching the style for the subsystem.  This
makes it easier for people to identify relevant patches.

[toc] | [prev] | [next] | [standalone]

#1494215

On Fri, Sep 30, 2016 at 12:15 PM, Fabien Lahoudere
<fabien.lahoudere@collabora.co.uk> wrote:
> Entries are needed in the spidev ID list to configure configure it from a
> device tree. Add entries for the following devices:

> - "exar,xra1403" : a GPIO expander

Looks like a suitable target for a drivers/gpio/ driver.
Datasheet available at
https://www.exar.com/product/interface/bridges/bridging-solutions/spi-gpio-expanders/xra1403

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

[toc] | [prev] | [next] | [standalone]

#1496200

Hi,

On 30/09/16 19:35, Geert Uytterhoeven wrote:
> On Fri, Sep 30, 2016 at 12:15 PM, Fabien Lahoudere
> <fabien.lahoudere@collabora.co.uk> wrote:
>> Entries are needed in the spidev ID list to configure configure it from a
>> device tree. Add entries for the following devices:
>
>> - "exar,xra1403" : a GPIO expander
>
> Looks like a suitable target for a drivers/gpio/ driver.
> Datasheet available at
> https://www.exar.com/product/interface/bridges/bridging-solutions/spi-gpio-expanders/xra1403
>

Good idea, however we have to measure impact of this change on the project.

Thanks

Fabien

> Gr{oetje,eeting}s,
>
>                         Geert
>
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
>
> In personal conversations with technical people, I call myself a hacker. But
> when I'm talking to journalists I just say "programmer" or something like that.
>                                 -- Linus Torvalds
>

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web