Re: [PATCH v3 5/7] net: cpsw: Add am33xx MACID readout

2014-08-18 Thread Markus Pargmann
Hi, On Sat, Aug 16, 2014 at 11:46:48AM -0500, Wolfram Sang wrote: diff --git a/drivers/net/ethernet/ti/Makefile b/drivers/net/ethernet/ti/Makefile index 9cfaab8152be..5a31c2b322ee 100644 --- a/drivers/net/ethernet/ti/Makefile +++ b/drivers/net/ethernet/ti/Makefile @@ -8,5 +8,6 @@

Re: [PATCH v3 5/7] net: cpsw: Add am33xx MACID readout

2014-08-18 Thread Markus Pargmann
On Sat, Aug 16, 2014 at 11:53:18AM -0500, Wolfram Sang wrote: + mac_addr[5] = (macid_lo 8) 0xff; + mac_addr[4] = macid_lo 0xff; + mac_addr[3] = (macid_hi 24) 0xff; + mac_addr[2] = (macid_hi 16) 0xff; + mac_addr[1] = (macid_hi 8) 0xff; + mac_addr[0] = macid_hi

Re: [PATCH] pwm: Fix period and polarity in pwm_get() for non-perfect matches

2014-08-18 Thread Thierry Reding
On Wed, Aug 13, 2014 at 05:18:53PM +0200, Geert Uytterhoeven wrote: If pwm_get() finds a look-up entry with a perfect match (both dev_id and con_id match), the loop is aborted, and p still points to the correct struct pwm_lookup. If only an entry with a matching dev_id or con_id is found,

Re: [PATCH] pwm: Fix period and polarity in pwm_get() for non-perfect matches

2014-08-18 Thread Geert Uytterhoeven
Hi Thierry, On Mon, Aug 18, 2014 at 10:20 AM, Thierry Reding thierry.red...@gmail.com wrote: Could we achieve the same by storing a pointer to the best match and then use that instead of p? Perhaps something like this: struct pwm_lookup *entry; ... if

Re: [PATCH] pwm: Fix period and polarity in pwm_get() for non-perfect matches

2014-08-18 Thread Thierry Reding
On Mon, Aug 18, 2014 at 10:38:00AM +0200, Geert Uytterhoeven wrote: Hi Thierry, On Mon, Aug 18, 2014 at 10:20 AM, Thierry Reding thierry.red...@gmail.com wrote: Could we achieve the same by storing a pointer to the best match and then use that instead of p? Perhaps something like this:

Re: [PATCH 07/15] tty: serial: 8250_dma: enqueue RX dma again on completion.

2014-08-18 Thread One Thousand Gnomes
if (!up-dma || dma_err) status = serial8250_rx_chars(up, status); + + if (dma_err port-type == PORT_OMAP_16750) + serial8250_rx_dma(up, 0); Can we stick to a 'has dma' flag and port-rx_dma() type usages so that we don't

Re: [PATCH 0/5] MMC cleanup of dev_pm_ops and .owner

2014-08-18 Thread Ulf Hansson
On 12 August 2014 18:14, Peter Griffin peter.grif...@linaro.org wrote: This series cleans up a few platform drivers in how they are declaring there dev_pm_ops structs, and gets rid of a few now redundant #else conditions. Also it removes the .owner field of drivers which use

Re: [PATCH 05/15] tty: serial: Add 8250-core based omap driver

2014-08-18 Thread Heikki Krogerus
Hi, On Fri, Aug 15, 2014 at 07:42:33PM +0200, Sebastian Andrzej Siewior wrote: diff --git a/drivers/tty/serial/8250/8250_core.c b/drivers/tty/serial/8250/8250_core.c index cc90c19..ab003b6 100644 --- a/drivers/tty/serial/8250/8250_core.c +++ b/drivers/tty/serial/8250/8250_core.c @@ -264,6

Re: [PATCH 06/15] tty: serial: 8250_dma: handle error on TX submit

2014-08-18 Thread Heikki Krogerus
On Fri, Aug 15, 2014 at 07:42:34PM +0200, Sebastian Andrzej Siewior wrote: diff --git a/include/linux/serial_8250.h b/include/linux/serial_8250.h index b161eee..02e82dc 100644 --- a/include/linux/serial_8250.h +++ b/include/linux/serial_8250.h @@ -85,6 +85,7 @@ struct uart_8250_port {

Re: [PATCH 1/5] mmc: remove .owner field for drivers using module_platform_driver

2014-08-18 Thread Michal Simek
Hi Peter, On 08/12/2014 06:14 PM, Peter Griffin wrote: This patch removes the superflous .owner field for drivers which use the module_platform_driver API, as this is overriden in platform_driver_register anyway. Signed-off-by: Peter Griffin peter.grif...@linaro.org ---

Re: [PATCH v7] 8250-core based serial driver for OMAP + DMA

2014-08-18 Thread Peter Hurley
On 08/15/2014 04:28 PM, Tony Lindgren wrote: * Sebastian Andrzej Siewior bige...@linutronix.de [140815 12:16]: On 08/15/2014 08:17 PM, Lennart Sorensen wrote: Are you saying that with the new driver you have to respond to the RX irq faster than before to avoid overflows? It is not quite

Lock Directory failure

2014-08-18 Thread Felipe Balbi
[5.306169] [6c617578] *pgd= [5.309969] Internal error: Oops: 805 [#1] SMP ARM [5.314972] Modules linked in: [5.318170] CPU: 0 PID: 1092 Comm: mount Not tainted 3.17.0-rc1-next-20140818-2-gd236526 #636 [5.327433] task: de6d0dc0 ti: de35 task.ti: de35 [5.333082] PC

Re: [PATCH v7] 8250-core based serial driver for OMAP + DMA

2014-08-18 Thread Felipe Balbi
On Mon, Aug 18, 2014 at 11:15:17AM -0400, Peter Hurley wrote: On 08/15/2014 04:28 PM, Tony Lindgren wrote: * Sebastian Andrzej Siewior bige...@linutronix.de [140815 12:16]: On 08/15/2014 08:17 PM, Lennart Sorensen wrote: Are you saying that with the new driver you have to respond to the

[PATCH 2/2] clk: ti: dra7-atl: Provide error check for incoming parameters in set_rate

2014-08-18 Thread Nishanth Menon
Check for valid parameters in check rate. Else, we end up getting errors. This occurs as part of the inital clock tree update of child clock nodes where new_rate could be 0 for non functional clocks. Fixes: 9ac33b0ce81fa48 ( CLK: TI: Driver for DRA7 ATL (Audio Tracking Logic)) Signed-off-by:

[PATCH 1/2] clk: ti: divider: Provide error check for incoming parameters in set_rate

2014-08-18 Thread Nishanth Menon
Check for valid parameters in check rate. Else, we end up getting errors like: [0.00] Division by zero in kernel. [0.00] CPU: 0 PID: 0 Comm: swapper/0 Not tainted 3.17.0-rc1 #1 [0.00] [c0015160] (unwind_backtrace) from [c0011978] (show_stack+0x10/0x14) [0.00]

[PATCH 0/2] clk: ti: Fix up error checks in set_rate (and div 0 error)

2014-08-18 Thread Nishanth Menon
Hi, The following patches are based on v3.17-rc1 Prior to this series: http://slexy.org/view/s20QH6PW4x (notice the /0 div error spam at initial boot log) After this series: http://slexy.org/view/s20tPNXPf4 Nishanth Menon (2): clk: ti: divider: Provide error check for incoming parameters in

RE: Questions about enabling dps(dynamic power switching) and SLM

2014-08-18 Thread Deepa Raj
Hi Tony, I have few more questions on OMAP3: 1) As you know there are power scripts in drivers/mfd folder for PMIC chip (Triton2). Is this mandatory to use these scripts like there are P1,P2,P3 areas are there in PMIC. 2) My understanding was when regulator_enable and disable are called from

Re: [PATCH v3 5/7] net: cpsw: Add am33xx MACID readout

2014-08-18 Thread Mugunthan V N
On Saturday 16 August 2014 08:48 PM, Markus Pargmann wrote: + mac_addr[5] = (macid_lo 8) 0xff; + mac_addr[4] = macid_lo 0xff; + mac_addr[3] = (macid_hi 24) 0xff; + mac_addr[2] = (macid_hi 16) 0xff; + mac_addr[1] = (macid_hi 8) 0xff; + mac_addr[0] = macid_hi

Re: [PATCH] ARM: dts: DRA7: fix interrupt-cells for GPIO

2014-08-18 Thread Nishanth Menon
On 07/30/2014 02:20 PM, Nishanth Menon wrote: GPIO modules are also interrupt sources. However, they require both the GPIO number and IRQ type to function properly. By declaring that GPIO uses interrupt-cells=1, we essentially do not allow users of the nodes to use the interrupt property

[PATCH] ARM: OMAP2+: board-generic: add support for AM57xx family

2014-08-18 Thread Nishanth Menon
AM57xx processor family are variants of DRA7 family of processors and targetted at industrial and non-automotive applications. Signed-off-by: Nishanth Menon n...@ti.com --- NOTE: there is no discrenable features yet defined in register set at this point in time. Tested on AM5728 compatible

Re: [PATCH v3 5/7] net: cpsw: Add am33xx MACID readout

2014-08-18 Thread Steven Rostedt
On Mon, 18 Aug 2014 23:54:26 +0530 Mugunthan V N mugunthan...@ti.com wrote: On Saturday 16 August 2014 08:48 PM, Markus Pargmann wrote: + mac_addr[5] = (macid_lo 8) 0xff; + mac_addr[4] = macid_lo 0xff; + mac_addr[3] = (macid_hi 24) 0xff; + mac_addr[2] = (macid_hi 16) 0xff;

Re: [PATCH v3 5/7] net: cpsw: Add am33xx MACID readout

2014-08-18 Thread Mugunthan V N
On Tuesday 19 August 2014 01:11 AM, Steven Rostedt wrote: On Mon, 18 Aug 2014 23:54:26 +0530 Mugunthan V N mugunthan...@ti.com wrote: On Saturday 16 August 2014 08:48 PM, Markus Pargmann wrote: + mac_addr[5] = (macid_lo 8) 0xff; + mac_addr[4] = macid_lo 0xff; + mac_addr[3] =

[PATCH 0/2] Input: palmas: add support for palmas power button

2014-08-18 Thread Nishanth Menon
Many Palmas PMIC variants have support for power button feature. This feature depends on certain One Time Program (OTP) and board pull configurations (POWERHOLD signal). However, on many platforms such as DRA72-evm, OMAP5-uevm, this may be used to generate input events similar to

[PATCH 1/2] doc: dt/bindings: input: introduce palmas power button description

2014-08-18 Thread Nishanth Menon
Many palmas family of PMICs have support for interrupt based power button. This allows the device to notify the processor of external push button events over the shared palmas interrupt. Document the hardware support for the same. Signed-off-by: Nishanth Menon n...@ti.com ---

[PATCH 2/2] Input: misc: introduce palmas-pwrbutton

2014-08-18 Thread Nishanth Menon
Many palmas family of PMICs have support for interrupt based power button. This allows the device to notify the processor of external push button events over the shared palmas interrupt. However, this event is generated only during a press operation. Software is supposed to poll(sigh!) for

Re: [PATCH v3 5/7] net: cpsw: Add am33xx MACID readout

2014-08-18 Thread Steven Rostedt
On Tue, 19 Aug 2014 01:28:09 +0530 Mugunthan V N mugunthan...@ti.com wrote: This will fail for DRA7xx not in AM33xx OK, is there a way to test the difference? -- Steve -- To unsubscribe from this list: send the line unsubscribe linux-omap in the body of a message to majord...@vger.kernel.org

Re: Lock Directory failure

2014-08-18 Thread Hugh Dickins
address 6c617578 [5.303340] pgd = de5c [5.306169] [6c617578] *pgd= [5.309969] Internal error: Oops: 805 [#1] SMP ARM [5.314972] Modules linked in: [5.318170] CPU: 0 PID: 1092 Comm: mount Not tainted 3.17.0-rc1-next-20140818-2-gd236526 #636 [5.327433] task

[PATCH v2 0/8] Beaglebone-Black HDMI audio

2014-08-18 Thread Jyri Sarha
Changes since v1 of the patch set: - Move gpio-clock under drivers drivers/clk/ti/ and change compatible string to ti,gpio-clock to get it accepted - Drop already applied ASoC: mcasp: Fix implicit BLCK divider setting - Fix typo from ARM: dts: am33xx: Add external clock provider commit message

[PATCH v2 8/8] ARM: OMAP2+: omap2plus_defconfig: Enable BeagleBone Black HDMI audio support

2014-08-18 Thread Jyri Sarha
Select CONFIG_SND_EDMA_SOC=m and CONFIG_SND_AM335X_SOC_NXPTDA_EVM=m for BBB HDMI audio. Signed-off-by: Jyri Sarha jsa...@ti.com --- arch/arm/configs/omap2plus_defconfig |2 ++ 1 file changed, 2 insertions(+) diff --git a/arch/arm/configs/omap2plus_defconfig

[PATCH v2 7/8] ARM: OMAP2+: omap2plus_defconfig: TDA998X HDMI trough tilcdc,slave

2014-08-18 Thread Jyri Sarha
Select CONFIG_DRM=m, CONFIG_DRM_I2C_NXP_TDA998X=m, and CONFIG_DRM_TILCDC=m, for Beaglebone-Black HDMI video support. Signed-off-by: Jyri Sarha jsa...@ti.com --- arch/arm/configs/omap2plus_defconfig |3 +++ 1 file changed, 3 insertions(+) diff --git a/arch/arm/configs/omap2plus_defconfig

[PATCH v2 6/8] ARM: dts: am335x-boneblack: Add HDMI audio support

2014-08-18 Thread Jyri Sarha
Adds mcasp0_pins, clk_mcasp0_fixed, clk_mcasp0, mcasp0, hdmi_audio, and sound nodes. Signed-off-by: Jyri Sarha jsa...@ti.com --- arch/arm/boot/dts/am335x-boneblack.dts | 54 1 file changed, 54 insertions(+) diff --git a/arch/arm/boot/dts/am335x-boneblack.dts

[PATCH v2 4/8] ASoC: davinci: HDMI audio build for AM33XX and TDA998x

2014-08-18 Thread Jyri Sarha
Adds configuration option for HDMI audio support for AM33XX based boards with NXP TDA998x HDMI transmitter. The audio is connected to NXP TDA998x trough McASP running in i2s mode. Signed-off-by: Jyri Sarha jsa...@ti.com --- sound/soc/davinci/Kconfig | 12 1 file changed, 12

[PATCH v2 5/8] ARM: dts: am33xx: Add external clock provider

2014-08-18 Thread Jyri Sarha
Add external clock provider for am33xx devices. Signed-off-by: Jyri Sarha jsa...@ti.com --- arch/arm/boot/dts/am33xx.dtsi | 10 ++ 1 file changed, 10 insertions(+) diff --git a/arch/arm/boot/dts/am33xx.dtsi b/arch/arm/boot/dts/am33xx.dtsi index 3a0a161..d2cc397 100644 ---

[PATCH v2 1/8] clk: ti: add gpio controlled clock

2014-08-18 Thread Jyri Sarha
The added ti,gpio-clock is a basic clock that can be enabled and disabled trough a gpio output. The DT binding document for the clock is also added. For EPROBE_DEFER handling the registering of the clock has to be delayed until of_clk_get() call time. Signed-off-by: Jyri Sarha jsa...@ti.com ---

[PATCH v2 3/8] ASoC: davinci-evm: HDMI audio support for TDA998x trough McASP I2S bus

2014-08-18 Thread Jyri Sarha
Add machine driver support for BeagleBone-Black HDMI audio. BBB has NXP TDA998X HDMI transmitter connected to McASP port in I2S mode. The 44100 Hz sample-rate and it's multiples can not be accurately produced on BBB. The only supported sample format is SNDRV_PCM_FORMAT_S32_LE. The 8 least

[PATCH v2 2/8] drm/tilcdc: Add I2S HDMI audio config for tda998x

2014-08-18 Thread Jyri Sarha
The configuration is needed for HDMI audio. The swap and mirr parameters have to be correctly set in the configuration in order to have proper colors in the HDMI picture. Signed-off-by: Jyri Sarha jsa...@ti.com --- drivers/gpu/drm/tilcdc/tilcdc_slave.c | 24 +++- 1 file

Re: [PATCH v3 5/7] net: cpsw: Add am33xx MACID readout

2014-08-18 Thread Javier Martinez Canillas
Hello Mugunthan, On Mon, Aug 18, 2014 at 9:58 PM, Mugunthan V N mugunthan...@ti.com wrote: Thus, for this patchset, as is: Tested-by: Steven Rostedt rost...@goodmis.org This will fail for DRA7xx not in AM33xx cpsw_am33xx_cm_get_macid() checks for of_machine_is_compatible(ti,am33xx) and

Re: [PATCH 2/2] Input: misc: introduce palmas-pwrbutton

2014-08-18 Thread Dmitry Torokhov
Hi NIshanth, On Mon, Aug 18, 2014 at 03:13:30PM -0500, Nishanth Menon wrote: Many palmas family of PMICs have support for interrupt based power button. This allows the device to notify the processor of external push button events over the shared palmas interrupt. However, this event is

Re: [PATCH 1/2] doc: dt/bindings: input: introduce palmas power button description

2014-08-18 Thread Dmitry Torokhov
On Mon, Aug 18, 2014 at 03:13:29PM -0500, Nishanth Menon wrote: Many palmas family of PMICs have support for interrupt based power button. This allows the device to notify the processor of external push button events over the shared palmas interrupt. Document the hardware support for the