[PATCH 4/4] clk: mvebu: armada-375: Fix the description of the SAR in the comment

2014-08-29 Thread Gregory CLEMENT
For dealing with the code we use the SAR1 and not the SAR0. The code was correct, and now the comments too. Signed-off-by: Gregory CLEMENT gregory.clem...@free-electrons.com --- drivers/clk/mvebu/armada-375.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/clk

[PATCH 1/4] clk: mvebu: Fix clk frequency value if SSCG is enabled

2014-08-29 Thread Gregory CLEMENT
in an new optional function related to each SoC: is_sscg_enabled(). If this function is not present then no correction is done on the clock frequency. Signed-off-by: Gregory CLEMENT gregory.clem...@free-electrons.com --- drivers/clk/mvebu/common.c | 74

[PATCH 3/4] ARM: mvebu: add SSCG to Armada 370 Device Tree

2014-08-29 Thread Gregory CLEMENT
The Armada 370 SoC has a Spread Spectrum Clock Generator. This commit adds the description of this generator to the Device Tree describing this SoC. Signed-off-by: Gregory CLEMENT gregory.clem...@free-electrons.com --- arch/arm/boot/dts/armada-370.dtsi | 4 1 file changed, 4 insertions

[PATCH 0/4] clk:mvebu: Improve clock drift

2014-08-29 Thread Gregory CLEMENT
the mvebu tree. The last one is just to fix a typo I found while I was reading the clk code. Thanks, Gregory CLEMENT (4): clk: mvebu: Fix clk frequency value if SSCG is enabled clk: mvebu: armada-370: Fix timer drift caused by the SSCG deviation ARM: mvebu: add SSCG to Armada 370 Device Tree

Re: [PATCH 1/4] clk: mvebu: Fix clk frequency value if SSCG is enabled

2014-08-29 Thread Gregory CLEMENT
Hi Sebastian, On 29/08/2014 14:48, Sebastian Hesselbarth wrote: On 08/29/2014 01:43 PM, Gregory CLEMENT wrote: When the SSCG (Spread Spectrum Clock Generator) is enabled, it shifts the frequency of the clock. The percentage is no more than 1% but when the clock is used for a timer it leads

Re: [PATCH 2/4] clk: mvebu: armada-370: Fix timer drift caused by the SSCG deviation

2014-08-29 Thread Gregory CLEMENT
Hi Thomas, On 29/08/2014 15:08, Thomas Petazzoni wrote: Dear Gregory CLEMENT, On Fri, 29 Aug 2014 13:43:38 +0200, Gregory CLEMENT wrote: +bool a370_is_sscg_enabled(void __iomem *sar) Minor nit: static is missing here. OK I will fix it in the next version Thanks, Gregory

Re: [PATCH 1/4] clk: mvebu: Fix clk frequency value if SSCG is enabled

2014-09-01 Thread Gregory CLEMENT
Hi Leigh, On 01/09/2014 00:25, Leigh Brown wrote: Hi Gregory, On 2014-08-29 12:43, Gregory CLEMENT wrote: When the SSCG (Spread Spectrum Clock Generator) is enabled, it shifts the frequency of the clock. The percentage is no more than 1% but when the clock is used for a timer it leads

Re: [PATCH 0/8] Armada XP pinctrl consolidation and ix4-300d fixes

2014-10-06 Thread Gregory CLEMENT
Hi Sebastian, [...] NR_IRQS:16 nr_irqs:16 16 L2C: device tree omits to specify unified cache Jason, Thomas, Gregory, we should add a cache-unified to the l2cc nodes for all SoCs. Right, I take care of it. Thanks, Gregory -- Gregory Clement, Free Electrons Kernel, drivers, real

Re: [PATCH 0/8] Armada XP pinctrl consolidation and ix4-300d fixes

2014-10-06 Thread Gregory CLEMENT
. -- Gregory Clement, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info

[PATCH v2 1/4] ARM: mvebu: Clean-up the Armada XP support

2014-10-30 Thread Gregory CLEMENT
tree. In a few releases, when the old device tree will be obsolete, we will be able to remove the smp field and then the armada-370-xp.h header. Signed-off-by: Gregory CLEMENT gregory.clem...@free-electrons.com --- arch/arm/mach-mvebu/armada-370-xp.h | 6 -- arch/arm/mach-mvebu/board-v7.c

[PATCH v2 4/4] ARM: mvebu: Implement the CPU hotplug support for the Armada 38x SoCs

2014-10-30 Thread Gregory CLEMENT
in the -smp_secondary_init() hook. This commit has been tested using CPU hotplug through sysfs (/sys/devices/system/cpu/cpuX/online) and using kexec. Signed-off-by: Gregory CLEMENT gregory.clem...@free-electrons.com --- arch/arm/mach-mvebu/platsmp-a9.c | 53

[PATCH v2 0/4] Hot plug support for the Armada 38x SoCs

2014-10-30 Thread Gregory CLEMENT
to the function power_up_scu function to keep it private and not pollute the global namespace. Gregory CLEMENT (4): ARM: mvebu: Clean-up the Armada XP support ARM: mvebu: Move SCU power up in a function ARM: mvebu: Fix the secondary startup for Cortex A9 SoC ARM: mvebu: Implement the CPU

[PATCH v2 2/4] ARM: mvebu: Move SCU power up in a function

2014-10-30 Thread Gregory CLEMENT
This will allow reusing the same function in the secondary_startup for the Cortex A9 SoC. Signed-off-by: Gregory CLEMENT gregory.clem...@free-electrons.com --- arch/arm/mach-mvebu/pmsu_ll.S | 20 +--- 1 file changed, 13 insertions(+), 7 deletions(-) diff --git a/arch/arm/mach

[PATCH v2 3/4] ARM: mvebu: Fix the secondary startup for Cortex A9 SoC

2014-10-30 Thread Gregory CLEMENT
During the secondary startup the SCU was assumed to be in normal mode. It is not always the case, and especially after a kexec. This commit adds the needed sequence to put the SCU in normal mode. Signed-off-by: Gregory CLEMENT gregory.clem...@free-electrons.com --- arch/arm/mach-mvebu/headsmp-a9

[PATCH V2 2/4] clk: mvebu: armada-370: Fix timer drift caused by the SSCG deviation

2014-09-02 Thread Gregory CLEMENT
it was reduced at around 50ppm (around 4s per day). Signed-off-by: Gregory CLEMENT gregory.clem...@free-electrons.com --- drivers/clk/mvebu/armada-370.c | 8 1 file changed, 8 insertions(+) diff --git a/drivers/clk/mvebu/armada-370.c b/drivers/clk/mvebu/armada-370.c index bef198a83863

[PATCH V2 3/4] ARM: mvebu: add SSCG to Armada 370 Device Tree

2014-09-02 Thread Gregory CLEMENT
The Armada 370 SoC has a Spread Spectrum Clock Generator. This commit adds the description of this generator to the Device Tree describing this SoC. Signed-off-by: Gregory CLEMENT gregory.clem...@free-electrons.com --- arch/arm/boot/dts/armada-370.dtsi | 4 1 file changed, 4 insertions

[PATCH V2 0/4] clk: mvebu: Improve clock drift

2014-09-02 Thread Gregory CLEMENT
clear that the formula in datasheet is erroneous - added a fix_sscg_deviation() callback in order to be able to deal with the Dove case which is different from the other mvebu SoCs Gregory CLEMENT (4): clk: mvebu: Fix clk frequency value if SSCG is enabled clk: mvebu: armada-370: Fix

[PATCH V2 1/4] clk: mvebu: Fix clk frequency value if SSCG is enabled

2014-09-02 Thread Gregory CLEMENT
in an new optional function related to each SoC: is_sscg_enabled(). The fix is done with the other new optional function related to each SoC: fix_sscg_deviation. If one these functions are not present then no correction is done on the clock frequency. Signed-off-by: Gregory CLEMENT gregory.clem...@free

[PATCH V2 4/4] clk: mvebu: armada-375: Fix the description of the SAR in the comment

2014-09-02 Thread Gregory CLEMENT
For dealing with the code we use the SAR1 and not the SAR0. The code was correct, and now the comments too. Signed-off-by: Gregory CLEMENT gregory.clem...@free-electrons.com --- drivers/clk/mvebu/armada-375.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/clk

Re: [PATCH v4 6/7] ARM: mvebu: add PHY support to the dts for the USB controllers on Armada 375

2014-11-22 Thread Gregory CLEMENT
Hi Jason, On 22/11/2014 03:08, Jason Cooper wrote: On Thu, Nov 13, 2014 at 12:47:48PM +0100, Gregory CLEMENT wrote: Now that the USB cluster node has been added, use it as a PHY provider for the USB controller linked to it: the first EHCI and the xHCI. Signed-off-by: Gregory CLEMENT

Re: [PATCH v2 4/6] phy: add support for USB cluster on the Armada 375 SoC

2014-11-12 Thread Gregory CLEMENT
/majordomo-info.html -- Gregory Clement, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org

[PATCH v3 6/6] ARM: mvebu: add PHY support to the dts for the USB controllers on Armada 375

2014-11-12 Thread Gregory CLEMENT
Now that the USB cluster node has been added, use it as a PHY provider for the USB controller linked to it: the first EHCI and the xHCI. Signed-off-by: Gregory CLEMENT gregory.clem...@free-electrons.com --- arch/arm/boot/dts/armada-375.dtsi | 5 + 1 file changed, 5 insertions(+) diff --git

[PATCH v3 4/6] phy: add support for USB cluster on the Armada 375 SoC

2014-11-12 Thread Gregory CLEMENT
-by: Gregory CLEMENT gregory.clem...@free-electrons.com Signed-off-by: Thomas Petazzoni thomas.petazz...@free-electrons.com --- drivers/phy/Kconfig | 6 ++ drivers/phy/Makefile | 1 + drivers/phy/phy-armada375-usb2.c | 145 +++ include

[PATCH v3 1/6] phy: Use PTR_ERR_OR_ZERO to fix warning raised by coccinelle

2014-11-12 Thread Gregory CLEMENT
Use PTR_ERR_OR_ZERO rather than if(IS_ERR(...)) + PTR_ERR Generated by: coccinelle/api/ptr_ret.cocci Signed-off-by: Gregory CLEMENT gregory.clem...@free-electrons.com --- drivers/phy/phy-berlin-sata.c | 5 + drivers/phy/phy-hix5hd2-sata.c | 5 + drivers/phy/phy-miphy365x.c| 5

[PATCH v3 5/6] ARM: mvebu: add Device Tree description of USB cluster controller on Armada 375

2014-11-12 Thread Gregory CLEMENT
On Armada 375, the USB cluster allows to control the cluster composed of the USB2 and USB3 host controllers. Acked-by: Kishon Vijay Abraham I kis...@ti.com Signed-off-by: Gregory CLEMENT gregory.clem...@free-electrons.com --- arch/arm/boot/dts/armada-375.dtsi | 6 ++ 1 file changed, 6

[PATCH v3 0/6] Add support for USB cluster(PHY muxer) on the Armada 375

2014-11-12 Thread Gregory CLEMENT
Andrew Lunn - Move the DT binding documentation of the Armada 375 USB cluster into the phy-mvebu.txt file. - Made the armada375_usb_phy_xlate more robust if there is a phy_put and then a phy_get. Andrew Lunn (1): Phy: DT binding documentation for Marvell MVEBU SATA phy. Gregory CLEMENT (5

[PATCH v3 3/6] Phy: DT binding documentation for the Armada 375 USB cluster binding

2014-11-12 Thread Gregory CLEMENT
Armada 375 comes with an USB2 host and device controller and an USB3 controller. The USB cluster control register allows to manage common features of both USB controllers. This commit adds the Device Tree binding documentation for this piece of hardware. Signed-off-by: Gregory CLEMENT

[PATCH v3 2/6] Phy: DT binding documentation for Marvell MVEBU SATA phy.

2014-11-12 Thread Gregory CLEMENT
From: Andrew Lunn and...@lunn.ch Describe the binding for the Marvell MVEBU SATA phy. This driver can be used at least with Kirkwood, Dove and maybe others. Additionally, update the SATA binding with the properties to link to the phy nodes. Signed-off-by: Andrew Lunn and...@lunn.ch ---

Re: [PATCH v3 4/6] phy: add support for USB cluster on the Armada 375 SoC

2014-11-13 Thread Gregory CLEMENT
*usb_cluster_phy. Then you can invoke platform_get_drvdata here to get *usb_cluster_phy. While fixing this also add yourself as Maintainer of this file. I am taking into account your comment and I am going to send a new version soon. Thanks, Gregory Thanks Kishon -- Gregory Clement

[PATCH v4 1/7] phy: Use PTR_ERR_OR_ZERO to fix warning raised by coccinelle

2014-11-13 Thread Gregory CLEMENT
Use PTR_ERR_OR_ZERO rather than if(IS_ERR(...)) + PTR_ERR Generated by: coccinelle/api/ptr_ret.cocci Signed-off-by: Gregory CLEMENT gregory.clem...@free-electrons.com --- drivers/phy/phy-berlin-sata.c | 5 + drivers/phy/phy-hix5hd2-sata.c | 5 + drivers/phy/phy-miphy365x.c| 5

[PATCH v4 5/7] ARM: mvebu: add Device Tree description of USB cluster controller on Armada 375

2014-11-13 Thread Gregory CLEMENT
On Armada 375, the USB cluster allows to control the cluster composed of the USB2 and USB3 host controllers. Acked-by: Kishon Vijay Abraham I kis...@ti.com Signed-off-by: Gregory CLEMENT gregory.clem...@free-electrons.com --- arch/arm/boot/dts/armada-375.dtsi | 6 ++ 1 file changed, 6

[PATCH v4 4/7] phy: add support for USB cluster on the Armada 375 SoC

2014-11-13 Thread Gregory CLEMENT
-by: Gregory CLEMENT gregory.clem...@free-electrons.com Signed-off-by: Thomas Petazzoni thomas.petazz...@free-electrons.com --- drivers/phy/Kconfig | 6 ++ drivers/phy/Makefile | 1 + drivers/phy/phy-armada375-usb2.c | 158 +++ include

[PATCH v4 7/7] MAINTAINERS: add entry for the Armada 375 USB cluster PHY driver

2014-11-13 Thread Gregory CLEMENT
Add MAINTAINERS add entry for the Armada 375 USB cluster PHY driver. Signed-off-by: Gregory CLEMENT gregory.clem...@free-electrons.com --- MAINTAINERS | 6 ++ 1 file changed, 6 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index ea4d0058fd1b..ea35fa243502 100644 --- a/MAINTAINERS

[PATCH v4 3/7] Phy: DT binding documentation for the Armada 375 USB cluster binding

2014-11-13 Thread Gregory CLEMENT
Armada 375 comes with an USB2 host and device controller and an USB3 controller. The USB cluster control register allows to manage common features of both USB controllers. This commit adds the Device Tree binding documentation for this piece of hardware. Signed-off-by: Gregory CLEMENT

[PATCH v4 0/7] Add support for USB cluster(PHY muxer) on the Armada 375

2014-11-13 Thread Gregory CLEMENT
(1): Phy: DT binding documentation for Marvell MVEBU SATA phy. Gregory CLEMENT (6): phy: Use PTR_ERR_OR_ZERO to fix warning raised by coccinelle Phy: DT binding documentation for the Armada 375 USB cluster binding phy: add support for USB cluster on the Armada 375 SoC ARM: mvebu: add

[PATCH v4 6/7] ARM: mvebu: add PHY support to the dts for the USB controllers on Armada 375

2014-11-13 Thread Gregory CLEMENT
Now that the USB cluster node has been added, use it as a PHY provider for the USB controller linked to it: the first EHCI and the xHCI. Signed-off-by: Gregory CLEMENT gregory.clem...@free-electrons.com --- arch/arm/boot/dts/armada-375.dtsi | 5 + 1 file changed, 5 insertions(+) diff --git

[PATCH v4 2/7] Phy: DT binding documentation for Marvell MVEBU SATA phy.

2014-11-13 Thread Gregory CLEMENT
From: Andrew Lunn and...@lunn.ch Describe the binding for the Marvell MVEBU SATA phy. This driver can be used at least with Kirkwood, Dove and maybe others. Additionally, update the SATA binding with the properties to link to the phy nodes. Signed-off-by: Andrew Lunn and...@lunn.ch ---

Re: [PATCH v2 0/4] Hot plug support for the Armada 38x SoCs

2014-11-13 Thread Gregory CLEMENT
Hi Jason, On 30/10/2014 12:39, Gregory CLEMENT wrote: Hi, This patch set is the second version of the series adding the hot plug and also kexec support for the Armada 38x Socs. If nobody object we could push them in linux-next. The first patch was done in order to have the same code

Re: [linux-sunxi] Re: [PATCH 3/6] ARM: dts: sunxi: Add Allwinner A80 dtsi

2014-09-24 Thread Gregory CLEMENT
it in. I'll look around for examples. You can have a look on what we did for Armada XP: arch/arm/boot/dts/armada-370-xp.dtsi arch/arm/boot/dts/armada-xp.dtsi arch/arm/boot/dts/armada-xp-gp.dts I created a skeleton64.dtsi for this case Gregory -- Gregory Clement, Free Electrons Kernel, drivers

Re: [linux-sunxi] Re: [PATCH 3/6] ARM: dts: sunxi: Add Allwinner A80 dtsi

2014-09-24 Thread Gregory CLEMENT
. Grégory -- Gregory Clement, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More

[no subject]

2014-12-26 Thread Gregory CLEMENT
a device node to get the regulator instead using the device object. The first patch is not related to the second one, but it is little improvement. Gregory Gregory CLEMENT (2): regulator: core: Add a sanity check on the regulator_ enable/disable functions regulator: core: Add the device

[PATCH 2/2] regulator: core: Add the device tree version to the regulator_get family

2014-12-26 Thread Gregory CLEMENT
associated to a child node which is not a device, it is the case of the SATA ports of an ahci controller for instance. Signed-off-by: Gregory CLEMENT gregory.clem...@free-electrons.com --- drivers/regulator/core.c | 114 + drivers/regulator/devres.c

[PATCH 1/2] regulator: core: Add a sanity check on the regulator_ enable/disable functions

2014-12-26 Thread Gregory CLEMENT
These two functions use the pointer passed in parameter without any check. By adding a NULL pointer check, it allows using those functions from a driver in a more generic way. It is useful especially for the disable case if the regulator is optional. Signed-off-by: Gregory CLEMENT gregory.clem

[PATCH 0/3] ata: libahci: Allow using a regulator for each port

2014-12-27 Thread Gregory CLEMENT
to the regulator_get family, https://lkml.org/lkml/2014/12/26/155 I refactored the PHY support in the same time, so I would like to have a feedback from the Berlin platform to ensure that no regressions was introduced. Thanks, Grégory Gregory CLEMENT (3): ata: libahci: Clean-up the ahci_platform_en

[PATCH 3/3] ata: libahci: Allow to use multiple regulators

2014-12-27 Thread Gregory CLEMENT
family Signed-off-by: Gregory CLEMENT gregory.clem...@free-electrons.com --- drivers/ata/ahci.h | 2 +- drivers/ata/ahci_imx.c | 14 +-- drivers/ata/libahci_platform.c | 206 - include/linux/ahci_platform.h | 2 + 4 files changed

[PATCH 1/3] ata: libahci: Clean-up the ahci_platform_en/disable_phys functions

2014-12-27 Thread Gregory CLEMENT
The phy_ functions handle the NULL pointer case, so there is no need to skip them if there is a NULL pointer. Moreover, after the error label there is already no check on the pointer. This patch removes the unnecessary tests and brings some consistency. Signed-off-by: Gregory CLEMENT gregory.clem

[PATCH 2/3] Documentation: bindings: Add the regulator property to the sub-nodes AHCI bindings

2014-12-27 Thread Gregory CLEMENT
It is now possible to use a regulator property for each port of the AHCI controller. Signed-off-by: Gregory CLEMENT gregory.clem...@free-electrons.com --- Documentation/devicetree/bindings/ata/ahci-platform.txt | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/Documentation

Re: [PATCH v2 0/6] Add support for USB cluster(PHY muxer) on the Armada 375

2014-11-03 Thread Gregory CLEMENT
Hi Jason, On 01/11/2014 23:46, Jason Cooper wrote: Gregory, On Fri, Oct 24, 2014 at 05:24:08PM +0200, Gregory CLEMENT wrote: Hello, this is the second version of a series I initially submitted in May: https://lkml.org/lkml/2014/5/16/743 This series adds support for the USB cluster which

Re: [PATCH 03/17] irqchip: irq-armada-370-xp: use proper return value for -set_affinity()

2014-11-03 Thread Gregory CLEMENT
-370-xp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) Reviewed-by: Gregory CLEMENT gregory.clem...@free-electrons.com Thanks, Gregory -- Gregory Clement, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free

Re: [PATCH 04/17] irqchip: irq-armada-370-xp: suspend/resume support

2014-11-03 Thread Gregory CLEMENT
, right before entering suspend, all interrupts are assigned to the boot CPU. So what about /proc/irq/*/smp_affinity ? Do this files still represent accurate information? Thanks, Gregory -- Gregory Clement, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting

Re: [PATCH 05/17] clocksource: time-armada-370-xp: add suspend/resume support

2014-11-03 Thread Gregory CLEMENT
, timer_local_ctrl_reg; We don't need to restore any other timer than the timer 0, but it would worth mentioning that we save and restore only this one by naming the variable timer0_ctrl_reg and timer0_local_ctrl_reg Besides this and the typo: Acked-by: Gregory CLEMENT gregory.clem...@free-electrons.com

Re: [PATCH 09/17] clk: mvebu: add suspend/resume for gatable clocks

2014-11-04 Thread Gregory CLEMENT
-- 1 file changed, 28 insertions(+), 2 deletions(-) Acked-by: Gregory CLEMENT gregory.clem...@free-electrons.com Thanks, Gregory -- Gregory Clement, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free

[PATCH] xhci: Fix the lack of support for the Handle Port Configure Error

2015-02-06 Thread Gregory CLEMENT
-electrons.com: ported from 3.10 and added more explanations(from Shimmer) in the commit log] Signed-off-by: Guang Shen gs...@marvell.com Signed-off-by: Shimmer Huang shimm...@marvell.com Signed-off-by: Nadav Haklai nad...@marvell.com Reviewed-by: Yehuda Yitschak yehu...@marvell.com Signed-off-by: Gregory

Re: [PATCH 1/3] ARM: Marvell: Remove init cpus function

2015-01-19 Thread Gregory CLEMENT
a real benefit as the DT should be correctly filled. Remove this callback. In the course of the factorization of the smp functions, this function has been more and more reduced. As stated, the remaining part is not so useful now, it makes sens to remove it now. Acked-by: Gregory CLEMENT

Re: [PATCH v4 4/4] ARM: mvebu: Armada 385 GP: Add regulators to the SATA port

2015-01-16 Thread Gregory CLEMENT
Hi Hans, On 16/01/2015 09:17, Hans de Goede wrote: Hi, On 15-01-15 15:09, Gregory CLEMENT wrote: Add the regulators to each SATA port. Signed-off-by: Gregory CLEMENT gregory.clem...@free-electrons.com --- arch/arm/boot/dts/armada-388-gp.dts | 126

Re: [PATCH v4 4/4] ARM: mvebu: Armada 385 GP: Add regulators to the SATA port

2015-01-16 Thread Gregory CLEMENT
Hi Mark and Hans, On 16/01/2015 13:37, Mark Brown wrote: On Fri, Jan 16, 2015 at 11:10:18AM +0100, Hans de Goede wrote: On 16-01-15 10:27, Gregory CLEMENT wrote: + reg_sata0: pwr-sata0 { + compatible = regulator-fixed; + regulator-name = pwr_en_sata0; + enable

Re: [PATCH v3 3/4] ata: libahci: Allow using multiple regulators

2015-01-15 Thread Gregory CLEMENT
Hi Hans, On 15/01/2015 09:46, Hans de Goede wrote: Hi, On 13-01-15 15:22, Gregory CLEMENT wrote: The current implementation of the libahci allows using multiple PHYs but not multiple regulators. This patch adds the support of multiple regulators. Until now it was mandatory to have a PHY

[PATCH v4 0/4] ata: libahci: Allow using a regulator for each port

2015-01-15 Thread Gregory CLEMENT
of the regulator framework and instead associate each regulator of a port with an unique name. - Added the acked-by on the clean-up patch Gregory CLEMENT (4): ata: libahci: Clean-up the ahci_platform_en/disable_phys functions Documentation: bindings: Add the regulator property to the sub-nodes

[PATCH v4 4/4] ARM: mvebu: Armada 385 GP: Add regulators to the SATA port

2015-01-15 Thread Gregory CLEMENT
Add the regulators to each SATA port. Signed-off-by: Gregory CLEMENT gregory.clem...@free-electrons.com --- arch/arm/boot/dts/armada-388-gp.dts | 126 1 file changed, 126 insertions(+) diff --git a/arch/arm/boot/dts/armada-388-gp.dts b/arch/arm/boot/dts

[PATCH v4 3/4] ata: libahci: Allow using multiple regulators

2015-01-15 Thread Gregory CLEMENT
-by: Gregory CLEMENT gregory.clem...@free-electrons.com --- drivers/ata/ahci.h | 2 +- drivers/ata/ahci_imx.c | 14 +-- drivers/ata/libahci_platform.c | 230 + include/linux/ahci_platform.h | 2 + 4 files changed, 173 insertions

[PATCH v4 1/4] ata: libahci: Clean-up the ahci_platform_en/disable_phys functions

2015-01-15 Thread Gregory CLEMENT
The phy_ functions handle the NULL pointer case, so there is no need to skip them if there is a NULL pointer. Moreover, after the error label there is already no check on the pointer. This patch removes the unnecessary tests and brings some consistency. Signed-off-by: Gregory CLEMENT gregory.clem

[PATCH v4 2/4] Documentation: bindings: Add the regulator property to the sub-nodes AHCI bindings

2015-01-15 Thread Gregory CLEMENT
It is now possible to use a regulator property for each port of the AHCI controller. Signed-off-by: Gregory CLEMENT gregory.clem...@free-electrons.com --- Documentation/devicetree/bindings/ata/ahci-platform.txt | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git

Re: [PATCH v4] ARM: mvebu: remove two selects of ARM_ERRATA_753970

2015-01-21 Thread Gregory CLEMENT
boards if ARCH_MULTI_V7 select ARM_ERRATA_720789 - select ARM_ERRATA_753970 select ARM_GIC select ARMADA_38X_CLK select HAVE_ARM_SCU -- Gregory Clement, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support

Re: next-20150120 broken on Tegra by ata: libahci: Allow using multiple regulators

2015-01-21 Thread Gregory CLEMENT
and it didn't introduce any regression on the board I tested. Gregory -- Gregory Clement, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com -- To unsubscribe from this list: send the line unsubscribe linux-kernel

Re: [PATCH 1/2] regulator: core: Add a sanity check on the regulator_ enable/disable functions

2015-01-06 Thread Gregory CLEMENT
Hi Mark, On 29/12/2014 16:40, Mark Brown wrote: On Fri, Dec 26, 2014 at 06:26:38PM +0100, Gregory CLEMENT wrote: These two functions use the pointer passed in parameter without any check. By adding a NULL pointer check, it allows using those functions from a driver in a more generic way

Re: [PATCH 1/2] regulator: core: Add a sanity check on the regulator_ enable/disable functions

2015-01-06 Thread Gregory CLEMENT
Hi Mark, On 06/01/2015 13:00, Mark Brown wrote: On Tue, Jan 06, 2015 at 12:36:02PM +0100, Gregory CLEMENT wrote: Hi Mark, On 29/12/2014 16:40, Mark Brown wrote: On Fri, Dec 26, 2014 at 06:26:38PM +0100, Gregory CLEMENT wrote: No, especially in the case of regulator_enable

Re: [PATCH v2 2/4] Documentation: bindings: Add the regulator property to the sub-nodes AHCI bindings

2015-01-12 Thread Gregory CLEMENT
-- Gregory Clement, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info

Re: [PATCH v2 2/4] Documentation: bindings: Add the regulator property to the sub-nodes AHCI bindings

2015-01-09 Thread Gregory CLEMENT
Hi Hans, On 09/01/2015 16:46, Hans de Goede wrote: Hi, On 09-01-15 11:39, Gregory CLEMENT wrote: It is now possible to use a regulator property for each port of the AHCI controller. Signed-off-by: Gregory CLEMENT gregory.clem...@free-electrons.com --- Documentation/devicetree/bindings

Re: [PATCH] spi: orion: Add multiple chip select support for Armada 370 and 375

2015-01-06 Thread Gregory CLEMENT
); master-min_speed_hz = DIV_ROUND_UP(tclk_hz, devdata-max_divisor); + master-num_chipselect = devdata-num_cs; r = platform_get_resource(pdev, IORESOURCE_MEM, 0); spi-base = devm_ioremap_resource(pdev-dev, r); -- Gregory Clement, Free Electrons Kernel, drivers, real-time

[PATCH v3 1/4] ata: libahci: Clean-up the ahci_platform_en/disable_phys functions

2015-01-13 Thread Gregory CLEMENT
The phy_ functions handle the NULL pointer case, so there is no need to skip them if there is a NULL pointer. Moreover, after the error label there is already no check on the pointer. This patch removes the unnecessary tests and brings some consistency. Signed-off-by: Gregory CLEMENT gregory.clem

[PATCH v3 4/4] ARM: mvebu: Armada 385 GP: Add regulators to the SATA port

2015-01-13 Thread Gregory CLEMENT
Add the regulators to each SATA port. Signed-off-by: Gregory CLEMENT gregory.clem...@free-electrons.com --- arch/arm/boot/dts/armada-388-gp.dts | 126 1 file changed, 126 insertions(+) diff --git a/arch/arm/boot/dts/armada-388-gp.dts b/arch/arm/boot/dts

[PATCH v3 2/4] Documentation: bindings: Add the regulator property to the sub-nodes AHCI bindings

2015-01-13 Thread Gregory CLEMENT
It is now possible to use a regulator property for each port of the AHCI controller. Signed-off-by: Gregory CLEMENT gregory.clem...@free-electrons.com --- Documentation/devicetree/bindings/ata/ahci-platform.txt | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git

[PATCH v3 0/4] ata: libahci: Allow using a regulator for each port

2015-01-13 Thread Gregory CLEMENT
. - Added the acked-by on the clean-up patch Gregory CLEMENT (4): ata: libahci: Clean-up the ahci_platform_en/disable_phys functions Documentation: bindings: Add the regulator property to the sub-nodes AHCI bindings ata: libahci: Allow using multiple regulators ARM: mvebu: Armada 385 GP: Add

[PATCH v3 3/4] ata: libahci: Allow using multiple regulators

2015-01-13 Thread Gregory CLEMENT
-by: Gregory CLEMENT gregory.clem...@free-electrons.com --- drivers/ata/ahci.h | 2 +- drivers/ata/ahci_imx.c | 14 +-- drivers/ata/libahci_platform.c | 227 + include/linux/ahci_platform.h | 2 + 4 files changed, 170 insertions

[PATCH v2 4/4] ARM: mvebu: Armada 385 GP: Add regulators to the SATA port

2015-01-09 Thread Gregory CLEMENT
Add the regulators to each SATA port. Signed-off-by: Gregory CLEMENT gregory.clem...@free-electrons.com --- arch/arm/boot/dts/armada-388-gp.dts | 126 1 file changed, 126 insertions(+) diff --git a/arch/arm/boot/dts/armada-388-gp.dts b/arch/arm/boot/dts

[PATCH v2 3/4] ata: libahci: Allow using multiple regulators

2015-01-09 Thread Gregory CLEMENT
. To preserve the bisectability the change in the ahci_imx driver was done in the same patch. Signed-off-by: Gregory CLEMENT gregory.clem...@free-electrons.com --- drivers/ata/ahci.h | 2 +- drivers/ata/ahci_imx.c | 14 +-- drivers/ata/libahci_platform.c | 218

[PATCH v2 0/4] ata: libahci: Allow using a regulator for each port

2015-01-09 Thread Gregory CLEMENT
patch Gregory CLEMENT (4): ata: libahci: Clean-up the ahci_platform_en/disable_phys functions Documentation: bindings: Add the regulator property to the sub-nodes AHCI bindings ata: libahci: Allow using multiple regulators ARM: mvebu: Armada 385 GP: Add regulators to the SATA port

[PATCH v2 2/4] Documentation: bindings: Add the regulator property to the sub-nodes AHCI bindings

2015-01-09 Thread Gregory CLEMENT
It is now possible to use a regulator property for each port of the AHCI controller. Signed-off-by: Gregory CLEMENT gregory.clem...@free-electrons.com --- Documentation/devicetree/bindings/ata/ahci-platform.txt | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git

[PATCH v2 1/4] ata: libahci: Clean-up the ahci_platform_en/disable_phys functions

2015-01-09 Thread Gregory CLEMENT
The phy_ functions handle the NULL pointer case, so there is no need to skip them if there is a NULL pointer. Moreover, after the error label there is already no check on the pointer. This patch removes the unnecessary tests and brings some consistency. Signed-off-by: Gregory CLEMENT gregory.clem

Re: [PATCH 3/3] ata: libahci: Allow to use multiple regulators

2015-01-06 Thread Gregory CLEMENT
(*hpriv-phys); ... sz = (nports ? nports : 1) * sizeof(*hpriv-target_pwrs); I will do it. Thanks again for your review, Gregory -- Gregory Clement, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com

Re: PM: knowing the system state in the device callback

2015-03-17 Thread Gregory CLEMENT
at machine level, how could you also doing it at device level? Thanks, Gregory Regards, -- Gregory Clement, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com -- To unsubscribe from this list: send the line

Re: [patch 0/7] dt: dtb version: add version info to dtb

2015-03-19 Thread Gregory CLEMENT
/ -- To unsubscribe from this list: send the line unsubscribe devicetree in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html -- Gregory Clement, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting

Re: [PATCH 2/2] MAINTAINERS: Correct rtc armada38x pattern entry

2015-03-09 Thread Gregory CLEMENT
Hi Joe, On 08/03/2015 20:15, Joe Perches wrote: commit c6a95dbee793 (MAINTAINERS: add the RTC driver for the Armada38x) typoed the pattern, fix it. Indeed it was a typo that I though I had fixed, but maybe I didn't actually send the patch. So of course: Acked-by: Gregory CLEMENT gregory.clem

Re: [PATCH] n_tty: use kmalloc() instead of vmalloc() to avoid crash on armada-xp

2015-03-11 Thread Gregory CLEMENT
On 11/03/2015 16:01, Stas Sergeev wrote: 11.03.2015 16:14, Russell King - ARM Linux пишет: On Wed, Mar 11, 2015 at 01:44:57PM +0100, Gregory CLEMENT wrote: Hi Stas, On 10/03/2015 17:54, Stas Sergeev wrote: Hello, the patch below is needed for a successful boot on armada-xp. I am really

Re: linux-next: Tree for Mar 11 (arm:multi_v5_defconfig build failure)

2015-03-11 Thread Gregory CLEMENT
this patch soon in the mvebu/for-next branch. Indeed I planed to do it today I apologize for the inconvenience. Simon -- Gregory Clement, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com

Re: [PATCH] n_tty: use kmalloc() instead of vmalloc() to avoid crash on armada-xp

2015-03-11 Thread Gregory CLEMENT
n_tty_data *ldata; /* Currently a malloc failure here can panic */ -ldata = vmalloc(sizeof(*ldata)); +ldata = kmalloc(sizeof(*ldata), GFP_KERNEL); if (!ldata) goto err; -- Gregory Clement, Free Electrons Kernel, drivers, real-time and embedded Linux development

[PATCH 2/5] ARM: mvebu: Add standby support

2015-03-30 Thread Gregory CLEMENT
SoCs. There is issues with the the Armada 375, and the support would be added (if possible) in a future patch. Signed-off-by: Gregory CLEMENT gregory.clem...@free-electrons.com --- arch/arm/mach-mvebu/common.h | 5 ++-- arch/arm/mach-mvebu/pm-board.c | 13 ++ arch/arm/mach-mvebu/pm.c

[PATCH 0/5] Add standby support for the recent mvebu SoCs

2015-03-30 Thread Gregory CLEMENT
up the system. The last patch warns the user about it. The first patch is a clean-up found while working on this series All the patch are independents either for building or for running. Thanks, Gregory Gregory CLEMENT (5): ARM: mvebu: Use __init for the PM initialization functions ARM

[PATCH 1/5] ARM: mvebu: Use __init for the PM initialization functions

2015-03-30 Thread Gregory CLEMENT
mvebu_pm_init and mvebu_armada_xp_gp_pm_init are only called during boot, so flag them with __init and save some memory. Signed-off-by: Gregory CLEMENT gregory.clem...@free-electrons.com --- arch/arm/mach-mvebu/pm-board.c | 2 +- arch/arm/mach-mvebu/pm.c | 2 +- 2 files changed, 2

[PATCH 5/5] ARM: mvebu: Warn about the wake-ups sources not taken into account in suspend

2015-03-30 Thread Gregory CLEMENT
, the wake source won't be taken into consideration. Signed-off-by: Gregory CLEMENT gregory.clem...@free-electrons.com --- arch/arm/mach-mvebu/pm.c | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm/mach-mvebu/pm.c b/arch/arm/mach-mvebu/pm.c index 547310230edd..94966f1e4b14 100644

[PATCH 3/5] ARM: mvebu: Allow using the GIC for wakeup in standby mode

2015-03-30 Thread Gregory CLEMENT
On the Armada 375/38x/39x SoCs, in standby mode the SoC stay powered and it is possible to wake-up from any interrupt sources. This patch adds flag to the GIC irqchip driver to let linux know this. Signed-off-by: Gregory CLEMENT gregory.clem...@free-electrons.com --- arch/arm/mach-mvebu/board-v7

[PATCH 4/5] irqchip: armada-370-xp: Allow using wakeup source

2015-03-30 Thread Gregory CLEMENT
On the Armada 370/XP SoCs, in standby mode the SoC stay powered and it is possible to wake-up from any interrupt sources. This patch adds flag to the MPIC irqchip driver to let linux know this. Signed-off-by: Gregory CLEMENT gregory.clem...@free-electrons.com --- drivers/irqchip/irq-armada-370

[PATCH] sata_mv: Manage SATA port multiplier while searching for active links

2015-03-30 Thread Gregory CLEMENT
and title] Signed-off-by: Lior Amsalem al...@marvell.com Reviewed-by: Nadav Haklai nad...@marvell.com Signed-off-by: Gregory CLEMENT gregory.clem...@free-electrons.com --- drivers/ata/sata_mv.c | 11 ++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/drivers/ata/sata_mv.c b

Re: [PATCH] sata_mv: Manage SATA port multiplier while searching for active links

2015-03-31 Thread Gregory CLEMENT
Hi Tejun, On 30/03/2015 19:35, Tejun Heo wrote: Hello, On Mon, Mar 30, 2015 at 07:32:45PM +0200, Gregory CLEMENT wrote: From: Nadav Haklai nad...@marvell.com With this patch, when searching for active link, first search through the port multiplier links. And then fall to the default ap

Re: [PATCH 3/5] ARM: mvebu: Allow using the GIC for wakeup in standby mode

2015-04-01 Thread Gregory CLEMENT
Hi Marcin, On 01/04/2015 10:34, Marcin Wojtas wrote: Hi Gregory, 2015-03-30 16:04 GMT+02:00 Gregory CLEMENT gregory.clem...@free-electrons.com: On the Armada 375/38x/39x SoCs, in standby mode the SoC stay powered and it is possible to wake-up from any interrupt sources. This patch adds

Re: [PATCH 0/5] Add standby support for the recent mvebu SoCs

2015-04-01 Thread Gregory CLEMENT
in standby mode but for the mvebu SoCs, no interrupt can wake up the system. The last patch warns the user about it. The first patch is a clean-up found while working on this series All the patch are independents either for building or for running. Thanks, Gregory Gregory CLEMENT (5): ARM

Re: [PATCH v4 2/2] ARM: mvebu: a385-db-ap: Enable the NAND

2015-03-03 Thread Gregory CLEMENT
; usb-phy = usb3_phy; -- Gregory Clement, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message

Re: [PATCH v4 1/2] mtd: nand: pxa3xx: Fix PIO FIFO draining

2015-03-02 Thread Gregory CLEMENT
and then push it to arm-soc. Thanks, Gregory Brian -- Gregory Clement, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body

Re: [PATCH 3/8] ARM: dts: dove: Fix uart[23] reg property

2015-02-23 Thread Gregory CLEMENT
Hi Sebastian, On 17/02/2015 19:52, Sebastian Hesselbarth wrote: Fix Dove's register addresses of uart2 and uart3 nodes that seem to be broken since ages due to a copy-and-paste error. Signed-off-by: Sebastian Hesselbarth sebastian.hesselba...@gmail.com Acked-by: Gregory CLEMENT gregory.clem

Re: [PATCH 4/8] ARM: dts: dove: Always include gpio and interrupt-controller headers

2015-02-23 Thread Gregory CLEMENT
Hi Sebastian, On 17/02/2015 19:52, Sebastian Hesselbarth wrote: We want to enforce the use of named flags in GPIO and interrupt specifiers, include the corresponding headers to Dove's SoC dtsi. Signed-off-by: Sebastian Hesselbarth sebastian.hesselba...@gmail.com Acked-by: Gregory CLEMENT

<    1   2   3   4   5   6   7   8   9   10   >