On 02/09/2012 01:19 PM, Rickard Andersson wrote:
On 02/09/2012 12:25 PM, Daniel Lezcano wrote:
On 02/09/2012 11:52 AM, Rickard Andersson wrote:
Hi Daniel!
Please see my embedded comments.
Also should we not keep the functions in pm.c file? I think Linus W advised us not to move stuff around at this stage.
The pm.c file does not exists for mach-ux500. May be I misunderstood Linus but I thought he was saying to that we should "not move stuff around" by letting this code in the db8500-prcmu.c file for the moment and let Mattias do its cleanups.
On our internal tree we have mach-ux500/pm/pm.c and this file contains the decouple-gic functions as well as other power management related help functions. By moving the code to drivers/mfd/db8500-prcmu.c I think we are moving code around. Especially since it not clear how db8500-prcmu.c will be handled in the future, db8500-prcmu.c should most likely be split into several parts where one part only handles the communication with the PRCMU firmware.
Rickard,
the reference is the mainstream code. Today there is no pm.c file and everything is located in db8500-prcmu.c. Except if I misunderstood, if Linus, who is the maintainer of the ux500 SoC, prefers to have this portion of code in this file until Mattias does its cleanups, that does not really matter if it is in pm.c or db8500-prcmu.c for the moment.
That does not mean this code won't be moved to pm.c in the future.
Thanks -- Daniel