RE: [PATCH v4 3/3] can: c_can: Add runtime PM support to Bosch C_CAN/D_CAN controller

2012-08-02 Thread Hiremath, Vaibhav
On Thu, Aug 02, 2012 at 18:43:11, AnilKumar, Chimata wrote: > Add Runtime PM support to C_CAN/D_CAN controller. The runtime PM > APIs control clocks for C_CAN/D_CAN IP and prevent access to the > register of C_CAN/D_CAN IP when clock is turned off. > > Signed-off-by: AnilKumar Ch > --- > drivers

Re: [PATCH v2 1/2] arm/dts: AM33XX: Add D_CAN device tree data

2012-08-02 Thread Vaibhav Hiremath
On 7/25/2012 5:53 PM, AnilKumar Ch wrote: > Add Bosch D_CAN controller device tree data to AM33XX dtsi file > by adding d_can device node with all the necessary parameters. > > Signed-off-by: AnilKumar Ch > --- > arch/arm/boot/dts/am33xx.dtsi |5 + > 1 file changed, 5 insertions(+) >

Re: [PATCH V2] dtc: Add ability to delete nodes and properties

2012-08-02 Thread David Gibson
On Tue, Jun 12, 2012 at 05:10:20PM -0600, Stephen Warren wrote: > From: Stephen Warren > > dtc currently allows the contents of properties to be changed, and the > contents of nodes to be added to. There are situations where removing > properties or nodes may be useful. This change implements the

RE: [PATCH 1/2] net: davinci_mdio: enable and disable clock

2012-08-02 Thread Hiremath, Vaibhav
On Fri, Aug 03, 2012 at 10:52:40, Daniel Mack wrote: > On 03.08.2012 07:16, Vaibhav Hiremath wrote: > > > > > > On 8/3/2012 1:13 AM, Daniel Mack wrote: > >> Make the driver control the device clocks. Appearantly, the Davinci > >> platform probes this driver with the clock all powered up, but on O

Re: How to read arguments from a gpio handle

2012-08-02 Thread Stephen Warren
On 08/02/2012 09:39 PM, Leela Krishna Amudala wrote: > Hello, > > Can some one please tell me how to read the args from the gpio handle > > For example: > Considerlcd-reset-gpio = <&gpx0 1 2 3 4>; as my phandle. > > The 4 args in the above handle denotes > > <[phandle of the gpio controller

Re: [PATCH 1/2] net: davinci_mdio: enable and disable clock

2012-08-02 Thread Daniel Mack
On 03.08.2012 07:16, Vaibhav Hiremath wrote: > > > On 8/3/2012 1:13 AM, Daniel Mack wrote: >> Make the driver control the device clocks. Appearantly, the Davinci >> platform probes this driver with the clock all powered up, but on OMAP, >> this isn't the case. >> >> Signed-off-by: Daniel Mack >>

Re: [PATCH 1/2] net: davinci_mdio: enable and disable clock

2012-08-02 Thread Vaibhav Hiremath
On 8/3/2012 1:13 AM, Daniel Mack wrote: > Make the driver control the device clocks. Appearantly, the Davinci > platform probes this driver with the clock all powered up, but on OMAP, > this isn't the case. > > Signed-off-by: Daniel Mack > --- > drivers/net/ethernet/ti/davinci_mdio.c | 16

Re: How to read arguments from a gpio handle

2012-08-02 Thread Daniel Mack
Hi, On 03.08.2012 05:39, Leela Krishna Amudala wrote: > Can some one please tell me how to read the args from the gpio handle > > For example: > Considerlcd-reset-gpio = <&gpx0 1 2 3 4>; as my phandle. > > The 4 args in the above handle denotes > > <[phandle of the gpio controller node] >

How to read arguments from a gpio handle

2012-08-02 Thread Leela Krishna Amudala
Hello, Can some one please tell me how to read the args from the gpio handle For example: Considerlcd-reset-gpio = <&gpx0 1 2 3 4>; as my phandle. The 4 args in the above handle denotes <[phandle of the gpio controller node] [pin number within the gpio controller] [mux function]

Re: [PATCH v3 2/2] ARM: EXYNOS: Add Gscaler device from DT

2012-08-02 Thread Sylwester Nawrocki
On 08/02/2012 06:33 PM, Olof Johansson wrote: > On Wed, Aug 1, 2012 at 12:03 PM, Sylwester Nawrocki > wrote: > >> It wouldn't be clear what specific SoCs the "samsung,exynos5-gsc" compatible >> string applies to, would it ? I believe there are already minor differences >> in GScaler parameters o

Re: am33xx: default to status = "disabled"?

2012-08-02 Thread Arnd Bergmann
On Thursday 02 August 2012, Daniel Mack wrote: > currently, all devices in arch/arm/boot/dts/am33xx.dtsi are enabled by > default. However, depending on the actual board dts, only some of the > devices should actually be initialized, given that they only make sense > if their pins are actually wire

am33xx: default to status = "disabled"?

2012-08-02 Thread Daniel Mack
Hi, currently, all devices in arch/arm/boot/dts/am33xx.dtsi are enabled by default. However, depending on the actual board dts, only some of the devices should actually be initialized, given that they only make sense if their pins are actually wired on the board. On other platform, such devices a

Re: [PATCH v7 00/11] usb: musb: adding multi instance support

2012-08-02 Thread Daniel Mack
Hi Ravi, On 02.08.2012 14:12, Ravi Babu wrote: > This series of patches adds, > a) Multi instances support in musb driver > b) DT support for musb_dsps glue layer > c) DT support for NOP transceiver > > AM33xx and TI81xx has dual musb controller and has two usb PHY of same type. > This patch seri

Re: [PATCH 1/2] net: davinci_mdio: enable and disable clock

2012-08-02 Thread Daniel Mack
On 02.08.2012 22:20, Paul Walmsley wrote: > Hi > > On Thu, 2 Aug 2012, Daniel Mack wrote: > >> Make the driver control the device clocks. Appearantly, the Davinci >> platform probes this driver with the clock all powered up, but on OMAP, >> this isn't the case. >> >> Signed-off-by: Daniel Mack >

Re: [PATCH 1/2] net: davinci_mdio: enable and disable clock

2012-08-02 Thread Paul Walmsley
Hi On Thu, 2 Aug 2012, Daniel Mack wrote: > Make the driver control the device clocks. Appearantly, the Davinci > platform probes this driver with the clock all powered up, but on OMAP, > this isn't the case. > > Signed-off-by: Daniel Mack > --- > drivers/net/ethernet/ti/davinci_mdio.c | 16 +

Re: [PATCH 1/2] net: davinci_mdio: enable and disable clock

2012-08-02 Thread Daniel Mack
On 02.08.2012 21:53, Russell King - ARM Linux wrote: > On Thu, Aug 02, 2012 at 09:43:35PM +0200, Daniel Mack wrote: >> Make the driver control the device clocks. Appearantly, the Davinci >> platform probes this driver with the clock all powered up, but on OMAP, >> this isn't the case. > > Hmm, thi

Re: [PATCH 1/2] net: davinci_mdio: enable and disable clock

2012-08-02 Thread Russell King - ARM Linux
On Thu, Aug 02, 2012 at 09:43:35PM +0200, Daniel Mack wrote: > Make the driver control the device clocks. Appearantly, the Davinci > platform probes this driver with the clock all powered up, but on OMAP, > this isn't the case. Hmm, this looks like it could do with improvement, especially as we're

Re: [PATCH v2] of: Add videomode helper

2012-08-02 Thread Stephen Warren
On 07/05/2012 08:51 AM, Rob Herring wrote: > On 07/04/2012 02:56 AM, Sascha Hauer wrote: >> This patch adds a helper function for parsing videomodes from the devicetree. >> The videomode can be either converted to a struct drm_display_mode or a >> struct fb_videomode. >> diff --git a/Documentation

[PATCH 2/2] net: davinci_mdio: add DT bindings

2012-08-02 Thread Daniel Mack
Signed-off-by: Daniel Mack --- .../devicetree/bindings/net/davinci_mdio.txt | 24 + drivers/net/ethernet/ti/davinci_mdio.c | 39 ++ 2 files changed, 63 insertions(+) create mode 100644 Documentation/devicetree/bindings/net/davinci_mdio.txt diff

[PATCH 1/2] net: davinci_mdio: enable and disable clock

2012-08-02 Thread Daniel Mack
Make the driver control the device clocks. Appearantly, the Davinci platform probes this driver with the clock all powered up, but on OMAP, this isn't the case. Signed-off-by: Daniel Mack --- drivers/net/ethernet/ti/davinci_mdio.c | 16 ++-- 1 file changed, 14 insertions(+), 2 deleti

Re: [PATCH v2] of: Add videomode helper

2012-08-02 Thread Stephen Warren
On 07/04/2012 01:56 AM, Sascha Hauer wrote: > This patch adds a helper function for parsing videomodes from the devicetree. > The videomode can be either converted to a struct drm_display_mode or a > struct fb_videomode. > diff --git a/Documentation/devicetree/bindings/video/displaymode > b/Docum

Re: [RFC][PATCH v3 1/3] runtime interpreted power sequences

2012-08-02 Thread Mark Brown
On Thu, Aug 02, 2012 at 10:21:57AM +0200, Thierry Reding wrote: > On Thu, Aug 02, 2012 at 05:00:13PM +0900, Alex Courbot wrote: > > The problem is, how do we turn these phandles into the resource of > > interest. The type of the resource can be infered by the name of the > > property. The hard par

Re: [PATCH] mfd: add MAX8907 core driver

2012-08-02 Thread Mark Brown
On Thu, Aug 02, 2012 at 11:11:21AM -0600, Stephen Warren wrote: > Samuel, please don't apply this just yet though - it looks like I need > to make some minor changes to the header file and DT binding > documentation to add in a definition for one more regulator. I'll repost > the amended version a

Re: [PATCH] mfd: add MAX8907 core driver

2012-08-02 Thread Stephen Warren
On 08/02/2012 10:15 AM, Mark Brown wrote: > On Wed, Aug 01, 2012 at 02:48:05PM -0600, Stephen Warren wrote: >> From: Gyungoh Yoo >> >> The MAX8907 is an I2C-based power-management IC containing voltage >> regulators, a reset controller, a real-time clock, and a touch-screen >> controller. > > Rev

Device tree backporting for ARM on 2.6.32 kernel

2012-08-02 Thread Pankaj Pandey
Dear All, We are working on device tree backporting from 3.0 kernel to 2.6.32-kernel. We have backported the device tree support for arm on 2.6.32. It is able to read machine name, memory size and command line parameter from devicetree file for arm based target board. But it failed to read driv

Re: [PATCH] regulator: tps6586x: add support for SYS rail

2012-08-02 Thread Stephen Warren
On 08/02/2012 05:16 AM, Laxman Dewangan wrote: > Device have SYS rail which is always ON. It is system > power bus. LDO5 and LDO_RTC get powered through this rail > internally. Add support for this rail and make the > LDO5/LDO_RTC input supply to "sys". > Update document accordingly. I believe you

Re: [PATCH v3 2/2] ARM: EXYNOS: Add Gscaler device from DT

2012-08-02 Thread Olof Johansson
On Wed, Aug 1, 2012 at 12:03 PM, Sylwester Nawrocki wrote: > It wouldn't be clear what specific SoCs the "samsung,exynos5-gsc" compatible > string applies to, would it ? I believe there are already minor differences > in GScaler parameters on currently available exynos5 SoC. The variant data > st

Re: [PATCH] mfd: add MAX8907 core driver

2012-08-02 Thread Mark Brown
On Wed, Aug 01, 2012 at 02:48:05PM -0600, Stephen Warren wrote: > From: Gyungoh Yoo > > The MAX8907 is an I2C-based power-management IC containing voltage > regulators, a reset controller, a real-time clock, and a touch-screen > controller. Reviewed-by: Mark Brown __

Re: [PATCH] regulator: tps6586x: add support for SYS rail

2012-08-02 Thread Mark Brown
On Thu, Aug 02, 2012 at 05:21:38PM +0530, Laxman Dewangan wrote: > On Thursday 02 August 2012 05:10 PM, Mark Brown wrote: > >Is the system rail actually regulated or is it just a nominal 5V? > >Normally it's just the raw, unregulated input switched in with FETs or > >whatever. > It is unregulated

[PATCH v4 2/3] can: c_can: Add device tree support to Bosch C_CAN/D_CAN controller

2012-08-02 Thread AnilKumar Ch
Add device tree support to C_CAN/D_CAN controller and usage details are added to device tree documentation. Driver was tested on AM335x EVM. Signed-off-by: AnilKumar Ch --- .../devicetree/bindings/net/can/c_can.txt | 37 + drivers/net/can/c_can/c_can_platform.c

[PATCH v4 3/3] can: c_can: Add runtime PM support to Bosch C_CAN/D_CAN controller

2012-08-02 Thread AnilKumar Ch
Add Runtime PM support to C_CAN/D_CAN controller. The runtime PM APIs control clocks for C_CAN/D_CAN IP and prevent access to the register of C_CAN/D_CAN IP when clock is turned off. Signed-off-by: AnilKumar Ch --- drivers/net/can/c_can/c_can_platform.c |8 1 file changed, 8 inserti

[PATCH v3 0/3] Add DT support to C_CAN/D_CAN controller

2012-08-02 Thread AnilKumar Ch
This patch series adds the device tree support and Runtime PM support to C_CAN/D_CAN controller. First patch cleans up the device names used in c_can driver. These patches have been tested on AM335x EVM using some additional patches to add device tree data to EVM dts files and to initialize D_CAN

[PATCH v4 1/3] can: c_can: Modify c_can device names

2012-08-02 Thread AnilKumar Ch
Modify c_can device names from *_CAN_DEVTYPE to BOSCH_*_CAN to make use of same names for array indexes in c_can_id_table[] as well as device names. This patch also add indexes to c_can_id_table array. Signed-off-by: AnilKumar Ch --- drivers/net/can/c_can/c_can.h |5 +++-- drivers/

RE: [PATCH v3 1/3] can: c_can: Add device tree support to Bosch C_CAN/D_CAN controller

2012-08-02 Thread AnilKumar, Chimata
Hi Arnd, Thanks for the review On Thu, Aug 02, 2012 at 17:03:59, Arnd Bergmann wrote: > On Thursday 02 August 2012 16:32:17 AnilKumar Ch wrote: > > +- interrupt-parent : The parent interrupt controller > > + > > +Optional properties: > > +- ti,hwmods: Must be "d_can" or "c_can", n

Re: [PATCH v3 0/3] Add DT support to C_CAN/D_CAN controller

2012-08-02 Thread Marc Kleine-Budde
On 08/02/2012 01:39 PM, AnilKumar, Chimata wrote: [...] > AnilKumar Ch (3): > can: c_can: Add device tree support to Bosch C_CAN/D_CAN controller > can: c_can: Modify c_can device names in c_can_pci driver You break bisectability here. After patch 1 the pci driver will not

Re: [PATCH] regulator: tps6586x: add support for SYS rail

2012-08-02 Thread Mark Brown
On Thu, Aug 02, 2012 at 04:46:33PM +0530, Laxman Dewangan wrote: > +static const unsigned int tps6586x_sys_voltages[] = { > + 500, > +}; Is the system rail actually regulated or is it just a nominal 5V? Normally it's just the raw, unregulated input switched in with FETs or whatever. _

RE: [PATCH v3 0/3] Add DT support to C_CAN/D_CAN controller

2012-08-02 Thread AnilKumar, Chimata
Marc, On Thu, Aug 02, 2012 at 16:53:38, Marc Kleine-Budde wrote: > On 08/02/2012 01:21 PM, AnilKumar, Chimata wrote: > > Marc, > > > > On Thu, Aug 02, 2012 at 16:43:04, Marc Kleine-Budde wrote: > >> On 08/02/2012 01:02 PM, AnilKumar Ch wrote: > >>> This patch series adds the device tree support a

Re: [PATCH v3 1/3] can: c_can: Add device tree support to Bosch C_CAN/D_CAN controller

2012-08-02 Thread Arnd Bergmann
On Thursday 02 August 2012 16:32:17 AnilKumar Ch wrote: > +- interrupt-parent : The parent interrupt controller > + > +Optional properties: > +- ti,hwmods: Must be "d_can" or "c_can", n being the > + instance number interrupt-parent should be optional, not m

Re: [PATCH 0/5] usb: phy: samsung: Introducing usb phy driver for samsung SoCs

2012-08-02 Thread Arnd Bergmann
On Thursday 02 August 2012, Praveen Paneri wrote: > Yes! I understand this problem and this is the reason these patches > were sitting in my system for couple of weeks. In a discussion with > Thomas an idea of using the existing regulator framework to > enable/disable numerous PHYs came up. For ex

Re: [PATCH v3 0/3] Add DT support to C_CAN/D_CAN controller

2012-08-02 Thread Marc Kleine-Budde
On 08/02/2012 01:21 PM, AnilKumar, Chimata wrote: > Marc, > > On Thu, Aug 02, 2012 at 16:43:04, Marc Kleine-Budde wrote: >> On 08/02/2012 01:02 PM, AnilKumar Ch wrote: >>> This patch series adds the device tree support and Runtime PM support >>> to C_CAN/D_CAN controller. >>> >>> These patches hav

RE: [PATCH v3 0/3] Add DT support to C_CAN/D_CAN controller

2012-08-02 Thread AnilKumar, Chimata
Marc, On Thu, Aug 02, 2012 at 16:43:04, Marc Kleine-Budde wrote: > On 08/02/2012 01:02 PM, AnilKumar Ch wrote: > > This patch series adds the device tree support and Runtime PM support > > to C_CAN/D_CAN controller. > > > > These patches have been tested on AM335x EVM using some additional > > pa

Re: [PATCH v3 0/3] Add DT support to C_CAN/D_CAN controller

2012-08-02 Thread Marc Kleine-Budde
On 08/02/2012 01:02 PM, AnilKumar Ch wrote: > This patch series adds the device tree support and Runtime PM support > to C_CAN/D_CAN controller. > > These patches have been tested on AM335x EVM using some additional > patches to add device tree data to EVM dts files and to initialize > D_CAN RAM.

[PATCH v3 3/3] can: c_can: Add runtime PM support to Bosch C_CAN/D_CAN controller

2012-08-02 Thread AnilKumar Ch
Add Runtime PM support to C_CAN/D_CAN controller. The runtime PM APIs control clocks for C_CAN/D_CAN IP and prevent access to the register of C_CAN/D_CAN IP when clock is turned off. Signed-off-by: AnilKumar Ch --- drivers/net/can/c_can/c_can_platform.c |8 1 file changed, 8 inserti

[PATCH v3 2/3] can: c_can: Modify c_can device names in c_can_pci driver

2012-08-02 Thread AnilKumar Ch
Modify c_can device names from *_CAN_DEVTYPE to BOSCH_*_CAN to make use of same names in platform_device_id struct and of_device_id struct. Signed-off-by: AnilKumar Ch --- drivers/net/can/c_can/c_can_pci.c |6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/net/c

[PATCH v3 1/3] can: c_can: Add device tree support to Bosch C_CAN/D_CAN controller

2012-08-02 Thread AnilKumar Ch
Add device tree support to C_CAN/D_CAN controller and usage details are added to device tree documentation. Driver was tested on AM335x EVM. Signed-off-by: AnilKumar Ch --- .../devicetree/bindings/net/can/c_can.txt | 37 + drivers/net/can/c_can/c_can.h

[PATCH v3 0/3] Add DT support to C_CAN/D_CAN controller

2012-08-02 Thread AnilKumar Ch
This patch series adds the device tree support and Runtime PM support to C_CAN/D_CAN controller. These patches have been tested on AM335x EVM using some additional patches to add device tree data to EVM dts files and to initialize D_CAN RAM. D_CAN raminit is controlled from control module register

RE: [PATCH v2 1/2] can: c_can: Add device tree support to Bosch C_CAN/D_CAN controller

2012-08-02 Thread AnilKumar, Chimata
Hi Marc, On Thu, Aug 02, 2012 at 13:29:44, Marc Kleine-Budde wrote: > On 07/25/2012 04:12 PM, AnilKumar, Chimata wrote: > > Marc, > > > > On Wed, Jul 25, 2012 at 19:17:52, Marc Kleine-Budde wrote: > >> On 07/25/2012 02:18 PM, AnilKumar Ch wrote: > >>> Add device tree support to C_CAN/D_CAN contro

Re: [RFC][PATCH v3 1/3] runtime interpreted power sequences

2012-08-02 Thread Thierry Reding
On Thu, Aug 02, 2012 at 05:27:44PM +0900, Alex Courbot wrote: > On Thu 02 Aug 2012 05:21:57 PM JST, Thierry Reding wrote: > >* PGP Signed by an unknown key > > > >On Thu, Aug 02, 2012 at 05:00:13PM +0900, Alex Courbot wrote: > >>On 07/31/2012 07:45 AM, Stephen Warren wrote: > >>>Oh I see. That's a

Re: [RFC][PATCH v3 1/3] runtime interpreted power sequences

2012-08-02 Thread Thierry Reding
On Thu, Aug 02, 2012 at 05:00:13PM +0900, Alex Courbot wrote: > On 07/31/2012 07:45 AM, Stephen Warren wrote: > >Oh I see. That's a little confusing. Why not just reference the relevant > >resources directly in each step; something more like: > > > > gpio@1 { > > act

Re: [PATCH v2 1/2] can: c_can: Add device tree support to Bosch C_CAN/D_CAN controller

2012-08-02 Thread Marc Kleine-Budde
On 07/25/2012 04:12 PM, AnilKumar, Chimata wrote: > Marc, > > On Wed, Jul 25, 2012 at 19:17:52, Marc Kleine-Budde wrote: >> On 07/25/2012 02:18 PM, AnilKumar Ch wrote: >>> Add device tree support to C_CAN/D_CAN controller and usage details >>> are added to device tree documentation. Driver was tes