devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
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
> 

  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).