Re: FEATURES prints

[Kevin Hilman <khilman@deeprootsystems.com>]

Tony Lindgren <tony@atomide.com> writes:

> * Kevin Hilman <khilman@deeprootsystems.com> [091013 15:01]:
>> Nishanth Menon <nm@ti.com> writes:
>> 
>> > Folks,
>> >
>> > With the addition of FEATURES in l-o, the following prints:
>> >  - l2cache : Y
>> >  - iva : Y
>> >  - sgx : Y
>> >  - neon : Y
>> >  - isp : Y
>> >
>> > comes up on SDP3430 -> now that we will introduce half a dozen
>> > features here and there, we will soon clutter this up. we should
>> > introduce a sysfs entry + remove the above noise..
>> >
>> 
>> Like Nishanth, I don't like the multi-line noise here.  The patch
>> below results in a single line output like this instead
>> 
>> OMAP3430/3530 ES3.0 (l2cache iva sgx neon isp )
>> 
>> Not sure why we need to dump features that are not there, but if that
>> s considered important, maybe prefixing each feature with a '+' or '-'
>> would still allow this to be collapsed into a single line.
>> 
>> Even with this, I think adding the display of these features into an
>> OMAP-specific section of /proc/cpuinfo would be even better.
>> 
>> Comments?
>
> I like. Looks like this patch needs to be refreshed.
>

OK, will send refreshed version.

Kevin