lkml.org 
[lkml]   [2015]   [Sep]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 1/3] Documentation: dt: keystone: provide SoC specific compatible flags
On 09/22/2015 12:08 PM, Nishanth Menon wrote:
> Keystone2 devices are used on more platforms than just Texas
> Instruments reference evaluation platforms called EVMs. Providing a
> generic compatible "ti,keystone" is not sufficient to differentiate
> various SoC definitions possible on various platforms. So, provide
> compatible matches for each SoC family by itself.
>
> This allows SoC specific logic to be run time handled based on
> of_machine_is_compatible("ti,k2hk") or as needed for the dependent
> processor instead of needing to use board dependent compatibles that
> are needed now.
>
> Signed-off-by: Nishanth Menon <nm@ti.com>
> ---
> .../devicetree/bindings/arm/keystone/keystone.txt | 20 +++++++++++++++++---
> 1 file changed, 17 insertions(+), 3 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/arm/keystone/keystone.txt b/Documentation/devicetree/bindings/arm/keystone/keystone.txt
> index 59d7a46f85eb..800d2d02e27b 100644
> --- a/Documentation/devicetree/bindings/arm/keystone/keystone.txt
> +++ b/Documentation/devicetree/bindings/arm/keystone/keystone.txt
> @@ -9,12 +9,26 @@ Required properties:
> the form "ti,keystone-*". Generic devices like gic, arch_timers, ns16550
> type UART should use the specified compatible for those devices.
>
> +SoC families:
> +
> +- Keystone 2 generic SoC:
> + compatible = "ti,keystone"
> +
> +SoCs:
> +
> +- Keystone 2 Hawking/Kepler
> + compatible = ti,k2hk", "ti,keystone"
> +- Keystone 2 Lamarr
> + compatible = ti,k2l", "ti,keystone"
> +- Keystone 2 Edison
> + compatible = ti,k2e", "ti,keystone"
> +
> Boards:
> - Keystone 2 Hawking/Kepler EVM
> - compatible = "ti,k2hk-evm","ti,keystone"
> + compatible = "ti,k2hk-evm", "ti,k2hk", "ti,keystone"
>
> - Keystone 2 Lamarr EVM
> - compatible = "ti,k2l-evm","ti,keystone"
> + compatible = "ti,k2l-evm", "ti, k2l", "ti,keystone"
>
> - Keystone 2 Edison EVM
> - compatible = "ti,k2e-evm","ti,keystone"
> + compatible = "ti,k2e-evm", "ti,k2e", "ti,keystone"
>
DTS takes care of the difference in the hardware and If there SoC
specific customization required outside this, then it is best to include
this as part of that change. In the past, I believe we didn't do it due
to the same reason as above.

Murali

--
Murali Karicheri
Linux Kernel, Keystone


\
 
 \ /
  Last update: 2015-09-23 20:21    [W:0.067 / U:0.792 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site