<div class="gmail_quote">On Mon, Jan 9, 2012 at 12:03,  <span dir="ltr"><<a href="mailto:przanoni@gmail.com">przanoni@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

From: Paulo Zanoni <<a href="mailto:paulo.r.zanoni@intel.com">paulo.r.zanoni@intel.com</a>><br>
<br>
This is a command-line tool that allows us to display and modify the<br>
InfoFrames we send.<br>
<br>
Signed-off-by: Paulo Zanoni <<a href="mailto:paulo.r.zanoni@intel.com">paulo.r.zanoni@intel.com</a>><br></blockquote><div><br>Very interesting!<br><br>I think that it would be more interesting to use command line options instead of interactive input - something like:<br>

-d, --dump<br>-c, --change [parameters]<br>-dd, --dip-disable<br>-de, --dip-enable [parameters]<br>-hd, --hdmi-disable<br>-he, --hdmi-enable [parameters]<br><br>So this could be scripted and used non-interactively. This would also simplify your change_avi_infoframe and parse_infoframe_option_s - you could just pass all the values as parameters instead.<br>

<br></div></div>Other than that,<br>Reviewed-by: Eugeni Dodonov <<a href="mailto:eugeni.dodonov@intel.com">eugeni.dodonov@intel.com</a>><br><br>-- <br>Eugeni Dodonov<a href="http://eugeni.dodonov.net/" target="_blank"><br>

</a><br>