On Tue, 5 Aug 2014, Varka Bhadram wrote:
On Tuesday 05 August 2014 07:58 PM, Brian Austin wrote:
On Mon, 4 Aug 2014, Varka Bhadram wrote:
- 0 = Left/right channels VMON[11:0], IMON[11:0], VPMON[7:0].
- 1 = Left/right channels VMON[11:0], IMON[11:0], STATUS.
- 2 = (Default) left/right channels VMON[15:0], IMON [15:0].
- 3 = Left/right channels VPMON[7:0], STATUS.
- cirrus,sdout-share : SDOUT sharing. Determines whether one or two
CS35L32
- devices are on board sharing SDOUT.
- 0 = (Default) One IC.
- 1 = Two IC's.
- cirrus,battery-recovery : Low battery nominal recovery threshold,
rising VP.
- 0 = 3.1V
- 1 = 3.2V
- 2 = 3.3V (Default)
- 3 = 3.4V
- cirrus,battery-threshold : Low battery nominal threshold, falling
VP.
- 0 = 3.1V
- 1 = 3.2V
- 2 = 3.3V
- 3 = 3.4V (Default)
- 4 = 3.5V
- 5 = 3.6V
Proper indentation for all the properties... These are not readable like this...
Required properties:
- compatible: "cirrus,cs35l32"
- reg: the I2C address of the device for I2C. Address
is determined by the level of the AD0 pin. Level 0 is 0x40 while Level 1 is 0x41.
- VA-supply,VP-supply: power supplies for the device, as covered in
Documentation/devicetree/bindings/regulator/regulator.txt.
Optional properties:
- reset-gpios: a GPIO spec for the reset pin. If specified,
it will be deasserted before communication to the codec starts.
- cirrus,boost-manager:Boost voltage control. 0 = Automatically managed. Boost-converter output voltage is the higher of the two: Class G or adaptive LED voltage. 1 = Automatically managed irrespective of audio,
adapting for low-power
-- Regards, Varka Bhadram.
Well when I look at your example that makes it more difficult for me to read ;)
sorry... this the problem due to my email client...
I think if there is a mandatory style we are looking for I would expect it to be documented. Maybe I have missed that? I take my bindings file from examples already in the mainline and find this to be acceptable and readable for me.
see this ex:http://lxr.free-electrons.com/source/Documentation/devicetree/bindings/sound...
It looks nice of course, but IMO doesn't really make it any more readable for me. Again, if there is a certain style required I would expect it to be documented or caught by checkpatch. And most of the bindings files in the tree look similar to my format.
Thanks, Brian