Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1232130

Re: [PATCH 1/3] Documentation: dt: keystone: provide SoC specific compatible flags

Show all headers | View raw

On 09/24/2015 09:05 AM, Murali Karicheri wrote:
> On 09/23/2015 02:19 PM, santosh shilimkar wrote:
>> Nishant,
>>
>> On 9/22/2015 9:08 AM, 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>
>>> ---
>> You need to expand that 'not sufficient' for me. Unless there is
>> genuine case to support this, I would want to avoid this churn.
>>
> 
> I agree. If there are run time check required in code to treat the 
> variants of Keystone SoC differently, then this change is needed. At 
> this time, SoC DTS captures these differences. IMHO, If a future 
> Keystone SoC support required SoC specific compatibility string to 
> customize SoC specific initialization code, then it can be introduced at 
> that time. I am not sure why this is introduced with out an example usage.

a) See
https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/Documentation/devicetree/usage-model.txt#n116
-> all we have today is board, soc generic family. The generic
compatible flag of ti,keystone to mark that this is a keystone2 device
does not identify that what kind of soc it is. which is what this
series introduces.

b) there is no realistic way for user space applications such as a
test framework or other SoC based applications in a generic distro to
detect the right SoC this platform is running on - imagine writing SoC
specific code that depends on board compatible flags - there is never
an end to that story as the number of boards expand.

c) Ideally we will try never to introduce code that will have to
depend on SoC compatible flag based match in kernel - but that is not
a reason not to follow guidelines provided by devicetree documentation
to provide SoC specific compatible flags.

-- 
Regards,
Nishanth Menon
--
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/

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH 1/3] Documentation: dt: keystone: provide SoC specific compatible flags Nishanth Menon <nm@ti.com> - 2015-09-22 18:10 +0200
  Re: [PATCH 1/3] Documentation: dt: keystone: provide SoC specific  compatible flags Murali Karicheri <m-karicheri2@ti.com> - 2015-09-23 20:10 +0200
    Re: [PATCH 1/3] Documentation: dt: keystone: provide SoC specific  compatible flags Nishanth Menon <nm@ti.com> - 2015-09-23 21:20 +0200
  Re: [PATCH 1/3] Documentation: dt: keystone: provide SoC specific  compatible flags santosh shilimkar <santosh.shilimkar@oracle.com> - 2015-09-23 20:30 +0200
    Re: [PATCH 1/3] Documentation: dt: keystone: provide SoC specific  compatible flags Murali Karicheri <m-karicheri2@ti.com> - 2015-09-24 16:10 +0200
      Re: [PATCH 1/3] Documentation: dt: keystone: provide SoC specific  compatible flags Nishanth Menon <nm@ti.com> - 2015-09-24 16:30 +0200
        Re: [PATCH 1/3] Documentation: dt: keystone: provide SoC specific  compatible flags Murali Karicheri <m-karicheri2@ti.com> - 2015-09-24 18:00 +0200
          Re: [PATCH 1/3] Documentation: dt: keystone: provide SoC specific  compatible flags Nishanth Menon <nm@ti.com> - 2015-09-25 17:00 +0200
            Re: [PATCH 1/3] Documentation: dt: keystone: provide SoC specific  compatible flags santosh shilimkar <santosh.shilimkar@oracle.com> - 2015-09-25 17:20 +0200
              Re: [PATCH 1/3] Documentation: dt: keystone: provide SoC specific  compatible flags Nishanth Menon <nm@ti.com> - 2015-09-25 18:10 +0200
                Re: [PATCH 1/3] Documentation: dt: keystone: provide SoC specific  compatible flags santosh shilimkar <santosh.shilimkar@oracle.com> - 2015-09-25 18:20 +0200
                Re: [PATCH 1/3] Documentation: dt: keystone: provide SoC specific  compatible flags Nishanth Menon <menon.nishanth@gmail.com> - 2015-09-25 19:40 +0200
                Re: [PATCH 1/3] Documentation: dt: keystone: provide SoC specific  compatible flags santosh shilimkar <santosh.shilimkar@oracle.com> - 2015-10-02 18:20 +0200
                Re: [PATCH 1/3] Documentation: dt: keystone: provide SoC specific  compatible flags Nishanth Menon <nm@ti.com> - 2015-10-04 01:50 +0200
                Re: [PATCH 1/3] Documentation: dt: keystone: provide SoC specific  compatible flags "santosh.shilimkar@oracle.com" <santosh.shilimkar@oracle.com> - 2015-10-04 02:20 +0200
                Re: [PATCH 1/3] Documentation: dt: keystone: provide SoC specific  compatible flags Murali Karicheri <m-karicheri2@ti.com> - 2015-09-30 16:40 +0200

csiph-web