Re: [PATCHv2 7/8] ir-rx51: Convert latency constraints to PM QoS API

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Timo,

On Fri, Aug 24, 2012 at 10:39 PM, Tony Lindgren <tony@xxxxxxxxxxx> wrote:
> * Timo Kokkonen <timo.t.kokkonen@xxxxxx> [120824 08:11]:
>> Convert the driver from the obsolete omap_pm_set_max_mpu_wakeup_lat
>> API to the new PM QoS API. This allows the callback to be removed from
>> the platform data structure.
>>
>> The latency requirements are also adjusted to prevent the MPU from
>> going into sleep mode. This is needed as the GP timers have no means
>> to wake up the MPU once it has gone into sleep. The "side effect" is
>> that from now on the driver actually works even if there is no
>> background load keeping the MPU awake.
>>
>> Signed-off-by: Timo Kokkonen <timo.t.kokkonen@xxxxxx>
>
> This should get acked by Kevin ideally. Other than that:
>
> Acked-by: Tony Lindgren <tony@xxxxxxxxxxx>

...
@@ -268,10 +270,14 @@ static ssize_t lirc_rx51_write(struct file
*file, const char *buf,
                lirc_rx51->wbuf[count] = -1; /* Insert termination mark */

        /*
-        * Adjust latency requirements so the device doesn't go in too
-        * deep sleep states
+        * If the MPU is going into too deep sleep state while we are
+        * transmitting the IR code, timers will not be able to wake
+        * up the MPU. Thus, we need to set a strict enough latency
+        * requirement in order to ensure the interrupts come though
+        * properly.
         */
-       lirc_rx51->pdata->set_max_mpu_wakeup_lat(lirc_rx51->dev, 50);
+       pm_qos_add_request(&lirc_rx51->pm_qos_request,
+                       PM_QOS_CPU_DMA_LATENCY, 10);
Minor remark: it would be nice to have more detail on where the
latency number 10 comes from. Is it fixed, is it linked to the baud
rate etc?

Here is my ack for the PM QoS API part:
Acked-by: Jean Pihet <j-pihet@xxxxxx>

Regards,
Jean
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux