LOGIN   :::   RECOVER PASS   :::   GET ACCOUNT    
Browse
  • Projects
  • Code (CVS)
  • Forums
  • News
  • Articles
  • Polls
  •  
    OpenCores
  • FAQ
  • CVS HowTo
  • Mission
  • Media
  • Tools
  • Sponsors
  • Mirrors
  • Logos
  • Contact us
  •  
    Tools
  • Search
      
  • Download Cores (CVSGet)
  •  
    More
  • Wishbone
  • Perlilog
  • EDA tools
  • OpenTech CD
  •  
    Overview :: News :: Downloads :: Tracker    

    Unsigned serial divider: Tracker : Opencores serial_div_uu bug?

    Monitor this item

    You will be notified via email when status of this item is changed or if somebody adds a comment.

    Your email

      Opencores serial_div_uu bug?

    Type BUG
    Status OPENED
    Top
    Hi John,

    First off I wanted to say thanks for posting serial_div_uu on Opencores.com. It's very simple and well documented.

    I think I may have found a simple (but fatal) bug/typo on line 176 where it says:

    grand_divisor <= divisor_i << (N_PP+R_PP-S_PP-1);

    I think it should say:

    grand_divisor <= divisor_i << (M_PP+R_PP-S_PP-1);

    Shouldn't the divisor be shifted by the size of the dividend? The way it is written the core only works when the dividend size is equal to the divisor size.

    Thanks,
    -Joe Howard

     
    Stats

    1 people are monitoring this item

    Progress
     
    Submited date 01-Dec-2006
    Submited by howardj@e...
     
    Assigned date
    Assigned to
     
    Closed date
    Closed by

    Top

    Comments

    by alhadry1423@h... on 06-May-2008
    hadry
     
    by henning.spruth@f... on 27-Jan-2007
    Thanks to Joe Howard for diagnosing this - I wrote a simple testbench for the divider and was puzzled by the (seemingly wrong) results. The change seems to fix the issue. Henning Spruth
     

    Add your comment

    Your email:

    Retype key:
    Top

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