Bug #353

Fix for USRP1 FPGA conditional CIC inclusion (patch)

Added by Johnathan Corgan about 4 years ago. Updated about 4 years ago.

Status:Closed Start date:
Priority:Normal Due date:
Assignee:Eric Blossom % Done:

0%
Category:unknown
Target version:release-3.2.0
Resolution:duplicate

Description

Message-ID: <[email protected]>
Date: Tue, 17 Feb 2009 15:16:56 -0500
From: Paul Creekmore <[email protected]>
To: [email protected]
Subject: [Patch-gnuradio] USRP 1 CIC Decimator Removal Patch

Here is a patch to correct a bug in the USRP 1 FGPA code.  As written, 
the code will not build if RX_CIC_ON is not defined (if the user 
attempts to build a version without the CIC decimator), the compiler 
directives include a block of code fails to correctly rout the data and 
sample strobe directly to the halfband filter.

---Side note---
Incidentally, I've tested the USRP with this patch applied, both the CIC 
filter and halfband filter removed, and my quantization code (previously 
presented for review on the discussion list) in place and successfully 
achieved a 64 MHz sample rate for low quantization levels.  "Low" is 
defined by the quality of the USB connection and any host computer 
processing/hard drive speed limitations.

--Paul

cic_removal.patch - Patch (1.3 kB) Johnathan Corgan, 03/02/2009 06:49 pm

History

#1
Updated by Johnathan Corgan about 4 years ago

This patch was integrated into developer branch pcreekmore/quantization in r10680. It is now pending merge into the trunk as part of integrating the additional quantization options implemented in this branch.

#2
Updated by Eric Blossom about 4 years ago

  • Status changed from New to Closed
  • Resolution set to duplicate

Paul rolled this change into his developer branch that we will merge as part of #381. I'm closing this as a duplicate for lack of a better resolution.

Also available in: Atom PDF