Re: [PATCHv6 1/2] drivers: spi: Add qspi flash controller

[Sourav Poddar <sourav.poddar-l0cyMroinI0@public.gmane.org>]

On Monday 29 July 2013 04:39 PM, Felipe Balbi wrote:
> Hi,
>
> On Mon, Jul 29, 2013 at 04:34:41PM +0530, Sourav Poddar wrote:
>>>>>> +	irq = platform_get_irq(pdev, 0);
>>>>>> +	if (irq<    0) {
>>>>>> +		dev_err(&pdev->dev, "no irq resource?\n");
>>>>>> +		return irq;
>>>>>> +	}
>>>>>> +
>>>>>> +	spin_lock_init(&qspi->lock);
>>>>>> +
>>>>>> +	qspi->base = devm_ioremap_resource(&pdev->dev, r);
>>>>>> +	if (IS_ERR(qspi->base)) {
>>>>>> +		ret = PTR_ERR(qspi->base);
>>>>>> +		goto free_master;
>>>>>> +	}
>>>>>> +
>>>>>> +	ret = devm_request_threaded_irq(&pdev->dev, irq, ti_qspi_isr,
>>>>>> +			ti_qspi_threaded_isr, IRQF_NO_SUSPEND | IRQF_ONESHOT,
>>>>> why do you need IRQF_NO_SUSPEND ?
>>>>>
>>>> I should get away with this.
>>> why ? Do you need or do you *not* need it ? And in either case, why ?
>>>
>> I was thinking, this will keep the irqs up even when we are
>> hitting suspend, we will not be prepared to handle it. ?
> won't be prepared in what way ?
>
Our driver will be down, so the irq might go un-serviced.


------------------------------------------------------------------------------
See everything from the browser to the database with AppDynamics
Get end-to-end visibility with application monitoring from AppDynamics
Isolate bottlenecks and diagnose root cause in seconds.
Start your free trial of AppDynamics Pro today!
http://pubads.g.doubleclick.net/gampad/clk?id=48808831&iu=/4140/ostg.clktrk