Re: [PATCH V3 1/3] dts: change Marvell prefix to 'marvell'

2013-07-12 Thread Haojian Zhuang
On Fri, Jul 12, 2013 at 11:10 PM, Daniel Drake wrote: > On Thu, Jul 11, 2013 at 5:54 PM, Haojian Zhuang > wrote: >>> Well, Daniel Drake spoke up for OLPC. Does that count? >> >> We don't know they used DT on Marvell MMP2/MMP3. So they don't have DTS file >> in kernel, we could use both old name

Re: Fixed PHY Device Tree usage?

2013-07-12 Thread Florian Fainelli
Le 13/07/2013 00:44, Grant Likely a écrit : On Wed, Jul 10, 2013 at 6:23 PM, Florian Fainelli wrote: Hello Thomas, 2013/7/10 Thomas Petazzoni : Dear Florian Fainelli, On Wed, 10 Jul 2013 17:29:44 +0100, Florian Fainelli wrote: [snip] }; phy1: ethernet-

Re: Fixed PHY Device Tree usage?

2013-07-12 Thread Grant Likely
On Wed, Jul 10, 2013 at 6:23 PM, Florian Fainelli wrote: > Hello Thomas, > > 2013/7/10 Thomas Petazzoni : >> Dear Florian Fainelli, >> >> On Wed, 10 Jul 2013 17:29:44 +0100, Florian Fainelli wrote: > [snip] > >> >>> > }; >>> > >>> > phy1: ethernet-phy@1 { >>> >

Re: Sharing *.dtsi between Linux architectures?

2013-07-12 Thread Russell King - ARM Linux
On Fri, Jul 12, 2013 at 01:58:35PM -0600, Stephen Warren wrote: > Is there a (possibly just proposed) mechanism in place to allow *.dts > from multiple Linux architectures to share common *.dtsi files? Don't forget that the long term goal is to move these files out of the kernel source, which mean

Re: [PATCH] gps6507x.txt: Remove executable bits

2013-07-12 Thread Rob Herring
On 07/12/2013 12:31 PM, Joe Perches wrote: > Documentation shouldn't be executable. > > Signed-off-by: Joe Perches Acked-by: Rob Herring > --- > diff --git a/Documentation/devicetree/bindings/mfd/tps6507x.txt > b/Documentation/devicetree/bindings/mfd/tps6507x.txt > old mode 100755 > new mode

Re: Sharing *.dtsi between Linux architectures?

2013-07-12 Thread Jean-Christophe PLAGNIOL-VILLARD
On Jul 13, 2013, at 4:17 AM, Rob Herring wrote: > On 07/12/2013 02:58 PM, Stephen Warren wrote: >> Is there a (possibly just proposed) mechanism in place to allow *.dts >> from multiple Linux architectures to share common *.dtsi files? > > Nothing proposed yet. There was some discussion at Conn

Re: Sharing *.dtsi between Linux architectures?

2013-07-12 Thread Rob Herring
On 07/12/2013 02:58 PM, Stephen Warren wrote: > Is there a (possibly just proposed) mechanism in place to allow *.dts > from multiple Linux architectures to share common *.dtsi files? Nothing proposed yet. There was some discussion at Connect (which I missed part of). We're certainly going to star

Sharing *.dtsi between Linux architectures?

2013-07-12 Thread Stephen Warren
Is there a (possibly just proposed) mechanism in place to allow *.dts from multiple Linux architectures to share common *.dtsi files? As an example, consider two SoCs that are identical except for the CPU complex. One uses an ARMv7 CPU (DTs in arch/arm/boot/dts/) and the other uses some ARMv8 CPU

Re: [PATCH v3 2/2] iio: twl6030-gpadc: TWL6030, TWL6032 GPADC driver

2013-07-12 Thread Lars-Peter Clausen
A couple of comments inline. On 07/12/2013 09:18 AM, Oleksandr Kozaruk wrote: diff --git a/drivers/iio/adc/Kconfig b/drivers/iio/adc/Kconfig index ab0767e6..87d699e 100644 --- a/drivers/iio/adc/Kconfig +++ b/drivers/iio/adc/Kconfig @@ -157,4 +157,12 @@ config VIPERBOARD_ADC Say yes he

Re: [PATCH v3 2/2] iio: twl6030-gpadc: TWL6030, TWL6032 GPADC driver

2013-07-12 Thread Jonathan Cameron
On 07/12/2013 08:18 AM, Oleksandr Kozaruk wrote: > The GPADC is general purpose ADC found on TWL6030, > and TWL6032 PMIC, known also as Phoenix and PhoenixLite. > > The TWL6030 and TWL6032 have GPADC with 17 and 19 > channels respectively. Some channels have current > source and are used for measur

Re: DT binding review for Armada display subsystem

2013-07-12 Thread Daniel Drake
On Fri, Jul 12, 2013 at 12:39 PM, Jean-Francois Moine wrote: > - I think it is better to keep the names 'lcd' for the memory to dumb panel > sub-devices and 'dcon' for the dumb panel to LCD/VGA sub-device, as > named in the spec. I agree it is worth keeping the spec-defined names, if they do

Re: [PATCH 2/2] pwm: Add PWM polarity flag macros for DT

2013-07-12 Thread Stephen Warren
On 07/12/2013 11:24 AM, Thierry Reding wrote: > On Fri, Jul 12, 2013 at 08:40:07AM -0600, Stephen Warren wrote: >> On 07/12/2013 04:41 AM, Laurent Pinchart wrote: >>> Hi Stephen, > [...] >>> What about splitting it in three patches that >>> >>> - add the include/dt-bindings/pwm/pwm.h header, and u

[PATCH] gps6507x.txt: Remove executable bits

2013-07-12 Thread Joe Perches
Documentation shouldn't be executable. Signed-off-by: Joe Perches --- diff --git a/Documentation/devicetree/bindings/mfd/tps6507x.txt b/Documentation/devicetree/bindings/mfd/tps6507x.txt old mode 100755 new mode 100644 ___ devicetree-discuss mailing

Re: [PATCH V2] pci: exynos: split into two parts such as Synopsys part and Exynos part

2013-07-12 Thread Thierry Reding
On Fri, Jul 12, 2013 at 03:31:23PM +0530, Kishon Vijay Abraham I wrote: > Hi, > > On Thursday 11 July 2013 11:19 AM, Jingoo Han wrote: [...] > > /* set the number of lines as 4 */ > > - readl_rc(pp, dbi_base + PCIE_PORT_LINK_CONTROL, &val); > > + dw_pcie_readl_rc(pp, dbi_base + PCIE_PORT_L

Re: [PATCH 2/2] pwm: Add PWM polarity flag macros for DT

2013-07-12 Thread Thierry Reding
On Fri, Jul 12, 2013 at 08:40:07AM -0600, Stephen Warren wrote: > On 07/12/2013 04:41 AM, Laurent Pinchart wrote: > > Hi Stephen, [...] > > What about splitting it in three patches that > > > > - add the include/dt-bindings/pwm/pwm.h header, and update > > include/linux/pwm.h > > and Documentati

Re: [PATCH] iio: add APDS9300 ambilent light sensor driver

2013-07-12 Thread Lars-Peter Clausen
On 07/10/2013 03:08 PM, Oleksandr Kravchenko wrote: > From: Oleksandr Kravchenko > > This patch adds IIO driver for APDS9300 ambilent light sensor (ALS). s/ambilent/ambient/ > http://www.avagotech.com/docs/AV02-1077EN > > The driver allows to read raw data from ADC registers or calculate > lux

Re: [PATCH RFC 0/8] MPC512x DMA slave s/g support, OF DMA lookup

2013-07-12 Thread Lars-Peter Clausen
On 07/12/2013 05:26 PM, Gerhard Sittig wrote: [...] > Lars, there is a checkpatch warning about a line longer than 80 > characters in the common xlate part -- I didn't dare to change your > submission, and the part included here is verbatim from your patchwork > 2331091 (original submission) and 25

DT binding review for Armada display subsystem

2013-07-12 Thread Daniel Drake
Hi, Based on the outcomes of the "Best practice device tree design for display subsystems" discussion I have drafted a DT binding. Comments much appreciated. At a high level, it uses a "super node" as something for the driver to bind to, needed because there is no clear one device that controls a

Re: [PATCH V3 1/3] dts: change Marvell prefix to 'marvell'

2013-07-12 Thread Jason Cooper
On Fri, Jul 12, 2013 at 10:05:45AM -0600, Daniel Drake wrote: > On Fri, Jul 12, 2013 at 9:57 AM, Jason Cooper wrote: > > This also means we should do a patch for stable v3.5+ appending the > > "mrvl,..." string to the drivers that had it removed improperly, as > > Daniel discovered. Daniel, since

Re: [PATCH V3 1/3] dts: change Marvell prefix to 'marvell'

2013-07-12 Thread Daniel Drake
On Fri, Jul 12, 2013 at 9:57 AM, Jason Cooper wrote: > This also means we should do a patch for stable v3.5+ appending the > "mrvl,..." string to the drivers that had it removed improperly, as > Daniel discovered. Daniel, since you are probably most familiar (and > most able to test ;-) ), would

Re: [PATCH V3 1/3] dts: change Marvell prefix to 'marvell'

2013-07-12 Thread Jason Cooper
On Fri, Jul 12, 2013 at 09:10:49AM -0600, Daniel Drake wrote: > On Thu, Jul 11, 2013 at 5:54 PM, Haojian Zhuang > wrote: > >> Well, Daniel Drake spoke up for OLPC. Does that count? > > > > We don't know they used DT on Marvell MMP2/MMP3. So they don't have DTS file > > in kernel, we could use bot

[PATCH RFC 8/8] HACK mmc: mxcmmc: enable clocks for the MPC512x

2013-07-12 Thread Gerhard Sittig
Q&D HACK to enable SD card support without correct COMMON_CLK support, best viewed with 'git diff -w -b', NOT acceptable for mainline (NAKed) Signed-off-by: Gerhard Sittig --- drivers/mmc/host/mxcmmc.c | 41 +++-- 1 file changed, 27 insertions(+), 14 deletio

[PATCH RFC 7/8] dma: mpc512x: register for device tree channel lookup

2013-07-12 Thread Gerhard Sittig
register the controller for device tree based lookup of DMA channels (non-fatal for backwards compatibility with older device trees), provide the '#dma-cells' property in the shared mpc5121.dtsi file, and introduce a bindings document for the MPC512x DMA controller Signed-off-by: Gerhard Sittig -

[PATCH RFC 6/8] dma: of: Add common xlate function for matching by channel id

2013-07-12 Thread Gerhard Sittig
From: Lars-Peter Clausen From: Lars-Peter Clausen This patch adds a new common OF dma xlate callback function which will match a channel by it's id. The binding expects one integer argument which it will use to lookup the channel by the id. Unlike of_dma_simple_xlate this function is able to

[PATCH RFC 5/8] dma: mpc512x: use symbolic specifiers for DMA channels

2013-07-12 Thread Gerhard Sittig
for the DMA controller of the MPC512x SoC, DMA channels directly correspond to specific peripherals, since requester lines directly map to channels while no additional flexibility or mapping is involved introduce a dt-bindings header file for MPC512x DMA channels, and make the shared DT specs in t

[PATCH RFC 4/8] dts: mpc512x: prepare for preprocessor support

2013-07-12 Thread Gerhard Sittig
prepare C preprocessor support when processing MPC512x DTS files - switch from DTS syntax to CPP syntax for include specs - create a symlink such that DTS processing can reference includes Signed-off-by: Gerhard Sittig --- arch/powerpc/boot/dts/ac14xx.dts |2 +- arch/powerpc/boot/dt

[PATCH RFC 3/8] dma: mpc512x: support 'terminate all' control request

2013-07-12 Thread Gerhard Sittig
implement the TERMINATE_ALL request in the device_control() callback of the DMA engine driver for the MPC512x DMA controller reword variable initialization to better follow the code path and to avoid artificial diffs later on (this style change vanishes when this patch gets squashed with the devic

[PATCH RFC 2/8] dma: mpc512x: fix start condition in execute()

2013-07-12 Thread Gerhard Sittig
adjust the conditions how submitted DMA jobs get started: memory transfers need to get initiated by an explicit software request, all transfers which involve peripherals need to reference the external requester line Signed-off-by: Gerhard Sittig --- drivers/dma/mpc512x_dma.c |4 +++- 1 file

[PATCH RFC 1/8] powerpc: mpc512x_dma: add support for data transfers between memory and i/o memory

2013-07-12 Thread Gerhard Sittig
From: Alexander Popov From: Alexander Popov Data transfers between memory and i/o memory require more delicate TCD (Transfer Control Descriptor) configuration and DMA channel service requests via hardware. dma_device.device_control callback function is needed to configure DMA channel to work w

[PATCH RFC 0/8] MPC512x DMA slave s/g support, OF DMA lookup

2013-07-12 Thread Gerhard Sittig
blurb is below the stats Alexander Popov (1): powerpc: mpc512x_dma: add support for data transfers between memory and i/o memory Gerhard Sittig (6): dma: mpc512x: fix start condition in execute() dma: mpc512x: support 'terminate all' control request dts: mpc512x: prepare for preproces

Re: [PATCH V3 1/3] dts: change Marvell prefix to 'marvell'

2013-07-12 Thread Daniel Drake
On Thu, Jul 11, 2013 at 5:54 PM, Haojian Zhuang wrote: >> Well, Daniel Drake spoke up for OLPC. Does that count? > > We don't know they used DT on Marvell MMP2/MMP3. So they don't have DTS file > in kernel, we could use both old name & new name in driver. You are listed as one of the MMP maintai

Re: [RFC PATCH v4 03/18] Documentation: devicetree: arm: cpus/cpu nodes bindings updates

2013-07-12 Thread Rob Herring
On Fri, May 17, 2013 at 10:20 AM, Lorenzo Pieralisi wrote: > In order to extend the current cpu nodes bindings to newer CPUs > inclusive of AArch64 and to update support for older ARM CPUs this > patch updates device tree documentation for the cpu nodes bindings. Sorry for the long delay on this,

Re: [PATCH 2/2] pwm: Add PWM polarity flag macros for DT

2013-07-12 Thread Stephen Warren
On 07/12/2013 05:01 AM, Laurent Pinchart wrote: > Hi, > > On Thursday 11 July 2013 14:06:44 Stephen Warren wrote: >> On 07/11/2013 01:32 PM, Thierry Reding wrote: >>> On Thu, Jul 11, 2013 at 11:50:48AM -0600, Stephen Warren wrote: On 07/11/2013 09:36 AM, Thierry Reding wrote: > On Thu, Ju

Re: [PATCH 2/2] pwm: Add PWM polarity flag macros for DT

2013-07-12 Thread Stephen Warren
On 07/12/2013 04:41 AM, Laurent Pinchart wrote: > Hi Stephen, > > On Thursday 11 July 2013 11:40:37 Stephen Warren wrote: >> On 07/11/2013 08:37 AM, Laurent Pinchart wrote: >>> Define PWM_POLARITY_NORMAL and PWM_POLARITY_INVERTED macros in >>> include/dt-bindings/pwm/pwm.h to be used by device tre

Re: [PATCH v2] regulator: pfuze100: add pfuze100 regulator driver

2013-07-12 Thread Mark Brown
On Fri, Jul 12, 2013 at 12:54:15PM +0800, Robin Gong wrote: > Add pfuze100 regulator driver. This looks mostly good. A few small issues below but nothing major. > +enum pfuze_id { > + PFUZE_ID_PFUZE100, > + PFUZE_ID_INVALID, > +}; > +struct pfuze_chip { Missing blank line here - there a

Re: Reference voltages for ADC channels

2013-07-12 Thread Hector Palacios
Hello, On 07/08/2013 08:13 AM, Hector Palacios wrote: Greetings, The other day at linux-iio-u79uwxl29ty76z2rm5m...@public.gmane.org we discussed the possibility of having ADC channels voltage references moved to the DT [1]. In Freescale's i.MX28 and i.MX23, at least, the CPU has 16 ADC channe

Re: Fixed PHY Device Tree usage?

2013-07-12 Thread Thomas Petazzoni
Dear Florian Fainelli, On Fri, 12 Jul 2013 13:05:59 +0100, Florian Fainelli wrote: > I am talking about scanning the MDIO bus DT nodes, not the entire DT. > That job is already done by of_mdiobus_probe() to register PHY > devices, so having a central point where the knowledge of how to treat > PH

Re: Fixed PHY Device Tree usage?

2013-07-12 Thread Florian Fainelli
Hello Thomas, 2013/7/12 Thomas Petazzoni : >> Why not? Since we are already have to scan the entire MDIO bus we are >> attached to, when we encounter such a PHY node with the special >> "fixed" properties, we just call fixed_phy_add() with the right >> parameters and voila. Which is also the reas

Re: Fixed PHY Device Tree usage?

2013-07-12 Thread Thomas Petazzoni
Dear Florian Fainelli, On Wed, 10 Jul 2013 18:23:30 +0100, Florian Fainelli wrote: > >> - declare all PHY nodes in the system as sub nodes of their belonging > >> real hardware MDIO bus node > >> - flag specific PHY nodes as "fixed" with a "fixed-link" boolean for > >> instance > >> - if we see

Re: [PATCH v5 2/7] sound: sam9x5_wm8731: machine driver for at91sam9x5 wm8731 boards

2013-07-12 Thread Mark Brown
On Thu, Jul 11, 2013 at 06:15:54PM +0200, Richard Genoud wrote: > From: Nicolas Ferre > > Description of the Asoc machine driver for an at91sam9x5 based board ASoC. > +sam9x5 pins: > + * LOUT > + * ROUT > + * LHPOUT > + * RHPOUT > + * LLINEIN > + * RLINEIN > + * MICIN These aren't pins on the

Re: [PATCH v5 1/7] sound: codec: wm8731: add rates constraints

2013-07-12 Thread Mark Brown
On Thu, Jul 11, 2013 at 06:15:53PM +0200, Richard Genoud wrote: Please always try to use commit logs that look like normal commit logs for the subsystem. > switch (freq) { > - case 11289600: > case 1200: > + wm8731->constraints = &wm8731_constraints_1200; > +

Re: [PATCH 2/2] pwm: Add PWM polarity flag macros for DT

2013-07-12 Thread Laurent Pinchart
Hi, On Thursday 11 July 2013 14:06:44 Stephen Warren wrote: > On 07/11/2013 01:32 PM, Thierry Reding wrote: > > On Thu, Jul 11, 2013 at 11:50:48AM -0600, Stephen Warren wrote: > >> On 07/11/2013 09:36 AM, Thierry Reding wrote: > >>> On Thu, Jul 11, 2013 at 04:37:48PM +0200, Laurent Pinchart > >>>

Re: [PATCH 2/2] pwm: Add PWM polarity flag macros for DT

2013-07-12 Thread Laurent Pinchart
Hi Thierry, On Thursday 11 July 2013 08:36:00 Thierry Reding wrote: > On Thu, Jul 11, 2013 at 04:37:48PM +0200, Laurent Pinchart wrote: > [...] > > > diff --git a/Documentation/devicetree/bindings/pwm/atmel-tcb-pwm.txt > > b/Documentation/devicetree/bindings/pwm/atmel-tcb-pwm.txt index > > de0eae

Re: [PATCH 2/2] pwm: Add PWM polarity flag macros for DT

2013-07-12 Thread Laurent Pinchart
Hi Stephen, On Thursday 11 July 2013 11:40:37 Stephen Warren wrote: > On 07/11/2013 08:37 AM, Laurent Pinchart wrote: > > Define PWM_POLARITY_NORMAL and PWM_POLARITY_INVERTED macros in > > include/dt-bindings/pwm/pwm.h to be used by device tree sources. > > > > Documentation/devicetree/bindings/

Re: [PATCH] usb: phy: samsung-usb2: Toggle HSIC GPIO from device tree

2013-07-12 Thread Tomasz Figa
Hi, On Friday 12 of July 2013 09:48:54 Jingoo Han wrote: > On Friday, July 12, 2013 6:46 AM, Julius Werner wrote: > > Hi Jingoo, > > > > Yeah, I followed that discussion back then, but it seems to have > > stalled a little (at least the HSIC patches haven't been picked up in > > any kernel.org re

[PATCH v3] Input: sysrq - DT binding for key sequences

2013-07-12 Thread mathieu . poirier
From: "Mathieu J. Poirier" Adding a simple device tree binding for the specification of key sequences. Definition of the keys found in the sequence are located in 'include/uapi/linux/input.h'. For the sysrq driver, holding the sequence of keys down for a specific amount of time will reset the s

Re: [PATCH 0/2] ARM: sunxi: Convert DTSI to new CPU bindings

2013-07-12 Thread Maxime Ripard
Hi Lorenzo, On Fri, Jul 05, 2013 at 11:19:46AM +0100, Lorenzo Pieralisi wrote: > On Sun, Jun 30, 2013 at 10:48:46AM +0100, Lorenzo Pieralisi wrote: > > On Sat, Jun 29, 2013 at 08:38:19PM +0100, Russell King - ARM Linux wrote: > > > On Fri, Jun 28, 2013 at 01:05:42PM -0700, Olof Johansson wrote: >

Re: [PATCH 1/2] ARM i.MX53: mba53: Fix PWM backlight DT node

2013-07-12 Thread Shawn Guo
On Thu, Jul 11, 2013 at 04:37:47PM +0200, Laurent Pinchart wrote: > The i.MX53 PWM controller uses two cells to describe the PWM specifier. > Remove the extra unused values from the backlight DT node pwms property. > > Signed-off-by: Laurent Pinchart Applied, thanks. ___