[Skiboot] [PATCH] hdata: Make sure FW feature name is not NULL
    Stewart Smith 
    stewart at linux.ibm.com
       
    Thu Sep 13 18:52:15 AEST 2018
    
    
  
Vaibhav Jain <vaibhav at linux.vnet.ibm.com> writes:
> Hi Vasant,
>
> Bikeshedding a bit but a minor review comment:
>
> "hdata: Make sure FW feature name is not NULL"
>
> I think it should be "not Empty" rather than "not NULL" since you are
> checking for length and 'name' and 'feature->name' being a static arrays
> wont be NULL.
Fixed the bikeshed when merging.
Merged to master as of 9d9395c3d542b5b2949b6ba3d5c0a8f1618b77e0
-- 
Stewart Smith
OPAL Architect, IBM.
    
    
More information about the Skiboot
mailing list