[PATCH 2/2] clk: mvebu: armada-37xx-periph: switch to SPDX license identifier

2018-06-19 Thread Gregory CLEMENT
Adopt the SPDX license identifier headers to ease license compliance management. Signed-off-by: Gregory CLEMENT --- drivers/clk/mvebu/armada-37xx-periph.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/clk/mvebu/armada-37xx-periph.c b/drivers/clk/mvebu/armada

Re: [PATCH] arm64: dts: marvell: fix CP110 ICU node size

2018-07-05 Thread Gregory CLEMENT
ICU on Armada > 7K/8K") > Cc: sta...@vger.kernel.org # 4.14.x > Signed-off-by: Miquel Raynal > Reviewed-by: Thomas Petazzoni > Signed-off-by: Gregory CLEMENT See https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=2f872ddcdb1e8e2186162616cea4581b

Re: [PATCH] arm64: dts: marvell: fix CP110 ICU node size

2018-07-05 Thread Gregory CLEMENT
Hi, On jeu., juil. 05 2018, Gregory CLEMENT wrote: > Hi Miquel, > > On jeu., juil. 05 2018, Miquel Raynal wrote: > >> commit 2f872ddcdb1e8e2186162616cea4581b8403849d upstream. >> >> ICU size in CP110 is not 0x10 but at least 0x440 bytes long (from the

[PATCH net-next 3/6] net: mvneta: discriminate error cause for missed packet

2018-07-06 Thread Gregory CLEMENT
From: Yelena Krivosheev In order to improve the diagnostic in case of error, make the distinction between refill error and skb allocation error. Signed-off-by: Yelena Krivosheev [gregory: extract from a larger patch] Signed-off-by: Gregory CLEMENT --- drivers/net/ethernet/marvell/mvneta.c

Re: [PATCH net-next v3 00/10] net: mvpp2: phylink conversion

2018-05-17 Thread Gregory CLEMENT
onfig | 1 + > drivers/net/ethernet/marvell/mvpp2.c | 931 +++++++--- > drivers/phy/marvell/phy-mvebu-cp110-comphy.c | 17 +- > include/linux/phy/phy.h | 1 + > 7 files changed, 680 insertions(+), 355 deletions(-) > > -- > 2.17.0 > -- Gregory Clement, Bootlin (formerly Free Electrons) Embedded Linux and Kernel engineering http://bootlin.com

Re: [PATCH v3 1/6] dt-bindings: ap806: add the cluster clock node in the syscon file

2018-12-18 Thread Gregory CLEMENT
Hi Rob, On lun., déc. 17 2018, Rob Herring wrote: > On Sun, Dec 16, 2018 at 10:41:42AM +0100, Gregory CLEMENT wrote: >> Document the device tree binding for the cluster clock controllers found >> in the Armada 7K/8K SoCs. >> >> Signed-off-by: Gregory CLEMENT >&

Re: [PATCH v2 0/5] Add CPU clock support for Armada 7K/8K

2018-12-16 Thread Gregory CLEMENT
Hi Stephen, On ven., déc. 14 2018, Stephen Boyd wrote: > Quoting Gregory CLEMENT (2018-12-06 09:15:11) >> Hello, >> >> This is the second version of a series allowing to manage the cpu >> clock for Armada 7K/8K. For these SoCs, the CPUs share the same clock >

[PATCH v3 5/6] arm64: marvell: enable the Armada 7K/8K CPU clk driver

2018-12-16 Thread Gregory CLEMENT
This commit makes sure the driver for the Armada 7K/8K CPU clock is enabled. Signed-off-by: Gregory CLEMENT --- arch/arm64/Kconfig.platforms | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm64/Kconfig.platforms b/arch/arm64/Kconfig.platforms index 51bc479334a4..8a05870b1ba8 100644

[PATCH v3 0/6] Add CPU clock support for Armada 7K/8K

2018-12-16 Thread Gregory CLEMENT
other minor cleanup Gregory CLEMENT (6): dt-bindings: ap806: add the cluster clock node in the syscon file clk: mvebu: add helper file for Armada AP and CP clocks clk: mvebu: add CPU clock driver for Armada 7K/8K clk: mvebu: ap806: Fix clock name for the cluster arm64: marvell: enable the

[PATCH v3 4/6] clk: mvebu: ap806: Fix clock name for the cluster

2018-12-16 Thread Gregory CLEMENT
Actually, the clocks exposed for the cluster are not the CPU clocks, but the PLL clock used as entry clock for the CPU clocks. The CPU clock will be managed by a driver submitting in the following patches. Signed-off-by: Gregory CLEMENT --- drivers/clk/mvebu/ap806-system-controller.c | 4

[PATCH v3 2/6] clk: mvebu: add helper file for Armada AP and CP clocks

2018-12-16 Thread Gregory CLEMENT
Clock drivers for Armada AP and Armada CP use the same function to generate unique clock name. A third drivers is coming with the same need, so it's time to move this function in a common file. Signed-off-by: Gregory CLEMENT --- drivers/clk/mvebu/Kconfig | 5 driver

[PATCH v3 6/6] arm64: dts: marvell: Add cpu clock node on Armada 7K/8K

2018-12-16 Thread Gregory CLEMENT
Add cpu clock node on AP Signed-off-by: Gregory CLEMENT --- arch/arm64/boot/dts/marvell/armada-ap806-quad.dtsi | 4 arch/arm64/boot/dts/marvell/armada-ap806.dtsi | 6 ++ 2 files changed, 10 insertions(+) diff --git a/arch/arm64/boot/dts/marvell/armada-ap806-quad.dtsi b/arch

[PATCH v3 1/6] dt-bindings: ap806: add the cluster clock node in the syscon file

2018-12-16 Thread Gregory CLEMENT
Document the device tree binding for the cluster clock controllers found in the Armada 7K/8K SoCs. Signed-off-by: Gregory CLEMENT --- .../arm/marvell/ap806-system-controller.txt | 22 +++ 1 file changed, 22 insertions(+) diff --git a/Documentation/devicetree/bindings/arm

[PATCH v3 3/6] clk: mvebu: add CPU clock driver for Armada 7K/8K

2018-12-16 Thread Gregory CLEMENT
The CPU frequency is managed at the AP level for the Armada 7K/8K. The CPU frequency is modified by cluster: the CPUs of the same cluster have the same frequency. This patch adds the clock driver that will be used by CPUFreq, it is based on the work of Omri Itach . Signed-off-by: Gregory CLEMENT

[PATCH 3/3] pinctrl: dt-bindings: Fix the armada-37xx documentation

2018-12-21 Thread Gregory CLEMENT
While it was possible to configure the PCIe1 Wakeup pin, it was missing in the bidding, let's document it. Signed-off-by: Gregory CLEMENT --- .../bindings/pinctrl/marvell,armada-37xx-pinctrl.txt| 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/Document

[PATCH 1/3] pinctrl: armada-37xx: Correct mpp definitions

2018-12-21 Thread Gregory CLEMENT
) fix ptp group bit mask to BIT(11) | BIT(12) | BIT(13) add smi group with bit mask BIT(4) [gregory: split the pcie group in 2, as at hardware level they can be configured separately] Signed-off-by: Marek Behún Signed-off-by: Gregory CLEMENT --- .../pinctrl/marvell,armada-37xx-pinctrl.txt

[PATCH 2/3] pinctrl: armada-37xx: add missing pin: PCIe1 Wakeup

2018-12-21 Thread Gregory CLEMENT
Declare the PCIe1 Wakeup which was initially missing. Signed-off-by: Gregory CLEMENT --- drivers/pinctrl/mvebu/pinctrl-armada-37xx.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/pinctrl/mvebu/pinctrl-armada-37xx.c b/drivers/pinctrl/mvebu/pinctrl-armada-37xx.c index 267dfc530985

[PATCH 0/3] Few fix for pins configuration on Armada 37xx

2018-12-21 Thread Gregory CLEMENT
Hello, this small series fixes wrong pins configuration we currently have on Armada 37xx. For now it is especially annoying it we want to enable suspend to RAM for PCie. Gregory Gregory CLEMENT (2): pinctrl: armada-37xx: add missing pin: PCIe1 Wakeup pinctrl: dt-bindings: Fix the armada

Re: [PATCH 1/3] pinctrl: armada-37xx: Correct mpp definitions

2018-12-24 Thread Gregory CLEMENT
Hi Marek, On sam., déc. 22 2018, Marek Behun wrote: > On Fri, 21 Dec 2018 18:32:57 +0100 > Gregory CLEMENT wrote: > >> +PIN_GRP_GPIO("pcie1", 3, 1, BIT(5), "pcie"), >> +PIN_GRP_GPIO("pcie1_clkreq", 4, 1, BIT(9), "pcie")

Re: [PATCH] clk: armada-370: fix refcount leak in a370_clk_init()

2018-12-26 Thread Gregory CLEMENT
Hi Yangtao, On mer., déc. 26 2018, Yangtao Li wrote: > The of_find_compatible_node() returns a node pointer with refcount > incremented, but there is the lack of use of the of_node_put() when > done. Add the missing of_node_put() to release the refcount. Reviewed-by: Gregory CLEMENT

Re: [PATCH] clk: kirkwood: fix refcount leak in kirkwood_clk_init()

2018-12-26 Thread Gregory CLEMENT
Hi Yangtao, On mer., déc. 26 2018, Yangtao Li wrote: > The of_find_compatible_node() returns a node pointer with refcount > incremented, but there is the lack of use of the of_node_put() when > done. Add the missing of_node_put() to release the refcount. Reviewed-by: Gregory CLEMENT

Re: [PATCH] clk: armada-xp: fix refcount leak in axp_clk_init()

2018-12-26 Thread Gregory CLEMENT
Hi Yangtao, On mer., déc. 26 2018, Yangtao Li wrote: > The of_find_compatible_node() returns a node pointer with refcount > incremented, but there is the lack of use of the of_node_put() when > done. Add the missing of_node_put() to release the refcount. Reviewed-by: Gregory CLEMENT

Re: [PATCH] clk: mv98dx3236: fix refcount leak in mv98dx3236_clk_init()

2018-12-26 Thread Gregory CLEMENT
Hi Yangtao, On mer., déc. 26 2018, Yangtao Li wrote: > The of_find_compatible_node() returns a node pointer with refcount > incremented, but there is the lack of use of the of_node_put() when > done. Add the missing of_node_put() to release the refcount. Reviewed-by: Gregory CLEMENT

Re: [PATCH] clk: dove: fix refcount leak in dove_clk_init()

2018-12-26 Thread Gregory CLEMENT
Hi Yangtao, On mer., déc. 26 2018, Yangtao Li wrote: > The of_find_compatible_node() returns a node pointer with refcount > incremented, but there is the lack of use of the of_node_put() when > done. Add the missing of_node_put() to release the refcount. Reviewed-by: Gregory CLEMENT

Re: [PATCH v2 1/1] arm64: dts: marvell: armada-ap806: reserve PSCI area

2018-12-26 Thread Gregory CLEMENT
lls = <2>; > + #size-cells = <2>; > + ranges; > + > + psci-area@400 { > + reg = <0x0 0x400 0x0 0x20>; > + no-map; > + }; > + }; > + > ap806 { >

Re: [PATCH] clk: mvebu: armada-37xx-tbg: Balance devm_clk_get in probe

2018-10-10 Thread Gregory CLEMENT
Hi Stephen, On mer., oct. 10 2018, Stephen Boyd wrote: > Quoting Gregory CLEMENT (2018-09-14 08:34:21) >> The parent clock is get only to have its name, and then the clock is no >> more used, so we can safely free it using devm_clk_put. >> >> Sig

[PATCH v2] clk: mvebu: armada-37xx-tbg: Switch to clk_get and balance it in probe

2018-10-10 Thread Gregory CLEMENT
) and clk_put(). Signed-off-by: Gregory CLEMENT --- Changelog: v1 -> v2 - Use the non managed version of clk_get() and clk_put() (suggested by Stephen Boyd) - Modify the title and the commit log to reflect this change drivers/clk/mvebu/armada-37xx-tbg.c | 3 ++- 1 file changed, 2 insertio

Re: [PATCH 1/5] arm64: defconfig: Enable sound support for Samsung Chromebook Plus.

2018-07-18 Thread Gregory CLEMENT
NFIG_SND_SOC_SAMSUNG=y > CONFIG_SND_SOC_RCAR=m > CONFIG_SND_SOC_AK4613=m > +CONFIG_SND_SOC_DA7219=m > +CONFIG_SND_SOC_MAX98357A=m > +CONFIG_SND_SOC_RL6231=m > +CONFIG_SND_SOC_RT5514=m > +CONFIG_SND_SOC_RT5514_SPI=m > +CONFIG_SND_SOC_RT5645=m > CONFIG_SND_SIMPLE_CARD=

Re: [PATCH v2] ARM: dts: mvebu: Add device tree for ATL-x530 Board

2021-04-07 Thread Gregory CLEMENT
gt;; > + label = "u-boot-env"; > + }; > + partition@unused { > + reg = <0x0014 0x00e8>; > + label = "unused"; > + }; > + partition@idprom { > + reg = <0x00fc 0x0004>; > + label = "idprom"; > + }; > + }; > + }; > +}; > + > +&nand_controller { > + status = "okay"; > + > + nand@0 { > + reg = <0>; > + label = "pxa3xx_nand-0"; > + nand-rb = <0>; > + nand-on-flash-bbt; > + nand-ecc-strength = <4>; > + nand-ecc-step-size = <512>; > + > + marvell,nand-enable-arbiter; > + > + partitions { > + compatible = "fixed-partitions"; > + #address-cells = <1>; > + #size-cells = <1>; > + partition@user { > + reg = <0x 0x0f00>; > + label = "user"; > + }; > + partition@errlog { > + /* Maximum mtdoops size is 8MB, so set to that. > */ > + reg = <0x0f00 0x0080>; > + label = "errlog"; > + }; > + partition@nand-bbt { > + reg = <0x0f80 0x0080>; > + label = "nand-bbt"; > + }; > + }; > + }; > +}; > + > -- > 2.29.2 > -- Gregory Clement, Bootlin Embedded Linux and Kernel engineering http://bootlin.com

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 dri

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,

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

2015-01-06 Thread Gregory CLEMENT
t; -hpriv->phys[port] = devm_of_phy_get(dev, child, NULL); >> - if (IS_ERR(hpriv->phys[port])) { >> -rc = PTR_ERR(hpriv->phys[port]); >> -dev_err(dev, >> -

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

2015-01-06 Thread Gregory CLEMENT
_device *pdev) > master->mode_bits = SPI_CPHA | SPI_CPOL; > > master->transfer_one_message = orion_spi_transfer_one_message; > - master->num_chipselect = ORION_NUM_CHIPSELECTS; > master->bits_per_word_mask = SPI_BPW_MASK(8) | SPI_BPW_MASK(16); >

[no subject]

2014-12-26 Thread Gregory CLEMENT
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 --- drivers/regulator/core.c | 114 + drivers/regulator/devres.c | 70

[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 --- drivers

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

2014-12-27 Thread Gregory CLEMENT
o 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_p

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

2014-12-27 Thread Gregory CLEMENT
o the regulator_get family" Signed-off-by: Gregory CLEMENT --- 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, 151 insertions(+), 73

[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

[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 --- Documentation/devicetree/bindings/ata/ahci-platform.txt | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/Documentation/devicetree/bindings/ata/ahci

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

2015-01-23 Thread Gregory CLEMENT
DIV_ROUND_UP(info->oob_size, 4)); > + drain_fifo(info, > +info->oob_buff + info->oob_buff_pos, > +DIV_ROUND_UP(info->oob_size, 4)); > break; > default: >

[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 Acked-by

[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 --- 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/armada-388-gp.dts index 4df22bf91683

[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 --- Documentation/devicetree/bindings/ata/ahci-platform.txt | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/Documentation/devicetree/bindings/ata/ahci

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

2015-01-13 Thread Gregory CLEMENT
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 AHCI bindings ata: libahci: Allow using multiple regulators ARM: mvebu: Armada

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

2015-01-13 Thread Gregory CLEMENT
-by: Gregory CLEMENT --- 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(+), 75 deletions(-) diff --git a

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 n

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

2015-01-15 Thread Gregory CLEMENT
on 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

[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 --- Documentation/devicetree/bindings/ata/ahci-platform.txt | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/Documentation/devicetree/bindings/ata/ahci

[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 --- 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/armada-388-gp.dts index 4df22bf91683

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

2015-01-15 Thread Gregory CLEMENT
-by: Gregory CLEMENT --- 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(+), 75 deletions(-) diff --git a

[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 Acked-by

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

2015-01-09 Thread Gregory CLEMENT
tch 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 p

[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 --- drivers/ata/ahci.h | 2 +- drivers/ata/ahci_imx.c | 14 +-- drivers/ata/libahci_platform.c | 218 + include

[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 --- 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/armada-388-gp.dts index 4df22bf91683

[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 --- Documentation/devicetree/bindings/ata/ahci-platform.txt | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/Documentation/devicetree/bindings/ata/ahci

[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 Acked-by

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 >> --- >>

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

2015-01-12 Thread Gregory CLEMENT
re we made DT a first class citizen and generally they shouldn't be >> used these days. The other is that at least for regulators we have >> constant problems with people abusing the API in various ways, as a >> result the API has a goal of not helping undesirable usage pa

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 >> --- >> 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&q

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

2015-01-19 Thread Gregory CLEMENT
ode is valid, it does not give 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

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

2015-04-03 Thread Gregory CLEMENT
Hi Jason, On 03/04/2015 01:23, Jason Cooper wrote: > On Mon, Mar 30, 2015 at 04:04:37PM +0200, Gregory CLEMENT wrote: >> 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 th

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

2015-04-03 Thread Gregory CLEMENT
On 03/04/2015 16:29, Jason Cooper wrote: > On Fri, Apr 03, 2015 at 09:17:55AM +0200, Gregory CLEMENT wrote: >> Hi Jason, >> >> On 03/04/2015 01:23, Jason Cooper wrote: >>> On Mon, Mar 30, 2015 at 04:04:37PM +0200, Gregory CLEMENT wrote: >>>> On the Armada

Re: [PATCH] genirq: provide dummy set_irq_wake()

2015-04-14 Thread Gregory CLEMENT
sk = noop, >> +.irq_set_wake = noop_int_ret, >> }; >> EXPORT_SYMBOL_GPL(dummy_irq_chip); >> > > -- > To unsubscribe from this list: send the line "unsubscribe linux-usb" in > the body of a message to majord...@vger.kernel.org > More majordomo info

Re: [PATCH] genirq: provide dummy set_irq_wake()

2015-04-15 Thread Gregory CLEMENT
Hi Roger, On 15/04/2015 10:07, Roger Quadros wrote: > Hi Gregory, > > On 14/04/15 17:02, Gregory CLEMENT wrote: >> Hi Roger, >> >> On 14/04/2015 12:13, Roger Quadros wrote: >>> Hi Thomas, >>> >>> On 30/03/15 16:15, Roger Quadros wrote: >

Re: [PATCH v2] genirq: Set IRQCHIP_SKIP_SET_WAKE flag for dummy_irq_chip

2015-04-15 Thread Gregory CLEMENT
nux.kernel/1879035 > > Signed-off-by: Roger Quadros FWIW: Reviewed-by: Gregory CLEMENT > --- > kernel/irq/dummychip.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/kernel/irq/dummychip.c b/kernel/irq/dummychip.c > index 988dc58..2feb6fe 100644 > --- a/kernel/i

Re: [PATCH 0/2] crypto: add new driver for Marvell CESA

2015-04-17 Thread Gregory CLEMENT
map_resource() in the new driver, so if the old one is already loaded the memory region will be already hold and the new driver will simply fail during the probe. So for this part it is OK. However, the old driver doesn't try to reserve the region, it directly uses an ioremap(). So if the new driv

[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 --- arch/arm/mach-mvebu/pm-board.c | 2 +- arch/arm/mach-mvebu/pm.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git

[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 --- arch/arm/mach-mvebu/common.h | 5 ++-- arch/arm/mach-mvebu/pm-board.c | 13 ++ arch/arm/mach-mvebu/pm.c | 56

[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 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 --- drivers/irqchip/irq-armada-370-xp.c | 1 + 1 file changed, 1

[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 --- 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 --- a/arch/arm/mach-mvebu/pm.c +++ b

[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 --- arch/arm/mach-mvebu/board-v7.c | 7 +++ 1 file changed, 7

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

2015-03-30 Thread Gregory CLEMENT
-by: Lior Amsalem Reviewed-by: Nadav Haklai Signed-off-by: Gregory CLEMENT --- drivers/ata/sata_mv.c | 11 ++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/drivers/ata/sata_mv.c b/drivers/ata/sata_mv.c index f8c33e3772b8..7fac4d0fee86 100644 --- a/drivers/ata/sata_mv.c

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 >> >> With this patch, when searching for active link, first search through >> the port multiplier links

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 > : >> 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 >>

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

2015-04-01 Thread Gregory CLEMENT
ds, > Marcin > > 2015-03-30 16:04 GMT+02:00 Gregory CLEMENT > : >> Hi, >> >> Until now only one Armada XP based board supported suspend to >> ram. This suspend to ram mode was unusual because it involved shutting >> down the SoC and relied on a PIC to

Re: [PATCH 03/11] arm/mach-mvebu: remove legacy __cpuinit sections that crept in

2015-04-28 Thread Gregory CLEMENT
ce we want to clobber the stubs soon, get this removed now. > Note that there would normally be a corresponding removal of > a ".previous" directive for each __CPUINIT in asm files, but in > this case it appears that this single function file was never > paired off with one. >

Re: [PATCH] arch/arm/mach-mvebu/Kconfig: remove obsolete selects

2015-04-28 Thread Gregory CLEMENT
r the PL310 used on these two SoCs were the r3p3. At the end I don't know which one of the patches to apply... But in any case I will add Acked-by: Gregory CLEMENT Thanks, Gregory > > Signed-off-by: Valentin Rothberg > --- > I detected this issue with scripts/checkkconfig

Re: [PATCH] arch/arm/mach-mvebu/Kconfig: remove obsolete selects

2015-04-28 Thread Gregory CLEMENT
Hi Russell, On 28/04/2015 18:54, Russell King - ARM Linux wrote: > On Tue, Apr 28, 2015 at 06:41:41PM +0200, Gregory CLEMENT wrote: >> Hi Valentin, >> >> On 28/04/2015 18:02, Valentin Rothberg wrote: >>> ARM_ERRATA_753970 is not defined in Kconfig, so that both s

Re: [PATCH v9 23/30] PCI/mvebu: Use pci_common_init_dev() to simplify code

2015-04-08 Thread Gregory CLEMENT
u can add my: Reviewed-by: Gregory CLEMENT Tested-by: Gregory CLEMENT Thanks, Gregory > Signed-off-by: Yijing Wang > CC: Thomas Petazzoni > CC: Jason Cooper > --- > drivers/pci/host/pci-mvebu.c | 18 +- > 1 files changed, 1 insertions(+), 17 deletion

Re: [PATCH v9 03/30] PCI: Save domain in pci_host_bridge

2015-04-08 Thread Gregory CLEMENT
omain number in core by pci_bus_assign_domain_nr(). > For the mvebu part I tested on Armada XP, Armada 375 and Armada 38x SoCs, and I didn't saw any regression. So you can add my: Tested-by: Gregory CLEMENT Thanks, Gregory > Signed-off-by: Yijing Wang > --- > arch/alph

Re: [PATCH 0/2] crypto: add new driver for Marvell CESA

2015-04-17 Thread Gregory CLEMENT
Hi Maxime, On 17/04/2015 16:32, Maxime Ripard wrote: > On Fri, Apr 17, 2015 at 04:19:22PM +0200, Boris Brezillon wrote: >> Hi Gregory, >> >> On Fri, 17 Apr 2015 15:01:01 +0200 >> Gregory CLEMENT wrote: >> >>> Hi Boris, >>> >>> On 17/04

Re: [PATCH 0/2] crypto: add new driver for Marvell CESA

2015-04-17 Thread Gregory CLEMENT
On 17/04/2015 16:50, Maxime Ripard wrote: > On Fri, Apr 17, 2015 at 04:40:43PM +0200, Gregory CLEMENT wrote: >> Hi Maxime, >> >> On 17/04/2015 16:32, Maxime Ripard wrote: >>> On Fri, Apr 17, 2015 at 04:19:22PM +0200, Boris Brezillon wrote: >>>> Hi Grego

Re: [PATCH 0/2] crypto: add new driver for Marvell CESA

2015-04-17 Thread Gregory CLEMENT
On 17/04/2015 17:49, Maxime Ripard wrote: > On Fri, Apr 17, 2015 at 05:01:55PM +0200, Gregory CLEMENT wrote: >> On 17/04/2015 16:50, Maxime Ripard wrote: >>> On Fri, Apr 17, 2015 at 04:40:43PM +0200, Gregory CLEMENT wrote: >>>> Hi Maxime, >>>> >

Re: Fwd: Re: [PATCH 2/2] ARM: mvebu: dts: Add dts file for DLink DNS-327L

2015-04-20 Thread Gregory CLEMENT
Hi Andrew, On 13/04/2015 16:32, Andrew wrote: > Gregory CLEMENT писал 13.04.2015 17:16: >> Hi Andrew(s), >> >> On 12/04/2015 21:47, Andrew Lunn wrote: >>> On Sun, Apr 12, 2015 at 06:41:31PM +0300, Andrew wrote: >>>> Andrew Lunn ?? 12.04.2015 17:5

Re: Fwd: Re: [PATCH 2/2] ARM: mvebu: dts: Add dts file for DLink DNS-327L

2015-04-20 Thread Gregory CLEMENT
On 20/04/2015 17:15, Andrew wrote: > Gregory CLEMENT писал 20.04.2015 18:04: >> Hi Andrew, >> >> On 13/04/2015 16:32, Andrew wrote: >>> Gregory CLEMENT писал 13.04.2015 17:16: >>>> Hi Andrew(s), >>>> >>>> On 12/04/2015 21:47,

Re: [PATCH 0/2] crypto: add new driver for Marvell CESA

2015-04-13 Thread Gregory CLEMENT
iver on the "old" marvell SoCs (especially kirkwood and dove). If we go to this path, then the best solution would be to still update all the the dts, and modifying the old driver to be able to use the new binding: for my point of view the only adaptation should be related to the SRAM.

Re: Fwd: Re: [PATCH 2/2] ARM: mvebu: dts: Add dts file for DLink DNS-327L

2015-04-13 Thread Gregory CLEMENT
a370_idle_driver. Keep the > ARM_CPUIDLE_WFI_STATE but disable the "Deep Idle" state. Actually you don't have to modify the kernel you can setup the CPU idle level you want at runtime: echo 1 > /sys/devices/system/cpu/cpu0/cpuidle/state1/disable will disable the state1. By do

Re: [PATCH 0/2] ARM: mvebu: Add DLINK-DNS327L support

2015-04-13 Thread Gregory CLEMENT
if it was a typo on their datasheet. Thanks, Gregory > >Andrew > -- 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 "unsubscr

Re: [PATCH v5 1/1] ARM: mvebu: dts: Add dts file for DLink DNS-327L

2015-05-27 Thread Gregory CLEMENT
regulator-max-microvolt = <500>; >> +startup-delay-us = <400>; >> +enable-active-high; >> +regulator-always-on; >> +regulator-boot-on; >> + gpio = <&gpio1 24 GPIO

Re: [PATCH 3/3] ARM: mvebu: use improved armada spi device tree compatible name for each SoC

2015-05-27 Thread Gregory CLEMENT
Hi, On 26/05/2015 11:44, Gregory CLEMENT wrote: > Use the new compatible introduced in rder to beneficiate of a wider > and more accurate range of baud rates to be used. > As the driver part was applied, I applied this patch on mvebu/dt Thanks, Gregory > Signed-off-by: Gre

Re: [PATCH 2/3] spi: orion: Fix extended baud rates for each Armada SoCs

2015-05-27 Thread Gregory CLEMENT
Hi Mark, On 27/05/2015 19:45, Mark Brown wrote: > On Tue, May 26, 2015 at 02:20:08PM +0200, Gregory CLEMENT wrote: > >> Mark do you want a new patch or do you prefer fixing the typo >> yourself ? it should be > > I fixed it up locally (in general it's easier to

Re: [PATCH] i2c: mv64xxx: remove unreachable signal case handling

2015-06-17 Thread Gregory CLEMENT
bus locked, block: %d\n", >> + drv_data->block); > > And if so, shouldn't that also be always 1 in the output here? > >> mv64xxx_i2c_hw_init(drv_data); >> } >> } else > > Maybe (no

Re: [PATCH] ARM: orion5x: Use vsprintf %pM extension

2015-06-17 Thread Gregory CLEMENT
common.c >> +++ b/arch/arm/mach-orion5x/tsx09-common.c >> @@ -101,9 +101,7 @@ static int __init qnap_tsx09_check_mac_addr(const char >> *addr_str) >> addr[i] = byte; >> } >> >> -printk(KERN_INFO "tsx09: found ethernet mac addr

Re: [PATCH v7 04/14] crypto: add a new driver for Marvell's CESA

2015-06-19 Thread Gregory CLEMENT
vast majority of platforms, this is OK. It's only if you have 4 GB > of RAM or more that it causes problem. And then we switch to > mv_mbus_dram_info_nooverlap() as part of the -rc cycle. > > (2) is probably easier. Herbert, what do you think? > > Thomas > --

Re: [PATCH v2 02/16] arm64: dts: marvell: fix CP110 ICU node size

2018-05-23 Thread Gregory CLEMENT
oller@1e { > compatible = "marvell,cp110-icu"; > - reg = <0x1e 0x10>; > + reg = <0x1e 0x440>; > #interrupt-cells = <3>; > interrupt-contro

[PATCH] pinctrl: armada-37xx: Fix spurious irq management

2018-05-23 Thread Gregory CLEMENT
: 30ac0d3b0702 ("pinctrl: armada-37xx: Add edge both type gpio irq support") Cc: Signed-off-by: Terry Zhou Reviewed-by: Gregory CLEMENT Signed-off-by: Gregory CLEMENT --- drivers/pinctrl/mvebu/pinctrl-armada-37xx.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/pin

[PATCH] clk: mvebu: cp110: Fix clock tree representation

2018-02-28 Thread Gregory CLEMENT
than the EIP ones. Signed-off-by: Gregory CLEMENT --- drivers/clk/mvebu/cp110-system-controller.c | 94 - 1 file changed, 39 insertions(+), 55 deletions(-) diff --git a/drivers/clk/mvebu/cp110-system-controller.c b/drivers/clk/mvebu/cp110-system-controller.c index

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