Re: [PATCH] PCI: designware: bail out if host_init failed

2015-12-07 Thread Jisheng Zhang
Dear Bjorn, On Wed, 25 Nov 2015 14:01:03 -0600 Bjorn Helgaas wrote: > Hi Jisheng, > > On Thu, Nov 12, 2015 at 09:48:45PM +0800, Jisheng Zhang wrote: > > There's no reason to continue the initialization such as configure > > register, scan root bus etc. if customized host_init() failed. This > >

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

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

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

2015-12-07 Thread Peter Ujfalusi
On 12/08/2015 02:19 AM, Tony Lindgren wrote: > * Peter Ujfalusi [151204 02:46]: >> + >> +ti,tptcs = <_tptc0 7>, <_tptc1 5>, >> + <_tptc2 0>; >> + >> +ti,edma-memcpy-channels = /bits/ 16 <20 21>; > > Is

Re: [PATCH 04/10] omap_hwspinlock: Replace "hweight_long(i & 0xf) != 1" with "!is_power_of_2(i & 0xf)"

2015-12-07 Thread zhaoxiu.zeng
在 2015/12/7 19:08, Ohad Ben-Cohen 写道: > Hi, > > On Sun, Dec 6, 2015 at 12:33 PM, Zhaoxiu Zeng wrote: >> >> From: Zeng Zhaoxiu >> >> Signed-off-by: Zeng Zhaoxiu > > Please explain why do you think we should make this

Re: [patch] OMAPDSS: DSS: fix a warning message

2015-12-07 Thread Tomi Valkeinen
On 04/12/15 15:14, Dan Carpenter wrote: > 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

Re: [PATCH 1/2] usb: musb: core: Fix handling of the phy notifications

2015-12-07 Thread Tony Lindgren
* Kishon Vijay Abraham I [151206 23:03]: > On Tuesday 01 December 2015 11:07 AM, Tony Lindgren wrote: > > We currently can't unload omap2430 MUSB platform glue driver module and > > this cause issues for fixing the MUSB code further. The reason we can't > > remove omap2430 is

Re: BUG: TI CPSW driver hanging up when setting mac-address to early

2015-12-07 Thread Tony Lindgren
* Pascal Speck (Iktek) [151207 01:43]: > > [ 11.893212] Unhandled fault: external abort on non-linefetch (0x1008) This usually means that some clock is not requested and enabled properly, or something is not powered. How many Ethernet interfaces do you have? Regards, Tony

Re: [PATCH 04/10] omap_hwspinlock: Replace "hweight_long(i & 0xf) != 1" with "!is_power_of_2(i & 0xf)"

2015-12-07 Thread Ohad Ben-Cohen
On Mon, Dec 7, 2015 at 5:03 PM, zhaoxiu.zeng wrote: > is_power_of_2 is simple, and faster than "hweightN(x) == 1" on most > architectures. Thanks. I'm not sure that speed is a major concern here, since this code executes only once during the lifetime of the driver.

Re: Re: BUG: TI CPSW driver hanging up when setting mac-address to early

2015-12-07 Thread kernel
Hi Tony, there are two ethernet interfaces ( dual-emac-configuration ) used. One is connected to another 100mbit switch-ic ( refclk should come from switch ic ) via rmii, the other one is connected to a 1gbit fpga rgmii interface ( where the clock is served from the fpga ). On both interfaces it

BUG: TI CPSW driver hanging up when setting mac-address to early

2015-12-07 Thread Pascal Speck (Iktek)
( resent with plain text only ) Dear Mr. Mugunthan, Dear Linux-Omap List Sorry in advance if I not do everything right in BUG-reporting, this is the first time I try to do this. I just posted this Bug on the netdev-mailinglist also, but with no results until now. (Don't know if this was the

Re: [PATCH 04/10] omap_hwspinlock: Replace "hweight_long(i & 0xf) != 1" with "!is_power_of_2(i & 0xf)"

2015-12-07 Thread Ohad Ben-Cohen
Hi, On Sun, Dec 6, 2015 at 12:33 PM, Zhaoxiu Zeng wrote: > > From: Zeng Zhaoxiu > > Signed-off-by: Zeng Zhaoxiu Please explain why do you think we should make this change. Btw, the original code used is_power_of_2, but

[PATCH] mmc: pwrseq_simple: Fix regression with optional GPIOs

2015-12-07 Thread Tony Lindgren
Commit ce037275861e ("mmc: pwrseq_simple: use GPIO descriptors array API") changed the handling MMC power sequence so GPIOs no longer are optional. This broke SDIO WLAN at least for omap5 that can't yet use the reset GPIOs with pwrseq_simple as a wait is needed after enabling the SDIO device.

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

2015-12-07 Thread Tony Lindgren
* Enric Balletbo Serra [151204 06:40]: > 2015-12-03 20:27 GMT+01:00 Tony Lindgren : > > * Javier Martinez Canillas [151203 10:29]: > > > >> Since is working for the boards with the libertas chip, I preferred > >> to remove the DTS

Re: [PATCH] mmc: pwrseq_simple: Fix regression with optional GPIOs

2015-12-07 Thread Ulf Hansson
+Linus On 7 December 2015 at 23:54, Tony Lindgren wrote: > Commit ce037275861e ("mmc: pwrseq_simple: use GPIO descriptors array API") > changed the handling MMC power sequence so GPIOs no longer are optional. > > This broke SDIO WLAN at least for omap5 that can't yet use the

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

2015-12-07 Thread Tony Lindgren
* Peter Ujfalusi [151204 02:46]: > + > + ti,tptcs = <_tptc0 7>, <_tptc1 5>, > +<_tptc2 0>; > + > + ti,edma-memcpy-channels = /bits/ 16 <20 21>; Is this safe to change to just ti,edma-memcpy-channels =

Re: [PATCH] mmc: pwrseq_simple: Fix regression with optional GPIOs

2015-12-07 Thread Tony Lindgren
* Ulf Hansson [151207 16:20]: > +Linus > > On 7 December 2015 at 23:54, Tony Lindgren wrote: > > Commit ce037275861e ("mmc: pwrseq_simple: use GPIO descriptors array API") > > changed the handling MMC power sequence so GPIOs no longer are optional. > >

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

2015-12-07 Thread Bjorn Helgaas
[+cc Jingoo (exynos), Richard, Lucas (imx6), Pratyush (spear13xx)] On Fri, Dec 04, 2015 at 11:22:50PM +0200, Grygorii Strashko wrote: > 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

Re: [PATCH] mmc: pwrseq_simple: Fix regression with optional GPIOs

2015-12-07 Thread Javier Martinez Canillas
Hello Tony, On 12/07/2015 09:32 PM, Tony Lindgren wrote: > * Ulf Hansson [151207 16:20]: >> +Linus >> >> On 7 December 2015 at 23:54, Tony Lindgren wrote: >>> Commit ce037275861e ("mmc: pwrseq_simple: use GPIO descriptors array API") >>> changed the

[PATCH] usb: musb: core: Fix pm runtime for deferred probe

2015-12-07 Thread Tony Lindgren
If musb_init_controller fails at musb_platform_init, we have already called pm_runtime_irq_safe for musb and that causes the pm runtime count to be enabled for parent before the parent has completed initialization. This causes pm to stop working as on unload nothing gets idled. This issue can be