<html>
  <head>
    <meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    <div class="moz-cite-prefix">On 11/08/12 06:58, Arnaud Quette wrote:<br>
    </div>
    <blockquote
cite="mid:CAFm_bUgV4ycEzY=kb8umt_gpE+rAHfKuqy-g3oN5kyhn_dKVcg@mail.gmail.com"
      type="cite">Hi Massimo and Greg,<br>
      <div class="gmail_quote">
        <div> <br>
          @Greg: if you yet returned your unit, you now have a solution
          ;)<br>
        </div>
      </div>
    </blockquote>
    I have not yet gotten rid of it, so thank you very much!  It's a
    case of download, extract, apply patch, and compile on my RPi,
    correct?
    <blockquote
cite="mid:CAFm_bUgV4ycEzY=kb8umt_gpE+rAHfKuqy-g3oN5kyhn_dKVcg@mail.gmail.com"
      type="cite">
      <div class="gmail_quote">thanks for the effort for the patch!<br>
        <div> <br>
          welcome ;)<br>
          <br>
        </div>
        <blockquote class="gmail_quote" style="margin:0 0 0
          .8ex;border-left:1px #ccc solid;padding-left:1ex">
          <div class="im">
            > I took most of this afternoon looking at this issue,
            and this should be<br>
            > fixed with the attached patch<br>
            > It applies fine on 2.6.5, using the following command:<br>
            > # cd nut-2.6.5<br>
            > # patch -p0 < /path/to/xcp-rcv-loop.diff<br>
            > # make<br>
            ><br>
            > I've tested it on a 5110 only currently, so it's not
            applied to the<br>
            > development tree.<br>
            > for testing purpose, I've also used a byte frame size,
            to be in your<br>
            > condition (Ie, set the 4th param of
            usb_interrupt_read() to '8').<br>
            ><br>
            > please tell me back how it behave on RPi.<br>
            > note that I'll probably have to do some code
            adjustment, and obviously<br>
            > regression testing on other units, before committing
            the patch to the main<br>
            > source tree.<br>
            ><br>
            <br>
          </div>
          I applied your patch to the 2.6.5 code and run only the
          bcmxcp_usb<br>
          driver and... I'm not sure that everything is ok.<br>
        </blockquote>
        <div><br>
          sorry, I forgot to explicitly request the trace (debug level
          5) to validate on my side...<br>
           </div>
        <blockquote class="gmail_quote" style="margin:0 0 0
          .8ex;border-left:1px #ccc solid;padding-left:1ex">
          If I run the driver as "./bcmxcp_usb -u root -a ups" it
          launches, but<br>
          after a second I get a "Communications with UPS lost:
          get_answer: not<br>
          the right sequence received 0!!!". However the driver seems
          still to<br>
          keep running.<br>
          <br>
          I also tried to start the driver under debug (-DDDD). In
          attachment<br>
          you can see the output (until I typed ctrl-c), that also
          include some<br>
          some of the 'communication lost' messages.<br>
        </blockquote>
      </div>
      <br>
      the driver runs fine. you should get a good upsc output now.<br>
      the "Comm lost" msg are part of code adjustments needed, but the
      code already handle this fine, and recovers (check "entering
      get_answer(33)" for example).<br>
      <br>
      could you please confirm this by sending the upsc output, and
      starting upsmon to monitor on the RPi?<br>
    </blockquote>
    I'll let you know how I go!<br>
    <br>
    Thanks,<br>
    Greg<br>
  </body>
</html>