[PATCH 2/2] MIPS: BCM47xx: Include bcm47xx_sprom.h

2020-07-25 Thread Florian Fainelli
Now that bcm47xx_sprom.h contains a prototype for bcm47xx_fill_sprom, include that header file directly from bcm47xx.h. Signed-off-by: Florian Fainelli --- arch/mips/include/asm/mach-bcm47xx/bcm47xx.h | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/arch/mips/include/asm

[PATCH 0/2] firmware: bcm47xx_sprom: Fix -Wmissing-prototypes warnings

2020-07-25 Thread Florian Fainelli
Hi Thomas, This patch series fixes W=1 -Wmissing-prototypes warnings for the bcm47xx_sprom.c firmware file. Thanks! Florian Fainelli (2): firmware: bcm47xx_sprom: Fix -Wmissing-prototypes warnings MIPS: BCM47xx: Include bcm47xx_sprom.h arch/mips/include/asm/mach-bcm47xx/bcm47xx.h | 4

[PATCH 1/2] firmware: bcm47xx_sprom: Fix -Wmissing-prototypes warnings

2020-07-25 Thread Florian Fainelli
bcm47xx_sprom.h did not include a prototype for bcm47xx_fill_sprom() therefore add one, and make sure we do include that header to fix -Wmissing-prototypes warnings. Reported-by: kernel test robot Signed-off-by: Florian Fainelli --- drivers/firmware/broadcom/bcm47xx_sprom.c | 1 + include

Re: [RFC 0/7] Add support to process rx packets in thread

2020-07-24 Thread Florian Fainelli
On 7/23/20 11:20 PM, Rakesh Pillai wrote: > > >> -Original Message----- >> From: Florian Fainelli >> Sent: Friday, July 24, 2020 12:33 AM >> To: Rakesh Pillai ; 'Andrew Lunn' >> >> Cc: ath...@lists.infradead.org; linux-wirel...@vger.kernel

Re: [PATCH v4 0/9] bmips: add BCM63xx power domain controller

2020-07-24 Thread Florian Fainelli
On 6/13/20 1:21 AM, Álvaro Fernández Rojas wrote: > BCM6318, BCM6328, BCM6362 and BCM63268 SoCs have a power domain controller > to enable/disable certain components in order to save power. > > v4: Fix device tree bindings documentation. > v3: Introduce changes suggested by Florian: > - Use

[PATCH] soc: bcm: brcmstb: biuctrl: Enable Read-ahead cache

2020-07-24 Thread Florian Fainelli
time configuration. Signed-off-by: Florian Fainelli --- drivers/soc/bcm/brcmstb/biuctrl.c | 87 +++ 1 file changed, 77 insertions(+), 10 deletions(-) diff --git a/drivers/soc/bcm/brcmstb/biuctrl.c b/drivers/soc/bcm/brcmstb/biuctrl.c index 61731e01f94b..95602ece51d4

Re: [PATCH v9 02/12] ata: ahci_brcm: Fix use of BCM7216 reset controller

2020-07-24 Thread Florian Fainelli
On 7/24/20 1:33 PM, Jim Quinlan wrote: > From: Jim Quinlan > > A reset controller "rescal" is shared between the AHCI driver and the PCIe > driver for the BrcmSTB 7216 chip. Use > devm_reset_control_get_optional_shared() to handle this sharing. > > Signed-off-by: Jim Quinlan > > Fixes:

[PATCH] irqchip/irq-bcm7038-l1: Guard uses of cpu_logical_map

2020-07-24 Thread Florian Fainelli
cpu_logical_map is only defined for CONFIG_SMP builds, when we are in an UP configuration, the boot CPU is 0. Reported-by: kernel test robot Fixes: 6468fc18b006 ("irqchip/irq-bcm7038-l1: Add PM support") Signed-off-by: Florian Fainelli --- drivers/irqchip/irq-bcm7038-l1.c | 8 ++

Re: [RFT 12/16] memory: brcmstb_dpfe: Add separate entry for compile test

2020-07-24 Thread Florian Fainelli
drivers/memory/Kconfig > +++ b/drivers/memory/Kconfig > @@ -54,6 +54,17 @@ config ATMEL_EBI > tree is used. This bus supports NANDs, external ethernet controller, > SRAMs, ATA devices, etc. > > +config BRCMSTB_DPFE > + bool "Broadcom STB DPFE driver"

Re: [PATCH 11/16] memory: brcmstb_dpfe: Fix language typo

2020-07-24 Thread Florian Fainelli
On 7/24/20 11:23 AM, Krzysztof Kozlowski wrote: > Fix firwmare -> firmware. > > Signed-off-by: Krzysztof Kozlowski Acked-by: Florian Fainelli -- Florian

Re: [PATCH net-next v3 3/7] net: macb: parse PHY nodes found under an MDIO node

2020-07-24 Thread Florian Fainelli
On 7/24/20 3:50 AM, Codrin Ciubotariu wrote: > The MACB embeds an MDIO bus controller. For this reason, the PHY nodes > were represented as sub-nodes in the MACB node. Generally, the > Ethernet controller is different than the MDIO controller, so the PHYs > are probed by a separate MDIO driver.

Re: drivers/irqchip/irq-bcm7038-l1.c:312:13: error: implicit declaration of function 'cpu_logical_map'

2020-07-23 Thread Florian Fainelli
On 7/23/2020 9:07 PM, kernel test robot wrote: > Hi Justin, > > FYI, the error/warning still remains. > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git > master > head: d15be546031cf65a0fc34879beca02fd90fe7ac7 > commit: 6468fc18b00685c82408f40e9569c0d3527862b8

Re: [PATCH net-next v2 3/7] net: macb: parse PHY nodes found under an MDIO node

2020-07-23 Thread Florian Fainelli
On 7/23/20 3:08 PM, codrin.ciubota...@microchip.com wrote: > On 23.07.2020 21:59, Florian Fainelli wrote: >> EXTERNAL EMAIL: Do not click links or open attachments unless you know the >> content is safe >> >> On 7/21/20 10:13 AM, Codrin Ciubotariu wrote: >>> Th

Re: [RFC 0/7] Add support to process rx packets in thread

2020-07-23 Thread Florian Fainelli
On 7/23/20 11:21 AM, Rakesh Pillai wrote: > > >> -Original Message----- >> From: Florian Fainelli >> Sent: Tuesday, July 21, 2020 11:35 PM >> To: Andrew Lunn ; Rakesh Pillai >> Cc: ath...@lists.infradead.org; linux-wirel...@vger.kernel.org;

Re: [PATCH net-next v2 6/7] ARM: dts: at91: sama5d4: add an mdio sub-node to macb

2020-07-23 Thread Florian Fainelli
On 7/21/20 10:13 AM, Codrin Ciubotariu wrote: > Use the new macb bindings and add an mdio sub-node to contain all the > phy nodes. > > Signed-off-by: Codrin Ciubotariu Reviewed-by: Florian Fainelli -- Florian

Re: [PATCH net-next v2 7/7] ARM: dts: at91: sam9x60: add an mdio sub-node to macb

2020-07-23 Thread Florian Fainelli
On 7/21/20 10:13 AM, Codrin Ciubotariu wrote: > Use the new macb bindings and add an mdio sub-node to contain all the > phy nodes. > > Signed-off-by: Codrin Ciubotariu Reviewed-by: Florian Fainelli -- Florian

Re: [PATCH net-next v2 5/7] ARM: dts: at91: sama5d3: add an mdio sub-node to macb

2020-07-23 Thread Florian Fainelli
On 7/21/20 10:13 AM, Codrin Ciubotariu wrote: > Use the new macb bindings and add an mdio sub-node to contain all the > phy nodes. > > Signed-off-by: Codrin Ciubotariu Reviewed-by: Florian Fainelli -- Florian

Re: [PATCH net-next v2 3/7] net: macb: parse PHY nodes found under an MDIO node

2020-07-23 Thread Florian Fainelli
On 7/21/20 10:13 AM, Codrin Ciubotariu wrote: > The MACB embeds an MDIO bus controller. For this reason, the PHY nodes > were represented as sub-nodes in the MACB node. Generally, the > Ethernet controller is different than the MDIO controller, so the PHYs > are probed by a separate MDIO driver.

Re: [PATCH net-next v2 4/7] ARM: dts: at91: sama5d2: add an mdio sub-node to macb

2020-07-23 Thread Florian Fainelli
On 7/21/20 10:13 AM, Codrin Ciubotariu wrote: > Use the new macb bindings and add an mdio sub-node to contain all the > phy nodes. > > Signed-off-by: Codrin Ciubotariu Reviewed-by: Florian Fainelli -- Florian

Re: [PATCH net-next v2 2/7] dt-bindings: net: macb: use an MDIO node as a container for PHY nodes

2020-07-23 Thread Florian Fainelli
d to contain of the PHY nodes (and other MDIO devices' nodes). > > Signed-off-by: Codrin Ciubotariu Reviewed-by: Florian Fainelli -- Florian

Re: [PATCH net-next v2 1/7] net: macb: use device-managed devm_mdiobus_alloc()

2020-07-23 Thread Florian Fainelli
On 7/21/20 10:13 AM, Codrin Ciubotariu wrote: > Use the device-managed variant for the allocating the MDIO bus. This > cleans-up the code a little on the remove and error paths. > > Signed-off-by: Codrin Ciubotariu Reviewed-by: Florian Fainelli -- Florian

Re: [PATCH 23/23] MAINTAINERS: Add Krzysztof Kozlowski as maintainer of memory controllers

2020-07-23 Thread Florian Fainelli
On 7/23/20 12:37 AM, Krzysztof Kozlowski wrote: > The specific drivers in drivers/memory usually go via architecture (e.g. > ARM SoC) maintainers but the generic parts (of_memory.[ch]) lacked any > care. > > Signed-off-by: Krzysztof Kozlowski Acked-by: Florian Fainelli What

Re: [PATCH 17/23] memory: brcmstb_dpfe: Remove unneeded braces

2020-07-23 Thread Florian Fainelli
On 7/23/20 12:37 AM, Krzysztof Kozlowski wrote: > Single statement blocks don't need braces. Fixes checkpatch warning: > > WARNING: braces {} are not necessary for single statement blocks > > Signed-off-by: Krzysztof Kozlowski Acked-by: Florian Fainelli -- Florian

Re: [PATCH 16/23] memory: brcmstb_dpfe: Constify the contents of string

2020-07-23 Thread Florian Fainelli
On 7/23/20 12:37 AM, Krzysztof Kozlowski wrote: > The string itself can be made const for safety. > > Signed-off-by: Krzysztof Kozlowski Acked-by: Florian Fainelli -- Florian

Re: [RFC PATCH] net: dsa: qca8k: Add 802.1q VLAN support

2020-07-22 Thread Florian Fainelli
On 7/22/20 12:38 PM, Jonathan McDowell wrote: > On Tue, Jul 21, 2020 at 10:26:07AM -0700, Florian Fainelli wrote: >> On 7/21/20 10:16 AM, Jonathan McDowell wrote: >>> This adds full 802.1q VLAN support to the qca8k, allowing the use of >>> vlan_filtering and more compl

Re: [PATCH v6 net-next] net: hyperv: Add attributes to show TX indirection table

2020-07-21 Thread Florian Fainelli
On 7/21/20 12:21 PM, Jakub Kicinski wrote: > On Tue, 21 Jul 2020 04:58:59 + Chi Song wrote: >> An imbalanced TX indirection table causes netvsc to have low >> performance. This table is created and managed during runtime. To help >> better diagnose performance issues caused by imbalanced

Re: [RFC 0/7] Add support to process rx packets in thread

2020-07-21 Thread Florian Fainelli
On 7/21/20 10:25 AM, Andrew Lunn wrote: > On Tue, Jul 21, 2020 at 10:44:19PM +0530, Rakesh Pillai wrote: >> NAPI gets scheduled on the CPU core which got the >> interrupt. The linux scheduler cannot move it to a >> different core, even if the CPU on which NAPI is running >> is heavily loaded. This

Re: [PATCH] net: dsa: Add protocol support for 802.1AD when adding or deleting vlan for dsa switch and port

2020-07-21 Thread Florian Fainelli
On 7/21/20 4:02 AM, hongbo.w...@nxp.com wrote: > From: "hongbo.wang" > > the following command will be supported: > Add VLAN: > ip link add link swp1 name swp1.100 type vlan protocol 802.1ad id 100 > Delete VLAN: > ip link del link swp1 name swp1.100 > > when adding vlan, this patch

Re: [PATCH v2 6/7] usb: bdc: Halt controller on suspend

2020-07-21 Thread Florian Fainelli
suspend callbacks have completed. > > Adding a suspend hook to the BDC driver that halts the controller before > S2 entry thus preventing unwanted access to the BDC register space during > this transition. > > Signed-off-by: Danesh Petigara > Signed-off-by: Al Cooper Acked-by: Florian Fainelli -- Florian

Re: [PATCH v2 2/7] usb: bdc: Add compatible string for new style USB DT nodes

2020-07-21 Thread Florian Fainelli
On 7/21/20 7:43 AM, Al Cooper wrote: > Add compatible string for some newer boards that only have this > as there match sting. Remove unused compatible string "brcm,bdc-v0.16". > > Signed-off-by: Al Cooper Acked-by: Florian Fainelli -- Florian

Re: [PATCH v2 5/7] usb: bdc: driver runs out of buffer descriptors on large ADB transfers

2020-07-21 Thread Florian Fainelli
faster than the incoming > 16K transfers. This issue is fixed by doubling the number of BDs > that can be queued so that the entire 1M request can be queued > without running out of buffers. > > Signed-off-by: Al Cooper Acked-by: Florian Fainelli -- Florian

Re: [PATCH v2 4/7] usb: bdc: Adb shows offline after resuming from S2

2020-07-21 Thread Florian Fainelli
; cleared. The fix is to clear the endpoint flags for the disconnect > case only and not for S2 exit. > > Signed-off-by: Al Cooper Acked-by: Florian Fainelli -- Florian

Re: [PATCH v2 1/7] dt-bindings: usb: bdc: Update compatible strings

2020-07-21 Thread Florian Fainelli
On 7/21/20 7:43 AM, Al Cooper wrote: > Remove "brcm,bdc-v0.16" because it was never used on any system. > Add "brcm,bdc-udc-v2" which exists for any STB system with BDC. > > Signed-off-by: Al Cooper Acked-by: Florian Fainelli -- Florian

Re: [PATCH v2 3/7] bdc: Fix bug causing crash after multiple disconnects

2020-07-21 Thread Florian Fainelli
The fix is to only allow the endpoint commands > when the endpoint is connected. This will also fix issues that > showed up when using configfs to create gadgets. > > Signed-off-by: Sasi Kumar > Signed-off-by: Al Cooper Acked-by: Florian Fainelli -- Florian

Re: [RFC PATCH] net: dsa: qca8k: Add 802.1q VLAN support

2020-07-21 Thread Florian Fainelli
On 7/21/20 10:16 AM, Jonathan McDowell wrote: > This adds full 802.1q VLAN support to the qca8k, allowing the use of > vlan_filtering and more complicated bridging setups than allowed by > basic port VLAN support. > > Tested with a number of untagged ports with separate VLANs and then a > trunk

Re: [PATCH 1/2] net: dsa: Add flag for 802.1AD when adding VLAN for dsa switch and port

2020-07-20 Thread Florian Fainelli
On 7/20/2020 3:41 AM, hongbo.w...@nxp.com wrote: > From: "hongbo.wang" > > the following command can be supported: > ip link add link swp1 name swp1.100 type vlan protocol 802.1ad id 100 You should probably include the switch driver that is going to be benefiting from doing these changes in

Re: [PATCH v8 00/12] PCI: brcmstb: enable PCIe for STB chips

2020-07-20 Thread Florian Fainelli
On 7/15/20 7:35 AM, Jim Quinlan wrote: > Patchset Summary: > Enhance a PCIe host controller driver. Because of its unusual design > we are foced to change dev->dma_pfn_offset into a more general role > allowing multiple offsets. See the 'v1' notes below for more info. Christoph, Robin,

Re: [PATCH] bus: brcmstb_gisb: Shorten prints

2020-07-20 Thread Florian Fainelli
On 7/9/20 4:27 PM, Florian Fainelli wrote: > Do not print the full function name (brcmstb_gisb_arb_decode_addr) which > is quite long, and reduces our chances to printing a full line, instead > just use "GISB: " as a prefix for these prints. > > Signed-off-by: F

Re: [PATCH] ARM: brcmstb: Add debug UART entry for 72614

2020-07-20 Thread Florian Fainelli
On 7/9/20 4:24 PM, Florian Fainelli wrote: > Signed-off-by: Florian Fainelli Applied to soc/next, thanks! -- Florian

Re: [PATCH] ARM: bcm: Enable BCM7038_L1_IRQ for ARCH_BRCMSTB

2020-07-20 Thread Florian Fainelli
On 7/9/20 4:45 PM, Florian Fainelli wrote: > ARCH_BRCMSTB makes use of the irq-bcm7038-l1.c irqchip driver, enable > it. > > Signed-off-by: Florian Fainelli Applied to soc/next, thanks! -- Florian

[PATCH stable 4.14.y] arm64: entry: Place an SB sequence following an ERET instruction

2020-07-20 Thread Florian Fainelli
-by: Florian Fainelli --- arch/arm64/kernel/entry.S | 2 ++ arch/arm64/kvm/hyp/entry.S | 2 ++ arch/arm64/kvm/hyp/hyp-entry.S | 4 3 files changed, 8 insertions(+) diff --git a/arch/arm64/kernel/entry.S b/arch/arm64/kernel/entry.S index c1ffa95c0ad2..f70e0893ba51 100644 --- a/arch/arm64

[PATCH stable 4.19.y] arm64: entry: Place an SB sequence following an ERET instruction

2020-07-20 Thread Florian Fainelli
at the point of an ERET, this could potentially be used as part of a side-channel attack. This patch emits an SB sequence after each ERET so that speculation is held up on exception return. Signed-off-by: Will Deacon Signed-off-by: Florian Fainelli --- arch/arm64/kernel/entry.S | 2 ++ arch

Re: [PATCH stable v4.9 v2] arm64: entry: Place an SB sequence following an ERET instruction

2020-07-20 Thread Florian Fainelli
On 7/20/20 6:04 AM, Greg KH wrote: > On Thu, Jul 09, 2020 at 12:50:23PM -0700, Florian Fainelli wrote: >> From: Will Deacon >> >> commit 679db70801da9fda91d26caf13bf5b5ccc74e8e8 upstream >> >> Some CPUs can speculate past an ERET instruction and potentiall

Re: [PATCH] r6040: switch from 'pci_' to 'dma_' API

2020-07-20 Thread Florian Fainelli
sync_sg_for_device(e1, e2, e3, e4) > +dma_sync_sg_for_device(>dev, e2, e3, e4) > > @@ > expression e1, e2; > @@ > -pci_dma_mapping_error(e1, e2) > +dma_mapping_error(>dev, e2) > > @@ > expression e1, e2; > @@ > -pci_set_dma_mask(e1, e2) > +dma_set_mask(>dev, e2) > > @@ > expression e1, e2; > @@ > -pci_set_consistent_dma_mask(e1, e2) > +dma_set_coherent_mask(>dev, e2) > > Signed-off-by: Christophe JAILLET Acked-by: Florian Fainelli -- Florian

Re: [PATCH net] net: bcmgenet: add missed clk_disable_unprepare in bcmgenet_probe

2020-07-20 Thread Florian Fainelli
quot;net: bcmgenet: Use devm_clk_get_optional() to get the > clocks") > Signed-off-by: Zhang Changzhong Acked-by: Florian Fainelli -- Florian

Re: [PATCH net] net: bcmgenet: fix error returns in bcmgenet_probe()

2020-07-20 Thread Florian Fainelli
t;net: bcmgenet: enable NETIF_F_HIGHDMA flag") > Signed-off-by: Zhang Changzhong Acked-by: Florian fainelli -- Florian

[PATCH net-next v2 0/4] net: dsa: Setup dsa_netdev_ops

2020-07-19 Thread Florian Fainelli
using macros (Andrew) Florian Fainelli (4): net: Wrap ndo_do_ioctl() to prepare for DSA stacked ops net: dsa: Add wrappers for overloaded ndo_ops net: Call into DSA netdevice_ops wrappers net: dsa: Setup dsa_netdev_ops include/net/dsa.h| 71

[PATCH net-next v2 3/4] net: Call into DSA netdevice_ops wrappers

2020-07-19 Thread Florian Fainelli
Make the core net_device code call into our ndo_do_ioctl() and ndo_get_phys_port_name() functions via the wrappers defined previously Signed-off-by: Florian Fainelli --- net/core/dev.c | 5 + net/core/dev_ioctl.c | 5 + 2 files changed, 10 insertions(+) diff --git a/net/core

[PATCH net-next v2 4/4] net: dsa: Setup dsa_netdev_ops

2020-07-19 Thread Florian Fainelli
do tests like dev->netdev_ops == _ops which was not the case before since we were allocating an entirely new structure. Signed-off-by: Florian Fainelli --- include/net/dsa.h | 1 - net/dsa/master.c | 52 --- 2 files changed, 13 insertions(+), 40 del

[PATCH net-next v2 1/4] net: Wrap ndo_do_ioctl() to prepare for DSA stacked ops

2020-07-19 Thread Florian Fainelli
In preparation for adding another layer of call into a DSA stacked ops singleton, wrap the ndo_do_ioctl() call into dev_do_ioctl(). Reviewed-by: Andrew Lunn Signed-off-by: Florian Fainelli --- net/core/dev_ioctl.c | 24 +--- 1 file changed, 17 insertions(+), 7 deletions

[PATCH net-next v2 2/4] net: dsa: Add wrappers for overloaded ndo_ops

2020-07-19 Thread Florian Fainelli
-by: Florian Fainelli --- include/net/dsa.h | 70 +++ 1 file changed, 70 insertions(+) diff --git a/include/net/dsa.h b/include/net/dsa.h index 6fa418ff1175..343642ca4f63 100644 --- a/include/net/dsa.h +++ b/include/net/dsa.h @@ -86,6 +86,18 @@ struct

Re: [PATCH net-next 2/4] net: dsa: Add wrappers for overloaded ndo_ops

2020-07-19 Thread Florian Fainelli
On 7/19/2020 8:40 AM, Andrew Lunn wrote: >> +#if IS_ENABLED(CONFIG_NET_DSA) >> +#define dsa_build_ndo_op(name, arg1_type, arg1_name, arg2_type, arg2_name) \ >> +static int inline dsa_##name(struct net_device *dev, arg1_type arg1_name, \ >> + arg2_type arg2_name) \

Re: [PATCH net-next 3/4] net: Call into DSA netdevice_ops wrappers

2020-07-19 Thread Florian Fainelli
On 7/19/2020 9:04 AM, Andrew Lunn wrote: >> If we have the core network stack reference DSA as a module then we >> force DSA to be either built-in or not, which is not very practical, >> people would still want a modular choice to be possible. The static >> inline only wraps indirect function

Re: [PATCH net-next 3/4] net: Call into DSA netdevice_ops wrappers

2020-07-18 Thread Florian Fainelli
On 7/18/2020 2:18 PM, Vladimir Oltean wrote: > On Fri, Jul 17, 2020 at 08:05:32PM -0700, Florian Fainelli wrote: >> Make the core net_device code call into our ndo_do_ioctl() and >> ndo_get_phys_port_name() functions via the wrappers defined previously >> >> Sign

Re: [PATCH net-next 1/4] net: Wrap ndo_do_ioctl() to prepare for DSA stacked ops

2020-07-18 Thread Florian Fainelli
On 7/18/2020 1:30 PM, Vladimir Oltean wrote: > Hi Florian, > > On Fri, Jul 17, 2020 at 08:05:30PM -0700, Florian Fainelli wrote: >> In preparation for adding another layer of call into a DSA stacked ops >> singleton, wrap the ndo_do_ioctl() call into dev_do_ioct

[PATCH net-next 4/4] net: dsa: Setup dsa_netdev_ops

2020-07-17 Thread Florian Fainelli
do tests like dev->netdev_ops == _ops which was not the case before since we were allocating an entirely new structure. Signed-off-by: Florian Fainelli --- include/net/dsa.h | 1 - net/dsa/master.c | 52 --- 2 files changed, 13 insertions(+), 40 del

[PATCH net-next 0/4] net: dsa: Setup dsa_netdev_ops

2020-07-17 Thread Florian Fainelli
(no known in tree problems because of that though). Let me know if this approach looks reasonable to you and we might do the same with our ethtool_ops overloading as well. Thanks! Florian Fainelli (4): net: Wrap ndo_do_ioctl() to prepare for DSA stacked ops net: dsa: Add wrappers for overloaded

[PATCH net-next 1/4] net: Wrap ndo_do_ioctl() to prepare for DSA stacked ops

2020-07-17 Thread Florian Fainelli
In preparation for adding another layer of call into a DSA stacked ops singleton, wrap the ndo_do_ioctl() call into dev_do_ioctl(). Signed-off-by: Florian Fainelli --- net/core/dev_ioctl.c | 24 +--- 1 file changed, 17 insertions(+), 7 deletions(-) diff --git a/net/core

[PATCH net-next 2/4] net: dsa: Add wrappers for overloaded ndo_ops

2020-07-17 Thread Florian Fainelli
-by: Florian Fainelli --- include/net/dsa.h | 41 + 1 file changed, 41 insertions(+) diff --git a/include/net/dsa.h b/include/net/dsa.h index 6fa418ff1175..681ba2752514 100644 --- a/include/net/dsa.h +++ b/include/net/dsa.h @@ -86,6 +86,18 @@ struct dsa_device_ops

[PATCH net-next 3/4] net: Call into DSA netdevice_ops wrappers

2020-07-17 Thread Florian Fainelli
Make the core net_device code call into our ndo_do_ioctl() and ndo_get_phys_port_name() functions via the wrappers defined previously Signed-off-by: Florian Fainelli --- net/core/dev.c | 5 + net/core/dev_ioctl.c | 5 + 2 files changed, 10 insertions(+) diff --git a/net/core

Re: [PATCH 2/2] dt-bindings: net: dsa: qca8k: Add PORT0_PAD_CTRL properties

2020-07-17 Thread Florian Fainelli
On 7/17/2020 1:29 PM, Matthew Hagan wrote: > > > On 16/07/2020 23:09, Jakub Kicinski wrote: >> On Mon, 13 Jul 2020 21:50:26 +0100 Matthew Hagan wrote: >>> Add names and decriptions of additional PORT0_PAD_CTRL properties. >>> >>> Signed-off-by: Matthew Hagan >>> --- >>>

Re: [PATCH 2/2] dt-bindings: net: dsa: qca8k: Add PORT0_PAD_CTRL properties

2020-07-17 Thread Florian Fainelli
On 7/17/2020 12:26 PM, Matthew Hagan wrote: > > > On 16/07/2020 23:32, Andrew Lunn wrote: >> On Thu, Jul 16, 2020 at 03:09:25PM -0700, Jakub Kicinski wrote: >>> On Mon, 13 Jul 2020 21:50:26 +0100 Matthew Hagan wrote: Add names and decriptions of additional PORT0_PAD_CTRL properties.

Re: [PATCH] firmware: arm_scmi: Pass shmem address to SMCCC call

2020-07-17 Thread Florian Fainelli
On 7/17/2020 2:45 AM, Sudeep Holla wrote: > On Wed, Jul 15, 2020 at 03:43:24PM -0700, Florian Fainelli wrote: >> >> >> On 7/15/2020 9:55 AM, Daniele Alessandrelli wrote: >>> From: Daniele Alessandrelli >>> >>> Currently, when SMC/HVC is used

Re: [PATCH 5/7] usb: bdc: driver runs out of buffer descriptors on large ADB transfers

2020-07-17 Thread Florian Fainelli
faster than the incoming > 16K transfers. This issue is fixed by doubling the number of BDs > that can be queued so that the entire 1M request can be queued > without running out of buffers. > > Signed-off-by: Al Cooper > Signed-off-by: Florian Fainelli You can certainly

Re: [PATCH 7/7] usb: bdc: Use devm_clk_get_optional()

2020-07-17 Thread Florian Fainelli
On 7/17/2020 8:23 AM, Al Cooper wrote: > From: Florian Fainelli > > The BDC clock is optional and we may get an -EPROBE_DEFER error code > which would not be propagated correctly, fix this by using > devm_clk_get_optional(). > > Signed-off-by: Florian Fainelli S

Re: [PATCH 3/7] bdc: Fix bug causing crash after multiple disconnects

2020-07-17 Thread Florian Fainelli
On 7/17/2020 8:23 AM, Al Cooper wrote: > From: Sasi Kumar > > Multiple connects/disconnects can cause a crash on the second > disconnect. The driver had a problem where it would try to send > endpoint commands after it was disconnected which is not allowed > by the hardware. The fix is to

Re: [PATCH 2/7] usb: bdc: Add compatible string for new style USB DT nodes

2020-07-17 Thread Florian Fainelli
On 7/17/2020 8:23 AM, Al Cooper wrote: > Add compatible string for some newer boards that only have this > as there match sting. > > Signed-off-by: Al Cooper This looks good to me, however you should also include it in the binding document. -- Florian

Re: [PATCH net 3/3] net: bcmgenet: restore HFB filters on resume

2020-07-17 Thread Florian Fainelli
add WAKE_FILTER support") > Signed-off-by: Doug Berger Acked-by: Florian Fainelli -- Florian

Re: [PATCH net 2/3] net: bcmgenet: test RBUF_ACPI_EN when resuming

2020-07-16 Thread Florian Fainelli
RBUF_ACPI_EN flag when Wake > on Filter is enabled. A clear flag indicates that the GENET > hardware must have been reset so the remainder of the > hardware programming is bypassed. > > Fixes: f50932cca632 ("net: bcmgenet: add WAKE_FILTER support") > Signed-off-by: Doug Berger Acked-by: Florian Fainelli -- Florian

Re: [PATCH net 1/3] net: bcmgenet: test MPD_EN when resuming

2020-07-16 Thread Florian Fainelli
e MPD_EN flag when Wake on > Magic Packet is enabled. A clear flag indicates that the > GENET hardware must have been reset so the remainder of the > hardware programming is bypassed. > > Fixes: 1a1d5106c1e3 ("net: bcmgenet: move clk_wol management to bcmgenet_wol") >

Re: [PATCH 2/2] dt-bindings: net: dsa: qca8k: Add PORT0_PAD_CTRL properties

2020-07-16 Thread Florian Fainelli
On 7/13/2020 1:50 PM, Matthew Hagan wrote: > Add names and decriptions of additional PORT0_PAD_CTRL properties. > > Signed-off-by: Matthew Hagan > --- > Documentation/devicetree/bindings/net/dsa/qca8k.txt | 8 > 1 file changed, 8 insertions(+) > > diff --git

Re: [PATCH] firmware: arm_scmi: Pass shmem address to SMCCC call

2020-07-16 Thread Florian Fainelli
On 7/16/2020 7:13 AM, Daniele Alessandrelli wrote: > Hi Florian, > > Thanks for you feedback. > > On Wed, 2020-07-15 at 15:43 -0700, Florian Fainelli wrote: >> >> On 7/15/2020 9:55 AM, Daniele Alessandrelli wrote: >>> From: Daniele Alessandrelli >

Re: [PATCH net-next] net: phy: sfp: Cotsworks SFF module EEPROM fixup

2020-07-15 Thread Florian Fainelli
On 7/15/2020 8:32 PM, Chris Healy wrote: > > > On Wed, Jul 15, 2020 at 8:10 PM Florian Fainelli <mailto:f.faine...@gmail.com>> wrote: > > > > On 7/14/2020 10:59 AM, Chris Healy wrote: > > Some Cotsworks SFF have invalid data in the first fe

Re: [PATCH net-next] net: phy: sfp: Cotsworks SFF module EEPROM fixup

2020-07-15 Thread Florian Fainelli
On 7/14/2020 10:59 AM, Chris Healy wrote: > Some Cotsworks SFF have invalid data in the first few bytes of the > module EEPROM. This results in these modules not being detected as > valid modules. > > Address this by poking the correct EEPROM values into the module > EEPROM when the model/PN

Re: [PATCH] bcm963xx_tag.h: fix duplicated word

2020-07-15 Thread Florian Fainelli
Le 2020-07-15 à 18:28, Randy Dunlap a écrit : > From: Randy Dunlap > > Change doubled word "is" to "it is". > > Signed-off-by: Randy Dunlap > Cc: Florian Fainelli > Cc: bcm-kernel-feedback-l...@broadcom.com > Cc: linux-m...@vger.kernel.org Acked

Re: [PATCH] firmware: arm_scmi: Pass shmem address to SMCCC call

2020-07-15 Thread Florian Fainelli
On 7/15/2020 9:55 AM, Daniele Alessandrelli wrote: > From: Daniele Alessandrelli > > Currently, when SMC/HVC is used as transport, the base address of the > shared memory used for communication is not passed to the SMCCC call. > This means that such an address must be hard-coded into the

Re: [PATCH v3 1/9] dt-bindings: reset: Add a binding for the RPi Firmware reset controller

2020-07-14 Thread Florian Fainelli
e running on the RPi VideoCore can be used to reset and >>>> initialize HW controlled by the firmware. >>>> >>>> Signed-off-by: Nicolas Saenz Julienne >>>> Reviewed-by: Florian Fainelli >>>> >>>> --- >>>> Changes

Re: [PATCH] net: xilinx: fix potential NULL dereference in temac_probe()

2020-07-13 Thread Florian Fainelli
On 7/13/2020 7:23 PM, Xu Wang wrote: > platform_get_resource() may return NULL, add proper > check to avoid potential NULL dereferencing. > > Signed-off-by: Xu Wang > --- If you use devm_ioremap_resource() you can remove the !res check entirely which would be equally acceptable as a fix. >

Re: [v2 PATCH] usb: gadget: bdc: use readl_poll_timeout() to simplify code

2020-07-12 Thread Florian Fainelli
On 7/12/2020 8:30 PM, Chunfeng Yun wrote: > Use readl_poll_timeout() to poll register status > > Cc: Florian Fainelli > Signed-off-by: Chunfeng Yun > --- > v2 changes, suggested by Stephen: > 1. use unsigned int instead of int for @usec parameter > 2. add dev_log

Re: [PATCH] pinctrl: initialise nsp-mux earlier.

2020-07-11 Thread Florian Fainelli
On 7/11/2020 2:07 PM, Linus Walleij wrote: > On Wed, Jul 1, 2020 at 6:44 AM Florian Fainelli wrote: >> On 6/30/2020 9:37 PM, Mark Tomlinson wrote: > >>> That was one of my thoughts too. I found someone had tried that >>> earlier, but it was rejected: >>&g

[PATCH] MAINTAINERS: Add entry for Broadcom BDC driver

2020-07-09 Thread Florian Fainelli
The Broadcom BDC driver did not have a MAINTAINERS entry which made it escape review from Al and myself, add an entry so the relevant mailing lists and people are copied. Signed-off-by: Florian Fainelli --- MAINTAINERS | 8 1 file changed, 8 insertions(+) diff --git a/MAINTAINERS b

[PATCH] ARM: bcm: Enable BCM7038_L1_IRQ for ARCH_BRCMSTB

2020-07-09 Thread Florian Fainelli
ARCH_BRCMSTB makes use of the irq-bcm7038-l1.c irqchip driver, enable it. Signed-off-by: Florian Fainelli --- arch/arm/mach-bcm/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm/mach-bcm/Kconfig b/arch/arm/mach-bcm/Kconfig index 6aa938b949db..52ce6febf6ea 100644 --- a/arch

[PATCH] irqchip/irq-bcm7038-l1: Allow building on ARM 32-bit

2020-07-09 Thread Florian Fainelli
We need to have a definition for cpu_logical_map[] which on ARM platforms is provided by asm/smp_plat.h. This header is not automatically included from linux/smp.h and untangling it is a bit difficult. Signed-off-by: Florian Fainelli --- drivers/irqchip/irq-bcm7038-l1.c | 3 +++ 1 file changed

[PATCH] bus: brcmstb_gisb: Shorten prints

2020-07-09 Thread Florian Fainelli
Do not print the full function name (brcmstb_gisb_arb_decode_addr) which is quite long, and reduces our chances to printing a full line, instead just use "GISB: " as a prefix for these prints. Signed-off-by: Florian Fainelli --- drivers/bus/brcmstb_gisb.c | 4 ++-- 1 file changed, 2

[PATCH] ARM: brcmstb: Add debug UART entry for 72614

2020-07-09 Thread Florian Fainelli
Signed-off-by: Florian Fainelli --- arch/arm/include/debug/brcmstb.S | 24 +--- 1 file changed, 13 insertions(+), 11 deletions(-) diff --git a/arch/arm/include/debug/brcmstb.S b/arch/arm/include/debug/brcmstb.S index 132a20c4a676..79223209d3f4 100644 --- a/arch/arm/include

[PATCH 6/6] irqchip/brcmstb-l2: Match UPG_AUX_AON_INTR2 compatible

2020-07-09 Thread Florian Fainelli
The UPG_AUX_AON_INTR2 Level 2 interrupt controller node is defined with the "brcm,upg-aux-aon-l2-intc" compatible string in Device Tree and behaves as an edge triggered standard Broadcom STB L2 interrupt controller. Signed-off-by: Florian Fainelli --- drivers/irqchip/irq-brcmstb-l2.c

[PATCH 4/6] irqchip/brcmstb-l2: Match HIF_SPI_INTR2 compatible

2020-07-09 Thread Florian Fainelli
From: Kamal Dasu The HIF_SPI_INTR2 Level 2 interrupt controller node is defined with the "brcm,hif-spi-l2-intc" compatible string in Device Tree and behaves as an edge triggered standard Broadcom STB L2 interrupt controller. Signed-off-by: Kamal Dasu Signed-off-by: Floria

[PATCH 5/6] dt-bindings: interrupt-controller: Document UPG auxiliary L2

2020-07-09 Thread Florian Fainelli
Define the compatible string brcm,upg-aux-aon-l2-intc which is used by the Broadcom STB UPG auxiliary always-on interrupt controller. Signed-off-by: Florian Fainelli --- .../devicetree/bindings/interrupt-controller/brcm,l2-intc.txt| 1 + 1 file changed, 1 insertion(+) diff --git

[PATCH 2/6] irqchip/brcmstb-l2: Set controller as wake-up source

2020-07-09 Thread Florian Fainelli
e_irq_wake() is called early on. Signed-off-by: Justin Chen Signed-off-by: Florian Fainelli --- drivers/irqchip/irq-brcmstb-l2.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/irqchip/irq-brcmstb-l2.c b/drivers/irqchip/irq-brcmstb-l2.c index 0298ede67e51..157fad184bdc 100644 ---

[PATCH 3/6] dt-bindings: interrupt-controller: Document Broadcom STB HIF L2

2020-07-09 Thread Florian Fainelli
Add documentation for the brcm,hif-spi-l2-intc compatible string to the brcm,l2-intc.txt binding document. Signed-off-by: Florian Fainelli --- .../devicetree/bindings/interrupt-controller/brcm,l2-intc.txt | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/Documentation

[PATCH 0/6] irqchip: Broadcom STB interrupt controller updates

2020-07-09 Thread Florian Fainelli
Hi Marc, This patch series contains a number of updates for Broadcom STB L2 interrupt controllers to enable them as wake-up interrupt controllers, and add missing compatible strings that should be matched. Thanks! Florian Fainelli (3): dt-bindings: interrupt-controller: Document Broadcom STB

[PATCH 1/6] irqchip/bcm7120-l2: Set controller as wake-up source

2020-07-09 Thread Florian Fainelli
e_irq_wake() is called early on. Signed-off-by: Justin Chen Signed-off-by: Florian Fainelli --- drivers/irqchip/irq-bcm7120-l2.c | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/drivers/irqchip/irq-bcm7120-l2.c b/drivers/irqchip/irq-bcm7120-l2.c index 586df3587be0..c7

[PATCH stable v4.9 v2] arm64: entry: Place an SB sequence following an ERET instruction

2020-07-09 Thread Florian Fainelli
] Signed-off-by: Florian Fainelli --- Changes in v2: - added missing hunk in hyp/entry.S per Will's feedback arch/arm64/kernel/entry.S | 2 ++ arch/arm64/kvm/hyp/entry.S | 2 ++ arch/arm64/kvm/hyp/hyp-entry.S | 4 3 files changed, 8 insertions(+) diff --git a/arch/arm64/kernel

Re: BUG: KASAN: global-out-of-bounds in is_affected_midr_range_list on arm64

2020-07-08 Thread Florian Fainelli
On 7/8/2020 9:36 PM, Naresh Kamboju wrote: > While running LTP cpuhotplug test on mainline 5.8.0-rc4 the kernel BUG noticed > on arm64 Juno-r2 KASAN config enabled kernel. > > steps to reproduce: > - boot KASAN enabled Juno-r2 device > - cd /opt/ltp > - ./runltp -f cpuhotplug > > metadata: >

[PATCH] arm64: Add missing sentinel to erratum_1463225

2020-07-08 Thread Florian Fainelli
: a9e821b89daa ("arm64: Add KRYO4XX gold CPU cores to erratum list 1463225 and 1418040") Signed-off-by: Florian Fainelli --- arch/arm64/kernel/cpu_errata.c | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm64/kernel/cpu_errata.c b/arch/arm64/kernel/cpu_errata.c index 8e

Re: [PATCH][next] net: systemport: fix double shift of a vlan_tci by VLAN_PRIO_SHIFT

2020-07-08 Thread Florian Fainelli
than > the u16 so the end result is always zero. Remove the second explicit > shift as this is extraneous. > > Fixes: 6e9fdb60d362 ("net: systemport: Add support for VLAN transmit > acceleration") > Addresses-Coverity: ("Operands don't affect result") >

[PATCH net-next 2/2] net: phy: Uninline PHY ethtool statistics operations

2020-07-08 Thread Florian Fainelli
Now that we have moved the PHY ethtool statistics to be dynamically registered, we no longer need to inline those for ethtool. This used to be done to avoid cross symbol referencing and allow ethtool to be decoupled from PHYLIB entirely. Signed-off-by: Florian Fainelli --- drivers/net/phy/phy.c

[PATCH net-next 0/2] net: phy: Uninline PHY ethtool statistics

2020-07-08 Thread Florian Fainelli
since we could have DSA register its own ethtool_phy_ops instance instead of overloading the ethtool_ops. Florian Fainelli (2): net: phy: Define PHY statistics ethtool_phy_ops net: phy: Uninline PHY ethtool statistics operations drivers/net/phy/phy.c| 48

[PATCH net-next 1/2] net: phy: Define PHY statistics ethtool_phy_ops

2020-07-08 Thread Florian Fainelli
Extend ethtool_phy_ops to include the 3 function pointers necessary for implementing PHY statistics. In a subsequent change we will uninline those functions. Signed-off-by: Florian Fainelli --- drivers/net/phy/phy_device.c | 3 +++ include/linux/ethtool.h | 7 +++ 2 files changed, 10

[PATCH net-next] net: dsa: loop: Print when registration is successful

2020-07-07 Thread Florian Fainelli
We have a number of error conditions that can lead to the driver not probing successfully, move the print when we are sure dsa_register_switch() has suceeded. This avoids repeated prints in case of probe deferral for instance. Signed-off-by: Florian Fainelli --- drivers/net/dsa/dsa_loop.c | 11

<    5   6   7   8   9   10   11   12   13   14   >