Message ID | 1350404650.1242.20.camel@hornet |
---|---|
State | New |
Headers | show |
On Tue, Oct 16, 2012 at 05:24:10PM +0100, Pawel Moll wrote: > Hi Arnd, Olof, > > Would you be so kind to pull the updates for Versatile Express? It's the > same stuff as discussed last month (I've just added a couple of fixes > for bugs reported by Tixy) plus the hwmon driver (the maintainer acked > it and chosen to get it merged through the arm-soc tree). Hi Pawel, The branch contents looks good, but I wonder if you would mind splitting it up across the same kind of topics that we tend to organize arm-soc in? Suggestions below. > Pawel Moll (8): > hwmon: Versatile Express hwmon driver > mfd: Versatile Express config infrastructure > mfd: Versatile Express system registers driver > ARM: vexpress: Reset driver The above would go in a drivers branch > ARM: vexpress: Add config bus components and clocks to DTs > ARM: vexpress: Start using new Versatile Express infrastructure > ARM: vexpress: Remove motherboard dependencies in the DTS files > ARM: vexpress: Make the debug UART detection more specific I would suggest taking the two DT patches (#1 and #3 in the list) and splitting them off in a dt branch, and the rest in a boards or soc branch. So, total of three branches: drivers dt soc If these are the only vexpress patches you expect at all for 3.8, we can just take it as it is in one more broad branch. But since we're early in the cycle I'm guessing you might end up having more patches coming in, so starting to sort them in topics is a good idea. Thanks! -Olof
On Tue, 2012-10-16 at 19:34 +0100, Olof Johansson wrote: > Hi Pawel, > The branch contents looks good, but I wonder if you would mind splitting it up > across the same kind of topics that we tend to organize arm-soc in? Sure, no problem. > Suggestions below. I can make those two a "drivers" branch indeed: > > Pawel Moll (8): > > hwmon: Versatile Express hwmon driver > > ARM: vexpress: Reset driver I can also split this one into "fixes": > ARM: vexpress: Make the debug UART detection more specific But this series constitutes one logical change and must be applied in order (the sysreg driver is an essential part), so I'll keep them together on a "soc" branch: > > mfd: Versatile Express config infrastructure > > mfd: Versatile Express system registers driver > > ARM: vexpress: Add config bus components and clocks to DTs > > ARM: vexpress: Start using new Versatile Express infrastructure > > ARM: vexpress: Remove motherboard dependencies in the DTS files If this makes sense, I'll send three pull requests soon. Cheers! Paweł