Hi,
On 05/08/2014 06:40 PM, Kevin Hilman wrote:
On Thu, May 8, 2014 at 8:31 AM, Kevin Hilman khil...@linaro.org wrote:
Roger Quadros rog...@ti.com writes:
Hi,
Nishant pointed me to a booting issue with omap4-panda-es on linux-next but
I'm observing
similar issues, although less frequent,
: OMAP2+: hwmod: Fix SOFTRESET logic was merged to.
On OMAP4+ we should always try to wait until IP reset is finished before
proceed.
Reviewed-by: Grygorii Strashko grygorii.stras...@ti.com
Error log example (k3.4):
[ 5766.575347] [ cut here ]
[ 5766.580749] WARNING
On 01/20/2014 10:06 PM, Nishanth Menon wrote:
Move all OMAP4 PM errata initializations to centralized location in
omap4_pm_init_early. This allows for users to utilize the erratas
in various submodules as needed.
Tested-by: Grygorii Strashko grygorii.stras...@ti.com
This patch fixes build
On 11/07/2013 05:34 PM, Nishanth Menon wrote:
On 10/30/2013 03:21 PM, Daniel Mack wrote:
[...]
diff --git a/arch/arm/common/edma.c b/arch/arm/common/edma.c
index 8e1a024..f15cdb9 100644
--- a/arch/arm/common/edma.c
+++ b/arch/arm/common/edma.c
[...]
+static int edma_pm_suspend(struct device
-by: Taras Kondratiuk taras.kondrat...@linaro.org
Signed-off-by: Grygorii Strashko grygorii.stras...@ti.com
---
Changes in V2:
- get rid of using omap4_mpuss_read_prev_context_state();
- move call of gic_dist_enable() in omap-mpuss-lowpower.c
- add call of gic_timer_retrigger()
Based on: v3.12-rc5
arch
PandaES and Tablet/Blaze with
SOM OMAP4460 ES1.0 when CONFIG_CPU_IDLE is enabled.
- GIC distributor is disabled
- scheduling is not performed any more
Grygorii Strashko (2):
Revert ARM: OMAP4+: CPUidle: Deprecate use of
omap4_mpuss_read_prev_context_state()
ARM: OMAP4460: cpuidle: WA for ROM
khil...@linaro.org
Reported-and-Tested-by: Taras Kondratiuk taras.kondrat...@linaro.org
Signed-off-by: Grygorii Strashko grygorii.stras...@ti.com
---
arch/arm/mach-omap2/common.h |1 +
arch/arm/mach-omap2/cpuidle44xx.c | 34
+-
arch/arm/mach-omap2/omap4
+: CPUidle: Deprecate use of
omap4_mpuss_read_prev_context_state().
Cc: Santosh Shilimkar santosh.shilim...@ti.com
Cc: Kevin Hilman khil...@linaro.org
Reported-and-Tested-by: Taras Kondratiuk taras.kondrat...@linaro.org
Signed-off-by: Grygorii Strashko grygorii.stras...@ti.com
---
arch/arm/mach-omap2
Sorry, forgot to mention that patches are based on 4a10c2a Linux 3.12-rc2
On 10/17/2013 12:23 PM, Grygorii Strashko wrote:
The same workaround as ff999b8a0983ee15668394ed49e38d3568fc6859
ARM: OMAP4460: Workaround for ROM bug because of CA9 r2pX GIC control register
change.
need to be applied
On 10/17/2013 04:57 PM, Santosh Shilimkar wrote:
On Thursday 17 October 2013 05:24 AM, Grygorii Strashko wrote:
On OMAP4+ devices, GIC register context is lost when MPUSS hits the
OSWR. On the CPU wakeup path, ROM code gets executed and one of the
steps in it is to restore the saved context
On 08/20/2013 11:18 AM, Samuel Ortiz wrote:
On Tue, Aug 20, 2013 at 08:13:54AM +0100, Graeme Gregory wrote:
On 20/08/13 02:01, Samuel Ortiz wrote:
Hi Grygorii,
On Thu, Jul 25, 2013 at 04:15:46PM +0300, Grygorii Strashko wrote:
This patch series intorduces twl6030-irq module rework to use
On 08/19/2013 03:11 PM, Wolfram Sang wrote:
Hi,
Which means your original patch starts to make a lot more sense. I
wonder is this is really what we should be doing though - breaking out
of the loop, I mean.
Yup, that is fine. I applied the old patch with Acks from Hein and
Felipe to -next.
On 08/06/2013 11:11 AM, Rajendra Nayak wrote:
Hi Grygorii,
On Monday 05 August 2013 05:14 PM, Grygorii Strashko wrote:
On 08/04/2013 07:27 PM, Rajendra Nayak wrote:
From: R Sricharan r.sricha...@ti.com
The DRA7xx is a high-performance, infotainment application device,
based on enhanced OMAP
On 08/04/2013 07:27 PM, Rajendra Nayak wrote:
From: R Sricharan r.sricha...@ti.com
The DRA7xx is a high-performance, infotainment application device,
based on enhanced OMAP architecture integrated on a 28-nm technology.
Since DRA7 is a platform supported only using DT, the cpu detection
is
irq_domain:
The irq_find_mapping() is used in twl6030_mmc_card_detect_config()
to get virtual IRQ number.
Based on v3.11-rc1
Tested generation of RTC_ALARM(3) and PWRON(0) IRQs on OMAP4430/TWL6030 and
OMAP4470/TWL6032.
Grygorii Strashko (3):
mfd: twl6030-irq: add error check when IRQs are masked
of irq_domain. So, IRQ mapping and descriptors
allocation will be performed dynamically basing on DT configuration.
The error message will be reported in case if unmapped IRQ is received by
TWL6030 (virq==0).
Signed-off-by: Grygorii Strashko grygorii.stras...@ti.com
---
drivers/mfd/twl6030-irq.c | 119
if nested IRQ is marked
as IRQS_PENDING.
Signed-off-by: Naga Venkata Srikanth V vnv.srika...@samsung.com
Signed-off-by: Oleg_Kosheliev oleg.koshel...@ti.com
Signed-off-by: Grygorii Strashko grygorii.stras...@ti.com
---
drivers/mfd/twl6030-irq.c | 150
Create struct twl6030_irq and place all local variables inside it.
Also allocate twl6030_irq structure dynamically during initialization.
Signed-off-by: Grygorii Strashko grygorii.stras...@ti.com
---
drivers/mfd/twl6030-irq.c | 101 ++---
1 file changed
From: Oleksandr Dmytryshyn oleksandr.dmytrys...@ti.com
The TWL6032 PMIC has different IRQ status bits meaning.
Hence, add interrupt mapping table for the twl6032.
Signed-off-by: Oleksandr Dmytryshyn oleksandr.dmytrys...@ti.com
Signed-off-by: Grygorii Strashko grygorii.stras...@ti.com
Add a missed check for errors when TWL IRQs are masked
initially on probe and report an error in case of failure.
Signed-off-by: Grygorii Strashko grygorii.stras...@ti.com
---
Hi Graeme,
I've tried to use only one write to STS_A register to clear IRQ status bits,
but,
unfortunatelly
On 07/23/2013 09:08 PM, Graeme Gregory wrote:
On 23/07/13 17:07, Grygorii Strashko wrote:
Add a missed check for errors when TWL IRQs are masked
initially on probe and report an error in case of failure.
Signed-off-by: Grygorii Strashko grygorii.stras...@ti.com
---
drivers/mfd/twl6030-irq.c
On 07/24/2013 01:49 PM, Lee Jones wrote:
On Tue, 23 Jul 2013, Grygorii Strashko wrote:
From: Naga Venkata Srikanth V vnv.srika...@samsung.com
1) Removed request_irq() and replaced it with request_threaded_irq().
2) Removed generic_handle_irq() and replaced it with
handle_nested_irq
On 07/24/2013 03:50 PM, Lee Jones wrote:
+ if (ret) {
+ pr_warn(%s: I2C error %d reading PIH ISR\n, __func__, ret);
Does the user really care which function we're returning from.
Would it be better if you replace '__func__' with the device name?
This module hasn't been
On 07/24/2013 02:35 PM, Lee Jones wrote:
On Tue, 23 Jul 2013, Grygorii Strashko wrote:
Since the TWL6030 PMIC is used with OMAP4 SoCs only and OMAP4 legacy
boot is dropped there are no needs to allocate the range of IRQ
descriptors during system boot to support TWL6030 IRQs.
Hence, convert
On 07/24/2013 02:52 PM, Lee Jones wrote:
On Tue, 23 Jul 2013, Grygorii Strashko wrote:
From: Oleksandr Dmytryshyn oleksandr.dmytrys...@ti.com
This patch adds interrupt mapping table for the twl6032.
Repeating the $SUBJECT line is never helpful.
Signed-off-by: Oleksandr Dmytryshyn
oleg.koshel...@ti.com
Signed-off-by: Grygorii Strashko grygorii.stras...@ti.com
---
drivers/mfd/twl6030-irq.c | 146 +++--
1 file changed, 49 insertions(+), 97 deletions(-)
diff --git a/drivers/mfd/twl6030-irq.c b/drivers/mfd/twl6030-irq.c
index 277a8db..b6030d9
on OMAP4430/TWL6030 and
OMAP4470/TWL6032.
Grygorii Strashko (2):
mfd: twl6030-irq: add error check when IRQs are masked initially
mfd: twl6030-irq: convert to use linear irq_domain
Naga Venkata Srikanth V (1):
mfd: twl6030-irq: migrate to IRQ threaded handler
Oleksandr Dmytryshyn (1):
mfd
From: Oleksandr Dmytryshyn oleksandr.dmytrys...@ti.com
This patch adds interrupt mapping table for the twl6032.
Signed-off-by: Oleksandr Dmytryshyn oleksandr.dmytrys...@ti.com
Signed-off-by: Grygorii Strashko grygorii.stras...@ti.com
---
drivers/mfd/twl6030-irq.c | 49
of irq_domain. So, IRQ mapping and descriptors
allocation will be performed dynamically basing on DT configuration.
The error message will be reported in case if unmapped IRQ is received by
TWL6030 (virq==0).
Signed-off-by: Grygorii Strashko grygorii.stras...@ti.com
---
drivers/mfd/twl6030-irq.c | 114
Add a missed check for errors when TWL IRQs are masked
initially on probe and report an error in case of failure.
Signed-off-by: Grygorii Strashko grygorii.stras...@ti.com
---
drivers/mfd/twl6030-irq.c | 13 +
1 file changed, 9 insertions(+), 4 deletions(-)
diff --git a/drivers
Hi Tony, Stephen
On 07/19/2013 10:39 AM, Tony Lindgren wrote:
* Stephen Warren swar...@wwwdotorg.org [130718 12:33]:
On 07/18/2013 01:36 AM, Tony Lindgren wrote:
* Stephen Warren swar...@wwwdotorg.org [130717 14:30]:
On 07/16/2013 03:05 AM, Tony Lindgren wrote:
...
Why shouldn't e.g. a
On 07/19/2013 05:39 PM, Sergei Shtylyov wrote:
Hello.
On 19-07-2013 13:27, Oleksandr Kozaruk wrote:
GPADC is the general purpose ADC present on twl6030.
The dt data is interrupt used to trigger end of ADC
conversion.
Signed-off-by: Oleksandr Kozaruk oleksandr.koza...@ti.com
---
On 07/18/2013 11:09 AM, Tony Lindgren wrote:
* Grygorii Strashko grygorii.stras...@ti.com [130717 09:48]:
Hi,
On 07/17/2013 06:32 PM, Tony Lindgren wrote:
* Grygorii Strashko grygorii.stras...@ti.com [130717 04:49]:
Add dynamic active/idle pin states for uart3/4 which will be applied
when
On 07/18/2013 11:14 AM, Tony Lindgren wrote:
* Grygorii Strashko grygorii.stras...@ti.com [130717 10:11]:
On 07/17/2013 06:38 PM, Tony Lindgren wrote:
* Grygorii Strashko grygorii.stras...@ti.com [130717 04:49]:
Before switching to DT pinctrl states of OMAP IPs have been handled by hwmod
Hi Tony,
On 07/18/2013 12:04 PM, Tony Lindgren wrote:
* Grygorii Strashko grygorii.stras...@ti.com [130718 02:01]:
On 07/18/2013 11:09 AM, Tony Lindgren wrote:
Don't think it's debug code - IO chain need to be rearmed after each
PRCM IO IRQ - otherwise IO wakeup events may be lost (at least
/2670291/
- mmc: omap_hsmmc: Remux pins to support SDIO interrupt and PM runtime
https://patchwork.kernel.org/patch/2690191/
- [PATCH 00/11] drivers: Add Pinctrl PM support
https://lkml.org/lkml/2013/5/31/210
Grygorii Strashko (3):
pinctrl: rollback check for !dev-pins in pinctrl_pm_select*() APIs
Signed-off-by: Grygorii Strashko grygorii.stras...@ti.com
---
arch/arm/boot/dts/omap4-sdp.dts | 34 ++
1 file changed, 30 insertions(+), 4 deletions(-)
diff --git a/arch/arm/boot/dts/omap4-sdp.dts b/arch/arm/boot/dts/omap4-sdp.dts
index 7951b4e..2624504 100644
...@linaro.org
Cc: Stephen Warren swar...@wwwdotorg.org
Signed-off-by: Grygorii Strashko grygorii.stras...@ti.com
---
Seems this one can be squashed in series
[PATCH 0/4] improved support for runtime muxing for pinctrl
http://www.spinics.net/lists/arm-kernel/msg258827.html
drivers/pinctrl/core.c
SDIO interrupt and PM runtime
https://patchwork.kernel.org/patch/2690191/
- [PATCH 00/11] drivers: Add Pinctrl PM support
https://lkml.org/lkml/2013/5/31/210
CC: Linus Walleij linus.wall...@linaro.org
Cc: Stephen Warren swar...@wwwdotorg.org
Signed-off-by: Grygorii Strashko grygorii.stras
to do the muxing for
runtime PM.
I've posted my patch and tested this patch series.
Seems everything is ok (except patch pinctrl: Remove duplicate code in
pinctrl_pm_select_state functions).
See http://www.spinics.net/lists/arm-kernel/msg259180.html
So, Tested-By: Grygorii Strashko
On 07/17/2013 02:57 PM, Roger Quadros wrote:
Hi Grygorii,
On 07/17/2013 02:41 PM, Grygorii Strashko wrote:
Hi Tony, Kevin
This patch series introduces dynamic pinctrl handling in OMAP device framework
in the same way as it was before switching to DT.
This allow OMAP devices driver's
Hi,
On 07/17/2013 06:32 PM, Tony Lindgren wrote:
* Grygorii Strashko grygorii.stras...@ti.com [130717 04:49]:
Add dynamic active/idle pin states for uart3/4 which will be applied
when uart3/4 state is switched from active to idle and back by Runtime
PM or during system suspend.
This is good
On 07/17/2013 06:38 PM, Tony Lindgren wrote:
* Grygorii Strashko grygorii.stras...@ti.com [130717 04:49]:
Before switching to DT pinctrl states of OMAP IPs have been handled by hwmod
framework. After switching to DT-boot the pinctrl handling was dropped from
hwmod framework
Hi Rajendra,
On 07/11/2013 12:17 PM, Rajendra Nayak wrote:
On Wednesday 10 July 2013 09:37 PM, Felipe Balbi wrote:
how about something like below ? It makes omap_device/hwmod and
pm_runtime agree on the initial state of the device and will prevent
-runtime_resume() from being called on first
Hi Hein, Felipe
On 07/16/2013 12:42 PM, Felipe Balbi wrote:
Hi,
On Tue, Jul 16, 2013 at 05:33:47PM +0800, Hein Tibosch wrote:
On 7/16/2013 5:03 PM, Felipe Balbi wrote:
Hi,
On Tue, Jul 16, 2013 at 04:19:35PM +0800, Hein Tibosch wrote:
Hi Vikram,
On a OMAP4460, i2c-bus-3:
A driver (lm75)
Hi Felipe,
On 07/16/2013 02:27 PM, Felipe Balbi wrote:
Hi,
On Tue, Jul 16, 2013 at 02:01:11PM +0300, Grygorii Strashko wrote:
On a OMAP4460, i2c-bus-3:
A driver (lm75) is causing many 'timeout waiting for bus ready' errors.
SDA remains high (as it should), but SCL remains low after a NACK
Hi Tony,
This patch causes boot failure when I've applied my patch
[RFC] ARM: OMAP2+: omap_device: add pinctrl handling
https://lkml.org/lkml/2013/6/21/309
on top of it:
[0.264007] L310 cache controller enabled
[0.268310] l2x0: 16 ways, CACHE_ID 0x41c4, AUX_CTRL 0x7e47,
Cache
Hi Tony,
On 07/16/2013 04:41 PM, Tony Lindgren wrote:
* Grygorii Strashko grygorii.stras...@ti.com [130716 06:22]:
Hi Tony,
This patch causes boot failure when I've applied my patch
[RFC] ARM: OMAP2+: omap_device: add pinctrl handling
https://lkml.org/lkml/2013/6/21/309
on top of it:
Hmm
Hi All,
I have a question regarding this patch and IIO in general
- Does IIO provide sync mechanism with system wide suspend/resume or
this should be handled by each driver itself?
What if during system suspend iio_read_channel_raw() (or any other
consumer API) will be called after gpadc
://www.spinics.net/lists/arm-kernel/msg258062.html
[3] http://www.spinics.net/lists/arm-kernel/msg258040.html
CC: Tony Lindgren t...@atomide.com
CC: Rajendra Nayak rna...@ti.com
CC: Felipe Balbi ba...@ti.com
CC: Kevin Hilman khil...@linaro.org
Signed-off-by: Grygorii Strashko grygorii.stras...@ti.com
---
tested
after all its parameters are
configured.
CC: Tony Lindgren t...@atomide.com
CC: Rajendra Nayak rna...@ti.com
CC: Felipe Balbi ba...@ti.com
CC: Kevin Hilman khil...@linaro.org
Signed-off-by: Grygorii Strashko grygorii.stras...@ti.com
---
tested on OMAP4 SDP with and without earlyprintk enabled
On 07/11/2013 01:16 PM, Felipe Balbi wrote:
in order to make HWMOD and pm_runtime agree on the
initial state of the device, we will unidle the device
and call pm_runtime_set_active() to tell pm_runtime
that the device is really active.
By the time driver's probe() is reached, a call to
Hi Linus,
On 07/10/2013 11:36 PM, Linus Walleij wrote:
On Thu, Jun 27, 2013 at 4:01 PM, Grygorii Strashko
grygorii.stras...@ti.com wrote:
I think, In the future the OMAP pinctrl configurations would be manged in
more flexible way then now (thanks to pinctrl PM helpers and you;))
- Idle state
On 07/11/2013 12:17 PM, Rajendra Nayak wrote:
On Wednesday 10 July 2013 09:37 PM, Felipe Balbi wrote:
how about something like below ? It makes omap_device/hwmod and
pm_runtime agree on the initial state of the device and will prevent
-runtime_resume() from being called on first
Hi,
On 07/11/2013 09:32 AM, Tony Lindgren wrote:
* Felipe Balbi ba...@ti.com [130710 09:18]:
On Wed, Jul 10, 2013 at 07:07:04PM +0300, Felipe Balbi wrote:
how about something like below ? It makes omap_device/hwmod and
pm_runtime agree on the initial state of the device and will prevent
Hi Peter,
On 07/11/2013 10:00 PM, Peter Barada wrote:
On 07/09/2013 11:55 AM, Tony Lindgren wrote:
* Peter Barada peter.bar...@logicpd.com [130709 08:14]:
I'm working with a 3.0.8 kernel in Android ICS and have run into a
problem using wl12xx bluetooth after a suspend/resume cycle.
GPIO_162
On 07/09/2013 10:41 PM, Felipe Balbi wrote:
Hi,
On Tue, Jul 09, 2013 at 09:59:28PM +0300, Grygorii Strashko wrote:
Imagine the device is marked as suspended even though it's fully enabled
(it hasn't been suspended by hwmod due to NO_IDLE flag). In that case
your context structure is all zeroes
Hi,
On 07/09/2013 09:42 AM, Felipe Balbi wrote:
Hi,
On Tue, Jul 09, 2013 at 11:03:54AM +0530, Rajendra Nayak wrote:
On Monday 08 July 2013 07:05 PM, Felipe Balbi wrote:
Hi,
On Mon, Jul 08, 2013 at 06:50:01PM +0530, Rajendra Nayak wrote:
I wonder if this is because the timeouts get now
Hi Marc,
Could you try these patches first:
https://patchwork.kernel.org/patch/2685541/
http://patchwork.ozlabs.org/patch/252639/
Then try this patch alone:
https://patchwork.kernel.org/patch/2689171/
Also could you provide your .config? Do you have lm75 (or any other
hwmon devices) enabled?
Hi Linus,
On 06/26/2013 10:31 PM, Linus Walleij wrote:
On Wed, Jun 26, 2013 at 3:20 PM, Grygorii Strashko
grygorii.stras...@ti.com wrote:
The Sleep pinctrl state is optional - if sleep state isn't defined
then Idle pinctrl state will be used during suspend.
Why? If we have a clear cut
On 06/27/2013 05:45 PM, Tony Lindgren wrote:
* Grygorii Strashko grygorii.stras...@ti.com [130627 07:12]:
As you can see, from 12 pins only 3 pins need to be reconfigured
while switching from active-idle states and back (and as I
mentioned above for OMAP idle == sleep now).
Regarding OFF
Hi All,
On 06/25/2013 09:58 AM, Tony Lindgren wrote:
* Linus Walleij linus.wall...@linaro.org [130624 05:13]:
On Fri, Jun 21, 2013 at 5:03 PM, Grygorii Strashko
grygorii.stras...@ti.com wrote:
Hence, add pinctrl handling in omap_device core:
1) on PM runtime resume
- switch pinctrl state
Picture format fixed. Sorry for that.
On 06/26/2013 04:20 PM, Grygorii Strashko wrote:
Hi All,
On 06/25/2013 09:58 AM, Tony Lindgren wrote:
* Linus Walleij linus.wall...@linaro.org [130624 05:13]:
On Fri, Jun 21, 2013 at 5:03 PM, Grygorii Strashko
grygorii.stras...@ti.com wrote:
Hence, add
://lkml.org/lkml/2013/5/31/210
CC: Hebbar Gururaja gururaja.heb...@ti.com
CC: Linus Walleij linus.wall...@linaro.org
CC: linux-arm-ker...@lists.infradead.org
CC: linux-omap@vger.kernel.org
CC: linux-ker...@vger.kernel.org
Signed-off-by: Grygorii Strashko grygorii.stras...@ti.com
---
Hi Kevin, Tony
On 06/07/2013 12:09 PM, Wolfram Sang wrote:
Class based instantiation can cause huge delays when booting. This
mechanism was used when it was not possible to describe slaves on I2C
busses. We now have other mechanisms, so most embedded I2C will not need
classes and it was explicitly not
Hi Wolfram,
On 06/19/2013 01:15 PM, Wolfram Sang wrote:
On Fri, Jun 07, 2013 at 11:09:26AM +0200, Wolfram Sang wrote:
Class based instantiation can cause huge delays when booting. This
mechanism was used when it was not possible to describe slaves on I2C
busses. We now have other mechanisms,
undefined (remove I2C_CLASS_HWMON).
CC: Tony Lindgren t...@atomide.com
Signed-off-by: Grygorii Strashko grygorii.stras...@ti.com
---
drivers/i2c/busses/i2c-omap.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/i2c/busses/i2c-omap.c b/drivers/i2c/busses/i2c-omap.c
index
, so disable it by
leaving I2C adapter class field undefined (remove I2C_CLASS_HWMON and
I2C_CLASS_SPD) and add a deprecation warning to allow users, relying on this
mechanism, to switch to something better.
CC: Haavard Skinnemoen hskinnem...@gmail.com
Signed-off-by: Grygorii Strashko
Hi Felipe,
On 06/07/2013 10:05 PM, Felipe Balbi wrote:
Hi,
On Fri, Jun 07, 2013 at 09:46:06PM +0300, Grygorii Strashko wrote:
ARDY|NACK and ARDY|AL are set together in OMAP_I2C_STAT_REG, which will be
Have you seen that happen ever ? AL is Arbitration Lost, we never put
OMAP in a multi-master
Hi Felipe,
On 06/07/2013 10:02 PM, Felipe Balbi wrote:
Hi,
On Fri, Jun 07, 2013 at 09:46:05PM +0300, Grygorii Strashko wrote:
Add runtime check at the beginning of omap_i2c_isr/omap_i2c_isr_thread
to be sure that i2c is enabled, before performing IRQ handling and accessing
I2C IP registers
Hi Felipe,
On 06/07/2013 10:07 PM, Felipe Balbi wrote:
Hi,
On Fri, Jun 07, 2013 at 09:46:08PM +0300, Grygorii Strashko wrote:
The omap_i2c_isr() does the irq check and schedules threaded handler if any of
enabled IRQs is active, but currently the I2C IRQs are enabled just once,
when I2C IP
On 06/07/2013 11:51 PM, Kevin Hilman wrote:
Grygorii Strashko grygorii.stras...@ti.com writes:
From: Kevin Hilman khil...@deeprootsystems.com
Currently, runtime PM is used to keep the device enabled only during
active transfers and for a configurable runtime PM autosuspend timout
after
Hi Wolfram,
On 06/07/2013 12:06 PM, Wolfram Sang wrote:
3) Thinking about Mainline: To reach the same target - no I2C
detection - and taking
into account above assumption No changes in default behavior
the following will need to be done:
- change i2c-omap/i2c-gpio DT bindings and add parameter
hi Tomi,
On 06/07/2013 02:53 PM, Tomi Valkeinen wrote:
Hi,
I was testing DT boot with 3.10-rc1 and Pandaboard, and couldn't get the
DVI output's EDID reading to work. Testing with i2cget and i2cdump, I
see that I can read individual bytes with i2cget, but using i2cdump
doesn't work, it just
.
This patch series based on top of:
http://patchwork.ozlabs.org/patch/248188/
Grygorii Strashko (4):
i2c: omap: add runtime check in isr to be sure that i2c is enabled
i2c: omap: handle all irqs befor unblocking omap_i2c_xfer_msg()
i2c: omap: query STP always when NACK is received
i2c: omap: remove
.
This patch series based on top of:
http://patchwork.ozlabs.org/patch/248188/
Grygorii Strashko (4):
i2c: omap: add runtime check in isr to be sure that i2c is enabled
i2c: omap: handle all irqs befor unblocking omap_i2c_xfer_msg()
i2c: omap: query STP always when NACK is received
i2c: omap: remove
Hilman khil...@linaro.org
Signed-off-by: Grygorii Strashko grygorii.stras...@ti.com
---
This is clean-up patch.
drivers/i2c/busses/i2c-omap.c | 33 +
1 file changed, 1 insertion(+), 32 deletions(-)
diff --git a/drivers/i2c/busses/i2c-omap.c b/drivers/i2c/busses/i2c
: omap: ack IRQ in parts.
CC: Kevin Hilman khil...@linaro.org
Signed-off-by: Grygorii Strashko grygorii.stras...@ti.com
---
drivers/i2c/busses/i2c-omap.c |6 ++
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/i2c/busses/i2c-omap.c b/drivers/i2c/busses/i2c-omap.c
index
received
during the last message transmitting/recieving.
This may lead to stuck Bus in Bus Busy until I2C IP reset (idle/enable).
Hence, fix it by querying Stop condition (STP) always when NACK is received.
CC: Kevin Hilman khil...@linaro.org
CC: Felipe Balbi ba...@ti.com
Signed-off-by: Grygorii
IRQ_NONE;
}
Produce warning in case if ISR called when i2c is disabled
CC: Kevin Hilman khil...@linaro.org
Signed-off-by: Grygorii Strashko grygorii.stras...@ti.com
---
drivers/i2c/busses/i2c-omap.c | 10 ++
1 file changed, 10 insertions(+)
diff --git a/drivers/i2c/busses/i2c
()
|- oops is here - I2C module is in
idle state
Cc: Nishanth Menon n...@ti.com
Signed-off-by: Kevin Hilman khil...@linaro.org
Signed-off-by: Grygorii Strashko grygorii.stras...@ti.com
---
drivers/i2c/busses/i2c-omap.c |4
1 file changed, 4 insertions
On 06/05/2013 04:50 PM, Wolfram Sang wrote:
The similar patch already exists:
https://patchwork.kernel.org/patch/2448251/ - [v2,1/2] RTC: rtc-twl:
Fix rtc_reg_map initialization
from Peter Ujfalusi
So, I think it is best if you resend this patch after all the fixes it
needs are applied or you
-foundation.org
Cc: Tony Lindgren t...@atomide.com
Signed-off-by: Grygorii Strashko grygorii.stras...@ti.com
---
drivers/rtc/rtc-twl.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/rtc/rtc-twl.c b/drivers/rtc/rtc-twl.c
index bbda0fd..1698115 100644
Pls, ignore this patch. It duplicates:
https://patchwork.kernel.org/patch/2448251/ - [v2,1/2] RTC: rtc-twl: Fix
rtc_reg_map initialization
from Peter Ujfalusi
On 06/05/2013 02:19 PM, Grygorii Strashko wrote:
The twl-rtc has the following dependencies from other drivers during the boot
On 06/04/2013 09:29 PM, Kevin Hilman wrote:
Grygorii Strashko grygorii.stras...@ti.com writes:
Hi Kevin,
On 06/03/2013 11:59 PM, Kevin Hilman wrote:
Grygorii Strashko grygorii.stras...@ti.com writes:
The OMAP I2C driver has a relation to pinctrl-single driver. As result,
its probe
Hi Peter,
Tested on OMAP4460 SDP - see no issues during boot.
-grygorii
On 04/16/2013 11:30 AM, Peter Ujfalusi wrote:
Initialize the rtc_reg_map in platform_driver's probe function instead at
module_init time. This way we can make sure that the twl-core has been already
probed and initialized
On 06/04/2013 08:46 PM, Kevin Hilman wrote:
Grygorii Strashko grygorii.stras...@ti.com writes:
Hi Kevin,
On 06/01/2013 01:37 AM, Kevin Hilman wrote:
Currently, the RTC IRQ is never wakeup-enabled so is not capable of
bringing the system out of suspend.
On OMAP platforms, we have gotten
Hi
On 06/04/2013 08:49 PM, Wolfram Sang wrote:
On Tue, Jun 04, 2013 at 08:33:42PM +0300, Grygorii Strashko wrote:
Currently the I2C devices instantiation Method 3 Probe an I2C bus for
certain devices (see Documentation/i2c/instantiating-devices) is always
enabled for all platforms (boards
Hi Kevin,
On 06/01/2013 01:37 AM, Kevin Hilman wrote:
Currently, the RTC IRQ is never wakeup-enabled so is not capable of
bringing the system out of suspend.
On OMAP platforms, we have gotten by without this because the TWL RTC
is on an I2C-connected chip which is capable of waking up the OMAP
Hi Kevin,
On 06/03/2013 11:59 PM, Kevin Hilman wrote:
Grygorii Strashko grygorii.stras...@ti.com writes:
The OMAP I2C driver has a relation to pinctrl-single driver. As result,
its probe will be deferred during system boot until late init time,
because the pinctrl-single is initizalized
Hi Kevin, Gururaja
On 05/31/2013 08:34 PM, Kevin Hilman wrote:
Hebbar Gururaja gururaja.heb...@ti.com writes:
Amend the I2C omap pin controller to optionally take a pin control
handle and set the state of the pins to:
- default on boot, resume and before performing an i2c transfer
- idle
: linux-...@vger.kernel.org
CC: linux-ker...@vger.kernel.org
CC: linux-omap@vger.kernel.org
Signed-off-by: Grygorii Strashko grygorii.stras...@ti.com
---
drivers/i2c/Kconfig| 10 ++
drivers/i2c/i2c-core.c |6 ++
2 files changed, 16 insertions(+)
diff --git a/drivers/i2c
On 05/31/2013 01:13 PM, Hebbar Gururaja wrote:
Amend the I2C omap pin controller to optionally take a pin control
handle and set the state of the pins to:
- default on boot, resume and before performing an i2c transfer
- idle after initial default, after resume default, and after each
i2c xfer
On 05/30/2013 07:36 PM, Kevin Hilman wrote:
Tomi Valkeinen tomi.valkei...@ti.com writes:
When using DT, dss device does not have platform data. However,
dss_get_ctx_loss_count() uses dss device's platform data to find the
get_ctx_loss_count function pointer.
To fix this,
On 05/28/2013 09:33 PM, Kevin Hilman wrote:
Nishanth Menon n...@ti.com writes:
On Fri, May 24, 2013 at 5:15 PM, Kevin Hilman khil...@linaro.org wrote:
Kevin Hilman khil...@linaro.org writes:
Nishanth Menon n...@ti.com writes:
[...]
Actually 2 things:
a) patch seems to do the wrong thing
On 05/17/2013 01:58 AM, Tony Lindgren wrote:
* Samuel Ortiz sa...@linux.intel.com [130516 15:22]:
Hi Grygorii,
On Tue, Apr 23, 2013 at 04:19:10PM +0300, Grygorii Strashko wrote:
Shift TWL initialization to module/device init layer, because I2C now is
not initialized on subsys init layer
to few comments on cover letter of
the previous version by Grygorii Strashko.
As, he has mentioned that there are Audio playback use cases which
also requires no_idle_on_suspend and using them on mainline after
this series can cause regression.
What you think will be the right approach
of subsys init layer where initialization dependencies resolved
indirectly in drivers/Makefile now.
Grygorii Strashko (2):
i2c: omap: convert to module_platform_driver()
mfd: twl-core: convert to module_i2c_driver()
drivers/i2c/busses/i2c-omap.c | 14 +-
drivers/mfd/twl-core.c
@vger.kernel.org
Cc: linux-ker...@vger.kernel.org
Signed-off-by: Grygorii Strashko grygorii.stras...@ti.com
---
drivers/mfd/twl-core.c | 12 +---
1 file changed, 1 insertion(+), 11 deletions(-)
diff --git a/drivers/mfd/twl-core.c b/drivers/mfd/twl-core.c
index 4f3baad..f01340d 100644
-off-by: Grygorii Strashko grygorii.stras...@ti.com
---
drivers/i2c/busses/i2c-omap.c | 14 +-
1 file changed, 1 insertion(+), 13 deletions(-)
diff --git a/drivers/i2c/busses/i2c-omap.c b/drivers/i2c/busses/i2c-omap.c
index 4cc2f05..70d3fed 100644
--- a/drivers/i2c/busses/i2c-omap.c
201 - 300 of 319 matches
Mail list logo