
<html>

  <head>

    <meta content="text/html; charset=windows-1252"

      http-equiv="Content-Type">

  </head>

  <body bgcolor="#FFFFFF" text="#000000">

    <small>Hi Alistair,</small><br>

    <br>

    <small>Thanks for the review.</small><br>

    <br>

    <div class="moz-cite-prefix">On 07/01/2015 12:02 PM, Alistair Popple

      wrote:<br>

    </div>

    <blockquote cite="mid:6029281.FKgWxzJ7aI@mexican" type="cite">

      <pre wrap="">Hi Neelesh,


A few comments below. The main question I have is do we even need to implement 

a queue in this driver, given the FSP already has a message queue?</pre>

    </blockquote>

    <br>

    <small>Yes, FSP queues the messages, but will not be aware of when

      the response<br>

      received (it's asynchronous) so fsp-ipmi driver maintains the

      queue so that<br>

      it queues the next request only after the current completed.<br>

      Without the list, we might send multiple IPMI mbox commands for

      different<br>

      requests which FSP hardware doesn't expect.<br>

      Going through your other comments, will respond/fix.<br>

      <br>

      Thanks,<br>

      Neelesh.<br>

    </small><br>

    <blockquote cite="mid:6029281.FKgWxzJ7aI@mexican" type="cite">

      <pre wrap="">


Regards,


Alistair


On Wed, 24 Jun 2015 22:46:20 Neelesh Gupta wrote:

</pre>

      <br>

    </blockquote>

    <br>

  </body>

</html>