<font size=2 face="sans-serif">I don't like mfg_ as a prefix for this
one because it likely would have nothing to do with manufacturing.  I
would suggest some kind of --debug flag but that is already in use.  Maybe
"opdev_" for Open Power Developer?  </font>
<br><font size=2 face="sans-serif"><br>
--<br>
Dan Crowell<br>
Senior Software Engineer - Power Systems Enablement Firmware<br>
IBM Rochester: t/l 553-2987<br>
dcrowell@us.ibm.com</font>
<br>
<br>
<br>
<br><font size=1 color=#5f5f5f face="sans-serif">From:      
 </font><font size=1 face="sans-serif">Patrick Williams <patrick@stwcx.xyz></font>
<br><font size=1 color=#5f5f5f face="sans-serif">To:      
 </font><font size=1 face="sans-serif">Neelesh Gupta <neelegup@linux.vnet.ibm.com></font>
<br><font size=1 color=#5f5f5f face="sans-serif">Cc:      
 </font><font size=1 face="sans-serif">skiboot@lists.ozlabs.org,
Daniel M Crowell/Rochester/IBM@IBMUS, Bill Schwartz/Austin/IBM@IBMUS, Christopher
J Cain/Rochester/IBM@IBMUS</font>
<br><font size=1 color=#5f5f5f face="sans-serif">Date:      
 </font><font size=1 face="sans-serif">08/24/2015 05:02 PM</font>
<br><font size=1 color=#5f5f5f face="sans-serif">Subject:    
   </font><font size=1 face="sans-serif">Re: [Skiboot]
[PATCH v2 5/5] external/opal-prd: Add 'run' option to execute HBRT commands</font>
<br>
<hr noshade>
<br>
<br>
<br><tt><font size=2>On Tue, Aug 25, 2015 at 12:50:37AM +0530, Neelesh
Gupta wrote:<br>
> From: Vaidyanathan Srinivasan <svaidy@linux.vnet.ibm.com><br>
> <br>
> opal-prd client to accept run command and pass it as-is to hbrt.<br>
> Example:<br>
> opal-prd -d run test hbrt -t 1 -c "good cmd"<br>
> argv[0] = test<br>
> argv[1] = hbrt<br>
> argv[2] = -t<br>
> argv[3] = 1<br>
> argv[4] = -c<br>
> argv[5] = good cmd<br>
> <br>
> Above argc/argv passed to hbrt->run_command() and result out string<br>
> sent to the console.<br>
><br>
<br>
Is this one suppose to be a 'manufacturing' command as well?  I don't<br>
believe our intention is to use this for any permanent code.  This<br>
should be one of those "only use if you know what you are doing"<br>
commands.<br>
<br>
We may want to change the name of the Hostboot side to 'mfg_run_command'<br>
if we determine this is a 'manufacturing' command.<br>
<br>
~~<br>
Patrick Williams<br>
[attachment "signature.asc" deleted by Daniel M Crowell/Rochester/IBM]
</font></tt>
<br>