Re: "Generic" MMIO ADC

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On Mon, Jul 3, 2017 at 2:37 PM, Mike Looijmans <mike.looijmans@xxxxxxxx> wrote:

> I was thinking along these lines:
>
> ... {
>    compatible = "iio-adc-generic-mmio";
>    reg = <0x43210000 4096>;
>    interrupt = <&gic 42 0>;
>    iio,start-sequence = <0x0 0x3> <0x4 0xff>;
>    iio,stop-sequence = <0x0 0x0>;

Nope. We are not going to invent scripting languages inside device tree.
No sequences please. This is the start of what we call a jam table.

We had this discussion a long time ago when starting to convert all the ARM
platforms to device tree: sequences should be expressed in code, not in
the device tree. Because next thing that happens is that you want to encode
a for() loop or if() or while().

The reason is that device tree is already part of open firmware, which already
has a scripting language: FCode (based on forth). This is then used to create
BIOS routines along the lines of what you're suggesting. So if you insist on
writing a BIOS, by all means write a BIOS, but not inside DT, use FCode
and figure out how to bridge that.

But I think it's easier to just write a normal driver really.

Compare to ACPI, which has AML, for the very same purpose, and DSDT
for describing the hardware.

Since ACPI is now starting to share parts of the DT bindings), things would
get very weird if we did things like this. Should they also implement a
compatible interpreter, rather than call their (very much in-use) AML
snippets?

Yours,
Linus Walleij
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux