[Skiboot] [PATCH 05/15] FSP/LED: Define macro instead of hardcoding led type

Vasant Hegde hegdevasant at linux.vnet.ibm.com
Thu Apr 2 16:16:57 AEDT 2015


On 03/31/2015 11:37 AM, Vasant Hegde wrote:
> On 03/26/2015 08:55 AM, Stewart Smith wrote:
>> Vasant Hegde <hegdevasant at linux.vnet.ibm.com> writes:
>>> Define macors for LED related device tree property and
>>> its values.
>>>
>>> Macros which needs to be exposed to host is defined in opal.h
>>
>> I don't think we really need these. convention is that we just use the
>> strings around the place and define that in API, so keeping strings
>> rather than defines around is fine.
> 
> Stewart,
> 
> There are multiple places where I refer same string.. Hence converted them to
> macros..May be I can move all macros to led.h . .what you say ?

Ok .. I have moved all the definition to fsp-leds.h file. Also I have modified
SAI MBOX interface (patch 13/15) after having discussion with LED folks. Will
send out v2 soon.

-Vasant



More information about the Skiboot mailing list