Re: [RFC PATCH 0/2] usb: Reuse fsl driver code for synopsys usb controller

2014-04-20 Thread Greg KH
On Sun, Apr 20, 2014 at 09:57:03PM +0530, Punnaiah Choudary Kalluri wrote: > Zynq soc contains a dual role usb controller and this IP is from synopsys. We > observed that there is driver available for this controller from freescale and > decided to reuse this driver for zynq use. > > Here is the l

Re: [RFC PATCH 0/2] usb: Reuse fsl driver code for synopsys usb controller

2014-04-20 Thread Alan Stern
On Sun, 20 Apr 2014, Punnaiah Choudary Kalluri wrote: > Zynq soc contains a dual role usb controller and this IP is from synopsys. We > observed that there is driver available for this controller from freescale and > decided to reuse this driver for zynq use. > > Here is the link for zynq soc TRM

Re: [RFC PATCH 0/2] usb: Reuse fsl driver code for synopsys usb controller

2014-04-20 Thread Marc Kleine-Budde
On 04/20/2014 06:27 PM, Punnaiah Choudary Kalluri wrote: > Zynq soc contains a dual role usb controller and this IP is from synopsys. We > observed that there is driver available for this controller from freescale and > decided to reuse this driver for zynq use. Have a look drivers/usb/chipidea. I

[RFC PATCH 0/2] usb: Reuse fsl driver code for synopsys usb controller

2014-04-20 Thread Punnaiah Choudary Kalluri
Zynq soc contains a dual role usb controller and this IP is from synopsys. We observed that there is driver available for this controller from freescale and decided to reuse this driver for zynq use. Here is the link for zynq soc TRM. Please refer chapter 15 for usb controller related information.

[RFC PATCH 1/2] usb: gadget: fsl_udc: Add support for zynq usb device controller

2014-04-20 Thread Punnaiah Choudary Kalluri
Since zynq soc usb controller is a synopsys IP and there is a driver available for this controller from freescale in opensource, reusing this driver for zynq use. Signed-off-by: Punnaiah Choudary Kalluri --- drivers/usb/gadget/Kconfig|2 +- drivers/usb/gadget/Makefile |1 +

[RFC PATCH 2/2] usb: ehci-fsl: Add support for zynq usb host controller

2014-04-20 Thread Punnaiah Choudary Kalluri
Since zynq soc usb controller is a synopsys IP and there is a driver available for this controller from freescale in opensource, reusing this driver for zynq use. Signed-off-by: Punnaiah Choudary Kalluri --- drivers/usb/host/Kconfig|2 +- drivers/usb/host/ehci-fsl.c | 63 ++

[RFC PATCH 0/2] usb: Reuse fsl driver code for synopsys usb controller

2014-04-20 Thread Punnaiah Choudary Kalluri
Zynq soc contains a dual role usb controller and this IP is from synopsys. We observed that there is driver available for this controller from freescale and decided to reuse this driver for zynq use. Here is the link for zynq soc TRM. Please refer chapter 15 for usb controller related information.

[RFC PATCH 1/2] usb: gadget: fsl_udc: Add support for zynq usb device controller

2014-04-20 Thread Punnaiah Choudary Kalluri
Since zynq soc usb controller is a synopsys IP and there is a driver available for this controller from freescale in opensource, reusing this driver for zynq use. Signed-off-by: Punnaiah Choudary Kalluri --- drivers/usb/gadget/Kconfig|2 +- drivers/usb/gadget/Makefile |1 +

[RFC PATCH 2/2] usb: ehci-fsl: Add support for zynq usb host controller

2014-04-20 Thread Punnaiah Choudary Kalluri
Since zynq soc usb controller is a synopsys IP and there is a driver available for this controller from freescale in opensource, reusing this driver for zynq use. Signed-off-by: Punnaiah Choudary Kalluri --- drivers/usb/host/Kconfig|2 +- drivers/usb/host/ehci-fsl.c | 63 ++

[PATCH 2/2] powerpc/powernc: revert part of commit d905c5df(PPC: POWERNV: move iommu_add_device earlier)

2014-04-20 Thread Wei Yang
Here is a draft call flow: pci_device_add pcibios_add_device pci_dma_dev_setup pnv_pci_dma_dev_setup pnv_pci_ioda_dma_dev_setup set_iommu_table_base_and_group <--- here device_add When set_iommu_table_base_and_group() is invoked int pnv_pci_ioda_dma_de

[PATCH 1/2] powerpc/powernv: clear the refcount for pci_dev on powernv platform

2014-04-20 Thread Wei Yang
When pcibios_remove_pci_devices() is removing pci devices, it will release pci device respectively. When the refcount of the device is 0, the pci_dev structure will be destroyed. On PowerNV platform, the pci_dev will not be destroyed since the refcount is not 0. After applying the patch, this war

Re: [PATCH 1/2] powerpc/powernv: clear the refcount for pci_dev on powernv platform

2014-04-20 Thread Benjamin Herrenschmidt
On Mon, 2014-04-21 at 10:25 +0800, Wei Yang wrote: > When pcibios_remove_pci_devices() is removing pci devices, it will release > pci device respectively. When the refcount of the device is 0, the pci_dev > structure will be destroyed. > > On PowerNV platform, the pci_dev will not be destroyed sin

Re: [PATCH 2/2] powerpc/powernc: revert part of commit d905c5df(PPC: POWERNV: move iommu_add_device earlier)

2014-04-20 Thread Benjamin Herrenschmidt
On Mon, 2014-04-21 at 10:25 +0800, Wei Yang wrote: > Here is a draft call flow: > > pci_device_add >pcibios_add_device > pci_dma_dev_setup > pnv_pci_dma_dev_setup > pnv_pci_ioda_dma_dev_setup >set_iommu_table_base_and_group <--- here > device_add > >

[PATCH] powerpc/mm: Fix tlbie to add AVAL fields for 64K pages

2014-04-20 Thread Aneesh Kumar K.V
The if condition check was based on a draft ISA doc. Remove the same. Signed-off-by: Aneesh Kumar K.V --- arch/powerpc/mm/hash_native_64.c | 38 -- 1 file changed, 16 insertions(+), 22 deletions(-) diff --git a/arch/powerpc/mm/hash_native_64.c b/arch/powerpc/

Re: [PATCH 2/2] powerpc/powernc: revert part of commit d905c5df(PPC: POWERNV: move iommu_add_device earlier)

2014-04-20 Thread Wei Yang
On Mon, Apr 21, 2014 at 01:37:32PM +1000, Benjamin Herrenschmidt wrote: >On Mon, 2014-04-21 at 10:25 +0800, Wei Yang wrote: >> Here is a draft call flow: >> >> pci_device_add >>pcibios_add_device >> pci_dma_dev_setup >> pnv_pci_dma_dev_setup >> pnv_pci_ioda_dma_dev_s

Re: [PATCH] powerpc/mm: Fix tlbie to add AVAL fields for 64K pages

2014-04-20 Thread Benjamin Herrenschmidt
On Mon, 2014-04-21 at 10:37 +0530, Aneesh Kumar K.V wrote: > The if condition check was based on a draft ISA doc. Remove the same. Is this a real bug ? Do we need to send that back to stable ? Ben. > Signed-off-by: Aneesh Kumar K.V > --- > arch/powerpc/mm/hash_native_64.c | 38

Re: [PATCH 1/3] cpufreq: ppc: Add missing #include

2014-04-20 Thread Viresh Kumar
On 17 April 2014 15:23, Geert Uytterhoeven wrote: > If CONFIG_SMP=n, does not include , causing: > > drivers/cpufreq/ppc-corenet-cpufreq.c: In function 'corenet_cpufreq_cpu_init': > drivers/cpufreq/ppc-corenet-cpufreq.c:173:3: error: implicit declaration of > function 'get_hard_smp_processor_id'

Re: [PATCH 2/3] cpufreq: ppc: Fix integer overflow in expression

2014-04-20 Thread Viresh Kumar
On 17 April 2014 15:23, Geert Uytterhoeven wrote: > On 32-bit, "12 * NSEC_PER_SEC" doesn't fit in "unsigned long" > (NSEC_PER_SEC is a "long" constant), causing an integer overflow: > > drivers/cpufreq/ppc-corenet-cpufreq.c: In function 'corenet_cpufreq_cpu_init': > drivers/cpufreq/ppc-corenet-cpu

Re: [PATCH 3/3] cpufreq: ppc: Fix handling of non-existent clocks

2014-04-20 Thread Viresh Kumar
On 17 April 2014 15:23, Geert Uytterhoeven wrote: > If the clock doesn't exist, clk_get_rate() returns -EINVAL You clk_get_rate() isn't written well then, it should return zero. @Mike: I didn't see this clearly mentioned in clk.h, should we fix that? >, which becomes > a large number (freq is u3

Re: [PATCH 1/2] powerpc/powernv: clear the refcount for pci_dev on powernv platform

2014-04-20 Thread Wei Yang
On Mon, Apr 21, 2014 at 01:35:34PM +1000, Benjamin Herrenschmidt wrote: >On Mon, 2014-04-21 at 10:25 +0800, Wei Yang wrote: >> When pcibios_remove_pci_devices() is removing pci devices, it will release >> pci device respectively. When the refcount of the device is 0, the pci_dev >> structure will b

Re: [RFC PATCH 0/2] usb: Reuse fsl driver code for synopsys usb controller

2014-04-20 Thread punnaiah choudary kalluri
On Mon, Apr 21, 2014 at 12:22 AM, Marc Kleine-Budde wrote: > On 04/20/2014 06:27 PM, Punnaiah Choudary Kalluri wrote: >> Zynq soc contains a dual role usb controller and this IP is from synopsys. We >> observed that there is driver available for this controller from freescale >> and >> decided to

Re: [PATCH 2/2] powerpc/powernc: revert part of commit d905c5df(PPC: POWERNV: move iommu_add_device earlier)

2014-04-20 Thread Wei Yang
On Mon, Apr 21, 2014 at 01:22:13PM +0800, Wei Yang wrote: >On Mon, Apr 21, 2014 at 01:37:32PM +1000, Benjamin Herrenschmidt wrote: >>On Mon, 2014-04-21 at 10:25 +0800, Wei Yang wrote: >>> Here is a draft call flow: >>> >>> pci_device_add >>>pcibios_add_device >>> pci_dma_dev_setup >>>