Thread (43 messages) 43 messages, 6 authors, 2012-12-04
STALE4932d

[PATCH 2/5] ARM: PXA: Zipit Z2: Add USB host and device support

From: zonque@gmail.com (Daniel Mack)
Date: 2012-10-29 11:00:55

On 29.10.2012 11:52, Vasily Khoruzhick wrote:
On Mon, Oct 29, 2012 at 1:44 PM, Daniel Mack [off-list ref] wrote:
quoted
I haven't seen any patches for DT bindings of this yet.
quoted
I still don't understand how DT handles platform-specific
callbacks like set_power(). And what about pm_power_off hook?
They require consideration.

In general, a driver should be aware of everything that affects the
peripheral, including all the voltages and clocks it needs and also all
extra pins like for example reset GPIOs etc. And all that properties are
to be exported via DT as well (things that you would have normally set
up using pdata).

If drivers lack that kind of full awareness of their environment, they
need to be augmented, because - to stick with the example of reset GPIOs
- the driver of all parts is the only piece that knows when to driver
that pin and how.

About the hack to put the z2 to deep-sleep when it should actually power
down, I don't know what to do. But I would say that it's the userspace
that should be fixed here, if possible.
Well, there's an issue - Z2 does not preserve memory contents in deep sleep
(but it does in sleep), so userspace can't be fixed here unfortunatelly.
There's no another possibility to turn Z2 off, and plain sleep is too
power hungry.
So the only way to keep Z2 in low-power mode is fake power off, which just puts
Z2 in deep sleep.
Why can't the userspace trigger a deep sleep then instead of powering
off? Which details do I lack?
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help