On Sun, Mar 9, 2014 at 1:01 PM, Santosh Shilimkar <santosh.shilimkar@xxxxxx> wrote: > On Sunday 09 March 2014 01:39 PM, Olof Johansson wrote: >> On Tue, Feb 25, 2014 at 04:50:32PM -0500, Santosh Shilimkar wrote: >>> From: Murali Karicheri <m-karicheri2@xxxxxx> >>> >>> This patch add compatibility strings for k2hk, k2l and k2e EVMs >>> >>> Cc: Olof Johansson <olof@xxxxxxxxx> >>> Cc: Arnd Bergmann <arnd@xxxxxxxx> >>> Cc: Rob Herring <robh+dt@xxxxxxxxxx> >>> Cc: Kumar Gala <galak@xxxxxxxxxxxxxx> >>> Cc: Mark Rutland <mark.rutland@xxxxxxx> >>> Signed-off-by: Murali Karicheri <m-karicheri2@xxxxxx> >>> Signed-off-by: Santosh Shilimkar <santosh.shilimkar@xxxxxx> >>> --- >>> .../devicetree/bindings/arm/keystone/keystone.txt | 10 ++++++++++ >>> arch/arm/mach-keystone/keystone.c | 3 +++ >>> 2 files changed, 13 insertions(+) >>> >>> diff --git a/Documentation/devicetree/bindings/arm/keystone/keystone.txt b/Documentation/devicetree/bindings/arm/keystone/keystone.txt >>> index 63c0e6a..ad16e7a 100644 >>> --- a/Documentation/devicetree/bindings/arm/keystone/keystone.txt >>> +++ b/Documentation/devicetree/bindings/arm/keystone/keystone.txt >>> @@ -8,3 +8,13 @@ Required properties: >>> - compatible: All TI specific devices present in Keystone SOC should be in >>> the form "ti,keystone-*". Generic devices like gic, arch_timers, ns16550 >>> type UART should use the specified compatible for those devices. >>> + >>> +Boards: >>> +- Keystone 2 Hawking/Kepler EVM >>> + compatible = "ti,k2hk-evm" >>> + >>> +- Keystone 2 Lamarr EVM >>> + compatible = "ti,k2l-evm" >> >> Your previous email said Lammar. One, or maybe both, are wrong, but which one? >> > My email had a typo then. The one here is right. > >>> + >>> +- Keystone 2 Edison EVM >>> + compatible = "ti,k2e-evm" >>> diff --git a/arch/arm/mach-keystone/keystone.c b/arch/arm/mach-keystone/keystone.c >>> index 6e6bb7d..aa0d212 100644 >>> --- a/arch/arm/mach-keystone/keystone.c >>> +++ b/arch/arm/mach-keystone/keystone.c >>> @@ -47,6 +47,9 @@ static void __init keystone_init(void) >>> >>> static const char *keystone_match[] __initconst = { >>> "ti,keystone-evm", >>> + "ti,k2hk-evm", >>> + "ti,k2l-evm", >>> + "ti,k2e-evm", >> >> -evm indicate board-level compat fields. We'd be better off just keeping a >> "ti,keystone" top-level compatible here and just using that to probe. That >> way you don't have to touch the file for new boards in the future. >> > So far keystone-evm was k2hk(Keystone2 Hawking EVM) but with additional two > SOC's and board's, separate compatible was needed to differentiate. These > boards do have significant differences though current mainline code. > This mainly because the boards are build for those specific SOC families. > > I would like to keep the organisation that way so that we cab support any > future boards like below. > "ti,k2hk-evm" --> All the Keystone2 Hawking/Kepler boards > "ti,k2l-evm" --> All the Keystone2 Lamarr boards > "ti,k2e-evm" --> All the Keystone2 Edision boards > > I don't expect to touch this file till new SOC gets added with its > EVM. Right, but today they all need to be enumerated in your C file -- if you do a "ti,keystone" compatible field that you use on all of them then there will be no need for any code changes for these different boards, which is the whole point behind device-tree... -Olof -- 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