On Mon, Nov 29, 2010 at 7:31 PM, Sundar <span dir="ltr"><<a href="mailto:sunder.svit@gmail.com">sunder.svit@gmail.com</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
Hi Vincent,<br>
<div><div></div><div class="h5"><br>
> On Mon, Nov 29, 2010 at 3:19 PM, Vincent Guittot <<a href="mailto:vincent.guittot@linaro.org">vincent.guittot@linaro.org</a>> wrote:<br>
>><br>
>> This patch extracts the common code of the cpu hotplug feature across<br>
>> arm platforms. The goal is to only keep the specific stuff of the<br>
>> platform in the sub-architecture. I have created a hotplug.c file in<br>
>> the arm/common directory after studying the cpu hotplug code of<br>
>> omap2, realview, s5pv310, ux500 and tegra. I have extracted 3 main<br>
>> platform dependent functions:<br>
>> -platform_enter_lowpower which prepares the platform for low power.<br>
>> -platform_do_lowpower on which the cpu will loop until it becomes<br>
>> really plugged (spurious wake up). This function must returned the cpu<br>
>> Id in order to leave the unplug state.<br>
>> -platform_leave_lowpower which restore the platform context.<br>
>> An ux500 patch is available which uses the common/hotplug.c code.<br>
>> This patch is quite short because the idle / power down functions are<br>
>> not yet upstreamed<br>
>><br>
><br>
<br>
I had posted a patch which does exactly the same thing sometime ago,<br>
but it got dropped<br>
off from the radar. The patch set can be referenced<br>
@ <a href="http://www.spinics.net/lists/arm-kernel/msg97600.html" target="_blank">http://www.spinics.net/lists/arm-kernel/msg97600.html</a><br>
I remember getting it Acked by most of the platform maintainers, but<br>
Russell had some<br>
reservations on that.<br></div></div></blockquote><div><br></div><div>It is unfortunate that we didn't catch that work when we started on this. Let us continue discussion on the other sub-thread of the current posting.</div>
<div><br></div><div>/Amit </div></div>