From: Rob Herring <robherring2-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
To: Leela Krishna Amudala
<leelakrishna.a-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
Cc: devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org,
thomas.ab-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org
Subject: Re: <Query> Reg: Adding a gpio initialization sequence in dts is valid or not
Date: Wed, 08 Aug 2012 08:30:20 -0500 [thread overview]
Message-ID: <502269EC.2070008@gmail.com> (raw)
In-Reply-To: <CAL1wa8fjJg33vVM4ZOFuA2ACps55cahoK65MDf4ebAi9B6KUxg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
On 08/08/2012 04:52 AM, Leela Krishna Amudala wrote:
> Hello All,
>
> I came across that to set power to an LCD, few panels require a pull
> up/pull down operation on single GPIO line
> and few panels need some gpio line setting sequence with some delay in between.
>
> For my development board, user manual says that I have to do a
> sequence of gpio line settings with some delay in between
> to set power to LCD and currently I'm doing it as
>
> pull_up(gpio_line_5)
> some_delay(x)
> pull_down(gpio_line_5)
> some_delay(x)
> pull_up(gpio_line_5)
>
> I want to read this GPIO line from device tree and parse it and do the
> above sequence in the driver.
> I'm thinking to follow the below approach to get this done.
>
See the thread "runtime interpreted power sequences"
> Putting the sequence in dts file like below for an LCD which requires
> gpio sequence
> lcd-reset-gpio = <&gpx1 5 1 3 0>, (3 - for pull up)
> <&gpx1 5 1 1 0>, (1 - for pull down)
> <&gpx1 5 1 3 0>; (3 - for pull up)
> delay = x;
>
> and only one gpio entry in the case of LCD which doesn't require gpio
> sequence setting
> lcd-reset-gpio = <&gpx1 5 1 3 0>, (3 - pull up)
>
> The 4 arguments in the above handle denotes
>
> <[phandle of the gpio controller node]
> [pin number within the gpio controller]
> [mux function]
> [pull up/down]
> [drive strength]>
>
The gpio controller defines the number of gpio cells. So technically you
could do that if you're defining the gpio controller's binding, but I
don't think thats a good idea. If the binding is already defined, then
you can't really change it anyway.
Rob
> My board requires one gpio line to pull up->pull down->pull up to set
> power and similarly
> other boards may require two gpio lines to follow the sequence in that
> case number of entries
> in dts file will increase.
>
> I want to know whether the above approach is acceptable in device-tree or not.
>
> Best Wishes,
> Leela Krishna Amudala.
> _______________________________________________
> devicetree-discuss mailing list
> devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org
> https://lists.ozlabs.org/listinfo/devicetree-discuss
>
next prev parent reply other threads:[~2012-08-08 13:30 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-08-08 9:52 <Query> Reg: Adding a gpio initialization sequence in dts is valid or not Leela Krishna Amudala
[not found] ` <CAL1wa8fjJg33vVM4ZOFuA2ACps55cahoK65MDf4ebAi9B6KUxg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2012-08-08 13:30 ` Rob Herring [this message]
[not found] ` <502269EC.2070008-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2012-08-08 15:38 ` Stephen Warren
[not found] ` <50231B55.6070007@nvidia.com>
[not found] ` <50231B55.6070007-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2012-08-17 10:37 ` Leela Krishna Amudala
[not found] ` <CAL1wa8d-6qXOXS2+sVZAgnpGw0-x03WKpOhiVhSKZ3JoQZW1fw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2012-08-17 15:13 ` Stephen Warren
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=502269EC.2070008@gmail.com \
--to=robherring2-re5jqeeqqe8avxtiumwx3w@public.gmane.org \
--cc=devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org \
--cc=leelakrishna.a-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
--cc=thomas.ab-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).