Re: [PATCH v2 13/14] ARM: DTS: am33xx: Use the new DT bindings for the eDMA3

2015-12-04 Thread Peter Ujfalusi
On 12/04/2015 02:54 AM, Tony Lindgren wrote: > Hi Peter, > > * Peter Ujfalusi [151016 00:23]: > > I noticed something while changing dm81xx to use the edma_xbar.. > >> --- a/arch/arm/boot/dts/am33xx.dtsi >> +++ b/arch/arm/boot/dts/am33xx.dtsi >> + >> +

Re: [PATCH 2/2] pinctrl: single: remove misuse of IRQF_NO_SUSPEND flag

2015-12-04 Thread Sudeep Holla
On 04/12/15 11:18, Grygorii Strashko wrote: On 12/04/2015 12:54 PM, Sudeep Holla wrote: Hi Grygorii, On 04/12/15 10:44, Grygorii Strashko wrote: On 12/03/2015 11:37 PM, Tony Lindgren wrote: [...] And these both need to be applied together when we have a fix for the above as otherwise

Re: [PATCH 2/7] ARM: dts: Update edma bindings on dm814x to use edma_xbar

2015-12-04 Thread Peter Ujfalusi
On 12/04/2015 03:25 AM, Tony Lindgren wrote: > The edma is the same as on am33xx, except it has four tptc > instances. And we need the edma_xbar for at least mmc3, so > let's use the edma_xbar and the new binding as suggested by > Peter Ujfalusi . > > Cc: Peter Ujfalusi

[PATCH] ARM: DTS: am33xx: Use the new DT bindings for the eDMA3

2015-12-04 Thread Peter Ujfalusi
Switch to use the ti,edma3-tpcc and ti,edma3-tptc binding for the eDMA3 and enable the DMA even crossbar with ti,am335x-edma-crossbar. With the new bindings boards can customize and tweak the DMA channel priority to match their needs. With the new binding the memcpy is safe to be used since with

Re: [PATCH 2/2] pinctrl: single: remove misuse of IRQF_NO_SUSPEND flag

2015-12-04 Thread Grygorii Strashko
On 12/03/2015 11:37 PM, Tony Lindgren wrote: * Grygorii Strashko [151203 10:36]: I think, this patch should not break our wake-up functionality. It will just change the moment when pcs_irq_handler() will be called: before this change: - suspend_enter() -

[PATCH] ARM: DTS: am437x: Use the new DT bindings for the eDMA3

2015-12-04 Thread Peter Ujfalusi
Switch to use the ti,edma3-tpcc and ti,edma3-tptc binding for the eDMA3 and enable the DMA even crossbar with ti,am335x-edma-crossbar. With the new bindings boards can customize and tweak the DMA channel priority to match their needs. With the new binding the memcpy is safe to be used since with

Re: [PATCH 2/2] pinctrl: single: remove misuse of IRQF_NO_SUSPEND flag

2015-12-04 Thread Sudeep Holla
Hi Grygorii, On 04/12/15 10:44, Grygorii Strashko wrote: On 12/03/2015 11:37 PM, Tony Lindgren wrote: [...] And these both need to be applied together when we have a fix for the above as otherwise we'll get the lock recursion Sudeep mentioned in patch 2/2. Most probably below diff will

Re: [PATCH 7/7] ARM: dts: Enable emmc on hp t410

2015-12-04 Thread Peter Ujfalusi
On 12/04/2015 03:25 AM, Tony Lindgren wrote: > There's a 2GB emmc on hp t410 that's wired to the sd_2 interface. > Note that we also need to configure the evtmux using edma_xbar for > edma channels. > > Signed-off-by: Tony Lindgren One question inlined for the mmc3, other than

Re: [PATCH 2/2] pinctrl: single: remove misuse of IRQF_NO_SUSPEND flag

2015-12-04 Thread Grygorii Strashko
On 12/04/2015 12:54 PM, Sudeep Holla wrote: > Hi Grygorii, > > On 04/12/15 10:44, Grygorii Strashko wrote: >> On 12/03/2015 11:37 PM, Tony Lindgren wrote: > > [...] > >>> And these both need to be applied together when we have a fix for the >>> above >>> as otherwise we'll get the lock

Re: [PATCH 1/3] ARM: OMAP2+: Fix SoC detection for dra62x j5-eco

2015-12-04 Thread Tony Lindgren
* Matthijs van Duin [151203 18:44]: > On 4 December 2015 at 00:11, Tony Lindgren wrote: > > We can boot dra62x j5-eco using the dm814x code as the clocks and > > devices are mapped in the device tree. The dra62x is also known > > as jacinto 5. > >

Re: [PATCH 2/2] pinctrl: single: remove misuse of IRQF_NO_SUSPEND flag

2015-12-04 Thread Tony Lindgren
* Grygorii Strashko [151204 02:45]: > On 12/03/2015 11:37 PM, Tony Lindgren wrote: > >* Grygorii Strashko [151203 10:36]: > >> > >>I think, this patch should not break our wake-up functionality. > >>It will just change the moment when

Re: [PATCH 7/7] ARM: dts: Enable emmc on hp t410

2015-12-04 Thread Tony Lindgren
* Peter Ujfalusi [151204 01:21]: > On 12/04/2015 03:25 AM, Tony Lindgren wrote: > > There's a 2GB emmc on hp t410 that's wired to the sd_2 interface. > > Note that we also need to configure the evtmux using edma_xbar for > > edma channels. > > > > Signed-off-by: Tony

Re: [PATCH 2/2] pinctrl: single: remove misuse of IRQF_NO_SUSPEND flag

2015-12-04 Thread Tony Lindgren
* Tony Lindgren [151203 13:41]: > * Sudeep Holla [151203 11:00]: > > > > I have added irq_set_irq_wake(pcs_soc->irq, state) in pcs_irq_set_wake > > which ensures it's marked for wakeup. > > Hmm well see the error I pasted in this thread, maybe that

Re: [PATCH 2/2] pinctrl: single: remove misuse of IRQF_NO_SUSPEND flag

2015-12-04 Thread Sudeep Holla
On 04/12/15 15:40, Tony Lindgren wrote: * Tony Lindgren [151203 13:41]: * Sudeep Holla [151203 11:00]: I have added irq_set_irq_wake(pcs_soc->irq, state) in pcs_irq_set_wake which ensures it's marked for wakeup. Hmm well see the error I pasted in

Re: [PATCH 2/2] pinctrl: single: remove misuse of IRQF_NO_SUSPEND flag

2015-12-04 Thread Grygorii Strashko
On 12/04/2015 05:35 PM, Tony Lindgren wrote: > * Grygorii Strashko [151204 02:45]: >> On 12/03/2015 11:37 PM, Tony Lindgren wrote: >>> * Grygorii Strashko [151203 10:36]: I think, this patch should not break our wake-up functionality.

Re: [PATCH 2/2] ARM: dts: omap3-igep0030: Use MMC pwrseq to init SDIO WiFi

2015-12-04 Thread Enric Balletbo Serra
2015-12-03 19:02 GMT+01:00 Javier Martinez Canillas : > When the WiFi support was added to the IGEP0030 board, the MMC subsystem > did not provide a mechanism to define power sequence providers. So a DT > hack was used to toggle the WiFi chip reset and power down pins by

Re: [PATCH 0/2] ARM: dts: Use MMC pwrseq instead regulators for IGEP WiFi init

2015-12-04 Thread Enric Balletbo Serra
2015-12-03 20:27 GMT+01:00 Tony Lindgren : > * Javier Martinez Canillas [151203 10:29]: >> Hello Tony, >> >> On 12/03/2015 03:16 PM, Tony Lindgren wrote: >> > * Javier Martinez Canillas [151203 10:03]: >> >> Hello, >> >> >> >>

[patch] OMAPDSS: DSS: fix a warning message

2015-12-04 Thread Dan Carpenter
The WARN() macro has to take a condition. The current code will just print the stack trace and the function name instead of the intended warning message. Signed-off-by: Dan Carpenter diff --git a/drivers/video/fbdev/omap2/dss/dss.h

Re: [PATCH 1/2] ARM: dts: omap3-igep0020: Use MMC pwrseq to init SDIO WiFi

2015-12-04 Thread Enric Balletbo Serra
2015-12-03 19:02 GMT+01:00 Javier Martinez Canillas : > When the WiFi support was added to the IGEP0020 board, the MMC subsystem > did not provide a mechanism to define power sequence providers. So a DT > hack was used to toggle the WiFi chip reset and power down pins by

Re: [PATCH 2/2] pinctrl: single: remove misuse of IRQF_NO_SUSPEND flag

2015-12-04 Thread Tony Lindgren
* Grygorii Strashko [151204 08:00]: > On 12/04/2015 05:35 PM, Tony Lindgren wrote: > > * Grygorii Strashko [151204 02:45]: > >> On 12/03/2015 11:37 PM, Tony Lindgren wrote: > >>> * Grygorii Strashko [151203 10:36]: >

Re: [PATCH] ARM: DTS: am33xx: Use the new DT bindings for the eDMA3

2015-12-04 Thread Felipe Balbi
Hi, Peter Ujfalusi writes: > @@ -174,12 +182,44 @@ > }; > > edma: edma@4900 { > - compatible = "ti,edma3"; > - ti,hwmods = "tpcc", "tptc0", "tptc1", "tptc2"; > - reg =

Re: [PATCH 2/2] pinctrl: single: remove misuse of IRQF_NO_SUSPEND flag

2015-12-04 Thread Tony Lindgren
* Sudeep Holla [151204 08:27]: > > > On 04/12/15 16:19, Grygorii Strashko wrote: > >On 12/04/2015 05:44 PM, Sudeep Holla wrote: > >> > >> > >>On 04/12/15 15:40, Tony Lindgren wrote: > >>>* Tony Lindgren [151203 13:41]: > * Sudeep Holla

Re: [PATCH 2/2] pinctrl: single: remove misuse of IRQF_NO_SUSPEND flag

2015-12-04 Thread Tony Lindgren
* Grygorii Strashko [151204 08:31]: > On 12/04/2015 06:11 PM, Sudeep Holla wrote: > > > > > > On 04/12/15 15:59, Grygorii Strashko wrote: > >> > >> Sorry, I can't test it right now :( > >> Potential fix below: > > > > I had posted similar patch a while ago which Tony

Re: [PATCH 2/2] pinctrl: single: remove misuse of IRQF_NO_SUSPEND flag

2015-12-04 Thread Tony Lindgren
* Sudeep Holla [151204 08:16]: > Hi Tony, > > On 04/12/15 15:40, Tony Lindgren wrote: > >* Tony Lindgren [151203 13:41]: > >>* Sudeep Holla [151203 11:00]: > >>> > >>>I have added irq_set_irq_wake(pcs_soc->irq, state) in

Re: [PATCH 2/2] pinctrl: single: remove misuse of IRQF_NO_SUSPEND flag

2015-12-04 Thread Sudeep Holla
On 04/12/15 16:19, Grygorii Strashko wrote: On 12/04/2015 05:44 PM, Sudeep Holla wrote: On 04/12/15 15:40, Tony Lindgren wrote: * Tony Lindgren [151203 13:41]: * Sudeep Holla [151203 11:00]: I have added irq_set_irq_wake(pcs_soc->irq, state) in

Re: [PATCH 2/2] pinctrl: single: remove misuse of IRQF_NO_SUSPEND flag

2015-12-04 Thread Sudeep Holla
Hi Tony, On 04/12/15 15:40, Tony Lindgren wrote: * Tony Lindgren [151203 13:41]: * Sudeep Holla [151203 11:00]: I have added irq_set_irq_wake(pcs_soc->irq, state) in pcs_irq_set_wake which ensures it's marked for wakeup. Hmm well see the error I

Re: [PATCH 2/2] pinctrl: single: remove misuse of IRQF_NO_SUSPEND flag

2015-12-04 Thread Sudeep Holla
On 04/12/15 15:59, Grygorii Strashko wrote: Sorry, I can't test it right now :( Potential fix below: I had posted similar patch a while ago which Tony rejected. I might have made a mistake of not putting them together, though they were part of the same series[1], patch 12 and 16 diff

Re: [PATCH 2/2] pinctrl: single: remove misuse of IRQF_NO_SUSPEND flag

2015-12-04 Thread Grygorii Strashko
On 12/04/2015 05:44 PM, Sudeep Holla wrote: > > > On 04/12/15 15:40, Tony Lindgren wrote: >> * Tony Lindgren [151203 13:41]: >>> * Sudeep Holla [151203 11:00]: I have added irq_set_irq_wake(pcs_soc->irq, state) in pcs_irq_set_wake which

Re: [PATCH 2/2] pinctrl: single: remove misuse of IRQF_NO_SUSPEND flag

2015-12-04 Thread Grygorii Strashko
On 12/04/2015 06:11 PM, Sudeep Holla wrote: > > > On 04/12/15 15:59, Grygorii Strashko wrote: >> >> Sorry, I can't test it right now :( >> Potential fix below: > > I had posted similar patch a while ago which Tony rejected. > I might have made a mistake of not putting them together, though they

Re: [PATCH] ARM: DTS: am33xx: Use the new DT bindings for the eDMA3

2015-12-04 Thread Arnd Bergmann
On Friday 04 December 2015 10:47:07 Tony Lindgren wrote: > > Peter Ujfalusi writes: > > > @@ -174,12 +182,44 @@ > > > }; > > > > > > edma: edma@4900 { > > > - compatible = "ti,edma3"; > > > - ti,hwmods =

Re: [PATCH v2] PCI: dra7xx: mark dra7xx_pcie_msi irq as IRQF_NO_THREAD

2015-12-04 Thread Grygorii Strashko
On 12/04/2015 08:46 PM, Bjorn Helgaas wrote: > Hi Grygorii, > > On Fri, Nov 20, 2015 at 03:59:26PM +0200, Grygorii Strashko wrote: >> On -RT, TI DRA7 PCIe driver always produces below backtrace when the >> first PCI interrupt is triggered: >> >> [ cut here ] >> WARNING:

[PATCH 3/6] ARM: dts: Add usb support for dm814x and dra62x

2015-12-04 Thread Tony Lindgren
The usb is very much like on am33xx, we just put the some of the componends under scm where they belong. Cc: Felipe Balbi Signed-off-by: Tony Lindgren --- arch/arm/boot/dts/dm814x.dtsi | 125 ++ 1 file changed, 125

[PATCH 1/6] ARM: OMAP2+: Update 81xx clock and power domains for default, active and sgx

2015-12-04 Thread Tony Lindgren
These offsets seem to be common, so let's rename the defines. And let's set up the default_l3_slow_81xx_clkdm with active and default powerdomains for dm814x. These are needed for usb to work. Cc: Paul Walmsley Cc: Tero Kristo Signed-off-by: Tony Lindgren

[PATCH 0/6] Add usb support for dm814x and dra62x

2015-12-04 Thread Tony Lindgren
Hi, These patches add usb support for dm814x and dra62x. These patches depend on what's listed at the mmc patches at: http://marc.info/?l=linux-omap=144919234321418=2 Regards, Tony Tony Lindgren (6): ARM: OMAP2+: Update 81xx clock and power domains for default, active and sgx ARM:

[PATCH 2/6] ARM: OMAP2+: Add support for dm814x and dra62x usb

2015-12-04 Thread Tony Lindgren
The usb phys are different on dm814x compared to dm816x so we need to use the clkdcoldo output for usb. Cc: Paul Walmsley Signed-off-by: Tony Lindgren --- arch/arm/mach-omap2/omap_hwmod_81xx_data.c | 38 +++--- 1 file changed, 29

[PATCH 4/6] ARM: dts: Add usb support for dm814x-evm

2015-12-04 Thread Tony Lindgren
Add usb support for dm814x-evm Cc: Felipe Balbi Signed-off-by: Tony Lindgren --- arch/arm/boot/dts/dm8148-evm.dts | 24 1 file changed, 24 insertions(+) diff --git a/arch/arm/boot/dts/dm8148-evm.dts b/arch/arm/boot/dts/dm8148-evm.dts

[PATCH 5/6] ARM: dts: Add usb support for hp t410

2015-12-04 Thread Tony Lindgren
Add usb support for hp t410 and a fixed regulator for the hub power. Cc: Felipe Balbi Signed-off-by: Tony Lindgren --- arch/arm/boot/dts/dm8148-t410.dts | 35 +++ 1 file changed, 35 insertions(+) diff --git

[PATCH 6/6] ARM: dts: Add usb support for j5-eco evm

2015-12-04 Thread Tony Lindgren
Add usb support for j5-eco evm. Cc: Felipe Balbi Signed-off-by: Tony Lindgren --- arch/arm/boot/dts/dra62x-j5eco-evm.dts | 21 + 1 file changed, 21 insertions(+) diff --git a/arch/arm/boot/dts/dra62x-j5eco-evm.dts

Re: [PATCH] ARM: DTS: am33xx: Use the new DT bindings for the eDMA3

2015-12-04 Thread Tony Lindgren
* Arnd Bergmann [151204 13:38]: > On Friday 04 December 2015 10:47:07 Tony Lindgren wrote: > > > Peter Ujfalusi writes: > > > > @@ -174,12 +182,44 @@ > > > > }; > > > > > > > > edma: edma@4900 { > > > > -

Re: [PATCH v2] PCI: dra7xx: mark dra7xx_pcie_msi irq as IRQF_NO_THREAD

2015-12-04 Thread Bjorn Helgaas
Hi Grygorii, On Fri, Nov 20, 2015 at 03:59:26PM +0200, Grygorii Strashko wrote: > On -RT, TI DRA7 PCIe driver always produces below backtrace when the > first PCI interrupt is triggered: > > [ cut here ] > WARNING: CPU: 1 PID: 82 at kernel/irq/handle.c:150 >

Re: [PATCH] ARM: DTS: am33xx: Use the new DT bindings for the eDMA3

2015-12-04 Thread Tony Lindgren
* Felipe Balbi [151204 09:23]: > > Hi, > > Peter Ujfalusi writes: > > @@ -174,12 +182,44 @@ > > }; > > > > edma: edma@4900 { > > - compatible = "ti,edma3"; > > - ti,hwmods = "tpcc", "tptc0",