LOGIN   :::   RECOVER PASS   :::   GET ACCOUNT    
Browse
  • Projects
  • Code (CVS)
  • Forums
  • News
  • Articles
  • Polls
  •  
    OpenCores
  • FAQ
  • CVS HowTo
  • Mission
  • Media
  • Tools
  • Advertise
  • Mirrors
  • Logos
  • Contact us
  • Job Opportunity
  •  
    Tools
  • Search
      
  • Download Cores (CVSGet)
  •  
    More
  • Wishbone
  • Perlilog
  • EDA tools
  • OpenTech CD
  •  
    Navigation: All forums > Cores > Message List > Message Post

    Message

    Reply | Reply all
    Date Prev | Date Next | Thread Prev | Thread Next Date Index | Thread Index

    From: Igor Mohor<igor.mohor@g...>
    Date: Thu Mar 10 09:27:29 CET 2005
    Subject: [oc] CAN Controller bug in can_fifo.v
    Top
    Hi, Jeff.


    Yes, you are right. Sorry for that. I already updated the cvs version.
    Thanks for reporting.

    Best regards,
    Igor


    On Wed, 09 Mar 2005 13:33:56 -0600, Jeff Hanoch <jeff@l...> wrote:
    > I was using the CAN controller in an FPGA and was seeing some strange
    > behavior. We would halt the CPU debugger but the activity on the CAN
    > bus would continue. This filled up the FIFOs in the controller as
    > expected, but it would get into some strange state where we could not
    > read correct data.
    >
    > Upon further logic simulation I narrowed down what I think the problem
    > is... In the always statement that generates the rd_info_pointer, the
    > pointer is incremented when release_buffer is high and fifo_empty is
    > low. I believe that this should be info_empty instead of fifo_empty. I
    > have changed this and rerun simulations as well as rebuilding the FPGA
    > and it seems to have fixed the problem.
    >
    > Here's the code I changed (line 283 of can_fifo.v). What do you think?
    >
    > // rd_info_pointer
    > always @ (posedge clk or posedge rst)
    > begin
    > if (rst)
    > rd_info_pointer <= 6'h0;
    > // else if (release_buffer & (~fifo_empty))
    > else if (release_buffer & (~info_empty))
    > rd_info_pointer <=#Tp rd_info_pointer + 1'b1;
    > end
    >
    > Thanks,
    > Jeff
    >
    >
    > _______________________________________________
    > http://www.opencores.org/mailman/listinfo/cores
    >
    >
    >

    ReferenceAuthor
    [oc] CAN Controller bug in can_fifo.vJeff Hanoch

     
    Copyright (c) 1999 OPENCORES.ORG. All rights reserved.