[PATCH 0/2] ARM: dts: BCM5301X: bugfixes

2017-02-08 Thread Jon Mason
Bug fixes to get bcm953012k working again Jon Mason (2): ARM: dts: BCM5301X: Fix UARTs on bcm953012k ARM: dts: BCM5301X: Fix memory start address arch/arm/boot/dts/bcm953012k.dts | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) -- 2.7.4

[PATCH 0/2] ARM: dts: BCM5301X: bugfixes

2017-02-08 Thread Jon Mason
Bug fixes to get bcm953012k working again Jon Mason (2): ARM: dts: BCM5301X: Fix UARTs on bcm953012k ARM: dts: BCM5301X: Fix memory start address arch/arm/boot/dts/bcm953012k.dts | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) -- 2.7.4

[PATCH net-next v2 2/3] net: ethernet: bgmac: unify code of the same family

2017-02-08 Thread Jon Mason
BCM471X and BCM535X are of the same family (from what I can derive from internal documents). Group them into the case statement together, which results in more code reuse. Also, use existing helper variables to make the code a little more readable too. Signed-off-by: Jon Mason <jon

[PATCH net-next v2 2/3] net: ethernet: bgmac: unify code of the same family

2017-02-08 Thread Jon Mason
BCM471X and BCM535X are of the same family (from what I can derive from internal documents). Group them into the case statement together, which results in more code reuse. Also, use existing helper variables to make the code a little more readable too. Signed-off-by: Jon Mason --- drivers/net

[PATCH net-next v2 1/3] net: ethernet: bgmac: use #defines for MAX size

2017-02-08 Thread Jon Mason
The maximum frame size is really just the standard ethernet frame size and FCS. So use those existing defines to make the code a little more beautiful. Signed-off-by: Jon Mason <jon.ma...@broadcom.com> --- drivers/net/ethernet/broadcom/bgmac.h | 2 +- 1 file changed, 1 insertion(+), 1 de

[PATCH net-next v2 1/3] net: ethernet: bgmac: use #defines for MAX size

2017-02-08 Thread Jon Mason
The maximum frame size is really just the standard ethernet frame size and FCS. So use those existing defines to make the code a little more beautiful. Signed-off-by: Jon Mason --- drivers/net/ethernet/broadcom/bgmac.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers

[PATCH net-next v2 3/3] net: ethernet: bgmac: driver power manangement

2017-02-08 Thread Jon Mason
From: Joey Zhong <zho...@broadcom.com> Implement suspend/resume callbacks in the bgmac driver. This makes sure that we de-initialize and re-initialize the hardware correctly before entering suspend and when resuming. Signed-off-by: Joey Zhong <zho...@broadcom.com> Signed-off-b

[PATCH net-next v2 3/3] net: ethernet: bgmac: driver power manangement

2017-02-08 Thread Jon Mason
From: Joey Zhong Implement suspend/resume callbacks in the bgmac driver. This makes sure that we de-initialize and re-initialize the hardware correctly before entering suspend and when resuming. Signed-off-by: Joey Zhong Signed-off-by: Jon Mason --- drivers/net/ethernet/broadcom/bgmac

[PATCH net-next v2 0/3] net: ethernet: bgmac: PM support and clean-ups

2017-02-08 Thread Jon Mason
Changes in v2: * Reworked the PM patch with Florian's suggestions Add code to support Power Management (only tested on NS2), and add some code clean-ups Joey Zhong (1): net: ethernet: bgmac: driver power manangement Jon Mason (2): net: ethernet: bgmac: use #defines for MAX size net

[PATCH net-next v2 0/3] net: ethernet: bgmac: PM support and clean-ups

2017-02-08 Thread Jon Mason
Changes in v2: * Reworked the PM patch with Florian's suggestions Add code to support Power Management (only tested on NS2), and add some code clean-ups Joey Zhong (1): net: ethernet: bgmac: driver power manangement Jon Mason (2): net: ethernet: bgmac: use #defines for MAX size net

[PATCH 1/2] ARM: dts: BCM5301X: Add NAND entries to bcm953012k

2017-02-08 Thread Jon Mason
Add the NAND entry in the DTS Signed-off-by: Jon Mason <jon.ma...@broadcom.com> --- arch/arm/boot/dts/bcm953012k.dts | 32 1 file changed, 32 insertions(+) diff --git a/arch/arm/boot/dts/bcm953012k.dts b/arch/arm/boot/dts/bcm953012k.dts index ae31a58..c

[PATCH 1/2] ARM: dts: BCM5301X: Add NAND entries to bcm953012k

2017-02-08 Thread Jon Mason
Add the NAND entry in the DTS Signed-off-by: Jon Mason --- arch/arm/boot/dts/bcm953012k.dts | 32 1 file changed, 32 insertions(+) diff --git a/arch/arm/boot/dts/bcm953012k.dts b/arch/arm/boot/dts/bcm953012k.dts index ae31a58..cb673c7 100644 --- a/arch/arm/boot

[PATCH 0/2] ARM: dts: BCM5301X: QSPI and NAND

2017-02-08 Thread Jon Mason
Change the default SPI of the BCM5301X to be the new(-ish) iProc QSPI driver. Testing limited to internal Broadcom systems. So, this might\ need extended testing (though I epxect it to work). Also, Add NAND to bcm953012k Jon Mason (2): ARM: dts: BCM5301X: Add NAND entries to bcm953012k ARM

[PATCH net v3 2/2] net: ethernet: bgmac: mac address change bug

2017-02-08 Thread Jon Mason
From: Hari Vyas <ha...@broadcom.com> ndo_set_mac_address() passes struct sockaddr * as 2nd parameter to bgmac_set_mac_address() but code assumed u8 *. This caused two bytes chopping and the wrong mac address was configured. Signed-off-by: Hari Vyas <ha...@broadcom.com> Signed-off-b

[PATCH 0/2] ARM: dts: BCM5301X: QSPI and NAND

2017-02-08 Thread Jon Mason
Change the default SPI of the BCM5301X to be the new(-ish) iProc QSPI driver. Testing limited to internal Broadcom systems. So, this might\ need extended testing (though I epxect it to work). Also, Add NAND to bcm953012k Jon Mason (2): ARM: dts: BCM5301X: Add NAND entries to bcm953012k ARM

[PATCH net v3 2/2] net: ethernet: bgmac: mac address change bug

2017-02-08 Thread Jon Mason
From: Hari Vyas ndo_set_mac_address() passes struct sockaddr * as 2nd parameter to bgmac_set_mac_address() but code assumed u8 *. This caused two bytes chopping and the wrong mac address was configured. Signed-off-by: Hari Vyas Signed-off-by: Jon Mason Fixes: 4e209001b86 ("bgmac: writ

[PATCH net v3 1/2] net: ethernet: bgmac: init sequence bug

2017-02-08 Thread Jon Mason
it out of reset regardless of whether it was in reset or not). Also, removed unnecessary usleeps (as there is already a read present to flush the IDM writes). Signed-off-by: Zac Schroff <zschr...@broadcom.com> Signed-off-by: Jon Mason <jon.ma...@broadcom.com> Fixes: f6a95a24957 (&q

[PATCH net v3 1/2] net: ethernet: bgmac: init sequence bug

2017-02-08 Thread Jon Mason
it out of reset regardless of whether it was in reset or not). Also, removed unnecessary usleeps (as there is already a read present to flush the IDM writes). Signed-off-by: Zac Schroff Signed-off-by: Jon Mason Fixes: f6a95a24957 ("net: ethernet: bgmac: Add platform device support") --

[PATCH 1/2] ARM: dts: BCM5301X: Fix UARTs on bcm953012k

2017-02-08 Thread Jon Mason
The UARTs are outputting garbage on the console. This is due to a speed issue. We can simply use the clock speed (which is now defined in the DTSI file) and everything works fine. Signed-off-by: Jon Mason <jon.ma...@broadcom.com> Fixes: cdc36b22 ("ARM: dts: enable clock support f

[PATCH 1/2] ARM: dts: BCM5301X: Fix UARTs on bcm953012k

2017-02-08 Thread Jon Mason
The UARTs are outputting garbage on the console. This is due to a speed issue. We can simply use the clock speed (which is now defined in the DTSI file) and everything works fine. Signed-off-by: Jon Mason Fixes: cdc36b22 ("ARM: dts: enable clock support for BCM5301X") --- arch/ar

Re: [PATCH 3/3] net: ethernet: bgmac: driver power manangement

2017-02-06 Thread Jon Mason
On Fri, Feb 3, 2017 at 9:16 PM, Florian Fainelli <f.faine...@gmail.com> wrote: > On 02/03/2017 01:39 PM, Jon Mason wrote: >> From: Joey Zhong <zho...@broadcom.com> >> >> Implements suspend/resume, external phy 54810 is assumed >> to remain powered up during

Re: [PATCH 3/3] net: ethernet: bgmac: driver power manangement

2017-02-06 Thread Jon Mason
On Fri, Feb 3, 2017 at 9:16 PM, Florian Fainelli wrote: > On 02/03/2017 01:39 PM, Jon Mason wrote: >> From: Joey Zhong >> >> Implements suspend/resume, external phy 54810 is assumed >> to remain powered up during deep-sleep for wake-on-lane. > > s/wake-on-l

Re: [PATCH v2 1/2] net: ethernet: bgmac: init sequence bug

2017-02-03 Thread Jon Mason
On Fri, Feb 3, 2017 at 4:41 PM, Rafał Miłecki <ra...@milecki.pl> wrote: > On 02/03/2017 10:08 PM, Jon Mason wrote: >> >> @@ -61,15 +60,20 @@ static bool platform_bgmac_clk_enabled(struct bgmac >> *bgmac) >> >> static void platform_bgmac_cl

Re: [PATCH v2 1/2] net: ethernet: bgmac: init sequence bug

2017-02-03 Thread Jon Mason
On Fri, Feb 3, 2017 at 4:41 PM, Rafał Miłecki wrote: > On 02/03/2017 10:08 PM, Jon Mason wrote: >> >> @@ -61,15 +60,20 @@ static bool platform_bgmac_clk_enabled(struct bgmac >> *bgmac) >> >> static void platform_bgmac_clk_enable(struct bgmac *bgmac, u32 flags)

Re: [PATCH 2/3] net: ethernet: bgmac: unify code of the same family

2017-02-03 Thread Jon Mason
On Fri, Feb 3, 2017 at 4:48 PM, Rafał Miłecki <ra...@milecki.pl> wrote: > On 2017-02-03 22:39, Jon Mason wrote: >> >> BCM471X and BCM535X are of the same family (from what I can derive from >> internal documents). Group them into the case statement together, which &g

Re: [PATCH 2/3] net: ethernet: bgmac: unify code of the same family

2017-02-03 Thread Jon Mason
On Fri, Feb 3, 2017 at 4:48 PM, Rafał Miłecki wrote: > On 2017-02-03 22:39, Jon Mason wrote: >> >> BCM471X and BCM535X are of the same family (from what I can derive from >> internal documents). Group them into the case statement together, which >> results in more

[PATCH 2/3] net: ethernet: bgmac: unify code of the same family

2017-02-03 Thread Jon Mason
BCM471X and BCM535X are of the same family (from what I can derive from internal documents). Group them into the case statement together, which results in more code reuse. Also, use existing helper variables to make the code a little more readable too. Signed-off-by: Jon Mason <jon

[PATCH 2/3] net: ethernet: bgmac: unify code of the same family

2017-02-03 Thread Jon Mason
BCM471X and BCM535X are of the same family (from what I can derive from internal documents). Group them into the case statement together, which results in more code reuse. Also, use existing helper variables to make the code a little more readable too. Signed-off-by: Jon Mason --- drivers/net

[PATCH 3/3] net: ethernet: bgmac: driver power manangement

2017-02-03 Thread Jon Mason
From: Joey Zhong <zho...@broadcom.com> Implements suspend/resume, external phy 54810 is assumed to remain powered up during deep-sleep for wake-on-lane. Signed-off-by: Joey Zhong <zho...@broadcom.com> Signed-off-by: Jon Mason <jon.ma...@broadcom.com> --- drivers/net/ether

[PATCH net-next 0/3] net: ethernet: bgmac: PM support and clean-ups

2017-02-03 Thread Jon Mason
Add code to support Power Management (only tested on NS2), and add some code clean-ups Joey Zhong (1): net: ethernet: bgmac: driver power manangement Jon Mason (2): net: ethernet: bgmac: use #defines for MAX size net: ethernet: bgmac: unify code of the same family drivers/net/ethernet

[PATCH net-next 0/3] net: ethernet: bgmac: PM support and clean-ups

2017-02-03 Thread Jon Mason
Add code to support Power Management (only tested on NS2), and add some code clean-ups Joey Zhong (1): net: ethernet: bgmac: driver power manangement Jon Mason (2): net: ethernet: bgmac: use #defines for MAX size net: ethernet: bgmac: unify code of the same family drivers/net/ethernet

[PATCH 3/3] net: ethernet: bgmac: driver power manangement

2017-02-03 Thread Jon Mason
From: Joey Zhong Implements suspend/resume, external phy 54810 is assumed to remain powered up during deep-sleep for wake-on-lane. Signed-off-by: Joey Zhong Signed-off-by: Jon Mason --- drivers/net/ethernet/broadcom/bgmac-platform.c | 34 + drivers/net/ethernet/broadcom

[PATCH 1/3] net: ethernet: bgmac: use #defines for MAX size

2017-02-03 Thread Jon Mason
The maximum frame size is really just the standard ethernet frame size and FCS. So use those existing defines to make the code a little more beautiful. Signed-off-by: Jon Mason <jon.ma...@broadcom.com> --- drivers/net/ethernet/broadcom/bgmac.h | 2 +- 1 file changed, 1 insertion(+), 1 de

[PATCH 1/3] net: ethernet: bgmac: use #defines for MAX size

2017-02-03 Thread Jon Mason
The maximum frame size is really just the standard ethernet frame size and FCS. So use those existing defines to make the code a little more beautiful. Signed-off-by: Jon Mason --- drivers/net/ethernet/broadcom/bgmac.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers

[PATCH v2 1/2] net: ethernet: bgmac: init sequence bug

2017-02-03 Thread Jon Mason
From: Zac Schroff <zschr...@broadcom.com> Fix a bug in the 'bgmac' driver init sequence that blind writes for init sequence where it should preserve most bits other than the ones it is deliberately manipulating. Signed-off-by: Zac Schroff <zschr...@broadcom.com> Signed-off-by: Jon M

[PATCH v2 1/2] net: ethernet: bgmac: init sequence bug

2017-02-03 Thread Jon Mason
From: Zac Schroff Fix a bug in the 'bgmac' driver init sequence that blind writes for init sequence where it should preserve most bits other than the ones it is deliberately manipulating. Signed-off-by: Zac Schroff Signed-off-by: Jon Mason Fixes: f6a95a24957 ("net: ethernet: bgmac

[PATCH v2 0/2] net: ethernet: bgmac: bug fixes

2017-02-03 Thread Jon Mason
Changes in v2: * Reworked the first match to make it more obvious what portions of the register were being preserved (Per Rafal Mileki) * Style change to reorder the function variables in patch 2 (per Sergei Shtylyov) Bug fixes for bgmac driver Hari Vyas (1): net: ethernet: bgmac: mac

[PATCH v2 0/2] net: ethernet: bgmac: bug fixes

2017-02-03 Thread Jon Mason
Changes in v2: * Reworked the first match to make it more obvious what portions of the register were being preserved (Per Rafal Mileki) * Style change to reorder the function variables in patch 2 (per Sergei Shtylyov) Bug fixes for bgmac driver Hari Vyas (1): net: ethernet: bgmac: mac

[PATCH v2 2/2] net: ethernet: bgmac: mac address change bug

2017-02-03 Thread Jon Mason
From: Hari Vyas <ha...@broadcom.com> ndo_set_mac_address() passes struct sockaddr * as 2nd parameter to bgmac_set_mac_address() but code assumed u8 *. This caused two bytes chopping and the wrong mac address was configured. Signed-off-by: Hari Vyas <ha...@broadcom.com> Signed-off-b

[PATCH v2 2/2] net: ethernet: bgmac: mac address change bug

2017-02-03 Thread Jon Mason
From: Hari Vyas ndo_set_mac_address() passes struct sockaddr * as 2nd parameter to bgmac_set_mac_address() but code assumed u8 *. This caused two bytes chopping and the wrong mac address was configured. Signed-off-by: Hari Vyas Signed-off-by: Jon Mason Fixes: 4e209001b86 ("bgmac: writ

Re: [PATCH v2 3/4] phy: Add USB3 PHY support for Broadcom NSP SoC

2017-02-03 Thread Jon Mason
; we already have phy-bcm-ns-usb3.c! In fact Jon told me to rewrite my initial > driver to make is possible to reuse it on NSP and I did that! > > This is old comment from Jon: > > In 30 March 2016 at 23:31, Jon Mason <jon.ma...@broadcom.com> wrote: >> On Mon, Mar 28, 201

Re: [PATCH v2 3/4] phy: Add USB3 PHY support for Broadcom NSP SoC

2017-02-03 Thread Jon Mason
is possible to reuse it on NSP and I did that! > > This is old comment from Jon: > > In 30 March 2016 at 23:31, Jon Mason wrote: >> On Mon, Mar 28, 2016 at 9:46 PM, Florian Fainelli >> wrote: >>> >>> CC: bcm-kernel-feedback-list, Jon >> >> >&g

Re: [PATCH 1/2] net: ethernet: bgmac: init sequence bug

2017-02-02 Thread Jon Mason
On Thu, Feb 2, 2017 at 3:15 PM, Rafał Miłecki wrote: > On 2017-02-02 01:31, Zac Schroff wrote: >> >> How about BCMA_IOCTL_PRESERVE_ACROSS_INIT? > > > I think wireless drivers may still set some these bits during init. > > I've a simpler idea: make it bgmac specific. Call it sth

Re: [PATCH 1/2] net: ethernet: bgmac: init sequence bug

2017-02-02 Thread Jon Mason
On Thu, Feb 2, 2017 at 3:15 PM, Rafał Miłecki wrote: > On 2017-02-02 01:31, Zac Schroff wrote: >> >> How about BCMA_IOCTL_PRESERVE_ACROSS_INIT? > > > I think wireless drivers may still set some these bits during init. > > I've a simpler idea: make it bgmac specific. Call it sth like >

Re: [PATCH 1/2] net: ethernet: bgmac: init sequence bug

2017-02-02 Thread Jon Mason
On Wed, Feb 1, 2017 at 6:06 PM, Rafał Miłecki <ra...@milecki.pl> wrote: > On 02/01/2017 11:39 PM, Jon Mason wrote: >> >> From: Zac Schroff <zschr...@broadcom.com> >> >> Fix a bug in the 'bgmac' driver init sequence that blind writes for init >> sequ

Re: [PATCH 1/2] net: ethernet: bgmac: init sequence bug

2017-02-02 Thread Jon Mason
On Wed, Feb 1, 2017 at 6:06 PM, Rafał Miłecki wrote: > On 02/01/2017 11:39 PM, Jon Mason wrote: >> >> From: Zac Schroff >> >> Fix a bug in the 'bgmac' driver init sequence that blind writes for init >> sequence where it should preserve most bits other th

Re: [PATCH 2/2] net: ethernet: bgmac: mac address change bug

2017-02-02 Thread Jon Mason
On Wed, Feb 1, 2017 at 6:12 PM, Rafał Miłecki <ra...@milecki.pl> wrote: > On 02/01/2017 11:39 PM, Jon Mason wrote: >> >> From: Hari Vyas <ha...@broadcom.com> >> >> ndo_set_mac_address() passes struct sockaddr * as 2nd parameter to >> bgmac_set_mac_addr

Re: [PATCH 2/2] net: ethernet: bgmac: mac address change bug

2017-02-02 Thread Jon Mason
On Wed, Feb 1, 2017 at 6:12 PM, Rafał Miłecki wrote: > On 02/01/2017 11:39 PM, Jon Mason wrote: >> >> From: Hari Vyas >> >> ndo_set_mac_address() passes struct sockaddr * as 2nd parameter to >> bgmac_set_mac_address() but code assumed u8 *. This caused two by

[PATCH 2/2] net: ethernet: bgmac: mac address change bug

2017-02-01 Thread Jon Mason
From: Hari Vyas <ha...@broadcom.com> ndo_set_mac_address() passes struct sockaddr * as 2nd parameter to bgmac_set_mac_address() but code assumed u8 *. This caused two bytes chopping and the wrong mac address was configured. Signed-off-by: Hari Vyas <ha...@broadcom.com> Signed-off-b

[PATCH 1/2] net: ethernet: bgmac: init sequence bug

2017-02-01 Thread Jon Mason
From: Zac Schroff <zschr...@broadcom.com> Fix a bug in the 'bgmac' driver init sequence that blind writes for init sequence where it should preserve most bits other than the ones it is deliberately manipulating. Signed-off-by: Zac Schroff <zschr...@broadcom.com> Signed-off-by: Jon M

[PATCH 2/2] net: ethernet: bgmac: mac address change bug

2017-02-01 Thread Jon Mason
From: Hari Vyas ndo_set_mac_address() passes struct sockaddr * as 2nd parameter to bgmac_set_mac_address() but code assumed u8 *. This caused two bytes chopping and the wrong mac address was configured. Signed-off-by: Hari Vyas Signed-off-by: Jon Mason Fixes: 4e209001b86 ("bgmac: writ

[PATCH 1/2] net: ethernet: bgmac: init sequence bug

2017-02-01 Thread Jon Mason
From: Zac Schroff Fix a bug in the 'bgmac' driver init sequence that blind writes for init sequence where it should preserve most bits other than the ones it is deliberately manipulating. Signed-off-by: Zac Schroff Signed-off-by: Jon Mason Fixes: f6a95a24957 ("net: ethernet: bgmac

[PATCH 0/2] net: ethernet: bgmac: bug fixes

2017-02-01 Thread Jon Mason
Bug fixes for bgmac driver Hari Vyas (1): net: ethernet: bgmac: mac address change bug Zac Schroff (1): net: ethernet: bgmac: init sequence bug drivers/net/ethernet/broadcom/bgmac-platform.c | 10 +++--- drivers/net/ethernet/broadcom/bgmac.c | 6 +-

[PATCH 0/2] net: ethernet: bgmac: bug fixes

2017-02-01 Thread Jon Mason
Bug fixes for bgmac driver Hari Vyas (1): net: ethernet: bgmac: mac address change bug Zac Schroff (1): net: ethernet: bgmac: init sequence bug drivers/net/ethernet/broadcom/bgmac-platform.c | 10 +++--- drivers/net/ethernet/broadcom/bgmac.c | 6 +-

Re: [PATCH v2 0/4] Style fixes: open code obfuscating macros

2017-02-01 Thread Jon Mason
On Thu, Jan 26, 2017 at 3:00 PM, Logan Gunthorpe wrote: > Hi, > > It's been a couple weeks... Any thoughts on this? My apologies for not responding sooner. A large rework of the core NTB code was done prior to your patches (for IDT NTB support). Unfortunately, after those

Re: [PATCH v2 0/4] Style fixes: open code obfuscating macros

2017-02-01 Thread Jon Mason
On Thu, Jan 26, 2017 at 3:00 PM, Logan Gunthorpe wrote: > Hi, > > It's been a couple weeks... Any thoughts on this? My apologies for not responding sooner. A large rework of the core NTB code was done prior to your patches (for IDT NTB support). Unfortunately, after those changes this series

Re: [PATCH v3 5/9] NTB: Alter Scratchpads API to support multi-ports devices

2017-02-01 Thread Jon Mason
On Tue, Jan 10, 2017 at 7:13 PM, Serge Semin wrote: > Even though there is no any real NTB hardware, which would have both more > than two ports and Scratchpad registers, it is logically correct to have > Scratchpad API accepting a peer port index as well. Intel/AMD

Re: [PATCH v3 5/9] NTB: Alter Scratchpads API to support multi-ports devices

2017-02-01 Thread Jon Mason
On Tue, Jan 10, 2017 at 7:13 PM, Serge Semin wrote: > Even though there is no any real NTB hardware, which would have both more > than two ports and Scratchpad registers, it is logically correct to have > Scratchpad API accepting a peer port index as well. Intel/AMD drivers utilize > Primary and

[PATCH 1/2] PCI: iproc: allow the MPS to be set

2017-01-27 Thread Jon Mason
pcie_bus_configure_settings() needs to be called on each of the root ports to allow for the MPS to be configured for all of the children devices. Signed-off-by: Jon Mason <jon.ma...@broadcom.com> --- drivers/pci/host/pcie-iproc.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-)

[PATCH 1/2] PCI: iproc: allow the MPS to be set

2017-01-27 Thread Jon Mason
pcie_bus_configure_settings() needs to be called on each of the root ports to allow for the MPS to be configured for all of the children devices. Signed-off-by: Jon Mason --- drivers/pci/host/pcie-iproc.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/drivers/pci/host

[PATCH 2/2] PCI: Add quirk entry for NS2 PAXC PCI

2017-01-27 Thread Jon Mason
fixed" in device firmware, so we must fix them with a quirk. Signed-off-by: Jon Mason <jon.ma...@broadcom.com> --- drivers/pci/quirks.c | 21 + 1 file changed, 21 insertions(+) diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c index 1800bef..c5ef20a 100644 --- a/driv

[PATCH 2/2] PCI: Add quirk entry for NS2 PAXC PCI

2017-01-27 Thread Jon Mason
fixed" in device firmware, so we must fix them with a quirk. Signed-off-by: Jon Mason --- drivers/pci/quirks.c | 21 + 1 file changed, 21 insertions(+) diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c index 1800bef..c5ef20a 100644 --- a/drivers/pci/quirks.c +++ b/d

[PATCH 0/2] PCI: iproc: MPS fix and quirks

2017-01-27 Thread Jon Mason
A patch that corrects an issue where the MPS could not be set, and another that fixes some quirks in the PAXC Jon Mason (2): PCI: iproc: allow the MPS to be set PCI: Add quirk entry for NS2 PAXC PCI drivers/pci/host/pcie-iproc.c | 5 - drivers/pci/quirks.c | 21

[PATCH 0/2] PCI: iproc: MPS fix and quirks

2017-01-27 Thread Jon Mason
A patch that corrects an issue where the MPS could not be set, and another that fixes some quirks in the PAXC Jon Mason (2): PCI: iproc: allow the MPS to be set PCI: Add quirk entry for NS2 PAXC PCI drivers/pci/host/pcie-iproc.c | 5 - drivers/pci/quirks.c | 21

[PATCH] ARM: dts: NSP: Fix DT ranges error

2017-01-12 Thread Jon Mason
The range size for axi is 0x2 bytes too small, as the QSPI needs 0x11c408 + 0x004 (which is 0x0011c40c, not 0x0011c40a). No errors have been observed with this shortcoming, but fixing it for correctness. Signed-off-by: Jon Mason <jon.ma...@broadcom.com> --- arch/arm/boot/dts/bcm-nsp.dt

[PATCH] ARM: dts: NSP: Fix DT ranges error

2017-01-12 Thread Jon Mason
The range size for axi is 0x2 bytes too small, as the QSPI needs 0x11c408 + 0x004 (which is 0x0011c40c, not 0x0011c40a). No errors have been observed with this shortcoming, but fixing it for correctness. Signed-off-by: Jon Mason --- arch/arm/boot/dts/bcm-nsp.dtsi | 2 +- 1 file changed, 1

Re: [PATCH 0/2] Style fixes: open code obfuscating macros

2017-01-10 Thread Jon Mason
On Tue, Jan 10, 2017 at 4:55 PM, Jon Mason <jdma...@kudzu.us> wrote: > On Mon, Jan 02, 2017 at 06:30:34PM -0700, Logan Gunthorpe wrote: >> Hi, >> >> I had copied some poor code style from the NTB drivers into an unrelated >> driver. Upon review of my new code,

Re: [PATCH 0/2] Style fixes: open code obfuscating macros

2017-01-10 Thread Jon Mason
On Tue, Jan 10, 2017 at 4:55 PM, Jon Mason wrote: > On Mon, Jan 02, 2017 at 06:30:34PM -0700, Logan Gunthorpe wrote: >> Hi, >> >> I had copied some poor code style from the NTB drivers into an unrelated >> driver. Upon review of my new code, I learned it was not a

Re: [PATCH 0/2] Style fixes: open code obfuscating macros

2017-01-10 Thread Jon Mason
On Mon, Jan 02, 2017 at 06:30:34PM -0700, Logan Gunthorpe wrote: > Hi, > > I had copied some poor code style from the NTB drivers into an unrelated > driver. Upon review of my new code, I learned it was not a good idea > to sweep dirty things under the rug^W macro. See [1], where Gregg k-h >

Re: [PATCH 0/2] Style fixes: open code obfuscating macros

2017-01-10 Thread Jon Mason
On Mon, Jan 02, 2017 at 06:30:34PM -0700, Logan Gunthorpe wrote: > Hi, > > I had copied some poor code style from the NTB drivers into an unrelated > driver. Upon review of my new code, I learned it was not a good idea > to sweep dirty things under the rug^W macro. See [1], where Gregg k-h >

[GIT PULL] NTB bug fixes for v4.10

2016-12-24 Thread Jon Mason
Hello Linus, Here are a few NTB bug fixes, new HE support, and updates to AMD NTB for 4.10. Please consider pulling them. Thanks, Jon The following changes since commit 69973b830859bc6529a7a0468ba0d80ee5117826: Linux 4.9 (2016-12-11 11:17:54 -0800) are available in the git repository at:

[GIT PULL] NTB bug fixes for v4.10

2016-12-24 Thread Jon Mason
Hello Linus, Here are a few NTB bug fixes, new HE support, and updates to AMD NTB for 4.10. Please consider pulling them. Thanks, Jon The following changes since commit 69973b830859bc6529a7a0468ba0d80ee5117826: Linux 4.9 (2016-12-11 11:17:54 -0800) are available in the git repository at:

Re: [PATCH] NTB: Fix 'request_irq()' and 'free_irq()' inconsistancy

2016-12-21 Thread Jon Mason
On Mon, Dec 19, 2016 at 10:04:24AM -0700, Dave Jiang wrote: > > > On 12/18/2016 10:52 PM, Christophe JAILLET wrote: > > 'request_irq()' and 'free_irq()' should have the same 'dev_id'. > > > > Signed-off-by: Christophe JAILLET > > Acking the Intel driver portion.

Re: [PATCH] NTB: Fix 'request_irq()' and 'free_irq()' inconsistancy

2016-12-21 Thread Jon Mason
On Mon, Dec 19, 2016 at 10:04:24AM -0700, Dave Jiang wrote: > > > On 12/18/2016 10:52 PM, Christophe JAILLET wrote: > > 'request_irq()' and 'free_irq()' should have the same 'dev_id'. > > > > Signed-off-by: Christophe JAILLET > > Acking the Intel driver portion. > Acked-by: Dave Jiang

[PATCH 3/7] ARM: dts: NSP: Add QSPI support to missing boards

2016-12-13 Thread Jon Mason
QSPI device tree entries are present in bcm958625k, but missing from bcm958522er, bcm958525er, bcm958525xmc, bcm958622hr, bcm958623hr, bcm958625hr, and bcm988312hr. Duplicate the entry in bcm958625k for all of those that are missing it (as they are identical). Signed-off-by: Jon Mason <jon

[PATCH 5/7] ARM: dts: NSP: Add and enable amac2

2016-12-13 Thread Jon Mason
Add and enable the third AMAC ethernet interface in the device trees for the platforms where it is present. Also, enable amac1 on some of the platforms where that was missing. Signed-off-by: Jon Mason <jon.ma...@broadcom.com> --- arch/arm/boot/dts/bcm-nsp.dtsi| 9 + arch/ar

[PATCH 3/7] ARM: dts: NSP: Add QSPI support to missing boards

2016-12-13 Thread Jon Mason
QSPI device tree entries are present in bcm958625k, but missing from bcm958522er, bcm958525er, bcm958525xmc, bcm958622hr, bcm958623hr, bcm958625hr, and bcm988312hr. Duplicate the entry in bcm958625k for all of those that are missing it (as they are identical). Signed-off-by: Jon Mason --- arch

[PATCH 5/7] ARM: dts: NSP: Add and enable amac2

2016-12-13 Thread Jon Mason
Add and enable the third AMAC ethernet interface in the device trees for the platforms where it is present. Also, enable amac1 on some of the platforms where that was missing. Signed-off-by: Jon Mason --- arch/arm/boot/dts/bcm-nsp.dtsi| 9 + arch/arm/boot/dts/bcm958622hr.dts | 8

[PATCH 0/7] ARM: dts: NSP: DT Clean-ups and more

2016-12-13 Thread Jon Mason
The Northstar Plus device tree files have gotten messy and out of sync. The first 5 patches correct this, the next 2 add support for new things. Jon Mason (7): ARM: dts: NSP: DT Clean-ups ARM: dts: NSP: Correct NAND partition unit address ARM: dts: NSP: Add QSPI support to missing boards

[PATCH 0/7] ARM: dts: NSP: DT Clean-ups and more

2016-12-13 Thread Jon Mason
The Northstar Plus device tree files have gotten messy and out of sync. The first 5 patches correct this, the next 2 add support for new things. Jon Mason (7): ARM: dts: NSP: DT Clean-ups ARM: dts: NSP: Correct NAND partition unit address ARM: dts: NSP: Add QSPI support to missing boards

[PATCH 6/7] ARM: dts: NSP: Add Ethernet to NSP XMC

2016-12-13 Thread Jon Mason
Enable the ethernet in the NSP XMC (bcm958525xmc) device tree Signed-off-by: Jon Mason <jon.ma...@broadcom.com> --- arch/arm/boot/dts/bcm958525xmc.dts | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/arch/arm/boot/dts/bcm958525xmc.dts b/arch/arm/boot/dts/bcm958525x

[PATCH 6/7] ARM: dts: NSP: Add Ethernet to NSP XMC

2016-12-13 Thread Jon Mason
Enable the ethernet in the NSP XMC (bcm958525xmc) device tree Signed-off-by: Jon Mason --- arch/arm/boot/dts/bcm958525xmc.dts | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/arch/arm/boot/dts/bcm958525xmc.dts b/arch/arm/boot/dts/bcm958525xmc.dts index 89d9abc..3912269

[PATCH 7/7] ARM: dts: NSP: Add SD/MMC support

2016-12-13 Thread Jon Mason
Add SD/MMC support to the Broadcom NSP SVK and XMC. Signed-off-by: Jon Mason <jon.ma...@broadcom.com> --- arch/arm/boot/dts/bcm-nsp.dtsi | 9 +++ arch/arm/boot/dts/bcm958525xmc.dts | 6 +- arch/arm/boot/dts/bcm958625k.dts | 118 - 3 files chang

[PATCH 4/7] ARM: dts: NSP: Add BCM958625K switch ports

2016-12-13 Thread Jon Mason
Add the layout of the switch ports found on the BCM958625K reference board. The CPU port is hooked up to the AMAC0 Ethernet controller adapter. Signed-off-by: Jon Mason <jon.ma...@broadcom.com> --- arch/arm/boot/dts/bcm958625k.dts | 45 1 file chang

[PATCH 7/7] ARM: dts: NSP: Add SD/MMC support

2016-12-13 Thread Jon Mason
Add SD/MMC support to the Broadcom NSP SVK and XMC. Signed-off-by: Jon Mason --- arch/arm/boot/dts/bcm-nsp.dtsi | 9 +++ arch/arm/boot/dts/bcm958525xmc.dts | 6 +- arch/arm/boot/dts/bcm958625k.dts | 118 - 3 files changed, 90 insertions(+), 43

[PATCH 4/7] ARM: dts: NSP: Add BCM958625K switch ports

2016-12-13 Thread Jon Mason
Add the layout of the switch ports found on the BCM958625K reference board. The CPU port is hooked up to the AMAC0 Ethernet controller adapter. Signed-off-by: Jon Mason --- arch/arm/boot/dts/bcm958625k.dts | 45 1 file changed, 45 insertions(+) diff

[PATCH 2/7] ARM: dts: NSP: Correct NAND partition unit address

2016-12-13 Thread Jon Mason
The NAND partition unit address does not match the other NSP device tree files. This change makes them uniform. Signed-off-by: Jon Mason <jon.ma...@broadcom.com> --- arch/arm/boot/dts/bcm958625k.dts | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/arch/arm/bo

[PATCH 1/7] ARM: dts: NSP: DT Clean-ups

2016-12-13 Thread Jon Mason
to make it match. Finally, remove errant new lines. Signed-off-by: Jon Mason <jon.ma...@broadcom.com> --- arch/arm/boot/dts/bcm-nsp.dtsi | 46 +-- arch/arm/boot/dts/bcm958522er.dts | 1 - arch/arm/boot/dts/bcm958525er.dts | 1 - arch/arm/boot/dts/bcm958525xmc.dt

[PATCH 2/7] ARM: dts: NSP: Correct NAND partition unit address

2016-12-13 Thread Jon Mason
The NAND partition unit address does not match the other NSP device tree files. This change makes them uniform. Signed-off-by: Jon Mason --- arch/arm/boot/dts/bcm958625k.dts | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/arch/arm/boot/dts/bcm958625k.dts b/arch/arm

[PATCH 1/7] ARM: dts: NSP: DT Clean-ups

2016-12-13 Thread Jon Mason
to make it match. Finally, remove errant new lines. Signed-off-by: Jon Mason --- arch/arm/boot/dts/bcm-nsp.dtsi | 46 +-- arch/arm/boot/dts/bcm958522er.dts | 1 - arch/arm/boot/dts/bcm958525er.dts | 1 - arch/arm/boot/dts/bcm958525xmc.dts | 26 arch

Re: [PATCH V2 4/5] ARM: BCM5301X: Specify all RAM by including an extra block

2016-12-07 Thread Jon Mason
> > So it seems we're forced to access first 128 MiB using alias at 0x0 and > the rest using real base address + 128 MiB offset which is 0x8800. > > Signed-off-by: Rafał Miłecki <ra...@milecki.pl> Acked-by: Jon Mason <jon.ma...@broadcom.com> > --- > V2: Updat

Re: [PATCH V2 4/5] ARM: BCM5301X: Specify all RAM by including an extra block

2016-12-07 Thread Jon Mason
re forced to access first 128 MiB using alias at 0x0 and > the rest using real base address + 128 MiB offset which is 0x8800. > > Signed-off-by: Rafał Miłecki Acked-by: Jon Mason > --- > V2: Updated commit message, thanks Jon! > Included XWR-3100 > --- > arch/ar

[RFC] PCI: Change default MPS behavior

2016-12-06 Thread Jon Mason
, change the default value of pcie_bus_config to be PCIE_BUS_SAFE. This configures all systems to use better values for the MPS, at the expense of any potential HW errata that might not like being changed. Signed-off-by: Jon Mason <jon.ma...@broadcom.com> --- drivers/pci/pci.c | 2 +- 1 file chan

[RFC] PCI: Change default MPS behavior

2016-12-06 Thread Jon Mason
, change the default value of pcie_bus_config to be PCIE_BUS_SAFE. This configures all systems to use better values for the MPS, at the expense of any potential HW errata that might not like being changed. Signed-off-by: Jon Mason --- drivers/pci/pci.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion

Re: [PATCH 4/5] ARM: BCM5301X: Specify all RAM by including extra block

2016-12-06 Thread Jon Mason
On Tue, Dec 06, 2016 at 09:57:31PM +0100, Rafał Miłecki wrote: > On 6 December 2016 at 21:06, Jon Mason <jon.ma...@broadcom.com> wrote: > > On Tue, Dec 06, 2016 at 06:17:13PM +0100, Rafał Miłecki wrote: > >> From: Rafał Miłecki <ra...@milecki.pl> > >> > &

Re: [PATCH 4/5] ARM: BCM5301X: Specify all RAM by including extra block

2016-12-06 Thread Jon Mason
On Tue, Dec 06, 2016 at 09:57:31PM +0100, Rafał Miłecki wrote: > On 6 December 2016 at 21:06, Jon Mason wrote: > > On Tue, Dec 06, 2016 at 06:17:13PM +0100, Rafał Miłecki wrote: > >> From: Rafał Miłecki > >> > >> So far we were specifying only the first b

Re: [PATCH 4/5] ARM: BCM5301X: Specify all RAM by including extra block

2016-12-06 Thread Jon Mason
On Tue, Dec 06, 2016 at 06:17:13PM +0100, Rafał Miłecki wrote: > From: Rafał Miłecki > > So far we were specifying only the first block which is always limited > up to 128 MiB. There are many devices with 256 MiB and few with 512 MiB. Assuming that NS is like NSP (and I'm

Re: [PATCH 4/5] ARM: BCM5301X: Specify all RAM by including extra block

2016-12-06 Thread Jon Mason
On Tue, Dec 06, 2016 at 06:17:13PM +0100, Rafał Miłecki wrote: > From: Rafał Miłecki > > So far we were specifying only the first block which is always limited > up to 128 MiB. There are many devices with 256 MiB and few with 512 MiB. Assuming that NS is like NSP (and I'm pretty sure it is),

Re: [PATCH 3/5] ARM: BCM5301X: Set GPIO enabling USB power on Netgear R7000

2016-12-06 Thread Jon Mason
On Tue, Dec 06, 2016 at 06:17:12PM +0100, Rafał Miłecki wrote: > There is one GPIO controlling power for both USB ports. > > Signed-off-by: Rafał Miłecki > Signed-off-by: Rafał Miłecki Was the double Signed-off-by intentional? > --- >

Re: [PATCH 3/5] ARM: BCM5301X: Set GPIO enabling USB power on Netgear R7000

2016-12-06 Thread Jon Mason
On Tue, Dec 06, 2016 at 06:17:12PM +0100, Rafał Miłecki wrote: > There is one GPIO controlling power for both USB ports. > > Signed-off-by: Rafał Miłecki > Signed-off-by: Rafał Miłecki Was the double Signed-off-by intentional? > --- > arch/arm/boot/dts/bcm4709-netgear-r7000.dts | 8

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