Re: [PATCH v2 1/2] usb: ohci-at91: Forcibly suspend ports while USB suspend

2016-06-03 Thread Nicolas Ferre
e why you need this now, but that is irrelevant to having a chip >> specific >> compatible string. > > Maybe a property is better. It seems that it's not an issue to the binding that Rob is talking about but the description that you used: "for SAMA5D2 which can force su

Re: [PATCH v3 1/2] usb: ohci-at91: Forcibly suspend ports while USB suspend

2016-06-08 Thread Nicolas Ferre
+ * it under the terms of the GNU General Public License version 2 as > + * published by the Free Software Foundation. > + * > + */ > +#ifndef __AT91_SFR_H__ > +#define __AT91_SFR_H__ > + > +#define SFR_DDRCFG 0x04/* DDR Configuration Register */ > +/* 0x08 ~ 0x

Re: [PATCH v3 2/2] ARM: at91/dt: sama5d2: Use new compatible for ohci node

2016-06-08 Thread Nicolas Ferre
0x0040 0x100000>; > interrupts = <41 IRQ_TYPE_LEVEL_HIGH 2>; > clocks = <&uhphs_clk>, <&uhphs_clk>, <&uhpck>; > Bye, -- Nicolas Ferre -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH v3 1/2] usb: ohci-at91: Forcibly suspend ports while USB suspend

2016-06-08 Thread Nicolas Ferre
Le 08/06/2016 12:04, Nicolas Ferre a écrit : > Le 08/06/2016 06:15, Wenyou Yang a écrit : >> In order to the save power consumption, as a workaround, suspend >> forcibly the USB PORTA/B/C via set the SUSPEND_A/B/C bits of OHCI >> Interrupt Configuration Register in the SFRs w

Re: [PATCH] usb: ohci-at91: make at91_dt_syscon_sfr static

2016-06-17 Thread Nicolas Ferre
tatic? > > Signed-off-by: Ben Dooks > --- > Cc: Alan Stern > Cc: Greg Kroah-Hartman > Cc: linux-usb@vger.kernel.org > Cc: Nicolas Ferre > Cc: Alexandre Belloni Acked-by: Nicolas Ferre > Cc: Jean-Christophe Plagniol-Villard > --- > drivers/usb/host/ohci-a

Re: [PATCH v3 1/2] usb: ohci-at91: Forcibly suspend ports while USB suspend

2016-06-20 Thread Nicolas Ferre
his workaround/feature... well, it's simple, we don't look for the SFR, we don't use the bits, and we come back to the situation that we've always experienced ; with the same compatibility sting for OHCI as the IP never actually changed... In conclusion: try Alexandre'

Re: [PATCH v4] usb: ohci-at91: Forcibly suspend ports while USB suspend

2016-06-21 Thread Nicolas Ferre
be done before the USB clock is disabled, > resume after the USB clock is enabled. > > Signed-off-by: Wenyou Yang Acked-by: Nicolas Ferre Alan, can you take it as it doesn't have dependency on the at91 material anymore. Thanks, bye. > --- > > Changes in v4: > -

Re: [PATCH 22/25] usb/at91_udc: don't use [delayed_]work_pending()

2013-01-07 Thread Nicolas Ferre
Only compile tested. > > Signed-off-by: Tejun Heo > Cc: Andrew Victor > Cc: Nicolas Ferre Acked-by: Nicolas Ferre > Cc: Jean-Christophe Plagniol-Villard > Cc: Felipe Balbi > Cc: linux-usb@vger.kernel.org > --- > Please let me know how this patch should be routed. I can ta

Re: [PATCH 2/2] USB: gadget: at91_adc: fix pullup pin validity check

2013-01-07 Thread Nicolas Ferre
On 12/23/2012 07:08 PM, Jean-Christophe PLAGNIOL-VILLARD : > Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD Acked-by: Nicolas Ferre > Cc: linux-usb@vger.kernel.org > --- > drivers/usb/gadget/at91_udc.c |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > dif

Re: [PATCH] usb: gadget: udc: atmel: used managed kasprintf

2016-12-01 Thread Nicolas Ferre
Le 01/12/2016 à 11:26, Alexandre Belloni a écrit : > Use devm_kasprintf instead of simple kasprintf to free the allocated memory > when needed. > > Suggested-by: Peter Rosin > Signed-off-by: Alexandre Belloni Acked-by: Nicolas Ferre > --- > drivers/usb/gadget/udc

Re: [PATCH] usb: gadget: udc: atmel: fix endpoint name

2016-09-16 Thread Nicolas Ferre
008078) > ---[ end trace e7cf9dcebf4815a6 ]--- > > Fixes: c32b5bcfa3c4 ("ARM: dts: at91: Fix USB endpoint nodes") > Reported-by: Richard Genoud > Signed-off-by: Alexandre Belloni Acked-by: Nicolas Ferre Felipe, Greg, It is clearly a regression and material for 4.8-fixe

Re: [PATCH] usb: gadget: udc: atmel: fix endpoint name

2016-09-23 Thread Nicolas Ferre
Le 16/09/2016 à 10:36, Nicolas Ferre a écrit : > Le 15/09/2016 à 17:07, Alexandre Belloni a écrit : >> Since commit c32b5bcfa3c4 ("ARM: dts: at91: Fix USB endpoint nodes"), >> atmel_usba_udc fails with: >> >> [ cut here ] >>

Re: [PATCH] usb: Convert pr_warning to pr_warn

2016-09-28 Thread Nicolas Ferre
rs/usb/gadget/function/u_serial.c | 4 ++-- > drivers/usb/gadget/udc/at91_udc.h | 2 +- > drivers/usb/gadget/udc/atmel_usba_udc.c | 4 ++-- For AT91 and Atmel parts: Acked-by: Nicolas Ferre Thanks > drivers/usb/gadget/udc/fsl_usb2_udc.h | 2 +- > drivers/usb/gadget/

Re: [RFC/PATCH 35/45] usb: gadget: udc: atmel: remove unnecessary & operation

2016-10-03 Thread Nicolas Ferre
Le 28/09/2016 à 15:05, Felipe Balbi a écrit : > Now that usb_endpoint_maxp() only returns the lowest > 11 bits from wMaxPacketSize, we can remove the & > operation from this driver. > > Cc: Nicolas Ferre Acked-by: Nicolas Ferre > Cc: > Signed-off-by: Felipe Balbi &g

Re: [RFC/PATCH 17/45] usb: gadget: udc: atmel: make use of new usb_endpoint_maxp_mult()

2016-10-03 Thread Nicolas Ferre
Le 28/09/2016 à 15:05, Felipe Balbi a écrit : > We have introduced a helper to calculate multiplier > value from wMaxPacketSize. Start using it. > > Cc: Nicolas Ferre Acked-by: Nicolas Ferre > Cc: > Signed-off-by: Felipe Balbi > --- > drivers/usb/gadget/udc/atme

Re: [PATCH] dt-bindings: usb: atmel: fix a couple of copy-paste style typos

2016-10-18 Thread Nicolas Ferre
Le 18/10/2016 à 13:05, Peter Rosin a écrit : > Signed-off-by: Peter Rosin Acked-by: Nicolas Ferre We may take it in a future at91-x.y-dt branch which will go through arm-soc. Thanks > --- > Documentation/devicetree/bindings/usb/atmel-usb.txt | 10 +- > 1 file changed,

Re: [PATCH] usb: gadget: udc: atmel: fix endpoint name

2016-10-24 Thread Nicolas Ferre
Le 26/09/2016 à 09:18, Felipe Balbi a écrit : > > Hi, > > Greg Kroah-Hartman writes: >> On Fri, Sep 23, 2016 at 04:20:45PM +0200, Nicolas Ferre wrote: >>> Le 16/09/2016 à 10:36, Nicolas Ferre a écrit : >>>> Le 15/09/2016 à 17:07, Alexandre Belloni a écrit

Re: [PATCH 17/82] usb: gadget: udc: atmel: make use of new usb_endpoint_maxp_mult()

2016-11-03 Thread Nicolas Ferre
Le 31/10/2016 à 11:48, Felipe Balbi a écrit : > We have introduced a helper to calculate multiplier > value from wMaxPacketSize. Start using it. > > Cc: Nicolas Ferre Acked-by: Nicolas Ferre > Cc: > Signed-off-by: Felipe Balbi > --- > drivers/usb/gadget/udc/atmel_usb

Re: [PATCH 36/82] usb: gadget: udc: atmel: remove unnecessary & operation

2016-11-03 Thread Nicolas Ferre
Le 31/10/2016 à 11:48, Felipe Balbi a écrit : > Now that usb_endpoint_maxp() only returns the lowest > 11 bits from wMaxPacketSize, we can remove the & > operation from this driver. > > Cc: Nicolas Ferre Acked-by: Nicolas Ferre > Cc: > Signed-off-by: Felipe Balbi &g

Re: [PATCH 3/6] usb: ohci-at91: use __maybe_unused to hide pm functions

2016-03-02 Thread Nicolas Ferre
#ifdef and instead uses a __maybe_unused > annotation to let the compiler know it can silently drop > the function definition. > > Signed-off-by: Arnd Bergmann Acked-by: Nicolas Ferre Thanks Arnd, bye. > --- > drivers/usb/host/ohci-at91.c | 8 +++- > 1 file changed

Re: [PATCH 4/6] usb: ehci-atmel: use __maybe_unused to hide pm functions

2016-03-02 Thread Nicolas Ferre
#ifdef and instead uses a __maybe_unused > annotation to let the compiler know it can silently drop > the function definition. > > Signed-off-by: Arnd Bergmann Acked-by: Nicolas Ferre > --- > drivers/usb/host/ehci-atmel.c | 6 ++ > 1 file changed, 2 insertions(+), 4 de

Re: [PATCH 1/2] usb: gadget: udc: at91: use PTR_ERR_OR_ZERO()

2016-03-29 Thread Nicolas Ferre
Le 29/03/2016 11:28, Felipe Balbi a écrit : > coccicheck found this pattern which could be > converted to PTR_ERR_OR_ZERO(). No functional > changes. > > Signed-off-by: Felipe Balbi Acked-by: Nicolas Ferre Thanks Felipe for having taken care of this! Bye, > --- > d

Re: [PATCH linux-next v2 1/1] usb: gadget: udc: atmel: Update endpoint allocation scheme

2017-01-26 Thread Nicolas Ferre
t;= 8) > + ep->ept_cfg = USBA_BF(EPT_SIZE, > USBA_EPT_SIZE_8); > + else > + /* LSB is bit 1, not 0 */ > + ep->ept_cfg = > + USBA_BF(EPT_SIZE, fls(ep->fifo_size - 1) - 3); > + > + ep->ept_cfg |= USBA_BF(BK_NUMBER, ep->nr_banks); > + } > + > if (i) > list_add_tail(&ep->ep.ep_list, &udc->gadget.ep_list); > > diff --git a/drivers/usb/gadget/udc/atmel_usba_udc.h > b/drivers/usb/gadget/udc/atmel_usba_udc.h > index b03b2eb..9551b70 100644 > --- a/drivers/usb/gadget/udc/atmel_usba_udc.h > +++ b/drivers/usb/gadget/udc/atmel_usba_udc.h > @@ -275,6 +275,12 @@ struct usba_dma_desc { > u32 ctrl; > }; > > +struct usba_fifo_cfg { > + u8 hw_ep_num; > + u16 fifo_size; > + u8 nr_banks; > +}; > + > struct usba_ep { > int state; > void __iomem*ep_regs; > @@ -293,7 +299,7 @@ struct usba_ep { > unsigned intcan_isoc:1; > unsigned intis_isoc:1; > unsigned intis_in:1; > - > + unsigned long ept_cfg; > #ifdef CONFIG_USB_GADGET_DEBUG_FS > u32 last_dma_status; > struct dentry *debugfs_dir; > @@ -338,6 +344,8 @@ struct usba_udc { > int vbus_pin; > int vbus_pin_inverted; > int num_ep; > + int configured_ep; > + struct usba_fifo_cfg *fifo_cfg; > struct clk *pclk; > struct clk *hclk; > struct usba_ep *usba_ep; > Regards, -- Nicolas Ferre -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH linux-next v2 1/1] usb: gadget: udc: atmel: Update endpoint allocation scheme

2017-01-27 Thread Nicolas Ferre
Le 27/01/2017 à 14:47, Cristian Birsan a écrit : > I will send a fixup patch with updates based on the comments received from > Nicolas. > > On 01/26/2017 05:02 PM, Nicolas Ferre wrote: >> Le 23/01/2017 à 15:45, cristian.bir...@microchip.com a écrit : >>> From: Cris

Re: [PATCH 4.10-rc3 03/13] net: macb: fix build errors when linux/phy*.h is removed from net/dsa.h

2017-02-01 Thread Nicolas Ferre
et/cadence/macb.h:862:2: error: unknown type name > 'phy_interface_t' > phy_interface_t phy_interface; > ^~~ > > Add linux/phy.h to macb.h > > Signed-off-by: Russell King Acked-by: Nicolas Ferre > --- > drivers/net/ethernet/cadence/macb.h

Re: [PATCH] usb: gadget: udc: atmel: fix debug output

2017-02-01 Thread Nicolas Ferre
bug output the same way as the other code. >> >> Fixes: 741d2558bf0a ("usb: gadget: udc: atmel: Update endpoint allocation >> scheme") >> Signed-off-by: Arnd Bergmann > Acked-by: Alexandre Belloni Acked-by: Nicolas Ferre > >> --- >> dri

Re: [PATCH 0/4 linux-next] usb: gadget: udc: atmel: Endpoint allocation scheme fixes

2017-02-15 Thread Nicolas Ferre
+--- > 2 files changed, 35 insertions(+), 17 deletions(-) I'm okay with the whole series: Acked-by: Nicolas Ferre Thanks, regards, -- Nicolas Ferre -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH] usb: ohci-at91: Do not drop unhandled USB suspend control requests

2017-02-21 Thread Nicolas Ferre
t; sleep and will not wakeup when an USB devices is inserted. > > Fixes: 2e2aa1bc7eff90ec ("usb: ohci-at91: Forcibly suspend ports while USB > suspend") > Signed-off-by: Jelle Martijn Kok > Tested-by: Wenyou Yang > Cc: Wenyou Yang > Cc: Alexandre Belloni > Cc:

Re: [PATCH linux-next v2 0/4] usb: gadget: udc: atmel: Endpoint allocation scheme fixes

2017-03-29 Thread Nicolas Ferre
patch As for previous revision, the whole series: Acked-by: Nicolas Ferre Regards, > Cristian Birsan (4): > usb: gadget: udc: atmel: Check fifo configuration values against > device tree > usb: gadget: udc: atmel: Minor code cleanup > usb: gadget: udc: atmel: Use dev_war

Re: [PATCH 1/6] usb: gadget: atmel: remove unused DMA_ADDR_INVALID

2013-03-14 Thread Nicolas Ferre
ipe, I see mush more use of DMA_ADDR_INVALID in this driver... Is there another patch series that I am missing? Bye, -- Nicolas Ferre -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH 1/6] usb: gadget: atmel: remove unused DMA_ADDR_INVALID

2013-03-14 Thread Nicolas Ferre
On 03/14/2013 09:56 AM, Felipe Balbi : > DMA_ADDR_INVALID isn't (and shouldn't) be used > anymore, let's remove it. > > Signed-off-by: Felipe Balbi Acked-by: Nicolas Ferre Thanks Felipe. Bye. > --- > drivers/usb/gadget/atmel_usba_udc.h | 6 -- > 1 fi

[PATCH] USB: ohci-at91: fix PIO handling in relation with number of ports

2012-08-29 Thread Nicolas Ferre
0ee6d1e (USB: ohci-at91: change maximum number of ports) that did not modified the associatd board files. Reported-by: Klaus Falkner Signed-off-by: Nicolas Ferre Cc: Alan Stern Cc: Greg Kroah-Hartman Cc: linux-usb@vger.kernel.org Cc: Stable [3.4+] --- drivers/usb/host/ohci-at91.c | 10

Re: [PATCH] USB: ohci-at91: fix PIO handling in relation with number of ports

2012-09-04 Thread Nicolas Ferre
On 08/29/2012 11:49 AM, Nicolas Ferre : > If the number of ports present on the SoC/board is not the maximum > and that the platform data is not filled with all data, there is > an easy way to mess the PIO setup for this interface. > This quick fix addresses mis-configuration in USB h

Re: [PATCH] usb: gadget: at91_udc: fix dt support

2012-09-07 Thread Nicolas Ferre
On 09/07/2012 03:27 PM, Fabio Porcedda : > Don't fail the initialization check for the platform_data > if there is avaiable an associated device tree node. > > Signed-off-by: Fabio Porcedda Acked-by: Nicolas Ferre Thanks, bye, > --- > drivers/usb/gadget/at91_udc.c |

Re: [PATCH v2] usb: gadget: at91_udc: dt: fix platform_data check

2012-09-11 Thread Nicolas Ferre
pport"). > > Tested on a at91sam9260 based board (PRO3-EVK). > > Signed-off-by: Fabio Porcedda > Cc: Stable [v3.4+] > Cc: Jean-Christophe PLAGNIOL-VILLARD Acked-by: Nicolas Ferre Felipe, Greg, can we queue this one for 3.6-final ? Best regards, > --- > v2: > -

Re: [PATCH v2] usb: gadget: at91_udc: dt: fix platform_data check

2012-09-12 Thread Nicolas Ferre
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 09/12/2012 08:14 AM, Felipe Balbi : > Hi, > > On Tue, Sep 11, 2012 at 02:20:19PM +0200, Nicolas Ferre wrote: >> On 09/11/2012 02:07 PM, Fabio Porcedda : >>> Don't fail the initialization check for the platform_dat

Re: [RFC 00/50] ARM: at91: move to common clk framework

2013-06-07 Thread Nicolas Ferre
at91/clk-system.c create mode 100644 drivers/clk/at91/clk-usb.c create mode 100644 drivers/clk/at91/clk-utmi.c rename arch/arm/mach-at91/include/mach/at91_pmc.h => include/linux/clk/at91.h (76%) -- Nicolas Ferre -- To unsubscribe from this list: send the line "unsubscribe linux-usb&q

Re: [PATCH] usb: gadget: at91_udc: add usb_clk for transition to common clk framework

2013-08-12 Thread Nicolas Ferre
BREZILLON Acked-by: Nicolas Ferre --- drivers/usb/gadget/at91_udc.c | 17 - drivers/usb/gadget/at91_udc.h |2 +- 2 files changed, 17 insertions(+), 2 deletions(-) diff --git a/drivers/usb/gadget/at91_udc.c b/drivers/usb/gadget/at91_udc.c index fce8e4e..ae06585 100644

Re: [PATCH v2] USB: ohci-at91: add usb_clk for transition to common clk framework

2013-08-12 Thread Nicolas Ferre
IS_ENABLED(CONFIG_COMMON_CLK) to isolate new at91 clk support Signed-off-by: Boris BREZILLON Well, for this v2 as well: Acked-by: Nicolas Ferre (even if it is too late) Bye, --- drivers/usb/host/ohci-at91.c | 23 +-- 1 file changed, 21 insertions(+), 2 deletions

Re: [PATCH] usb: gadget: at91_udc: add usb_clk for transition to common clk framework

2013-08-13 Thread Nicolas Ferre
On 12/08/2013 22:42, boris brezillon : Hello Nicolas, On 12/08/2013 15:52, Nicolas Ferre wrote: On 01/08/2013 08:18, Boris BREZILLON : The AT91 PMC (Power Management Controller) provides an USB clock used by USB Full Speed host (ohci) and USB Full Speed device (udc). The usb drivers (ohci and

Re: [PATCH V5 5/9] USB: OHCI: make ohci-at91 a separate driver

2013-09-23 Thread Nicolas Ferre
Signed-off-by: Deepak Saxena Acked-by: Alan Stern Otherwise, seems correct. Acked-by: Nicolas Ferre Cc: Arnd Bergmann Cc: Greg KH Cc: linux-usb@vger.kernel.org V3->V4: - Removed extra space after "tristate". - Removed extra space between function name and '('

Re: [PATCH 36/51] DMA-API: usb: use dma_set_coherent_mask()

2013-09-23 Thread Nicolas Ferre
the correct API. Signed-off-by: Russell King --- drivers/usb/chipidea/ci_hdrc_imx.c |5 +++-- drivers/usb/dwc3/dwc3-exynos.c |5 +++-- drivers/usb/gadget/lpc32xx_udc.c |4 +++- drivers/usb/host/ehci-atmel.c |5 +++-- For Atmel driver: Acked-by: Nicolas Ferre

Re: [PATCH 37/51] DMA-API: usb: use new dma_coerce_mask_and_coherent()

2013-09-23 Thread Nicolas Ferre
On 20/09/2013 00:02, Russell King : Signed-off-by: Russell King --- drivers/usb/chipidea/ci_hdrc_imx.c |4 +--- drivers/usb/dwc3/dwc3-exynos.c |4 +--- drivers/usb/host/ehci-atmel.c |4 +--- For Atmel driver: Acked-by: Nicolas Ferre [..] diff --git a/drivers/usb

Re: [PATCH 4/7] pcmcia: at91_cf: fix deferred probe from __init

2013-09-23 Thread Nicolas Ferre
ails. Cc: Jean-Christophe PLAGNIOL-VILLARD Cc: Nicolas Ferre Acked-by: Nicolas Ferre Thanks Johan. Signed-off-by: Johan Hovold --- drivers/pcmcia/at91_cf.c | 11 +-- 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/drivers/pcmcia/at91_cf.c b/drivers/pcmcia/at91_cf.

Re: [PATCH v4 01/17] ARM: at91: move at91_pmc.h to include/linux/clk/at91_pmc.h

2013-10-16 Thread Nicolas Ferre
-by: Boris BREZILLON Acked-by: Nicolas Ferre --- arch/arm/mach-at91/at91rm9200.c|2 +- arch/arm/mach-at91/at91sam9260.c |2 +- arch/arm/mach-at91/at91sam9261.c |2 +- arch/arm/mach-at91/at91sam9263.c

Re: [PATCH v4 02/17] ARM: at91: add Kconfig options for common clk support

2013-10-16 Thread Nicolas Ferre
Acked-by: Nicolas Ferre Thanks --- arch/arm/mach-at91/Kconfig| 21 + arch/arm/mach-at91/Kconfig.non_dt |6 ++ arch/arm/mach-at91/Makefile |2 +- arch/arm/mach-at91/generic.h |3 ++- arch/arm/mach-at91/setup.c|6 --

Re: [PATCH v4 03/17] clk: at91: add PMC base support

2013-10-17 Thread Nicolas Ferre
*irqdomain; +}; + +static inline void pmc_lock(struct at91_pmc *pmc) +{ + spin_lock(&pmc->lock); +} + +static inline void pmc_unlock(struct at91_pmc *pmc) +{ + spin_unlock(&pmc->lock); +} + +static inline u32 pmc_read(struct at91_pmc *pmc, int offset) +{ + return r

Re: [PATCH v4 04/17] clk: at91: add PMC macro file for dt definitions

2013-10-17 Thread Nicolas Ferre
On 11/10/2013 10:41, Boris BREZILLON : This patch adds a new macro file for PMC macros. This macro file includes the definitions of SR (status register) bit offsets and will be use to reference PMC irqs. Signed-off-by: Boris BREZILLON Acked-by: Nicolas Ferre --- include/dt-bindings/clk

Re: [PATCH v4 05/17] clk: at91: add PMC main clock

2013-10-17 Thread Nicolas Ferre
BREZILLON Acked-by: Nicolas Ferre --- drivers/clk/at91/Makefile |1 + drivers/clk/at91/clk-main.c | 189 +++ drivers/clk/at91/pmc.c |5 ++ drivers/clk/at91/pmc.h |3 + 4 files changed, 198 insertions(+) create mode 100644

Re: [PATCH v4 07/17] clk: at91: add PMC master clock

2013-10-17 Thread Nicolas Ferre
clock output rate These characteristics are checked during rate change to avoid over/underclocking. These characteristics are described in atmel's SoC datasheet in "Electrical Characteristics" paragraph. Signed-off-by: Boris BREZILLON Acked-by: Nicolas Ferre --- drivers/cl

Re: [PATCH v4 08/17] clk: at91: add PMC system clocks

2013-10-17 Thread Nicolas Ferre
ned-off-by: Boris BREZILLON Acked-by: Nicolas Ferre --- drivers/clk/at91/Makefile |1 + drivers/clk/at91/clk-system.c | 193 + drivers/clk/at91/pmc.c|5 ++ drivers/clk/at91/pmc.h|3 + 4 files changed, 202 insertions(+

Re: [PATCH v4 06/17] clk: at91: add PMC pll clocks

2013-10-17 Thread Nicolas Ferre
Electrical Characteristics" paragraph. Signed-off-by: Boris BREZILLON Acked-by: Nicolas Ferre --- drivers/clk/at91/Makefile |2 +- drivers/clk/at91/clk-pll.c| 545 + drivers/clk/at91/clk-plldiv.c | 137 +++ drivers

Re: [PATCH v4 09/17] clk: at91: add PMC peripheral clocks

2013-10-17 Thread Nicolas Ferre
ff-by: Boris BREZILLON Acked-by: Nicolas Ferre --- drivers/clk/at91/Makefile |2 +- drivers/clk/at91/clk-peripheral.c | 407 + drivers/clk/at91/pmc.c|9 + drivers/clk/at91/pmc.h|5 + 4 files changed

Re: [PATCH v4 10/17] clk: at91: add peripheral clk macros for peripheral clk dt bindings

2013-10-17 Thread Nicolas Ferre
On 11/10/2013 11:53, Boris BREZILLON : This patch adds the peripheral divisors macros (for sam9x5 compatible IPs) which will be used by peripheral clk dt definitions. Signed-off-by: Boris BREZILLON Acked-by: Nicolas Ferre --- include/dt-bindings/clk/at91.h |6 ++ 1 file changed

Re: [PATCH v4 11/17] clk: at91: add PMC programmable clocks

2013-10-17 Thread Nicolas Ferre
igned to the appropriate peripheral (see atmel's datasheets). Signed-off-by: Boris BREZILLON Acked-by: Nicolas Ferre --- drivers/clk/at91/Makefile |2 + drivers/clk/at91/clk-programmable.c | 423 +++ drivers/clk/at91/pmc.c

Re: [PATCH v4 12/17] clk: at91: add PMC utmi clock

2013-10-17 Thread Nicolas Ferre
On 11/10/2013 12:22, Boris BREZILLON : This adds new at91 utmi clock implementation using common clk framework. This clock is a pll with a fixed factor (x40). It is used as a source for usb clock. Signed-off-by: Boris BREZILLON Acked-by: Nicolas Ferre --- arch/arm/mach-at91/Kconfig

Re: [PATCH v4 13/17] clk: at91: add PMC usb clock

2013-10-17 Thread Nicolas Ferre
0644 --- a/drivers/clk/at91/pmc.c +++ b/drivers/clk/at91/pmc.c @@ -279,6 +279,17 @@ static const struct of_device_id pmc_clk_ids[] __initdata = { .data = of_at91sam9x5_clk_utmi_setup, }, #endif + /* USB clock */ +#if defined(CONFIG_HAVE_AT91_USB_CLK) + { +

Re: [PATCH v4 14/17] clk: at91: add PMC smd clock

2013-10-17 Thread Nicolas Ferre
On 11/10/2013 13:27, Boris BREZILLON : This patch adds at91 smd (Soft Modem) clock implementation using common clk framework. Not used by any driver right now. Signed-off-by: Boris BREZILLON Acked-by: Nicolas Ferre --- arch/arm/mach-at91/Kconfig |5 ++ drivers/clk/at91/Makefile

Re: [PATCH v4 15/17] clk: at91: add PMC clk device tree binding doc.

2013-10-17 Thread Nicolas Ferre
On 11/10/2013 13:38, Boris BREZILLON : This patch adds new at91 clks dt bindings documentation. Signed-off-by: Boris BREZILLON It seems good Acked-by: Nicolas Ferre --- .../devicetree/bindings/clock/at91-clock.txt | 328 1 file changed, 328 insertions

Re: [PATCH v5 13/17] clk: at91: add PMC usb clock

2013-10-18 Thread Nicolas Ferre
On 17/10/2013 18:55, Boris BREZILLON : This patch adds new at91 usb clock implementation using common clk framework. This clock is used to clock usb ports (ohci, ehci and udc). Signed-off-by: Boris BREZILLON Acked-by: Nicolas Ferre Ok, this one is now replacing the older 13/17. Thanks

Re: [PATCH v4 00/17] ARM: at91: move to common clk framework

2013-10-18 Thread Nicolas Ferre
include/dt-bindings/clk/at91.h rename {arch/arm/mach-at91/include/mach => include/linux/clk}/at91_pmc.h (98%) -- Nicolas Ferre -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More majordomo info

Re: [PATCH] USB: ehci-atmel: add usb_clk for transition to CCF

2013-11-04 Thread Nicolas Ferre
configuration was formely done in mach-at91/clock.c, but will be bypassed when moving to common clk framework. This patch adds support for usb clock retrieval and configuration only if CCF is enabled (CONFIG_COMMON_CLK). Signed-off-by: Boris BREZILLON Acked-by: Nicolas Ferre Greg, as this patch is

Re: [PATCH v6 00/16] ARM: at91: move to common clk framework

2013-12-02 Thread Nicolas Ferre
de 100644 drivers/clk/at91/pmc.h create mode 100644 include/dt-bindings/clk/at91.h rename {arch/arm/mach-at91/include/mach => include/linux/clk}/at91_pmc.h (98%) -- 1.7.9.5 -- Nicolas Ferre -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of

Re: [PATCH] USB: ohci-at91: fix null pointer in ohci_hcd_at91_overcurrent_irq

2012-09-24 Thread Nicolas Ferre
80 (e5903028) > [7.81] ---[ end trace 85aa37ed128143b5 ]--- > [7.81] Kernel panic - not syncing: Fatal exception in interrupt > > Commit 6fffb77c (USB: ohci-at91: fix PIO handling in relation with number of > ports) started setting unused pins to EINVAL. But this exposed a

Re: [PATCH] USB: ohci-at91: fix null pointer in ohci_hcd_at91_overcurrent_irq

2012-09-25 Thread Nicolas Ferre
On 09/24/2012 07:34 PM, Greg KH : > On Mon, Sep 24, 2012 at 09:44:31AM +0200, Nicolas Ferre wrote: >>> This patches fixed the issue by adding the gpio valid check. >>> >>> Signed-off-by: Joachim Eastwood >> >> Acked-by: Nicolas Ferre >> >

Re: [PATCH] USB: ohci-at91: fix null pointer in ohci_hcd_at91_overcurrent_irq

2012-09-27 Thread Nicolas Ferre
On 09/25/2012 03:40 PM, Greg KH : > On Tue, Sep 25, 2012 at 11:48:43AM +0200, Nicolas Ferre wrote: >> On 09/24/2012 07:34 PM, Greg KH : >>> On Mon, Sep 24, 2012 at 09:44:31AM +0200, Nicolas Ferre wrote: >>>>> This patches fixed the issue by adding the gpio valid

Re: [PATCH 14/24] USB: ohci: merge ohci_finish_controller_resume with ohci_resume

2012-10-05 Thread Nicolas Ferre
lves the bit HCD_FLAG_HW_ACCESSIBLE, which is now handled by > ohci_resume(). > > Signed-off-by: Florian Fainelli > --- > drivers/usb/host/ohci-at91.c |2 +- Seems ok for AT91, so Acked-by: Nicolas Ferre Thanks Florian, bye, > drivers/usb/host/ohci-ep93xx.c |

Re: [PATCHv2 1/6] arch: Change defconfigs to point to g_mass_storage.

2012-11-02 Thread Nicolas Ferre
|2 +- > arch/arm/configs/at91sam9g20_defconfig |2 +- For AT91 related defconfigs: Acked-by: Nicolas Ferre > arch/arm/configs/corgi_defconfig |2 +- > arch/arm/configs/davinci_all_defconfig |2 +- > arch/arm/configs/h7202_de

Re: [RFC 3/3] ARM/AVR32: get rid of serial_at91.h

2012-11-16 Thread Nicolas Ferre
ttempt to remove the files, Andrew said that it was used internally in his former company. I had no serious reason to remove it, so we kept it back then. But now it seems the situation has evolved and we must consider the move to a single kernel image and the will to clean the include/asm/mach d

Re: [PATCH 119/493] usb: remove use of __devexit_p

2012-11-20 Thread Nicolas Ferre
i-atmel.c| 2 +- > drivers/usb/host/ohci-at91.c | 2 +- For Atmel: Acked-by: Nicolas Ferre [..] > diff --git a/drivers/usb/c67x00/c67x00-drv.c b/drivers/usb/c67x00/c67x00-drv.c > index 6f3b6e2..855d538 100644 > --- a/drivers/usb/c67x00/c67x00-drv.c > +++ b/drive

Re: [PATCH v3 5/7] USB: EHCI: make ehci-atmel a separate driver

2013-03-30 Thread Nicolas Ferre
ost/Kconfig >> index 01c1acb..8c564aa 100644 >> --- a/drivers/usb/host/ehci-atmel.c >> +++ b/drivers/usb/host/ehci-atmel.c I missed the patch itself but I reviewed it on the mailing-list archive. So, here is my: Acked-by: Nicolas Ferre Thanks a lot for having taking c

Re: [PATCH 1/7] usb: gadget: atmel_usba_udc: Remove redundant platform_set_drvdata()

2013-05-06 Thread Nicolas Ferre
On 06/05/2013 13:37, Sachin Kamat : Commit 0998d06310 (device-core: Ensure drvdata = NULL when no driver is bound) removes the need to set driver data field to NULL. Signed-off-by: Sachin Kamat Cc: Nicolas Ferre Acked-by: Nicolas Ferre --- drivers/usb/gadget/atmel_usba_udc.c |2

Re: [PATCH 1/7] USB: gadget: atmel_usba: move global struct usba_ep usba_ep to struct usba_udc

2013-05-23 Thread Nicolas Ferre
On 23/05/2013 04:23, Bo Shen : Hi J, On 5/21/2013 00:25, Jean-Christophe PLAGNIOL-VILLARD wrote: so we can have multiple usb gadget instance Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD Cc: Nicolas Ferre Cc: linux-usb@vger.kernel.org For this series, test OK on at91sam9m10g45ek and

Re: [PATCH 2/3] usb: ohci-at91: replace request_mem_region + ioremap by devm_request_and_ioremap

2013-12-03 Thread Nicolas Ferre
On 03/12/2013 15:07, Boris BREZILLON : Replace the request_mem_region + ioremap calls by the devm_request_and_ioremap call which does the same things but with device managed resources. Signed-off-by: Boris BREZILLON Tested-by: Robert Nelson Acked-by: Nicolas Ferre --- drivers/usb/host

Re: [PATCH 1/3] usb: ohci-at91: fix irq and iomem resource retrieval

2013-12-03 Thread Nicolas Ferre
avoid resource type mismatch. Signed-off-by: Boris BREZILLON Tested-by: Robert Nelson Acked-by: Nicolas Ferre --- drivers/usb/host/ohci-at91.c | 19 +++ 1 file changed, 11 insertions(+), 8 deletions(-) diff --git a/drivers/usb/host/ohci-at91.c b/drivers/usb/host/ohci

Re: [PATCH 3/3] usb: ohci-at91: use device managed clk retrieval

2013-12-03 Thread Nicolas Ferre
On 03/12/2013 15:07, Boris BREZILLON : Replace clk_get calls by devm_clk_get calls. Signed-off-by: Boris BREZILLON Tested-by: Robert Nelson Acked-by: Nicolas Ferre Thanks Boris for these fixes. Alan, Greg, can you take the whole series as fixes for 3.13? Thanks, best regards

Re: [PATCH 05/12] USB: ohci-at91: Use devm_*() functions

2013-12-11 Thread Nicolas Ferre
gt;regs); - - err2: - release_mem_region(hcd->rsrc_start, hcd->rsrc_len); - err1: usb_put_hcd(hcd); return retval; @@ -246,15 +224,7 @@ static void usb_hcd_at91_remove(struct usb_hcd *hcd, { usb_remove_hcd(hcd); at91_stop_hc(pdev); - iounmap(hcd

Re: [PATCH 1/2] usb: gadget: atmel_usba: Use devm_*() functions

2013-12-16 Thread Nicolas Ferre
On 16/12/2013 07:13, Jingoo Han : > Use devm_*() functions to make cleanup paths simpler. > > Signed-off-by: Jingoo Han Acked-by: Nicolas Ferre Thanks, bye, > --- > drivers/usb/gadget/atmel_usba_udc.c | 64 > ++- > 1 file changed,

Re: [PATCH 2/2] usb: gadget: atmel_usba: Fix sparse warning

2013-12-16 Thread Nicolas Ferre
it be static? > > Signed-off-by: Jingoo Han Acked-by: Nicolas Ferre Thanks. > --- > drivers/usb/gadget/atmel_usba_udc.c |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/usb/gadget/atmel_usba_udc.c > b/drivers/usb/gadget/atmel_

Re: [PATCH 1/2] USB: at91: fix the number of endpoint parameter

2014-01-17 Thread Nicolas Ferre
On 17/01/2014 03:59, Bo Shen : > In sama5d3 SoC, there are 16 endpoints. As the USBA_NR_ENDPOINTS > is only 7. So, fix it for sama5d3 SoC using the udc->num_ep. > > Signed-off-by: Bo Shen Acked-by: Nicolas Ferre > --- > > drivers/usb/gadget/atmel_usba_udc.c | 2

Re: [PATCH 2/2] USB: at91: using USBA_NR_DMAS for DMA channels

2014-01-17 Thread Nicolas Ferre
On 17/01/2014 03:59, Bo Shen : > When the SoC is earlier than sama5d3 SoC, which have the same number > endpoints and DMAs. However for sama5d3 SoC, it has different number > for endpoints and DMAs. So, define USBA_NR_DMAs for DMA channels > > Signed-off-by: Bo Shen Acked-by

Re: [PATCH 1/2] USB: at91: fix the number of endpoint parameter

2014-01-21 Thread Nicolas Ferre
->num_ep; > ---8<--- It seems to me pretty valid to use num_ep in this driver and not have to rely on another compatibility string just for this. The information is here, it is retrieved pretty cleanly so I vote for a simple use of it: if we introduce another information we will have to

[RESEND PATCH] usb: gadget: at91_udc: move prepare clk into process context

2014-11-19 Thread Nicolas Ferre
clock preparation and setting clock rate into process context (at91udc_probe). Signed-off-by: Ronald Wahl Acked-by: Alexandre Belloni Acked-by: Boris Brezillon Acked-by: Nicolas Ferre Cc: Felipe Balbi Cc: # v3.17+ --- Hi Felipe, I forgot to answer you on this patch. So I resend it now with the

Re: [PATCH 1/2] USB: gadget: udc: atmel: change setting for DMA

2014-12-18 Thread Nicolas Ferre
Shen Seems okay, even if I am not used to these aspects: Acked-by: Nicolas Ferre I would also add a "stable" tag to these patches, something like this: Fixes: 914a3f3b3754 (USB: add atmel_usba_udc driver) Cc: sta...@vger.kernel.org # always been there... Felipe, Can you take bo

Re: [PATCH 2/2] USB: gadget: udc: atmel: fix possible IN hang issue

2014-12-18 Thread Nicolas Ferre
ve_data() function, this bit RXRDY has been cleared. > So, after the receive_data() function return, this bit should > not be cleared again, or else it may cause the accessing FIFO > corrupt, which will make the data loss. > > Signed-off-by: Bo Shen Seems okay: Acked-by: Nicolas Fe

Re: [RESEND PATCH] usb: gadget: at91_udc: move prepare clk into process context

2014-12-19 Thread Nicolas Ferre
Le 19/12/2014 15:02, Ronald Wahl a écrit : > On 19.12.2014 14:51, Luis Henriques wrote: >> Hi Felipe, >> >> On Thu, Nov 20, 2014 at 01:50:49PM -0600, Felipe Balbi wrote: >>> On Wed, Nov 19, 2014 at 04:37:27PM +0100, Nicolas Ferre wrote: >>&g

[RESEND PATCH] USB: gadget: udc: atmel: fix possible oops when unloading module

2015-01-09 Thread Nicolas Ferre
g_cleanup [g_hid]) from [] (SyS_delete_module+0x118/0x19c) [] (SyS_delete_module) from [] (ret_fast_syscall+0x0/0x30) Code: bad PC value Signed-off-by: Songjun Wu [nicolas.fe...@atmel.com: reworked the commit message] Signed-off-by: Nicolas Ferre Fixes: 914a3f3b3754 ("USB: add atmel_usba_udc

Re: [PATCH v3 0/5] usb: atmel_usba_udc: Rework errata handling

2015-01-12 Thread Nicolas Ferre
er in commit message of patch 2 > > Changes since v1: > - cache INT_ENB value to speedup INT_ENB read operations To the whole series: Acked-by: Nicolas Ferre Thanks. > Boris Brezillon (5): > usb: atmel_usba_udc: Rework at91sam9rl errata handling > usb: atmel_usba_udc: Add

Re: [PATCH 00/12] Atmel matrix, SMC and UDC rework

2015-01-15 Thread Nicolas Ferre
gadget: at91_udc: Remove non-DT handling code > usb: gadget: at91_udc: Simplify probe and remove functions > usb: gadget: at91_udc: Rework for multi-platform kernel support > usb: gadget: at91_udc: Update DT binding documentation > usb: gadget: at91_udc: Allocate udc instance I

[PATCH RESEND] usb: at91-udc: fix irq and iomem resource retrieval

2014-02-12 Thread Nicolas Ferre
ILLON Acked-by: Nicolas Ferre Cc: stable # 3.4 --- drivers/usb/gadget/at91_udc.c | 10 -- 1 file changed, 10 deletions(-) diff --git a/drivers/usb/gadget/at91_udc.c b/drivers/usb/gadget/at91_udc.c index cea8c20a1425..1926925a52a9 100644 --- a/drivers/usb/gadget/at91_udc.c +++ b/driver

Re: [PATCH v2 1/2] USB: at91: fix the number of endpoint parameter

2014-02-19 Thread Nicolas Ferre
gt;num_ep, which get from platform data for non-dt kernel, > or parse from dt node. > > Signed-off-by: Bo Shen Acked-by: Nicolas Ferre > --- > Changes in v2: > - Make the commit message more clearer. > > drivers/usb/gadget/atmel_usba_udc.c | 2 +- > 1 file changed, 1 i

Re: [PATCH v2 2/2] USB: at91: using USBA_NR_DMAS for DMA channels

2014-02-19 Thread Nicolas Ferre
ints and DMA channels. > So, define a new micro USBA_NR_DMAs for DMA channels. And the s/micro/macro/ > USBA_NR_ENDPOINS is not used anymore, remove it at the same time. > > Signed-off-by: Bo Shen Acked-by: Nicolas Ferre > --- > Changes in v2: > - Make the commit message more

Re: [PATCH] usb: gadget: atmel_usba: fix crash when no endpoint are specified

2014-02-28 Thread Nicolas Ferre
mel_udc_of_init(struct > platform_device *pdev, > i++; > } > > + if (i == 0) { > + dev_err(&pdev->dev, "of_probe: no endpoint specified\n"); > + ret = -EINVAL; > + goto err; > + } > + It

Re: [PATCH RESEND] usb: at91-udc: fix irq and iomem resource retrieval

2014-03-12 Thread Nicolas Ferre
On 12/02/2014 14:03, Nicolas Ferre : > From: Jean-Jacques Hiblot > > When using dt resources retrieval (interrupts and reg properties) there is > no predefined order for these resources in the platform dev resource > table. Also don't expect the number of resource to be alway

Re: [PATCH RESEND] usb: at91-udc: fix irq and iomem resource retrieval

2014-03-12 Thread Nicolas Ferre
On 12/03/2014 16:31, Greg Kroah-Hartman : > On Wed, Mar 12, 2014 at 03:57:48PM +0100, Nicolas Ferre wrote: >> On 12/02/2014 14:03, Nicolas Ferre : >>> From: Jean-Jacques Hiblot >>> >>> When using dt resources retrieval (interrupts and reg properties) there

[PATCH RESEND] usb: at91-udc: fix irq and iomem resource retrieval

2014-03-12 Thread Nicolas Ferre
ILLON Acked-by: Nicolas Ferre Cc: stable # 3.4 --- drivers/usb/gadget/at91_udc.c | 10 -- 1 file changed, 10 deletions(-) diff --git a/drivers/usb/gadget/at91_udc.c b/drivers/usb/gadget/at91_udc.c index cea8c20a1425..1926925a52a9 100644 --- a/drivers/usb/gadget/at91_udc.c +++ b/driver

Re: [PATCH] usb: gadget: atmel_usba: always test udc->driver

2014-05-09 Thread Nicolas Ferre
-by: Alexandre Belloni Acked-by: Nicolas Ferre Thanks, > --- > drivers/usb/gadget/atmel_usba_udc.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/usb/gadget/atmel_usba_udc.c > b/drivers/usb/gadget/atmel_usba_udc.c > index 9f653

Re: [PATCH 5/5] usb: gadget: atmel_usba_udc: add missing ret value check

2015-07-08 Thread Nicolas Ferre
Le 07/07/2015 16:02, Robert Baldyga a écrit : > Add missing return value check. In case of error print debug message > and return error code. > > Signed-off-by: Robert Baldyga Yes, it's indeed missing: Acked-by: Nicolas Ferre Thanks, bye. > --- > drivers/usb/gadget/u

Re: [PATCH] usb: gadget: atmel: remove useless include

2015-09-18 Thread Nicolas Ferre
Le 10/08/2015 16:29, Alexandre Belloni a écrit : > Definitions from linux/platform_data/atmel.h are not used, remove the > include. > > Signed-off-by: Alexandre Belloni Acked-by: Nicolas Ferre > --- > drivers/usb/gadget/udc/atmel_usba_udc.c | 1 - > 1 file changed, 1

Re: [PATCH] usb: gadget: at91_udc: move at91_udc_data in at91_udc.h

2015-09-18 Thread Nicolas Ferre
Le 10/08/2015 16:46, Alexandre Belloni a écrit : > struct at91_udc_data is now only used inside the driver, move it to its > include. > > Signed-off-by: Alexandre Belloni Acked-by: Nicolas Ferre Thanks. > --- > drivers/usb/gadget/udc/at91_udc.h | 8 > includ

  1   2   >