Messages in this thread | | | Date | Tue, 28 Apr 2009 09:13:41 +0800 | Subject | Re: Patch to add mioa701 glue for voltage regulation | From | Eric Miao <> |
| |
On Tue, Apr 28, 2009 at 5:11 AM, Robert Jarzmik <robert.jarzmik@free.fr> wrote: > Liam Girdwood <lrg@slimlogic.co.uk> writes: > >> On Thu, 2009-04-23 at 20:30 +0200, Robert Jarzmik wrote: >>> Hi Mark and Eric, >>> >>> I have that patch which adds voltage regulation definitions to mioa701 >>> board. The trick is, this patch depends on two others : >>> >>> - one which will be merged through Mark's regulator tree. >>> This one is mandatory as a compiling dependency exists through include files. >>> >>> - one which will be merge through Eric pxa tree. >>> This is the cpufreq one, and has a "very weak" dependency, as only the >>> "vcc_core" name _is_ the dependency. >>> >>> I think the easiest way to solve the compiling dependency >>> (include/linux/regulator.max1586.h) is to make that patch go through regulator >>> tree as well for linux-next, even if it's arm machine specific, don't you ? >>> >> >> If fine with this going through regulator providing I get an ACK from >> Eric. > > Eric, > > Since Liam took the max1586 regulator through his tree, would you state if you > ack that please ? >
I'd really be happy to. However, my concern is that this patch modifies arch/arm/mach-pxa/mioa701.c _only_, so it would be better to go through my tree so that I can manage all the potential merge conflicts. There are several other changes to mioa701.c and I expect the content in arch/arm/mach-pxa/* to be heavily changed due to recent addition of pxa168/pxa910 support.
The issue of dependency, as was always before, can be resolved by putting this into 'pending' and monitoring the status of merge window before sending the PULL request.
Liam,
Sounds OK to you? -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |