Re: [RFC PATCH] PCI: rockchip: fix system hang up if activate CONFIG_DEBUG_SHIRQ

2017-08-10 Thread Shawn Lin
Hi Heiko On 2017/8/10 17:27, Heiko Stuebner wrote: Hi Shawn, Am Donnerstag, 10. August 2017, 16:21:13 CEST schrieb Shawn Lin: With CONFIG_DEBUG_SHIRQ enabled, the irq tear down routine would still access the irq handler registed as a shard irq. Per the comment within the function of

Re: [RFC PATCH] PCI: rockchip: fix system hang up if activate CONFIG_DEBUG_SHIRQ

2017-08-10 Thread Shawn Lin
Hi Heiko On 2017/8/10 17:27, Heiko Stuebner wrote: Hi Shawn, Am Donnerstag, 10. August 2017, 16:21:13 CEST schrieb Shawn Lin: With CONFIG_DEBUG_SHIRQ enabled, the irq tear down routine would still access the irq handler registed as a shard irq. Per the comment within the function of

Re: [RFC PATCH] PCI: rockchip: fix system hang up if activate CONFIG_DEBUG_SHIRQ

2017-08-10 Thread jeffy
Hi Heiko, On 08/10/2017 05:27 PM, Heiko Stuebner wrote: Hi Shawn, Am Donnerstag, 10. August 2017, 16:21:13 CEST schrieb Shawn Lin: >With CONFIG_DEBUG_SHIRQ enabled, the irq tear down routine >would still access the irq handler registed as a shard irq. >Per the comment within the function of

Re: [RFC PATCH] PCI: rockchip: fix system hang up if activate CONFIG_DEBUG_SHIRQ

2017-08-10 Thread jeffy
Hi Heiko, On 08/10/2017 05:27 PM, Heiko Stuebner wrote: Hi Shawn, Am Donnerstag, 10. August 2017, 16:21:13 CEST schrieb Shawn Lin: >With CONFIG_DEBUG_SHIRQ enabled, the irq tear down routine >would still access the irq handler registed as a shard irq. >Per the comment within the function of

Re: [RFC PATCH] PCI: rockchip: fix system hang up if activate CONFIG_DEBUG_SHIRQ

2017-08-10 Thread Heiko Stuebner
Hi Shawn, Am Donnerstag, 10. August 2017, 16:21:13 CEST schrieb Shawn Lin: > With CONFIG_DEBUG_SHIRQ enabled, the irq tear down routine > would still access the irq handler registed as a shard irq. > Per the comment within the function of __free_irq, it says > "It's a shared IRQ -- the driver

Re: [RFC PATCH] PCI: rockchip: fix system hang up if activate CONFIG_DEBUG_SHIRQ

2017-08-10 Thread Heiko Stuebner
Hi Shawn, Am Donnerstag, 10. August 2017, 16:21:13 CEST schrieb Shawn Lin: > With CONFIG_DEBUG_SHIRQ enabled, the irq tear down routine > would still access the irq handler registed as a shard irq. > Per the comment within the function of __free_irq, it says > "It's a shared IRQ -- the driver

Re: [RFC PATCH] PCI: rockchip: fix system hang up if activate CONFIG_DEBUG_SHIRQ

2017-08-10 Thread jeffy
Hi shawn, On 08/10/2017 05:14 PM, Shawn Lin wrote: Hi Jeffy On 2017/8/10 16:39, jeffy wrote: Hi shawn, On 08/10/2017 04:21 PM, Shawn Lin wrote: With CONFIG_DEBUG_SHIRQ enabled, the irq tear down routine would still access the irq handler registed as a shard irq. Per the comment within the

Re: [RFC PATCH] PCI: rockchip: fix system hang up if activate CONFIG_DEBUG_SHIRQ

2017-08-10 Thread jeffy
Hi shawn, On 08/10/2017 05:14 PM, Shawn Lin wrote: Hi Jeffy On 2017/8/10 16:39, jeffy wrote: Hi shawn, On 08/10/2017 04:21 PM, Shawn Lin wrote: With CONFIG_DEBUG_SHIRQ enabled, the irq tear down routine would still access the irq handler registed as a shard irq. Per the comment within the

Re: [RFC PATCH] PCI: rockchip: fix system hang up if activate CONFIG_DEBUG_SHIRQ

2017-08-10 Thread Shawn Lin
Hi Jeffy On 2017/8/10 16:39, jeffy wrote: Hi shawn, On 08/10/2017 04:21 PM, Shawn Lin wrote: With CONFIG_DEBUG_SHIRQ enabled, the irq tear down routine would still access the irq handler registed as a shard irq. Per the comment within the function of __free_irq, it says "It's a shared IRQ --

Re: [RFC PATCH] PCI: rockchip: fix system hang up if activate CONFIG_DEBUG_SHIRQ

2017-08-10 Thread Shawn Lin
Hi Jeffy On 2017/8/10 16:39, jeffy wrote: Hi shawn, On 08/10/2017 04:21 PM, Shawn Lin wrote: With CONFIG_DEBUG_SHIRQ enabled, the irq tear down routine would still access the irq handler registed as a shard irq. Per the comment within the function of __free_irq, it says "It's a shared IRQ --

Re: [RFC PATCH] PCI: rockchip: fix system hang up if activate CONFIG_DEBUG_SHIRQ

2017-08-10 Thread jeffy
Hi shawn, On 08/10/2017 04:21 PM, Shawn Lin wrote: With CONFIG_DEBUG_SHIRQ enabled, the irq tear down routine would still access the irq handler registed as a shard irq. Per the comment within the function of __free_irq, it says "It's a shared IRQ -- the driver ought to be prepared for an IRQ

Re: [RFC PATCH] PCI: rockchip: fix system hang up if activate CONFIG_DEBUG_SHIRQ

2017-08-10 Thread jeffy
Hi shawn, On 08/10/2017 04:21 PM, Shawn Lin wrote: With CONFIG_DEBUG_SHIRQ enabled, the irq tear down routine would still access the irq handler registed as a shard irq. Per the comment within the function of __free_irq, it says "It's a shared IRQ -- the driver ought to be prepared for an IRQ

[RFC PATCH] PCI: rockchip: fix system hang up if activate CONFIG_DEBUG_SHIRQ

2017-08-10 Thread Shawn Lin
With CONFIG_DEBUG_SHIRQ enabled, the irq tear down routine would still access the irq handler registed as a shard irq. Per the comment within the function of __free_irq, it says "It's a shared IRQ -- the driver ought to be prepared for an IRQ event to happen even now it's being freed". However

[RFC PATCH] PCI: rockchip: fix system hang up if activate CONFIG_DEBUG_SHIRQ

2017-08-10 Thread Shawn Lin
With CONFIG_DEBUG_SHIRQ enabled, the irq tear down routine would still access the irq handler registed as a shard irq. Per the comment within the function of __free_irq, it says "It's a shared IRQ -- the driver ought to be prepared for an IRQ event to happen even now it's being freed". However