[PATCH net-next v2 1/2] of: mdio: Fall back to mdiobus_register() with NULL device_node

2018-05-15 Thread Florian Fainelli
consistent with the stub version that we defined when CONFIG_OF=n. Signed-off-by: Florian Fainelli <f.faine...@gmail.com> --- drivers/of/of_mdio.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/of/of_mdio.c b/drivers/of/of_mdio.c index 8c0c92712fc9..d963baf8e53a

[PATCH net-next v2 0/2] of: mdio: Fall back to mdiobus_register() with NULL device_node

2018-05-15 Thread Florian Fainelli
a much less obvious behavior and specifically attempt to deal with CONFIG_ACPI. Changes in v2: - fixed build error in davincin_mdio.c (Grygorii) - reworked first patch a bit: commit message, subject and removed useless code comment Florian Fainelli (2): of: mdio: Fall back to mdiobus_register

[PATCH net-next v2 2/2] drivers: net: Remove device_node checks with of_mdiobus_register()

2018-05-15 Thread Florian Fainelli
A number of drivers have the following pattern: if (np) of_mdiobus_register() else mdiobus_register() which the implementation of of_mdiobus_register() now takes care of. Remove that pattern in drivers that strictly adhere to it. Signed-off-by: Florian Fainelli <f.fa

Re: [PATCH net-next 2/2] drivers: net: Remove device_node checks with of_mdiobus_register()

2018-05-15 Thread Florian Fainelli
On 05/15/2018 03:57 PM, Grygorii Strashko wrote: > > > On 05/15/2018 04:59 PM, Florian Fainelli wrote: >> A number of drivers have the following pattern: >> >> if (np) >> of_mdiobus_register() >> else >> mdiobus_register() >> >> wh

[PATCH net-next 0/2] of: mdio: Fall back to mdiobus_register() with np is NULL

2018-05-15 Thread Florian Fainelli
a much less obvious behavior and specifically attempt to deal with CONFIG_ACPI. Florian Fainelli (2): of: mdio: Fall back to mdiobus_register() with np is NULL drivers: net: Remove device_node checks with of_mdiobus_register() drivers/net/dsa/bcm_sf2.c | 8

[PATCH net-next 2/2] drivers: net: Remove device_node checks with of_mdiobus_register()

2018-05-15 Thread Florian Fainelli
A number of drivers have the following pattern: if (np) of_mdiobus_register() else mdiobus_register() which the implementation of of_mdiobus_register() now takes care of. Remove that pattern in drivers that strictly adhere to it. Signed-off-by: Florian Fainelli <f.fa

[PATCH net-next 1/2] of: mdio: Fall back to mdiobus_register() with np is NULL

2018-05-15 Thread Florian Fainelli
. This is also consistent with the stub version that we defined when CONFIG_OF=n. Signed-off-by: Florian Fainelli <f.faine...@gmail.com> --- drivers/of/of_mdio.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/drivers/of/of_mdio.c b/drivers/of/of_mdio.c index 8c0c92712fc9..2341dbf675bf

Re: [PATCH] usb: gadget: bdc: 64-bit pointer capability check

2017-06-16 Thread Florian Fainelli
efed421a94e6 ("usb: gadget: Add UDC driver for > Broadcom USB3.0 device controller IP BDC") > > Signed-off-by: Srinath Mannam <srinath.man...@broadcom.com> Reviewed-by: Florian Fainelli <f.faine...@gmail.com> -- Florian -- To unsubscribe from this list: send the line

Re: [PATCH v3 0/2] usb: Check for DMA capable buffer sanity

2017-05-05 Thread Florian Fainelli
On 04/25/2017 05:56 PM, Florian Fainelli wrote: > Changes in v3: > > - added check in usb_gadget_map_request_by_dev (Felipe), new patch > - improved commit message description (Clemens) > - added additiona checks for urb->setup_packet (Alan) > > Changes in v2: >

[PATCH v3 2/2] usb: udc: core: Error if req->buf is either from vmalloc or stack

2017-04-25 Thread Florian Fainelli
Check that req->buf is a valid DMA capable address, produce a warning and return an error if it's either coming from vmalloc space or is an on stack buffer. Signed-off-by: Florian Fainelli <f.faine...@gmail.com> --- drivers/usb/gadget/udc/core.c | 9 + 1 file changed, 9 i

[PATCH v3 0/2] usb: Check for DMA capable buffer sanity

2017-04-25 Thread Florian Fainelli
Changes in v3: - added check in usb_gadget_map_request_by_dev (Felipe), new patch - improved commit message description (Clemens) - added additiona checks for urb->setup_packet (Alan) Changes in v2: - moved the check from usb_start_wait_urb() to usb_hcd_map_urb_for_dma() Florian Fainelli

[PATCH v3 1/2] usb: core: Check URB setup_packet and transfer_buffer sanity

2017-04-25 Thread Florian Fainelli
buffer there is an existing is_vmalloc_addr() check so we just supplement that with an object_is_on_stack() check, produce a warning if that is the case and also return an error. Signed-off-by: Florian Fainelli <f.faine...@gmail.com> --- drivers/usb/core/hcd.c | 12 1 file changed, 1

[PATCH v2] usb: core: Warn if an URB's transfer_buffer is on stack

2017-04-23 Thread Florian Fainelli
We see a large number of fixes to several drivers to remove the usage of on-stack buffers feeding into USB transfer functions. Make it easier to spot the offenders by adding a warning in usb_hcd_map_urb_for_dma() checking that urb->transfer_buffer is not a stack object. Signed-off-by: Flor

Re: [PATCH] usb: core: Warn if an URB's transfer_buffer is on stack

2017-04-23 Thread Florian Fainelli
On 04/23/2017 09:01 AM, Alan Stern wrote: > On Sun, 23 Apr 2017, Greg Kroah-Hartman wrote: > >> On Sat, Apr 22, 2017 at 05:31:27PM -0400, Alan Stern wrote: >>> On Sat, 22 Apr 2017, Florian Fainelli wrote: >>> >>>> We see a large number of fi

[PATCH] usb: core: Warn if an URB's transfer_buffer is on stack

2017-04-22 Thread Florian Fainelli
We see a large number of fixes to several drivers to remove the usage of on-stack buffers feeding into USB transfer functions. Make it easier to spot the offenders by adding a warning in usb_start_wait_urb() for urb->transfer_buffer to be located on the stack. Signed-off-by: Florian Faine

[PATCH net-next] net: usbnet: Remove unused driver_name variable

2017-04-04 Thread Florian Fainelli
With GCC 6.3, we can get the following warning: drivers/net/usb/usbnet.c:85:19: warning: 'driver_name' defined but not used [-Wunused-const-variable=] static const char driver_name [] = "usbnet"; ^~~ Signed-off-by: Florian Fainelli <f.faine...@gmail.com&

Re: [PATCH RFC 0/7] phylib MMD accessor cleanups

2017-03-19 Thread Florian Fainelli
ause 45 phy, or clause 22 indirect accesses if the PHY > is clause 22. LGTM: Reviewed-by: Florian Fainelli <f.faine...@gmail.com> > > Note: confusingly, phy_read_mmd_indirect() vs phy_read_mmd() switches > the order of prtad and devad, which means that converting between t

Re: [PATCH net-next v2 00/12] net: dsa: remove unnecessary phy.h include

2017-02-10 Thread Florian Fainelli
On 02/10/2017 10:51 AM, David Miller wrote: > From: Kalle Valo <kv...@codeaurora.org> > Date: Thu, 09 Feb 2017 16:10:06 +0200 > >> Florian Fainelli <f.faine...@gmail.com> writes: >> >>>>> If not, for something like this it's a must: >>>&

Re: [PATCH net-next v2 00/12] net: dsa: remove unnecessary phy.h include

2017-02-08 Thread Florian Fainelli
On 02/08/2017 08:11 AM, Kalle Valo wrote: > David Miller <da...@davemloft.net> writes: > >> From: Florian Fainelli <f.faine...@gmail.com> >> Date: Tue, 7 Feb 2017 15:02:53 -0800 >> >>> I'm hoping this doesn't conflict with what's already in net-next.

[PATCH net-next v2 02/12] net: cgroups: fix build errors when linux/phy*.h is removed from net/dsa.h

2017-02-07 Thread Florian Fainelli
From: Russell King net/core/netprio_cgroup.c:303:16: error: expected declaration specifiers or '...' before string constant MODULE_LICENSE("GPL v2"); ^~~~ Add linux/module.h to fix this. Signed-off-by: Russell King

[PATCH net-next v2 00/12] net: dsa: remove unnecessary phy.h include

2017-02-07 Thread Florian Fainelli
Hi all, Including phy.h and phy_fixed.h into net/dsa.h causes phy*.h to be an unnecessary dependency for quite a large amount of the kernel. There's very little which actually requires definitions from phy.h in net/dsa.h - the include itself only wants the declaration of a couple of structures

[PATCH net-next v2 03/12] net: macb: fix build errors when linux/phy*.h is removed from net/dsa.h

2017-02-07 Thread Florian Fainelli
From: Russell King drivers/net/ethernet/cadence/macb.h:862:33: sparse: expected ; at end of declaration drivers/net/ethernet/cadence/macb.h:862:33: sparse: Expected } at end of struct-union-enum-specifier drivers/net/ethernet/cadence/macb.h:862:33: sparse: got

[PATCH net-next v2 04/12] net: lan78xx: fix build errors when linux/phy*.h is removed from net/dsa.h

2017-02-07 Thread Florian Fainelli
From: Russell King drivers/net/usb/lan78xx.c:394:33: sparse: expected ; at end of declaration drivers/net/usb/lan78xx.c:394:33: sparse: Expected } at end of struct-union-enum-specifier drivers/net/usb/lan78xx.c:394:33: sparse: got interface

[PATCH net-next v2 01/12] net: sunrpc: fix build errors when linux/phy*.h is removed from net/dsa.h

2017-02-07 Thread Florian Fainelli
From: Russell King Removing linux/phy.h from net/dsa.h reveals a build error in the sunrpc code: net/sunrpc/xprtrdma/svc_rdma_backchannel.c: In function 'xprt_rdma_bc_put': net/sunrpc/xprtrdma/svc_rdma_backchannel.c:277:2: error: implicit declaration of function

[PATCH net-next v2 05/12] net: bgmac: fix build errors when linux/phy*.h is removed from net/dsa.h

2017-02-07 Thread Florian Fainelli
From: Russell King drivers/net/ethernet/broadcom/bgmac.c:1015:17: error: dereferencing pointer to incomplete type 'struct mii_bus' drivers/net/ethernet/broadcom/bgmac.c:1185:2: error: implicit declaration of function 'phy_start'

[PATCH net-next v2 09/12] MIPS: Octeon: Remove unnecessary MODULE_*()

2017-02-07 Thread Florian Fainelli
From: Russell King octeon-platform.c can not be built as a module for two reasons: (a) the Makefile doesn't allow it: obj-y := cpu.o setup.o octeon-platform.o octeon-irq.o csrc-octeon.o (b) the multiple *_initcall() statements, each of which are translated

[PATCH net-next v2 12/12] net: dsa: remove unnecessary phy*.h includes

2017-02-07 Thread Florian Fainelli
- even with ccache, the time difference is noticable. Tested-by: Vivien Didelot <vivien.dide...@savoirfairelinux.com> Reviewed-by: Florian Fainelli <f.faine...@gmail.com> Signed-off-by: Russell King <rmk+ker...@armlinux.org.uk> --- drivers/net/dsa/mv88e6xxx/mv88e6xxx.h

[PATCH net-next v2 10/12] net: liquidio: fix build errors when linux/phy*.h is removed from net/dsa.h

2017-02-07 Thread Florian Fainelli
From: Russell King drivers/net/ethernet/cavium/liquidio/lio_vf_main.c:30: error: expected declaration specifiers or '...' before string constant drivers/net/ethernet/cavium/liquidio/lio_vf_main.c:30: warning: data definition has no type or storage class

[PATCH net-next v2 08/12] iscsi: fix build errors when linux/phy*.h is removed from net/dsa.h

2017-02-07 Thread Florian Fainelli
From: Russell King drivers/target/iscsi/iscsi_target_login.c:1135:7: error: implicit declaration of function 'try_module_get' [-Werror=implicit-function-declaration] Add linux/module.h to iscsi_target_login.c. Signed-off-by: Russell King

[PATCH net-next v2 06/12] net: fman: fix build errors when linux/phy*.h is removed from net/dsa.h

2017-02-07 Thread Florian Fainelli
From: Russell King drivers/net/ethernet/freescale/fman/fman_memac.c:519:21: error: dereferencing pointer to incomplete type 'struct fixed_phy_status' Add linux/phy_fixed.h to fman_memac.c Signed-off-by: Russell King ---

[PATCH net-next v2 07/12] net: mvneta: fix build errors when linux/phy*.h is removed from net/dsa.h

2017-02-07 Thread Florian Fainelli
From: Russell King drivers/net/ethernet/marvell/mvneta.c:2694:26: error: storage size of 'status' isn't known drivers/net/ethernet/marvell/mvneta.c:2695:26: error: storage size of 'changed' isn't known drivers/net/ethernet/marvell/mvneta.c:2695:9: error: variable

[PATCH net-next v2 11/12] net: ath5k: fix build errors when linux/phy*.h is removed from net/dsa.h

2017-02-07 Thread Florian Fainelli
From: Russell King Fix these errors reported by the 0-day builder by replacing the linux/export.h include with linux/module.h. In file included from include/linux/platform_device.h:14:0, from drivers/net/wireless/ath/ath5k/ahb.c:20:

Re: [PATCH 4.10-rc3 00/13] net: dsa: remove unnecessary phy.h include

2017-01-31 Thread Florian Fainelli
with what's already in net-next... For the entire series: Acked-by: Florian Fainelli <f.faine...@gmail.com> Thanks a lot for doing that. > > arch/mips/cavium-octeon/octeon-platform.c | 4 > drivers/net/dsa/mv88e6xxx/mv88e6xxx.h | 1 + > drivers/

Re: [RFC v2 4/5] DT bindings documentation for Synopsys UDC platform driver

2017-01-19 Thread Florian Fainelli
On 01/19/2017 02:36 PM, Scott Branden wrote: The driver stands alone from the SoC and does not need compatibility strings per SoC. New SoCs will use the exact same block. >>> >>> Even if you take the exact same block and put it in a different SoC, >>> that's still an integration work

Re: [RFC v2 4/5] DT bindings documentation for Synopsys UDC platform driver

2017-01-19 Thread Florian Fainelli
On 01/19/2017 12:07 PM, Scott Branden wrote: > Hi Florian, > > On 17-01-19 11:40 AM, Florian Fainelli wrote: >> On 01/19/2017 11:30 AM, Scott Branden wrote: >>> Hi Rob, >>> >>> On 17-01-19 09:36 AM, Rob Herring wrote: >>>> On Tue, Jan

Re: [RFC v2 4/5] DT bindings documentation for Synopsys UDC platform driver

2017-01-19 Thread Florian Fainelli
On 01/19/2017 11:30 AM, Scott Branden wrote: > Hi Rob, > > On 17-01-19 09:36 AM, Rob Herring wrote: >> On Tue, Jan 17, 2017 at 01:35:07PM +0530, Raviteja Garimella wrote: >>> This patch adds device tree bindings documentation for Synopsys >>> USB device controller platform driver. >> >> Bindings

Re: [PATCH RFC 0/7] Clean up PHY MMD accessors

2017-01-19 Thread Florian Fainelli
On 01/13/2017 07:20 AM, Russell King - ARM Linux wrote: > This series cleans up phylib's MMD accessors, so that we have a common > way of accessing the Clause 45 register set. > > The current situation is far from ideal - we have phy_(read|write)_mmd() > which accesses Clause 45 registers over

Re: [RFC v2 1/5] UDC: Split the driver into amd (pci) and Synopsys core driver

2017-01-19 Thread Florian Fainelli
On 01/19/2017 02:44 AM, Raviteja Garimella wrote: > Hi, > > On Thu, Jan 19, 2017 at 12:15 AM, Florian Fainelli <f.faine...@gmail.com> > wrote: >> On 01/17/2017 12:05 AM, Raviteja Garimella wrote: >>> This patch splits the amd5536udc driver into two -- one that

Re: [RFC v2 1/5] UDC: Split the driver into amd (pci) and Synopsys core driver

2017-01-18 Thread Florian Fainelli
On 01/17/2017 12:05 AM, Raviteja Garimella wrote: > This patch splits the amd5536udc driver into two -- one that does > pci device registration and the other file that does the rest of > the driver tasks like the gadget/ep ops etc for Synopsys UDC. > > This way of splitting helps in exporting

Re: [RFC 1/1] Changes to support the driver for platform device registration

2017-01-05 Thread Florian Fainelli
On 01/05/2017 12:23 AM, Raviteja Garimella wrote: > -- Add OF based platform device registration > -- Modify debug prints to be compatible with both pci and platform devices > -- Add members to 'struct udc' for extcon and phy support > -- Add checks to not process repeated calls to udc connect and

[PATCH 14/15] net: usb: ax88172x: Utilize phy_ethtool_nway_reset

2016-11-15 Thread Florian Fainelli
Signed-off-by: Florian Fainelli <f.faine...@gmail.com> --- drivers/net/usb/ax88172a.c | 10 +- 1 file changed, 1 insertion(+), 9 deletions(-) diff --git a/drivers/net/usb/ax88172a.c b/drivers/net/usb/ax88172a.c index 49a3bc107d05..6308386b09df 100644 --- a/drivers/net/usb/ax88

[PATCH 15/15] net: usb: lan78xx: Utilize phy_ethtool_nway_reset

2016-11-15 Thread Florian Fainelli
Signed-off-by: Florian Fainelli <f.faine...@gmail.com> --- drivers/net/usb/lan78xx.c | 7 +-- 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/drivers/net/usb/lan78xx.c b/drivers/net/usb/lan78xx.c index bcd9010c1f27..cf2857fa938f 100644 --- a/drivers/net/usb/lan78xx.c

Re: [PATCH V2 2/2] usb: phy: phy-brcm-usb: Add Broadcom STB USB Phy driver

2016-07-11 Thread Florian Fainelli
> +++ b/MAINTAINERS > @@ -2631,6 +2631,13 @@ S: Maintained > F: drivers/bcma/ > F: include/linux/bcma/ > > +BROADCOM STB USB PHY DRIVER > +M: Al Cooper <alcoop...@gmail.com> > +L: linux-usb@vger.kernel.org > +L: bcm-kernel-feedback-l...@broadcom

Re: [PATCH V2 1/2] soc: brcmstb: Add Product ID and Family ID helper functions

2016-07-11 Thread Florian Fainelli
On 05/27/2016 12:57 PM, Al Cooper wrote: > Signed-off-by: Al Cooper > --- > drivers/soc/brcmstb/common.c| 12 > include/linux/soc/brcmstb/brcmstb.h | 10 ++ > 2 files changed, 22 insertions(+) > > diff --git a/drivers/soc/brcmstb/common.c

Re: [PATCH] usb: phy: bcm-ns-usb3: new driver for USB 3.0 PHY on Northstar

2016-03-28 Thread Florian Fainelli
CC: bcm-kernel-feedback-list, Jon Le 28/03/2016 14:59, Rafał Miłecki a écrit : > Northstar is a family of SoCs used in home routers. They have USB 2.0 > and 3.0 controllers with PHYs that need to be properly initialized. > This driver provides PHY init support in a generic way and can be bound >

Re: [PATCH v3 5/6] usb: bcma: add USB 3.0 support

2015-12-01 Thread Florian Fainelli
On 25/10/15 15:52, Hauke Mehrtens wrote: > On 10/24/2015 12:40 AM, Florian Fainelli wrote: >> On 23/10/15 14:37, Hauke Mehrtens wrote: >>> From: Rafał Miłecki <zaj...@gmail.com> >>> >>> Signed-off-by: Rafał Miłecki <zaj...@gmail.com> >

Re: [PATCH v3 5/6] usb: bcma: add USB 3.0 support

2015-10-23 Thread Florian Fainelli
On 23/10/15 14:37, Hauke Mehrtens wrote: > From: Rafał Miłecki > > Signed-off-by: Rafał Miłecki > Signed-off-by: Hauke Mehrtens > --- [snip] > + > + switch (chipinfo->id) { > + case BCMA_CHIP_ID_BCM4707: All the code above cries

Re: [PATCH 8/9] bus: brcmstb_gisb: Honor the big-endian and native-endian DT properties

2015-05-28 Thread Florian Fainelli
Le 11/25/14 16:49, Kevin Cernekee a écrit : On chips strapped for BE, we'll need to use ioread32be/iowrite32be instead of ioread32/iowrite32. Signed-off-by: Kevin Cernekee cerne...@gmail.com Applied to soc/next, thanks! -- Florian -- To unsubscribe from this list: send the line unsubscribe

Re: [RFC PATCH 0/3] Enable connecting DSA-based switch to the USB RMII interface.

2015-04-21 Thread Florian Fainelli
On 21/04/15 10:39, Andrew Lunn wrote: I would however say that sysfs is the wrong API. The linux network stack uses netlink for most configuration activities. So i would suggest adding a netlink binding to DSA, and place the code in net/dsa/, not within an MDIO driver. I suppose we could do

Re: [RFC PATCH 0/3] Enable connecting DSA-based switch to the USB RMII interface.

2015-04-21 Thread Florian Fainelli
On 21/04/15 05:47, Andrew Lunn wrote: Hi Jan Interesting work, but i think the architecture is wrong. DSA needs an Ethernet device, an MDIO bus, and information about ports on the switch. That requirement is completely artificial as it is today, and just comes from arbitrary limitations

Re: [RFC PATCH 0/3] Enable connecting DSA-based switch to the USB RMII interface.

2015-04-21 Thread Florian Fainelli
On 21/04/15 10:30, Andrew Lunn wrote: My goal in reworking this weird DSA device/driver model is that you could just register your switch devices as an enhanced phy_driver/spi_driver/pci_driver etc..., such that libphy-ready drivers could just take advantage of that when they scan/detect their

Re: [RFC PATCH 1/3] net/dsa: Refactor dsa_probe()

2015-04-21 Thread Florian Fainelli
On 21/04/15 06:26, Jan Kaisrlik wrote: From: Jan Kaisrlik ja.kaisr...@gmail.com This patch refactors dsa_probe in order to simplify code in the patch 2/3. It does not look like you are working on the latest net-next tree, that part of the code has already been refactored to have separate

Re: USB device nodes in device tree

2015-04-04 Thread Florian Fainelli
Le 30/03/2015 12:35, Alan Stern a écrit : On Mon, 30 Mar 2015, Valentin Longchamp wrote: Hello, We are currently developing a board with an USB MFD device (I2C and GPIOs are to be supported). The device is soldered on the board and is the only one on the bus, so the bus is not really

Re: [PATCH 8/9] bus: brcmstb_gisb: Honor the big-endian and native-endian DT properties

2015-02-09 Thread Florian Fainelli
On 25/11/14 16:49, Kevin Cernekee wrote: On chips strapped for BE, we'll need to use ioread32be/iowrite32be instead of ioread32/iowrite32. Has of_device_is_big_endian() been merged in a tree now, I am not seeing it in Linus' tree, but have not look at Grant's tree yet. Thanks Signed-off-by:

Re: [PATCH 0/9] Extend various drivers to run on bi-endian BMIPS hosts

2014-11-25 Thread Florian Fainelli
might want to review now, merge later. For the entire series: Acked-by: Florian Fainelli f.faine...@gmail.com I will probably take the brcmstb_gisb.c changes separately and submit them to arm-soc as a drivers pull request since this driver has typically been routed that way. Thanks! Kevin

Re: [PATCH 3/3] USB: host: Introduce flag to enable use of 64-bit dma_mask for ehci-platform

2014-11-14 Thread Florian Fainelli
On 11/14/2014 07:23 AM, Alan Stern wrote: On Fri, 14 Nov 2014, Andreas Herrmann wrote: On Thu, Nov 13, 2014 at 08:44:17PM -0800, Florian Fainelli wrote: 2014-11-13 13:36 GMT-08:00 Andreas Herrmann andreas.herrm...@caviumnetworks.com: ehci-octeon driver used a 64-bit dma_mask. With removal

Re: [PATCH 3/3] USB: host: Introduce flag to enable use of 64-bit dma_mask for ehci-platform

2014-11-13 Thread Florian Fainelli
2014-11-13 13:36 GMT-08:00 Andreas Herrmann andreas.herrm...@caviumnetworks.com: ehci-octeon driver used a 64-bit dma_mask. With removal of ehci-octeon and usage of ehci-platform ehci dma_mask is now limited to 32 bits (coerced in ehci_platform_probe). Provide a flag in ehci platform data to

Re: [PATCH 3/3] usb host/MIPS: Remove hard-coded OCTEON platform information.

2014-05-29 Thread Florian Fainelli
2014-05-29 8:03 GMT-07:00 Alan Stern st...@rowland.harvard.edu: On Thu, 29 May 2014, Alex Smith wrote: From: David Daney david.da...@cavium.com The device tree will *always* have correct ehci/ohci clock configuration, so use it. This allows us to remove a big chunk of platform

Re: [PATCH net-next 07/14] r8152: combine PHY reset with set_speed

2014-02-18 Thread Florian Fainelli
Hi Hayes, 2014-02-18 5:49 GMT-08:00 Hayes Wang hayesw...@realtek.com: PHY reset is necessary after some hw settings. However, it would cause the linking down, and so does the set_speed function. Combine the PHY reset with set_speed function. That could reduce the frequency of linking down and

Re: [PATCH] usb: gadget: bcm63xx_udc: fix build failure on DMA channel code

2014-02-14 Thread Florian Fainelli
2014-01-21 17:30 GMT-08:00 Greg KH gre...@linuxfoundation.org: On Tue, Jan 21, 2014 at 11:20:09AM -0800, Florian Fainelli wrote: 2014/1/14 Florian Fainelli flor...@openwrt.org: Commit 3dc6475 (bcm63xx_enet: add support Broadcom BCM6345 Ethernet) changed the ENETDMA[CS] macros

Re: ohci-/ehci-platform: Change compatible string to ?hci-platform

2014-02-11 Thread Florian Fainelli
Le mardi 11 février 2014, 16:43:37 Arnd Bergmann a écrit : On Tuesday 11 February 2014 10:27:12 Alan Stern wrote: It might even be a good idea to change the xhci-platform string to match, it that doesn't cause too much trouble. The original xhci binding was contributed by Al Cooper, but I

Re: [PATCH] usb: gadget: bcm63xx_udc: fix build failure on DMA channel code

2014-01-21 Thread Florian Fainelli
2014/1/14 Florian Fainelli flor...@openwrt.org: Commit 3dc6475 (bcm63xx_enet: add support Broadcom BCM6345 Ethernet) changed the ENETDMA[CS] macros such that they are no longer macros, but actual register offset definitions. The bcm63xx_udc driver was not updated, and as a result, causes

Re: [PATCH 1/2] ohci-platform: Add support for controllers with big-endian regs / descriptors

2014-01-21 Thread Florian Fainelli
2014/1/21 Alan Stern st...@rowland.harvard.edu: On Tue, 21 Jan 2014, Hans de Goede wrote: Hi, On 01/21/2014 05:40 PM, Alan Stern wrote: On Tue, 21 Jan 2014, Hans de Goede wrote: Note this commit uses the same devicetree booleans for this as the ones already existing in the usb-ehci

Re: [PATCH] usb: gadget: bcm63xx_udc: fix build failure on DMA channel code

2014-01-21 Thread Florian Fainelli
2014/1/21 Greg KH gre...@linuxfoundation.org: On Tue, Jan 21, 2014 at 11:20:09AM -0800, Florian Fainelli wrote: 2014/1/14 Florian Fainelli flor...@openwrt.org: Commit 3dc6475 (bcm63xx_enet: add support Broadcom BCM6345 Ethernet) changed the ENETDMA[CS] macros such that they are no longer

Re: [PATCH v7 0/2] ohci and ehci-platform clks, phy and dt support

2014-01-16 Thread Florian Fainelli
Le mercredi 15 janvier 2014, 15:26:21 Alan Stern a écrit : On Wed, 15 Jan 2014, Hans de Goede wrote: Hi All, This version of my ohci and ehci-platform clks, phy and dt support patch-set, really fixes the 2 small bugs Alan found. All okay -- this time I can't find anything to complain

Re: [PATCH v7 2/2] ehci-platform: Add support for clks and phy passed through devicetree

2014-01-15 Thread Florian Fainelli
2014/1/15 Hans de Goede hdego...@redhat.com: Currently ehci-platform is only used in combination with devicetree when used with some Via socs. By extending it to (optionally) get clks and a phy from devicetree, and enabling / disabling those on power_on / off, it can be used more generically.

[PATCH] usb: gadget: bcm63xx_udc: fix build failure on DMA channel code

2014-01-14 Thread Florian Fainelli
, hence doing again what the macro implicitely did for us. CC: Kevin Cernekee cerne...@gmail.com CC: Jonas Gorski j...@openwrt.org CC: sta...@vger.kernel.org Signed-off-by: Florian Fainelli flor...@openwrt.org --- Felipe, This is against your branch as balbi/usb.git, and this fix should be applied

Re: [PATCH v2 1/2] ohci-platform: Add support for devicetree instantiation

2014-01-08 Thread Florian Fainelli
Hello, 2014/1/8 Hans de Goede hdego...@redhat.com: Add support for ohci-platform instantiation from devicetree, including optionally getting clks and a phy from devicetree, and enabling / disabling those on power_on / off. This should allow using ohci-platform from devicetree in various

[PATCH 3/5 v4] USB: enclose EHCI HCD drivers within an if USB_EHCI_HCD block

2013-04-09 Thread Florian Fainelli
...@rowland.harvard.edu Signed-off-by: Florian Fainelli flor...@openwrt.org --- Changes in v4: - added Alan's Acked-by tag - refreshed against latest usb-next Changes in v3: - move out USB_FSL_MPH_DR_OF from the USB_EHCI_HCD block - remove depends on USB_EHCI_HCD for USB_EHCI_BIG_ENDIAN_DESC drivers/usb/host

[PATCH 4/5 v4] USB: enclose all depends on USB_OHCI_HCD within an if USB_OHCI_HCD block

2013-04-09 Thread Florian Fainelli
This patch removes the various depends on USB_OHCI_HCD from the OHCI HCD drivers and enclose them within an if USB_OHCI_HCD / endif block. The Octeon OHCI HCD driver has been moved around to remain in this block. Acked-by: Alan Stern st...@rowland.harvard.edu Signed-off-by: Florian Fainelli flor

[PATCH 1/5 v4] USB: regroup all depends on USB within an if USB block

2013-04-09 Thread Florian Fainelli
item. No functionnal change is introduced. Acked-by: Alan Stern st...@rowland.harvard.edu Signed-off-by: Florian Fainelli flor...@openwrt.org --- Changes since v3: - added Alan's Acked-by tag - refreshed against usb-next Changes since v2: - properly keep USB_MUSB_HDRC dependency Changes since v1

[PATCH 5/5 v4] USB: enclose USB_XHCI_HCD related symbols within a if USB_XHCI_HCD block

2013-04-09 Thread Florian Fainelli
This patch encloses all symbols depending on USB_XHCI_HCD within an if USB_XHCI_HCD / endif block. Acked-by: Alan Stern st...@rowland.harvard.edu Signed-off-by: Florian Fainelli flor...@openwrt.org --- Changes in v4: - refreshed against latest usb-next Changes in v3: - added Alan's Acked-by tag

[PATCH 0/5 v4] USB: Kconfig cleanups

2013-04-09 Thread Florian Fainelli
These 5 patches contain my Kconfig cleanup on which I based the removal of the USB_ARCH_HAS_* patches. They have been suggested by Alan Stern as part of an earlier conversations. Let me know what you think about it so I can post subsequent work based on it. Florian Fainelli (5): USB: regroup

[PATCH 2/5 v4] USB: remove USB_EHCI_BIG_ENDIAN_{DESC,MMIO} depends on architecture symbol

2013-04-09 Thread Florian Fainelli
to be added for these configuration symbols to be selected. Now it is up to the EHCI driver and/or platform to select these configuration symbols accordingly. Acked-by: Alan Stern st...@rowland.harvard.edu Signed-off-by: Florian Fainelli flor...@openwrt.org --- Changes in v4: - refreshed against latest usb

Re: [PATCH 0/5 v3] USB: Kconfig cleanups

2013-04-08 Thread Florian Fainelli
Le 08/04/2013 18:32, Greg KH a écrit : On Thu, Apr 04, 2013 at 05:57:23PM +0200, Florian Fainelli wrote: These 5 patches contain my Kconfig cleanup on which I based the removal of the USB_ARCH_HAS_* patches. They have been suggested by Alan Stern as part of an earlier conversations. Let me

[PATCH 5/5 v3] USB: enclose USB_XHCI_HCD related symbols within a if USB_XHCI_HCD block

2013-04-04 Thread Florian Fainelli
This patch encloses all symbols depending on USB_XHCI_HCD within an if USB_XHCI_HCD / endif block. Acked-by: Alan Stern st...@rowland.harvard.edu Signed-off-by: Florian Fainelli flor...@openwrt.org --- Changes in v3: - added Alan's Acked-by tag drivers/usb/host/Kconfig |6 -- 1 file

[PATCH 0/5 v3] USB: Kconfig cleanups

2013-04-04 Thread Florian Fainelli
These 5 patches contain my Kconfig cleanup on which I based the removal of the USB_ARCH_HAS_* patches. They have been suggested by Alan Stern as part of an earlier conversations. Let me know what you think about it so I can post subsequent work based on it. Thanks! Fainelli (5): USB: regroup

[PATCH 1/5 v3] USB: regroup all depends on USB within an if USB block

2013-04-04 Thread Florian Fainelli
item. No functionnal change is introduced. Signed-off-by: Florian Fainelli flor...@openwrt.org --- Changes since v2: - properly keep USB_MUSB_HDRC dependency Changes since v1: - add missing if USB in drivers/usb/Kconfig before USB_USS720 Changes since RFC: - only remove the depends on USB

[PATCH 2/5 v3] USB: remove USB_EHCI_BIG_ENDIAN_{DESC,MMIO} depends on architecture symbol

2013-04-04 Thread Florian Fainelli
to be added for these configuration symbols to be selected. Now it is up to the EHCI driver and/or platform to select these configuration symbols accordingly. Acked-by: Alan Stern st...@rowland.harvard.edu Signed-off-by: Florian Fainelli flor...@openwrt.org --- Changes in v2: - added Alan's Acked-by tag

[PATCH 3/5 v3] USB: enclose EHCI HCD drivers within an if USB_EHCI_HCD block

2013-04-04 Thread Florian Fainelli
Thist patch removes the depends on USB_EHCI_HCD that the various USB EHCI HCD drivers use and encloses every driver within an if USB_EHCI_HCD / endif block. The EHCI HCD platform and Octeon drivers have been moved around to remain enclosed within this block. Signed-off-by: Florian Fainelli flor

[PATCH 4/5 v3] USB: enclose all depends on USB_OHCI_HCD within an if USB_OHCI_HCD block

2013-04-04 Thread Florian Fainelli
This patch removes the various depends on USB_OHCI_HCD from the OHCI HCD drivers and enclose them within an if USB_OHCI_HCD / endif block. The Octeon OHCI HCD driver has been moved around to remain in this block. Acked-by: Alan Stern st...@rowland.harvard.edu Signed-off-by: Florian Fainelli flor

Re: [PATCH 1/5 v2] USB: regroup all depends on USB within an if USB block

2013-04-03 Thread Florian Fainelli
Le 04/02/13 20:19, ba...@ti.com a écrit : Hi, On Tue, Apr 02, 2013 at 07:10:22PM +0200, Florian Fainelli wrote: diff --git a/drivers/usb/musb/Kconfig b/drivers/usb/musb/Kconfig index 05e5143..ab5a3b9 100644 --- a/drivers/usb/musb/Kconfig +++ b/drivers/usb/musb/Kconfig @@ -6,7 +6,6 @@ # (M

Re: [PATCH 1/5 v2] USB: regroup all depends on USB within an if USB block

2013-04-03 Thread Florian Fainelli
Le 04/02/13 20:06, Alan Stern a écrit : On Tue, 2 Apr 2013, Florian Fainelli wrote: This patch removes the depends on USB from all config symbols in drivers/usb/host/Kconfig and replace that with an if USB / endif block as suggested by Alan Stern. Some source ... Kconfig lines have been

Re: [PATCH 1/5 v2] USB: regroup all depends on USB within an if USB block

2013-04-03 Thread Florian Fainelli
Le 04/03/13 14:15, Felipe Balbi a écrit : On Wed, Apr 03, 2013 at 02:11:13PM +0200, Florian Fainelli wrote: Le 04/02/13 20:06, Alan Stern a écrit : On Tue, 2 Apr 2013, Florian Fainelli wrote: This patch removes the depends on USB from all config symbols in drivers/usb/host/Kconfig

[PATCH 1/5] USB: regroup all depends on USB within an if USB block

2013-04-02 Thread Florian Fainelli
item. No functionnal change is introduced. Signed-off-by: Florian Fainelli flor...@openwrt.org --- Changes since RFC: - only remove the depends on USB conditionnals and not more as the RFC patch did drivers/usb/Kconfig| 21 + drivers/usb/atm/Kconfig

[PATCH 0/5] USB: Kconfig cleanups

2013-04-02 Thread Florian Fainelli
the RFC from the subject. Thanks! Florian Fainelli (5): USB: regroup all depends on USB within an if USB block USB: remove USB_EHCI_BIG_ENDIAN_{DESC,MMIO} depends on architecture symbol USB: enclose EHCI HCD drivers within an if USB_EHCI_HCD block USB: enclose all depends

[PATCH 2/5] USB: remove USB_EHCI_BIG_ENDIAN_{DESC,MMIO} depends on architecture symbol

2013-04-02 Thread Florian Fainelli
to be added for these configuration symbols to be selected. Now it is up to the EHCI driver and/or platform to select these configuration symbols accordingly. Signed-off-by: Florian Fainelli flor...@openwrt.org --- Changes since RFC: - dropped default n for USB_EHCI_BIG_ENDIAN_{MMIO_DESC

[PATCH 5/5] USB: enclose USB_XHCI_HCD related symbols within a if USB_XHCI_HCD block

2013-04-02 Thread Florian Fainelli
This patch encloses all symbols depending on USB_XHCI_HCD within an if USB_XHCI_HCD / endif block. Signed-off-by: Florian Fainelli flor...@openwrt.org --- drivers/usb/host/Kconfig |6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/usb/host/Kconfig b/drivers/usb

[PATCH 3/5] USB: enclose EHCI HCD drivers within an if USB_EHCI_HCD block

2013-04-02 Thread Florian Fainelli
Thist patch removes the depends on USB_EHCI_HCD that the various USB EHCI HCD drivers use and encloses every driver within an if USB_EHCI_HCD / endif block. The EHCI HCD platform and Octeon drivers have been moved around to remain enclosed within this block. Signed-off-by: Florian Fainelli flor

[PATCH 4/5] USB: enclose all depends on USB_OHCI_HCD within an if USB_OHCI_HCD block

2013-04-02 Thread Florian Fainelli
This patch removes the various depends on USB_OHCI_HCD from the OHCI HCD drivers and enclose them within an if USB_OHCI_HCD / endif block. The Octeon OHCI HCD driver has been moved around to remain in this block. Signed-off-by: Florian Fainelli flor...@openwrt.org --- drivers/usb/host/Kconfig

[PATCH 2/5 v2] USB: remove USB_EHCI_BIG_ENDIAN_{DESC,MMIO} depends on architecture symbol

2013-04-02 Thread Florian Fainelli
to be added for these configuration symbols to be selected. Now it is up to the EHCI driver and/or platform to select these configuration symbols accordingly. Signed-off-by: Florian Fainelli flor...@openwrt.org --- Changes since RFC: - dropped default n for USB_EHCI_BIG_ENDIAN_{MMIO_DESC

Re: [PATCH 0/5] USB: Kconfig cleanups

2013-04-02 Thread Florian Fainelli
Le 04/02/13 19:05, Florian Fainelli a écrit : Hello Alan, Greg, These 5 patches contain my Kconfig cleanup on which I based the removal of the USB_ARCH_HAS_* patches. They have been suggested by Alan Stern as part of an earlier conversations. Let me know what you think about it so I can post

[PATCH 1/5 v2] USB: regroup all depends on USB within an if USB block

2013-04-02 Thread Florian Fainelli
item. No functionnal change is introduced. Signed-off-by: Florian Fainelli flor...@openwrt.org --- Changes since v1: - add missing if USB in drivers/usb/Kconfig before USB_USS720 Changes since RFC: - only remove the depends on USB conditionnals and not more as the RFC patch did drivers/usb

[PATCH 0/5 v2] USB: Kconfig cleanups

2013-04-02 Thread Florian Fainelli
the RFC from the subject. Thanks! Florian Fainelli (5): USB: regroup all depends on USB within an if USB block USB: remove USB_EHCI_BIG_ENDIAN_{DESC,MMIO} depends on architecture symbol USB: enclose EHCI HCD drivers within an if USB_EHCI_HCD block USB: enclose all depends

[PATCH 3/5 v2] USB: enclose EHCI HCD drivers within an if USB_EHCI_HCD block

2013-04-02 Thread Florian Fainelli
Thist patch removes the depends on USB_EHCI_HCD that the various USB EHCI HCD drivers use and encloses every driver within an if USB_EHCI_HCD / endif block. The EHCI HCD platform and Octeon drivers have been moved around to remain enclosed within this block. Signed-off-by: Florian Fainelli flor

[PATCH 4/5 v2] USB: enclose all depends on USB_OHCI_HCD within an if USB_OHCI_HCD block

2013-04-02 Thread Florian Fainelli
This patch removes the various depends on USB_OHCI_HCD from the OHCI HCD drivers and enclose them within an if USB_OHCI_HCD / endif block. The Octeon OHCI HCD driver has been moved around to remain in this block. Signed-off-by: Florian Fainelli flor...@openwrt.org --- drivers/usb/host/Kconfig

[PATCH 5/5 v2] USB: enclose USB_XHCI_HCD related symbols within a if USB_XHCI_HCD block

2013-04-02 Thread Florian Fainelli
This patch encloses all symbols depending on USB_XHCI_HCD within an if USB_XHCI_HCD / endif block. Signed-off-by: Florian Fainelli flor...@openwrt.org --- drivers/usb/host/Kconfig |6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/usb/host/Kconfig b/drivers/usb

[RFC PATCH 0/5] USB: Kconfig cleanups

2013-03-26 Thread Florian Fainelli
Hello Alan, Greg, These 5 patches contain my Kconfig cleanup on which I based the removal of the USB_ARCH_HAS_* patches. They have been suggested by Alan Stern as part of an earlier conversations. Let me know what you think about it so I can post subsequent work based on it. Thanks! Florian

[RFC PATCH 2/5] USB: remove USB_EHCI_BIG_ENDIAN_{DESC,MMIO} depends on architecture symbol

2013-03-26 Thread Florian Fainelli
to be added for these configuration symbols to be selected. Now it is up to the EHCI driver and/or platform to select these configuration symbols accordingly. Signed-off-by: Florian Fainelli flor...@openwrt.org --- arch/arm/Kconfig|2 ++ arch/mips/Kconfig |3

[RFC PATCH 5/5] USB: enclose USB_XHCI_HCD related symbols within a if USB_XHCI_HCD block

2013-03-26 Thread Florian Fainelli
This patch encloses all symbols depending on USB_XHCI_HCD within an if USB_XHCI_HCD / endif block. Signed-off-by: Florian Fainelli flor...@openwrt.org --- drivers/usb/host/Kconfig |6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/usb/host/Kconfig b/drivers/usb

  1   2   3   >