[Skiboot] [PATCH 1/2] opal-prd: Fix occ_reset call

Stewart Smith stewart at linux.vnet.ibm.com
Thu Sep 28 13:19:19 AEST 2017


Daniel M Crowell <dcrowell at us.ibm.com> writes:
> Seems fine (though I don't really have the context of what function this 
> is.  (I'm not fluent in patch reviews, not sure how you understand 
> anything with so little context....)

A bunch of code context can be just kept in the brain (it goes alongside
the ego, both take up a lot of room :) - but the other half is just
applying the patches and looking at the result - i'll freely pipe to
'git am -s' when doing close review - build, look, etc. Of course, that
requires a sane mail client, which certainly isn't the IBM default.

-- 
Stewart Smith
OPAL Architect, IBM.



More information about the Skiboot mailing list