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

2015-11-02 Thread Olof Johansson
Hi, 1) This seems to have broken BBB in -next for me, bisected down to this patch. For bootlog: http://arm-soc.lixom.net/bootlogs/next/next-20151102/bbb-arm-omap2plus_defconfig.html 2) Please avoid merging DT/platform code in your driver tree, Vinod, at least without an ack from the platform

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

2015-11-02 Thread Vinod Koul
On Mon, Nov 02, 2015 at 01:21:19AM -0800, Olof Johansson wrote: > Hi, > > 1) This seems to have broken BBB in -next for me, bisected down to this patch. > > For bootlog: > http://arm-soc.lixom.net/bootlogs/next/next-20151102/bbb-arm-omap2plus_defconfig.html > > 2)

[PATCH] ARM: OMAP: hwmod: AM33xx/43xx: Add HWMOD_INIT_NO_IDLE flag to tptc

2015-11-02 Thread Peter Ujfalusi
In Linux we do not have driver for TPTCs of eDMA3 since there is no need to do any configuration within TPTC for the eDMA3 to be operational. All configuration is via the TPCC. To prevent the omap_device_late_idle() to disable the TPTCs when they are no longer bind with the edma driver, the HWMOD_I

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

2015-11-02 Thread Peter Ujfalusi
Hi Olof, On 11/02/2015 11:21 AM, Olof Johansson wrote: > Hi, > > 1) This seems to have broken BBB in -next for me, bisected down to this patch. > > For bootlog: > http://arm-soc.lixom.net/bootlogs/next/next-20151102/bbb-arm-omap2plus_defconfig.html Aargh, I had the patch whic

Re: [PATCH] ARM: OMAP: hwmod: AM33xx/43xx: Add HWMOD_INIT_NO_IDLE flag to tptc

2015-11-02 Thread Vinod Koul
On Mon, Nov 02, 2015 at 12:11:00PM +0200, Peter Ujfalusi wrote: > In Linux we do not have driver for TPTCs of eDMA3 since there is no need to > do any configuration within TPTC for the eDMA3 to be operational. All > configuration is via the TPCC. > To prevent the omap_device_late_idle() to disable

Re: [PATCH] ARM: OMAP: hwmod: AM33xx/43xx: Add HWMOD_INIT_NO_IDLE flag to tptc

2015-11-02 Thread Peter Ujfalusi
On 11/02/2015 12:24 PM, Vinod Koul wrote: > On Mon, Nov 02, 2015 at 12:11:00PM +0200, Peter Ujfalusi wrote: >> In Linux we do not have driver for TPTCs of eDMA3 since there is no need to >> do any configuration within TPTC for the eDMA3 to be operational. All >> configuration is via the TPCC. >> To

[PATCH 2/3] pwm: Add PWM driver for OMAP using dual-mode timers

2015-11-02 Thread Neil Armstrong
Adds support for using a OMAP dual-mode timer with PWM capability as a Linux PWM device. The driver controls the timer by using the dmtimer API. Add a platform_data structure for each pwm-omap-dmtimer nodes containing the dmtimers functions in order to get driver not rely on platform specific func

[PATCH 3/3] arm: plat-omap: Add PWM dmtimer platform data quirks

2015-11-02 Thread Neil Armstrong
In order to set the currently platform dependent dmtimer functions pointers as platform data for the pwm-omap-dmtimer platform driver, add it to plat-omap auxdata_lookup table. Suggested-by: Tony Lindgren Signed-off-by: Neil Armstrong --- arch/arm/mach-omap2/pdata-quirks.c | 23

[PATCH 0/3] pwm: omap: Add PWM support using dual-mode timers

2015-11-02 Thread Neil Armstrong
This patch is based on an earlier patch by NeilBrown which is based on a older patch from Grant Erickson which provided PWM devices using the 'legacy' interface. The pwm driver was renamed to not be confused with the OMAP4 PWM dedicated hardware and was cleaned with the review from Thierry Reding.

[PATCH 1/3] arm: plat-omap: dmtimer: Add clock source from DT

2015-11-02 Thread Neil Armstrong
Add a function which sets the timer source from the clocks binding on dm_timer_prepare call. In case the clocks property is not valid, it falls back to the set_source() with 32_KHZ clock as default. Suggested-by: Tony Lindgren Signed-off-by: Neil Armstrong --- arch/arm/plat-omap/dmtimer.c | 31

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

2015-11-02 Thread Peter Ujfalusi
Vinod, On 11/02/2015 12:04 PM, Vinod Koul wrote: > On Mon, Nov 02, 2015 at 01:21:19AM -0800, Olof Johansson wrote: >> Hi, >> >> 1) This seems to have broken BBB in -next for me, bisected down to this >> patch. >> >> For bootlog: >> http://arm-soc.

Re: [PATCH] ARM: OMAP: hwmod: AM33xx/43xx: Add HWMOD_INIT_NO_IDLE flag to tptc

2015-11-02 Thread Peter Ujfalusi
On 11/02/2015 12:23 PM, Peter Ujfalusi wrote: > On 11/02/2015 12:24 PM, Vinod Koul wrote: >> On Mon, Nov 02, 2015 at 12:11:00PM +0200, Peter Ujfalusi wrote: >>> In Linux we do not have driver for TPTCs of eDMA3 since there is no need to >>> do any configuration within TPTC for the eDMA3 to be opera

[PATCH] dmaengine: edma: Add dummy driver skeleton for edma3-tptc

2015-11-02 Thread Peter Ujfalusi
The eDMA3 TPTC does not need any software configuration, but it is a separate IP block in the SoC. In order the omap hwmod core to be able to handle the TPTC resources correctly in regards of PM we need to have a driver loaded for it. This patch will add a dummy driver skeleton without probe or rem

Re: [PATCH] spi: ti-qspi: improve ->remove() callback

2015-11-02 Thread Grygorii Strashko
On 10/29/2015 03:57 PM, Felipe Balbi wrote: there's no need to call pm_runtime_get_sync() followed by pm_runtime_put(). We should, instead, just call pm_runtime_put_sync() and pm_runtime_disable(). Sry, but why do we need to call pm_runtime_put[_sync]() here? My be just pm_runtime_disable() wi

Re: [PATCH] spi: ti-qspi: improve ->remove() callback

2015-11-02 Thread Felipe Balbi
Grygorii Strashko writes: > On 10/29/2015 03:57 PM, Felipe Balbi wrote: >> there's no need to call pm_runtime_get_sync() >> followed by pm_runtime_put(). We should, instead, >> just call pm_runtime_put_sync() and pm_runtime_disable(). > > Sry, but why do we need to call pm_runtime_put[_sync]() he

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

2015-11-02 Thread Vinod Koul
for me, bisected down to this > >> patch. > >> > >> For bootlog: > >> http://arm-soc.lixom.net/bootlogs/next/next-20151102/bbb-arm-omap2plus_defconfig.html > >> > >> 2) Please avoid merging DT/platform code in your driver tree, Vinod, > >>

Re: [PATCH] spi: ti-qspi: improve ->remove() callback

2015-11-02 Thread Grygorii Strashko
On 11/02/2015 05:20 PM, Felipe Balbi wrote: > Grygorii Strashko writes: > >> On 10/29/2015 03:57 PM, Felipe Balbi wrote: >>> there's no need to call pm_runtime_get_sync() >>> followed by pm_runtime_put(). We should, instead, >>> just call pm_runtime_put_sync() and pm_runtime_disable(). >> >> Sry,

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

2015-11-02 Thread Peter Ujfalusi
;>>> >>>> 1) This seems to have broken BBB in -next for me, bisected down to this >>>> patch. >>>> >>>> For bootlog: >>>> http://arm-soc.lixom.net/bootlogs/next/next-20151102/bbb-arm-omap2plus_defconfig.html >>>> >&

Re: [PATCH] spi: ti-qspi: improve ->remove() callback

2015-11-02 Thread Felipe Balbi
hi, Grygorii Strashko writes: > On 11/02/2015 05:20 PM, Felipe Balbi wrote: >> Grygorii Strashko writes: >> >>> On 10/29/2015 03:57 PM, Felipe Balbi wrote: there's no need to call pm_runtime_get_sync() followed by pm_runtime_put(). We should, instead, just call pm_runtime_put_sy

OMAP baseline test results for v4.3-rc7

2015-11-02 Thread Paul Walmsley
Here are some basic OMAP test results for Linux v4.3-rc7. Logs and other details at: http://www.pwsan.com/omap/testlogs/test_v4.3-rc7/20151025181941/ Test summary Build: uImage: Pass ( 3/ 3): omap1_defconfig, omap1_defconfig_1510innovator_only, omap1_defc

Re: [PATCH] spi: ti-qspi: improve ->remove() callback

2015-11-02 Thread Grygorii Strashko
On 11/02/2015 06:06 PM, Felipe Balbi wrote: > > hi, > > Grygorii Strashko writes: >> On 11/02/2015 05:20 PM, Felipe Balbi wrote: >>> Grygorii Strashko writes: >>> On 10/29/2015 03:57 PM, Felipe Balbi wrote: > there's no need to call pm_runtime_get_sync() > followed by pm_runtime_pu

Re: [PATCH] spi: ti-qspi: improve ->remove() callback

2015-11-02 Thread Felipe Balbi
Hi, Grygorii Strashko writes: > On 11/02/2015 06:06 PM, Felipe Balbi wrote: >> >> hi, >> >> Grygorii Strashko writes: >>> On 11/02/2015 05:20 PM, Felipe Balbi wrote: Grygorii Strashko writes: > On 10/29/2015 03:57 PM, Felipe Balbi wrote: >> there's no need to call pm_runtim