* [PATCH 5/5] pcmcia: at91_cf: add support for DT
[not found] ` <1360863766-22511-6-git-send-email-manabian@gmail.com>
@ 2013-03-08 9:18 ` Nicolas Ferre
0 siblings, 0 replies; 2+ messages in thread
From: Nicolas Ferre @ 2013-03-08 9:18 UTC (permalink / raw)
To: linux-arm-kernel
On 02/14/2013 06:42 PM, Joachim Eastwood :
> Signed-off-by: Joachim Eastwood <manabian@gmail.com>
Seems good.
Acked-by: Nicolas Ferre <nicolas.ferre@atmel.com>
> ---
> .../devicetree/bindings/ata/atmel-at91_cf.txt | 19 +++++++++
> drivers/pcmcia/Kconfig | 2 +-
> drivers/pcmcia/at91_cf.c | 45 +++++++++++++++++++++-
> 3 files changed, 64 insertions(+), 2 deletions(-)
> create mode 100644 Documentation/devicetree/bindings/ata/atmel-at91_cf.txt
>
> diff --git a/Documentation/devicetree/bindings/ata/atmel-at91_cf.txt b/Documentation/devicetree/bindings/ata/atmel-at91_cf.txt
> new file mode 100644
> index 0000000..c1d22b3
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/ata/atmel-at91_cf.txt
> @@ -0,0 +1,19 @@
> +Atmel AT91RM9200 CompactFlash
> +
> +Required properties:
> +- compatible : "atmel,at91rm9200-cf".
> +- reg : should specify localbus address and size used.
> +- gpios : specifies the gpio pins to control the CF device. Detect
> + and reset gpio's are mandatory while irq and vcc gpio's are
> + optional and may be set to 0 if not present.
> +
> +Example:
> +compact-flash at 50000000 {
> + compatible = "atmel,at91rm9200-cf";
> + reg = <0x50000000 0x30000000>;
> + gpios = <&pioC 13 0 /* irq */
> + &pioC 15 0 /* detect */
> + 0 /* vcc */
> + &pioC 5 0 /* reset */
> + >;
> +};
> diff --git a/drivers/pcmcia/Kconfig b/drivers/pcmcia/Kconfig
> index 8fd255f..2adad5b 100644
> --- a/drivers/pcmcia/Kconfig
> +++ b/drivers/pcmcia/Kconfig
> @@ -288,7 +288,7 @@ config BFIN_CFPCMCIA
>
> config AT91_CF
> tristate "AT91 CompactFlash Controller"
> - depends on PCMCIA && ARCH_AT91RM9200
> + depends on PCMCIA && ARCH_AT91
> help
> Say Y here to support the CompactFlash controller on AT91 chips.
> Or choose M to compile the driver as a module named "at91_cf".
> diff --git a/drivers/pcmcia/at91_cf.c b/drivers/pcmcia/at91_cf.c
> index bce8a64..149b95c 100644
> --- a/drivers/pcmcia/at91_cf.c
> +++ b/drivers/pcmcia/at91_cf.c
> @@ -20,6 +20,9 @@
> #include <linux/platform_data/atmel.h>
> #include <linux/io.h>
> #include <linux/sizes.h>
> +#include <linux/of.h>
> +#include <linux/of_device.h>
> +#include <linux/of_gpio.h>
>
> #include <pcmcia/ss.h>
>
> @@ -211,6 +214,37 @@ static struct pccard_operations at91_cf_ops = {
>
> /*--------------------------------------------------------------------------*/
>
> +#if defined(CONFIG_OF)
> +static const struct of_device_id at91_cf_dt_ids[] = {
> + { .compatible = "atmel,at91rm9200-cf" },
> + { /* sentinel */ }
> +};
> +MODULE_DEVICE_TABLE(of, at91_cf_dt_ids);
> +
> +static int at91_cf_dt_init(struct platform_device *pdev)
> +{
> + struct at91_cf_data *board;
> +
> + board = devm_kzalloc(&pdev->dev, sizeof(*board), GFP_KERNEL);
> + if (!board)
> + return -ENOMEM;
> +
> + board->irq_pin = of_get_gpio(pdev->dev.of_node, 0);
> + board->det_pin = of_get_gpio(pdev->dev.of_node, 1);
> + board->vcc_pin = of_get_gpio(pdev->dev.of_node, 2);
> + board->rst_pin = of_get_gpio(pdev->dev.of_node, 3);
> +
> + pdev->dev.platform_data = board;
> +
> + return 0;
> +}
> +#else
> +static int at91_cf_dt_init(struct platform_device *pdev)
> +{
> + return -ENODEV;
> +}
> +#endif
> +
> static int __init at91_cf_probe(struct platform_device *pdev)
> {
> struct at91_cf_socket *cf;
> @@ -218,7 +252,15 @@ static int __init at91_cf_probe(struct platform_device *pdev)
> struct resource *io;
> int status;
>
> - if (!board || !gpio_is_valid(board->det_pin) || !gpio_is_valid(board->rst_pin))
> + if (!board) {
> + status = at91_cf_dt_init(pdev);
> + if (status)
> + return status;
> +
> + board = pdev->dev.platform_data;
> + }
> +
> + if (!gpio_is_valid(board->det_pin) || !gpio_is_valid(board->rst_pin))
> return -ENODEV;
>
> io = platform_get_resource(pdev, IORESOURCE_MEM, 0);
> @@ -360,6 +402,7 @@ static struct platform_driver at91_cf_driver = {
> .driver = {
> .name = "at91_cf",
> .owner = THIS_MODULE,
> + .of_match_table = of_match_ptr(at91_cf_dt_ids),
> },
> .remove = __exit_p(at91_cf_remove),
> .suspend = at91_cf_suspend,
>
--
Nicolas Ferre
^ permalink raw reply [flat|nested] 2+ messages in thread
* [PATCH 0/5] pcmcia: at91_cf: clean up and add DT support
[not found] <1360863766-22511-1-git-send-email-manabian@gmail.com>
[not found] ` <1360863766-22511-6-git-send-email-manabian@gmail.com>
@ 2013-03-08 9:21 ` Nicolas Ferre
1 sibling, 0 replies; 2+ messages in thread
From: Nicolas Ferre @ 2013-03-08 9:21 UTC (permalink / raw)
To: linux-arm-kernel
On 02/14/2013 06:42 PM, Joachim Eastwood :
> Hi
>
> This series clean up at91_cf a bit and add DT bindings.
>
> Patch series tested on custom RM9200 board in both with DT and
> with old platform data.
>
>
> Joachim Eastwood (5):
> pcmcia: at91_cf: fix gpio_get_value in at91_cf_get_status
> pcmcia: at91_cf: convert to dev_ print functions
> pcmcia: at91_cf: use devm_ functions for allocations
> pcmcia: at91_cf: clean up header includes
> pcmcia: at91_cf: add support for DT
>
> .../devicetree/bindings/ata/atmel-at91_cf.txt | 19 +++
> drivers/pcmcia/Kconfig | 2 +-
> drivers/pcmcia/at91_cf.c | 148 +++++++++++----------
> 3 files changed, 99 insertions(+), 70 deletions(-)
> create mode 100644 Documentation/devicetree/bindings/ata/atmel-at91_cf.txt
Joachim,
Thanks a lot for these patches. You can add on the whole series:
Acked-by: Nicolas Ferre <nicolas.ferre@atmel.com>
I hope that it can be taken by linux-pcmcia.
Best regards,
PS: make sure to add linux-arm-kernel mailing-list as well: it always
interests people there to have updates on AT91 peripherals...
--
Nicolas Ferre
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2013-03-08 9:21 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <1360863766-22511-1-git-send-email-manabian@gmail.com>
[not found] ` <1360863766-22511-6-git-send-email-manabian@gmail.com>
2013-03-08 9:18 ` [PATCH 5/5] pcmcia: at91_cf: add support for DT Nicolas Ferre
2013-03-08 9:21 ` [PATCH 0/5] pcmcia: at91_cf: clean up and add DT support Nicolas Ferre
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).