Re: [PATCH V7 1/8] mfd: add device-tree binding doc for PMIC max77620/max20024

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





On Wednesday 10 February 2016 06:53 PM, Lee Jones wrote:
On Tue, 09 Feb 2016, Laxman Dewangan wrote:

On Tuesday 09 February 2016 09:12 PM, Lee Jones wrote:
On Sat, 30 Jan 2016, Laxman Dewangan wrote:

+	Normal mode also called as active mode on which all step-down
+		regulators, all linear regulators, GPIOs, and the 32kHz
+		oscillator are in normal active mode.
+	sleep mode: Regulators/GPIOs/clock can go on OFF state based on
"can go on OFF state"?
Regulator/GPIO has two states, enable and disable. If sleep mode is
configured for these resource and external signal triggers to sleep
then this get disabled.
It's the English that I'm unhappy with.

"can go on OFF state" doesn't sound right.

+			source gets the control signal for ON and OFF.
+	Power on slot: 	Slot number on which resource is ON once FPS source
+			get ON signal.
Can you find another way of explaining this please?
Hmm..
Does it look fine:
There is 8 slots for each FPS on which resource can get enabled.
This property provides the slot number on which resource gets
enabled after FPS sequence started.
It's a bit better, yes.  Although it's still a little difficult to
read.  In fact, I can't even recommend a suitable alternative, since I
still do not understand exactly what it is you're trying to say.

I am sorry if I am not able to make it clear. Better I post the datasheet content so that it is easy to understand and then we can have better text here.


The Flexible Power Sequencer (FPS) allows each regulator to power up under hardware or software control. Additionally, each regulator can power on independently or among a group of other regulators with an adjustable power-up and power-down delays (sequencing). GPIO1, GPIO2, and GPIO3 can be programmed to be part of a sequence allowing external regulators to be sequenced along with internal regulators. nRST_IO can be programmed to be part of a sequence.

The flexible sequencing structure consists of two hardware enable inputs (EN0, EN1), and 3 master sequencing timers. Each master sequencing timer is programmable through its configuration register to have a hardware enable source or a software enable source (CNFGFPSx). When enabled/disabled the master sequencing timer generates eight sequencing events. The time period between each event is programmable within the configuration register.

Each regulator, GPIO1, GPIO2, GPIO3, and nRST_IO has a flexible power sequence slave register (FPS_x) which allows its enable source to be specified as a flexible power sequencer timer or a software bit. When a FPSSRCx specifies the enable source to be a flexible power sequencer, the power up and power down delays are configured by FPSPUx[2:0] and FPSPDx[2:0].can be specified in that regulators flexible power sequencer configuration register.



+
+-maxim,enable-sleep: 		    Boolean, enable sleep state of PMIC
We already have bindings for sleeping.  Please use a generic one.
Which property? Saw sleep property with vendor prefix.
I guess there are too many '.*sleep.*' properties now, each with
slightly different syntax and meaning.  The situation is exacerbated
by one of the key examples is using the property with cells expected
i.e. non-bool.


My grep shows some sleep property and followig are near to which I am looking.

st,supports-sleepmode

pinctrl/ste,nomadik.txt:- ste,sleep: <0/1>
                                    0: sleep mode disable,
                                    1: sleep mode enable.


Should I made this as generic like

sleep: <0/1>
            0: sleep mode disable,
            1: sleep mode enable.





--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux