Hi,
2015-08-24 10:51 GMT+02:00 Charles Keepax ckeepax@opensource.wolfsonmicro.com:
What is with the spi_irq = -1 ?? is this a problem here?
The IRQ needs pulled from the ACPI as well, having no IRQ specified will not interfer with basic functionality of the driver but will become a problem eventually.
Thanks, Charles
ok, I will have an eye on it these days for the IRQ. For me its looking that the pxa2xx does not have an irq yet, nor an dma channel (thats should be fine, as I understand it correctly)
actually I cleanup and want to rework / begin with a new v2 patch series, but first it should drive this chip before beginning with a v2 patch.
one more question: the function devm_gpio_request_one() which is used here to request "arizona /RESET"
<code in arizona-core.c>
/* Start out with /RESET low to put the chip into reset */ ret = devm_gpio_request_one(arizona->dev, arizona->pdata.reset, GPIOF_DIR_OUT | GPIOF_INIT_LOW, "arizona /RESET"); </code>
the string arizona /RESET, where is this documented, where can I find it? It fails here, and my question is, is this string correct at all for my case? Should it be called only "reset" in my case??
All I have found is: Documentation/devicetree/bindings/mfd/arizona.txt: - wlf,reset : GPIO specifier for the GPIO controlling /RESET
in devres.c its documented as a label only: @label: a literal description string of this GPIO
I am curios about the answers, anyway I will change it, it looks faulty (but do not know why exactly at all :) )
cheers chris