On 10/01/2013 06:57 PM, Daniel Mack wrote:
That sound's like a good idea. Everything that make the driver smaller
and easier to understand is certainly a good thing :)
So we can drop this patch favor of your cleanup. However, I appreciate
if you did it on top of the second round of patches I
On 30.09.2013 18:25, Felipe Balbi wrote:
On Fri, Sep 27, 2013 at 08:47:10PM +0200, Daniel Mack wrote:
So are you happy with this v2?
It's just a cosmetic cleanup after all ...
I can't see v2 anywhere... The only version I have is the one with
DEV_PM_OPS still in it.
Seen them this time?
* Daniel Mack | 2013-10-01 15:31:11 [+0200]:
In cppi41_tear_down_chan(), bail out earlier in case td_seen is unset
instead of popping another descriptor when td_desc_seen is also unset.
My system ran into WARN() condition multiple times when
cppi41_tear_down_chan() was called for channels that
On 10/01/2013 03:19 PM, Balaji T K wrote:
Hi Roger,
It has been a while, but I would like to pickup this thread. We
have a couple of pandaboards used as test setup. These have an
SDIO adapter hooked up to expansion connector A using MMC2. I have
attached the patch file (just ignore
On Tue, Oct 01, 2013 at 10:17:53AM -0500, Jon Loeliger wrote:
Hi Rob,
On 01/10/2013 15:17, Rob Herring wrote:
On 10/01/2013 03:06 AM, Benoit Cousson wrote:
+ more DT maintainers folks
Hi all,
I know this is mostly boring user space code, but I was expecting a
little bit
Hi Sebastian,
On 02.10.2013 10:29, Sebastian Andrzej Siewior wrote:
* Daniel Mack | 2013-10-01 15:31:11 [+0200]:
diff --git a/drivers/dma/cppi41.c b/drivers/dma/cppi41.c
index 7747bf7..6decf34 100644
--- a/drivers/dma/cppi41.c
+++ b/drivers/dma/cppi41.c
@@ -586,6 +586,9 @@ static int
Hi Jon,
On 01/10/2013 17:17, Jon Loeliger wrote:
Hi Rob,
On 01/10/2013 15:17, Rob Herring wrote:
On 10/01/2013 03:06 AM, Benoit Cousson wrote:
+ more DT maintainers folks
Hi all,
I know this is mostly boring user space code, but I was expecting a
little bit of comments about at least the
* Daniel Mack | 2013-10-01 15:31:08 [+0200]:
Patch #3, however, gives me headaches. I can't fully explain what's
going on, but I can tell for sure that if fixes a problem that I stared
on for many hours.
The problem is that on resume, the musb core will detect that some of
the suspended USB
Hi Tony,
On 09/24/2013 11:53 AM, Roger Quadros wrote:
The platform data bits can be inferred from the other members of
struct usbhs_phy_data. So get rid of the platform_data member.
Build the platform data for the PHY device in usbhs_init_phys() instead.
Signed-off-by: Roger Quadros
Hi Tony,
On 09/24/2013 11:53 AM, Roger Quadros wrote:
The USB phy-nop nop driver expects the RESET line information
to be sent as a GPIO number via platform data. Adapt to that.
Signed-off-by: Roger Quadros rog...@ti.com
Need your Ack on this one as well.
cheers,
-roger
---
On 10/01/2013 01:29 PM, Luca Coelho wrote:
Hi,
On Tue, 2013-10-01 at 12:53 +0200, Arend van Spriel wrote:
On 10/01/2013 11:53 AM, Roger Quadros wrote:
On 10/01/2013 12:49 PM, Roger Quadros wrote:
Hi Arend,
On 10/01/2013 11:05 AM, Arend van Spriel wrote:
On 07/19/2013 12:57 PM, Arend van
On 10/02/2013 11:19 AM, Daniel Mack wrote:
Hi Sebastian,
Hi Daniel,
On 02.10.2013 10:29, Sebastian Andrzej Siewior wrote:
* Daniel Mack | 2013-10-01 15:31:11 [+0200]:
Thanks a lot for having a look! As I'm going to be off for a couple of
days now, and only be able to read my mails
On Tue, Oct 01, 2013 at 11:31:27PM +0200, Thierry Reding wrote:
On Tue, Oct 01, 2013 at 02:59:43PM -0600, Stephen Warren wrote:
OK, hopefully it (the regulator core) complains about the missing DT
property though; I assume you're using regulator_get() not
regulator_get_optional(), since
* Daniel Mack | 2013-10-01 15:39:53 [+0200]:
musb_port_reset() sleeps, so we can't call it from atomic context. It
is, however, called from places inside musb_hub_control() while
musb-lock is held, which leads to a scheduling while atomic warning.
I guess you are hit by the msleep(1) here.
Fix
On Wed, 2013-10-02 at 12:20 +0200, Arend van Spriel wrote:
On 10/01/2013 01:29 PM, Luca Coelho wrote:
Hi,
On Tue, 2013-10-01 at 12:53 +0200, Arend van Spriel wrote:
On 10/01/2013 11:53 AM, Roger Quadros wrote:
On 10/01/2013 12:49 PM, Roger Quadros wrote:
Hi Arend,
On 10/01/2013
* Daniel Mack | 2013-10-01 15:39:54 [+0200]:
Make musb_port_suspend() externally available, and call it when to host
goes into suspend. This allows the core to go into suspend while a
device is connected.
Sorry for asking this stupid question but what is usally happening when
the host goes to
Initially commit cb527ede1bf6ff2008a025606f25344b8ed7b4ac
i2c-omap: Double clear of ARDY status in IRQ handler
added a workaround for undocumented errata ProDB0017052.
But then commit 1d7afc95946487945cc7f5019b41255b72224b70
i2c: omap: ack IRQ in parts refactored code and missed
one of ARDY
On 02.10.2013 12:20, Sebastian Andrzej Siewior wrote:
* Daniel Mack | 2013-10-01 15:31:08 [+0200]:
Patch #3, however, gives me headaches. I can't fully explain what's
going on, but I can tell for sure that if fixes a problem that I stared
on for many hours.
The problem is that on resume,
From: Danke Xie d@sta.samsung.com
The current TWL 6030 IRQ handler assumes little endianness.
This change makes it endian-neutral.
Signed-off-by: Danke Xie d@sta.samsung.com
Signed-off-by: Taras Kondratiuk taras.kondrat...@linaro.org
---
drivers/mfd/twl6030-irq.c |6 --
1 file
On 02.10.2013 12:49, Sebastian Andrzej Siewior wrote:
* Daniel Mack | 2013-10-01 15:39:54 [+0200]:
Make musb_port_suspend() externally available, and call it when to host
goes into suspend. This allows the core to go into suspend while a
device is connected.
Sorry for asking this stupid
On 02.10.2013 12:46, Sebastian Andrzej Siewior wrote:
* Daniel Mack | 2013-10-01 15:39:53 [+0200]:
musb_port_reset() sleeps, so we can't call it from atomic context. It
is, however, called from places inside musb_hub_control() while
musb-lock is held, which leads to a scheduling while atomic
On 02.10.2013 12:25, Sebastian Andrzej Siewior wrote:
On 10/02/2013 11:19 AM, Daniel Mack wrote:
Once the system is booted up and the USB media is detected, I send the
system to sleep mode with cat mem /sys/power/state. After wakeup, I
access the media by mounting and unmounting it once, then
On 10/02/2013 01:14 PM, Daniel Mack wrote:
On 02.10.2013 12:49, Sebastian Andrzej Siewior wrote:
* Daniel Mack | 2013-10-01 15:39:54 [+0200]:
Make musb_port_suspend() externally available, and call it when to host
goes into suspend. This allows the core to go into suspend while a
device is
On 02.10.2013 14:01, Sebastian Andrzej Siewior wrote:
On 10/02/2013 01:14 PM, Daniel Mack wrote:
On 02.10.2013 12:49, Sebastian Andrzej Siewior wrote:
What happens if the device is unplugged while the host is suspended and
not there on resume?
That condition is detected and a full teardown
Recent commit 76bac1987ca181d54f8c9456d20997cb1020af2d
OMAP: UART: Fix the revision register read changed UART_MVR 16-bit
read to 32-bit.
But pre-OMAP4 devices have 16-bit MVR register.
Is it safe to assume that 32-bit access will return 0's in upper
16-bits in this case?
--
Regards,
Taras
On 10/02/2013 01:07 PM, Daniel Mack wrote:
No, as stated, the line numbers in the kernel message are somewhat off
due to added debugging code. What kicks in here is this one:
if (!c-td_desc_seen) {
desc_phys = cppi41_pop_desc(cdd, c-q_comp_num);
if
On Mon, Sep 23, 2013 at 04:11:30PM +0300, Roger Quadros wrote:
Hi Felipe,
On 09/18/2013 03:49 PM, Roger Quadros wrote:
usb_otg_ss_refclk960m is an optional functional clock to the
UBS_OTG_SS module. So manage it in the driver.
Just realized that usb_otg_ss_refclk960m is in fact
Benoit,
Sorry, I meant to ask earlier but forgot.
Shouldn't this development be based on the
upstream DTC repository and not the in-kernel
copy of the DTC?
Eventually, yes,
We should *start* there, though.
but here the main point is to discuss the schema that
will be used to
On 10/02/2013 04:11 PM, Felipe Balbi wrote:
On Mon, Sep 23, 2013 at 04:11:30PM +0300, Roger Quadros wrote:
Hi Felipe,
On 09/18/2013 03:49 PM, Roger Quadros wrote:
usb_otg_ss_refclk960m is an optional functional clock to the
UBS_OTG_SS module. So manage it in the driver.
Just realized that
On Tue, Oct 01, 2013 at 03:54:20PM -0500, Rob Herring wrote:
On Tue, Oct 1, 2013 at 10:06 AM, Benoit Cousson bcous...@baylibre.com wrote:
Hi Rob,
On 01/10/2013 15:17, Rob Herring wrote:
On 10/01/2013 03:06 AM, Benoit Cousson wrote:
+ more DT maintainers folks
Hi all,
I
On Tue, Oct 01, 2013 at 08:17:42AM -0500, Rob Herring wrote:
On 10/01/2013 03:06 AM, Benoit Cousson wrote:
+ more DT maintainers folks
Hi all,
I know this is mostly boring user space code, but I was expecting a
little bit of comments about at least the bindings syntax:-(
I'd
On Tue, Sep 24, 2013 at 06:52:07PM +0200, Benoit Cousson wrote:
From: Fabien Parent fpar...@baylibre.com
As noted elsewhere, please write patches against upstream dtc, not the
version in the kernel. git://git.kernel.org/pub/scm/utils/dtc/dtc.git
There are a few memory leaks in dtc which until
On Tue, Oct 01, 2013 at 04:22:24PM -0600, Stephen Warren wrote:
On 09/24/2013 10:52 AM, Benoit Cousson wrote:
Hi All,
Following the discussion that happened during LCE-2013 and the email
thread started by Tomasz few months ago [1], here is a first attempt
to introduce:
- a schema
On Wed, 2 Oct 2013, Daniel Mack wrote:
On 02.10.2013 14:01, Sebastian Andrzej Siewior wrote:
On 10/02/2013 01:14 PM, Daniel Mack wrote:
On 02.10.2013 12:49, Sebastian Andrzej Siewior wrote:
What happens if the device is unplugged while the host is suspended and
not there on resume?
On Wed, 2 Oct 2013 14:08:44 +0300
Taras Kondratiuk taras.kondrat...@linaro.org wrote:
From: Danke Xie d@sta.samsung.com
The current TWL 6030 IRQ handler assumes little endianness.
This change makes it endian-neutral.
Signed-off-by: Danke Xie d@sta.samsung.com
Signed-off-by:
On Tue, Oct 01, 2013 at 12:50:51PM -0600, Stephen Warren wrote:
On 09/23/2013 03:41 PM, Thierry Reding wrote:
[...]
+ if (gpio_is_valid(pb-enable_gpio)) {
+ if (pb-enable_gpio_flags PWM_BACKLIGHT_GPIO_ACTIVE_LOW)
+ gpio_set_value(pb-enable_gpio, 0);
+
Set the alias for ethernet0 and ethernet1 so that uBoot
can set the MAC address appropriately.
Currently uBoot cannot find the alias and there for does not set the
MAC address.
Signed-off-by: Dan Murphy dmur...@ti.com
---
arch/arm/boot/dts/am33xx.dtsi |2 ++
1 file changed, 2 insertions(+)
On Wed, Oct 02, 2013 at 11:54:50PM +1000, David Gibson wrote:
On Tue, Oct 01, 2013 at 03:54:20PM -0500, Rob Herring wrote:
I would expect the schema to replace
Documentation/devicetree/bindings/* over time. I think the thing that
needs to be worked out here is how to add free form
On 2 October 2013 19:43, Kim Phillips kim.phill...@linaro.org wrote:
On Wed, 2 Oct 2013 14:08:44 +0300
Taras Kondratiuk taras.kondrat...@linaro.org wrote:
From: Danke Xie d@sta.samsung.com
The current TWL 6030 IRQ handler assumes little endianness.
This change makes it endian-neutral.
On Wed, Oct 02, 2013 at 07:08:41PM +0100, Mark Brown wrote:
On Wed, Oct 02, 2013 at 11:54:50PM +1000, David Gibson wrote:
On Tue, Oct 01, 2013 at 03:54:20PM -0500, Rob Herring wrote:
I would expect the schema to replace
Documentation/devicetree/bindings/* over time. I think the thing
On Wednesday 02 October 2013 12:58 PM, Dan Murphy wrote:
Set the alias for ethernet0 and ethernet1 so that uBoot
can set the MAC address appropriately.
Currently uBoot cannot find the alias and there for does not set the
MAC address.
Signed-off-by: Dan Murphy dmur...@ti.com
Tested this
* Roger Quadros rog...@ti.com [131002 03:27]:
Hi Tony,
On 09/24/2013 11:53 AM, Roger Quadros wrote:
The platform data bits can be inferred from the other members of
struct usbhs_phy_data. So get rid of the platform_data member.
Build the platform data for the PHY device in
* Roger Quadros rog...@ti.com [131002 03:28]:
Hi Tony,
On 09/24/2013 11:53 AM, Roger Quadros wrote:
The USB phy-nop nop driver expects the RESET line information
to be sent as a GPIO number via platform data. Adapt to that.
Signed-off-by: Roger Quadros rog...@ti.com
Need your Ack
arm builds have been failing on and off with
arch/arm/mach-omap2/board-omap3evm.c:703: undefined reference to
`usb_nop_xceiv_register'
for several years. Current arm:allmodconfig build fails for this reason.
Problem is that board-omap3evm.c is always build into the kernel.
It calls
For few things we're still going to be needing platform
data for device tree based drivers. Let's set up auxdata
handling and do it in pdata-quirks.c so we have all the
legacy calls in one place.
Signed-off-by: Tony Lindgren t...@atomide.com
---
arch/arm/mach-omap2/board-generic.c |5 +
The wake-up interrupt bit is available on omap3/4/5 processors
unlike what we claim. Without fixing it we cannot use it on
omap3 and the system configured for wake-up events will just
hang on wake-up.
Cc: Grygorii Strashko grygorii.stras...@ti.com
Cc: Benoît Cousson bcous...@baylibre.com
Cc:
Hi all,
Here's a respin of the pinctrl-single related patches to
use chained irq as suggested by Linus W earlier. This series
does not try to provide any generic automated solution yet,
but works by doing a request_irq() for the wake-up pin, so
it should be flexible enough for both manual control
Hi Mark,
On Fri, Sep 27, 2013 at 05:06:38PM +0100, Kumar Gala wrote:
On Sep 17, 2013, at 2:30 PM, Suman Anna wrote:
HwSpinlock IP is present only on OMAP4 and other newer SoCs,
which are all device-tree boot only. This patch adds the
base support for parsing the DT nodes, and removes the
Hi Mark,
On 10/01/2013 03:36 AM, Mark Rutland wrote:
Hi Suman,
Apologies for replying to a subthread, due to an earlier mistake on my
part I don't have the original to hand.
No issues, thanks for your review.
On Fri, Sep 27, 2013 at 05:04:22PM +0100, Kumar Gala wrote:
On Sep 17, 2013,
* Tony Lindgren t...@atomide.com [131002 22:25]:
Hi all,
Here's a respin of the pinctrl-single related patches to
use chained irq as suggested by Linus W earlier. This series
does not try to provide any generic automated solution yet,
but works by doing a request_irq() for the wake-up pin,
Let's replace is_pinconf with flags and add struct pcs_soc_data
so we can support SoC specific features like pin wake-up events.
Done in collaboration with Roger Quadros rog...@ti.com.
Cc: Peter Ujfalusi peter.ujfal...@ti.com
Cc: Grygorii Strashko grygorii.stras...@ti.com
Cc: Prakash
Hi all,
Here's a respin of the
The wake-up interrupt bit is available on omap3/4/5 processors
unlike what we claim. Without fixing it we cannot use it on
omap3 and the system configured for wake-up events will just
hang on wake-up.
Cc: Grygorii Strashko grygorii.stras...@ti.com
Cc: Benoît Cousson bcous...@baylibre.com
Cc:
For few things we're still going to be needing platform
data for device tree based drivers. Let's set up auxdata
handling and do it in pdata-quirks.c so we have all the
legacy calls in one place.
Signed-off-by: Tony Lindgren t...@atomide.com
---
arch/arm/mach-omap2/board-generic.c |5 +
The pin control registers can have interrupts for example
for device wake-up. These interrupts can be treated as a
chained interrupt controller as suggested earlier by
Linus Walleij linus.wall...@linaro.org.
This patch adds support for interrupts in a way that
should be pretty generic, and works
Now pinctrl-single-omap can handle the wake-up events for us now
as long as the events are configured in the .dts files.
Done in collaboration with Roger Quadros rog...@ti.com.
Cc: Peter Ujfalusi peter.ujfal...@ti.com
Cc: Grygorii Strashko grygorii.stras...@ti.com
Cc: Prakash Manjunathappa
For omaps, we still have dependencies to the legacy code
for handling the PRM (Power Reset Management) interrupts,
and also for reconfiguring the io wake-up chain after
changes.
Let's pass the PRM interrupt and the rearm functions via
auxdata. Then when at some point we have a proper PRM
driver,
57 matches
Mail list logo
