On Wed, Aug 21, 2013 at 10:03:15PM -0700, Olof Johansson wrote: > On Sun, Aug 18, 2013 at 01:55:57PM +1000, Simon Horman wrote: > > On Wed, Aug 14, 2013 at 12:23:28AM -0700, Olof Johansson wrote: > > > On Wed, Jul 24, 2013 at 09:32:14AM +0900, Simon Horman wrote: > > > > Hi Olof, Hi Arnd, > > > > > > > > please consider the following pinmux updates for v3.12. > > > > > > > > Difference with v1 of this pull-request: > > > > * Remove bogus i2c portions of the first patch of the series. > > > > * Correct addressing mode used in dts of the first patch of the series. > > > > * Removal of egg from face: wip > > > > > > > > The following changes since commit ad81f0545ef01ea651886dddac4bef6cec930092: > > > > > > > > Linux 3.11-rc1 (2013-07-14 15:18:27 -0700) > > > > > > > > are available in the git repository at: > > > > > > > > git://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas.git tags/renesas-pinmux-for-v3.12 > > > > > > Pulled, but I noticed this. Can you please follow up with bindings revisit in > > > time for 3.12 final (i.e. if we need to take it after merge window that's > > > probably OK): > > > > > > pfc: pfc@e6060000 { > > > compatible = "renesas,pfc-r8a7790"; > > > reg = <0 0xe6060000 0 0x250>; > > > #gpio-range-cells = <3>; > > > }; > > > > > > sdhi0: sdhi@ee100000 { > > > compatible = "renesas,r8a7790-sdhi"; > > > > > > > > > I.e. different ordering on the compatible values looks a bit uncoordinated. > > > > Can I confirm that you are referring to the fact that > > assume you mean that "-sdhi" follows "r8a7790" whereas "pfc-" precedes > > "r8a7790"? > > > > If so, I'll work with Laurent and Guennadi (both CCed, both on vacation), > > and Magnus to resolve this inconsistency. > > Yes, exactly. Some platforms use <vendor>,<soc>-<ip>, others > <vendor>,<ip>-<soc>. I don't care which way is used, but it would be good if > everybody used the same convention to make it easier for developers. > > Preference for direction is something I leave up to the device tree maintainers > to declare. ;-) Thanks, got it. I'll get this sorted out. -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html