On Wednesday 18 April 2012 07:30 PM, Kevin Hilman wrote:
>>>> >>> if (ret < 0) {
>>>> >>> serial_omap_enable_ier_thri(up);
>>>> >>> + pm_runtime_mark_last_busy(&up->pdev->dev);
>>>> >>> + pm_runtime_put_autosuspend(&up->pdev->dev);
>>> >> Why the autosuspend version here?
>>> >>
>>> >> Kevin
>>> >>
>>> >>
>> > In case the request_dma fails we enable the thri( effectively like intr
>> > mode) so
>> > I thought of using the autosuspend version here .
>> >
>> > Do you prefer put version instead ?
> Not necessarily, it should just be well described in the changelog.
>
> Kevin
Yes agree completely. Will describe that in the changelog.
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
- [PATCH 0/7] UART: OMAP: Updates Shubhrajyoti D
- [PATCH 5/7] UART: OMAP: Prevent cutting of clocks ... Shubhrajyoti D
- [PATCH 6/7] UART: OMAP: Remove the hardcode serial... Shubhrajyoti D
- [PATCH 1/7] ARM: OMAP: UART: Make context_loss_cnt... Shubhrajyoti D
- [PATCH 7/7] UART: OMAP: Trivial optimisation of th... Shubhrajyoti D
- [PATCH 2/7] UART: OMAP: Cut the clock in the error... Shubhrajyoti D
- Re: [PATCH 2/7] UART: OMAP: Cut the clock in t... Kevin Hilman
- Re: [PATCH 2/7] UART: OMAP: Cut the clock ... Shubhrajyoti
- Re: [PATCH 2/7] UART: OMAP: Cut the cl... Kevin Hilman
- Re: [PATCH 2/7] UART: OMAP: Cut t... Shubhrajyoti
- Re: [PATCH 2/7] UART: OMAP: C... Shubhrajyoti Datta
- Re: [PATCH 2/7] UART: OMA... Kevin Hilman
- [PATCH 3/7] UART: OMAP: fix software flow control Shubhrajyoti D
- [PATCH 4/7] UART: OMAP: Remove the default setting... Shubhrajyoti D
