REVIEW 3 major objections 5 minor 25 references
Real-Time Piano Note Frequency Detection Using FPGA and FFT Core
T0 review · 3 major / 5 minor · reviewed 2026-08-05 · deepseek-v4-flash
Pith's one-line read This paper reports that a Spartan-3E FPGA, using a 512-point fixed-point FFT core in burst mode, samples analog piano audio, finds the dominant frequency bin, and displays the frequency on an LCD, with each key press producing a distinct bi
desk verdict A student project report on a standard FPGA FFT pitch detector with a fixable factor-of-two error in the resolution and an uncalibrated sample rate; workable as a teaching demo, not a research contribution. read the letter →
The pith
A machine-rendered reading of the paper's core claim, the machinery that carries it, and where it could break.
The reading
What carries the argument
The central mechanism is the sampling-rate chain feeding a 512-point FFT. A 5 MHz system clock is divided by a 73-clock spacing between ADC reads and a further 16:1 downsample, giving an effective sampling rate F_s ≈ 4.28 kHz; the FFT then yields bins spaced by Δf ≈ 8.36 Hz. The transform itself is a Radix-4 Burst I/O FFT core—it loads all 512 time samples in one burst, computes internally, then unloads bins in natural order—with 14-bit fixed-point inputs. A peak detector squares the real and imaginary parts of each output bin, keeps the largest magnitude and its index, and scales that index to a frequency in hertz for the character LCD. A finite state machine synchronizes the ADC, the FFT c
What would settle it
Feed a calibrated sine wave of known frequency, say 440.0 Hz, into the same DC-biased audio input while the system runs, and simultaneously capture the ADC sample clock with a logic analyzer or oscilloscope. If the LCD shows a frequency more than one bin width (about 8.36 Hz) away from 440 Hz, or the measured sample period is not exactly 73 × 16 / 5 MHz, then the nominal sampling-rate assumption is the failure point. Repeating this at 130.68 Hz and 293.68 Hz would test whether the displayed values reproduce the examples in the report.
Extended reading notes
Core claim
On the paper's own terms, the discovery is that a working real-time piano-note frequency detector can be assembled from standard FPGA blocks: a custom SPI master driving the board's ADC, a vendor FFT core configured for a 512-point Radix-4 Burst I/O transform, a peak-magnitude tracker that compares squared real and imaginary outputs, and a finite state machine that coordinates sampling, FFT feeding, and an LCD update. The report states that every key press produced a distinct frequency bin, confirming the sampling and FFT pipeline. Two captured waveforms show the detected fundamentals at approximately 293.68 Hz and 130.68 Hz. The design deliberately trades transform size and throughput for r
Load-bearing premise
The whole frequency readout assumes the ADC really samples at F_s = 5 MHz / (73 × 16) ≈ 4.28 kHz from the nominal 5 MHz board clock; the report never measures the actual sample timing, so a deviation there would bias every displayed frequency.
Editorial extensions
If this is right
- A single audio source can be reduced to its dominant frequency using only a small FPGA's logic and block RAM, with no software processor or operating system in the signal path.
- With the 512-point/4.28 kHz configuration, notes whose fundamentals are separated by more than about 8.36 Hz can be told apart; the report says this distinguishes common piano notes but leaves closely spaced or harmonic-rich cases unresolved.
- Raising the transform to 1024 points, or switching to Pipelined Streaming mode, fails on this board, so real-time operation on this class of FPGA depends on burst-mode FFT and deliberate downsampling.
- The detected frequency is a linear function of the assumed sampling rate, so any error in the nominal 5 MHz clock or the delay counts shifts every displayed frequency by the same proportion.
Reading between the lines
- The paper never verifies the sample clock, so a calibrated-tone test against a measured sample period would tell whether the displayed frequencies are accurate in absolute terms; this is an inference beyond the reported results.
- Because the bin spacing is 8.36 Hz, adjacent low-piano notes (e.g., C2 to C#2, about 3.9 Hz apart) would fall in the same or neighboring bins, suggesting low-note accuracy needs interpolation even though the mid-range detections in the report are distinct.
- The pipeline should apply to any periodic sound source with a fundamental below the 2.14 kHz Nyquist limit, not just pianos, since the piano only provides the test tones.
- Burst-mode FFT means the system takes snapshots rather than streaming continuously; the report's 'real-time' claim is about a complete sample-FFT-display cycle, not sample-by-sample throughput.
Signed reviews
Editorial analysis
A structured set of objections, weighed in public.
Referee Report
Summary. The manuscript reports an FPGA-based single-note frequency detector built on a Spartan-3E starter kit. Analog piano audio is captured through the board's LTC1407A ADC, downsampled to a nominal effective rate of 4.28 kHz, stored as 512 samples, processed by a Xilinx FFT IP core in Radix-4 Burst I/O mode, and reduced to a peak magnitude bin. The bin index is converted to a frequency in hertz and displayed on a character LCD. The paper describes the module-level design, the FFT core interface, resource-driven choices (1024-point and streaming FFT configurations were abandoned), and ChipScope debugging. The authors conclude that the system meets the core functional requirements of sampling, FFT processing, and dominant-frequency display, with oscilloscope waveforms as evidence.
Significance. If the frequency-detection claim were quantitatively validated, this would be a modest but useful educational demonstration of FPGA-based FFT signal processing on a resource-constrained board. The paper has several genuine strengths: the DFT/FFT mathematics in Section 3 are correct, the description of the Xilinx FFT core handshaking is careful and consistent with the product behavior, and the resource-driven decision to move from 1024-point/streaming implementations to a 512-point burst configuration is clearly documented. The use of ChipScope waveforms and the explicit derivation of the nominal sampling rate also show good engineering discipline. However, the manuscript does not supply reproducible code, a measured sample-rate calibration, or a quantitative comparison of detected frequencies to ground-truth piano-note frequencies. Those missing elements are load-bearing for the central claim.
major comments (3)
- [Section 4 vs. Section 5] There is a direct internal contradiction in the reported frequency resolution. Section 4 derives Δf = F_s/512 ≈ 8.36 Hz/bin for F_s ≈ 4.28 kHz. Section 5 states 'a resolution of approximately 4.28 Hz per FFT bin' for the same 512-point FFT and the same sampling rate of roughly 4.28 kHz. These values differ by a factor of two. A 4.28 Hz/bin resolution would require F_s ≈ 2.19 kHz, not 4.28 kHz. Since the displayed frequency is presumably computed as bin_index × F_s/512 (or an equivalent scaling), this discrepancy implies either the actual sampling rate is not 4.28 kHz or the HDL bin-to-Hz conversion divides by ~1024 instead of 512. This must be resolved because it scales every reported frequency and directly affects the accuracy claim.
- [Section 4 and Design Flow step 5] The effective sampling rate is taken as exactly F_s = 5 MHz/(73×16) ≈ 4.28 kHz from the nominal system clock and the chosen downsampling counter values, but the paper never measures the actual sample timing. Any deviation in the ADC SPI transaction length, clock divider behavior, or debounce logic would scale every displayed frequency by a constant factor. The 'Output Calibration' step in Section 2 mentions scaling bin indexes based on the known sampling rate, but no calibration data or measured sample intervals are reported. The authors should either measure the sample cadence with ChipScope or feed a known calibration tone (e.g., a 440 Hz sine) and report the observed bin index and displayed frequency versus the expected value. Without this, the frequency output is unvalidated even if the FFT pipeline works correctly.
- [Section 5 and Figures 2-4] The evidence presented supports only the weaker claim that distinct piano keys produce distinct FFT bin outputs. The conclusion that 'each key press translated into a distinct frequency bin output, confirming correct implementation of the FFT and sampling pipeline' does not establish that the displayed frequencies are accurate. Figures 2-4 show oscilloscope waveforms with annotated frequencies (293.68 Hz, 130.68 Hz, 350.26 Hz), but there is no tabulated comparison to the expected piano-note frequencies, no description of how the annotated values were measured, and no stated error metric. For the central claim of detecting and displaying the dominant piano-note frequency, the authors need to compare the FPGA's LCD output to ground-truth signal frequencies for several notes and report the errors.
minor comments (5)
- [Equation (2)] The inverse DFT formula is misstated: the left side is written as X(k), but the right side is the synthesis formula for x(n). The 1/N factor is present, but the variable mismatch makes the equation incorrect as written.
- [Figures 2-4 captions] It is unclear whether the annotated frequencies (293.68 Hz, 130.68 Hz, 350.26 Hz) are oscilloscope measurements, the FPGA's LCD output, or expected note frequencies. Please state the source of these values in the captions and in the text.
- [Section 5 self-citation block] The paragraph citing references [3]-[25] is largely unrelated to the technical content of this project, and the claim that FPGA-based systems are 'currently the best option' is unsupported by the cited analyses in the context of this work. This block should be removed or reduced to the few references that directly inform the design.
- [Section 1 and 5] The term 'real-time' is used loosely. The design uses pushbutton-triggered sampling and a burst FFT, which is not continuous streaming analysis. Please either quantify the end-to-end latency or qualify the real-time claim (e.g., 'near-real-time for single notes').
- [Section 4] The notation F_s is used with inconsistent capitalization (F_s, Fs, fs). Also, the 73×16 downsampling rationale is not explained beyond resource constraints; a sentence on why these particular values were chosen would help.
Circularity Check
No circular derivation for the frequency-detection result; only a minor, non-load-bearing self-citation block and an internal factor-of-two resolution inconsistency.
-
self citation load bearing
[Section 5, Conclusion paragraph (page 17)]
"Their analyses [3],[4] shows that FPGA-based systems are currently the best option to support applications and algorithms, such as the ones presented in this report. Also, their previous work on FPGA-based accelerators, architectures, and techniques for various compute and data-intensive applications ... demonstrated that FPGA-based systems are the best avenue to support and accelerate complex algorithms."
The paper's motivational claim that FPGA is 'the best option' is supported exclusively by references [3] and [4], which are prior publications by the same author (Perera), plus a long chain of further self-citations. This is a self-referential support chain. However, this claim is not load-bearing for the central frequency-detection derivation: the FFT bin selection, sampling-rate computation, and LCD frequency scaling are self-contained and do not depend on these citations. The circularity is therefore minor and does not infect the main result.
full rationale
The central claim—that the FPGA system samples the analog piano signal, runs a 512-point FFT, finds the peak bin, and displays a corresponding frequency—is derived from the paper's own stated parameters: F_s = 5 MHz / (73 × 16) ≈ 4.28 kHz and Δf = F_s / 512 ≈ 8.36 Hz/bin (Section 4). There is no fitted parameter that is later renamed as a prediction, no uniqueness theorem imported from prior work, and no ansatz smuggled in via citation. The displayed frequency is simply bin_index × F_s / N, which is a definitional scaling, not a prediction equivalent to an input. The main correctness gap is that F_s is never measured and Section 5 claims '4.28 Hz per FFT bin' when Section 4 computes 8.36 Hz/bin; this is an internal calibration/validity problem, not a circularity. The large block of self-citations in Section 5 is used only to assert that FPGAs are 'the best option,' which is not essential to the frequency-detection result. Thus the overall circularity score is 2, reflecting one minor non-load-bearing self-citation, with the core derivation self-contained.
Assumptions & free parameters
free parameters (1)
- Effective sampling rate F_s =
≈4281 Hz
assumptions (4)
- domain assumption The Xilinx FFT IP core outputs the correct DFT when driven according to its datasheet timing.
- domain assumption The ADC samples the analog input at uniform intervals determined by the SPI master's clock.
- domain assumption The piano note is quasi-stationary across the 512-sample capture window (~120 ms) so the highest FFT bin represents the fundamental.
- standard math DFT/FFT mathematical correctness and spectral binning.
Cite this review
Pith. "Pith review of Real-Time Piano Note Frequency Detection Using FPGA and FFT Core." pith.science (2026). https://pith.science/paper/B5AHCKHH
@misc{pith2026250900589,
author = {Pith},
title = {Pith review of: Real-Time Piano Note Frequency Detection Using FPGA and FFT Core},
year = {2026},
howpublished = {\url{https://pith.science/paper/B5AHCKHH}},
note = {Machine review of arXiv:2509.00589}
}
read the original abstract
Real-time frequency analysis of musical instruments, such as the piano, is an essential feature in areas like electronic tuners, music visualizers, and live sound monitoring. Traditional methods often rely on software-based digital signal processing (DSP), which may introduce latency and require significant computational power. In contrast, hardware platforms such as FPGAs (Field Programmable Gate Arrays) offer the ability to perform such analyses with greater speed and determinism due to their parallel processing capabilities. The primary objective of this project was to analyze analog audio signals from a digital piano using an FPGA-based real-time Fast Fourier Transform (FFT) system.
Figures
Reference graph
Works this paper leans on
-
[1]
AMD (Xilinx), LogiCORE IP Fast Fourier Transform (v7.1), Product Guide, DS260, Apr. 2022. [Online]. Available: https://docs.amd.com/v/u/en-US/xfft_ds260
work page 2022
-
[2]
AMD (Xilinx), Spartan-3E FPGA Starter Kit User Guide, UG230 (v1.0), Jan. 2006. [Online]. Available: https://docs.amd.com/v/u/en-US/ug230
work page 2006
-
[3]
Analysis of Single-Chip Hardware Support for Mobile and Embedded Applications,
D.G. Perera and K.F. Li, "Analysis of Single-Chip Hardware Support for Mobile and Embedded Applications," in Proc. of IEEE Pacific Rim Int. Conf. on Communication, Computers, and Signal Processing, (PacRim’13), pp. 369-376, Victoria, BC, Canada, August 2013
work page 2013
-
[4]
Analysis of Computation Models and Application Characteristics Suitable for Reconfigurable FPGAs
D.G. Perera and K.F. Li, “Analysis of Computation Models and Application Characteristics Suitable for Reconfigurable FPGAs”, in Proc. of 10th IEEE Int. Conf. on P2P, Parallel, Grid, Cloud, and Internet Computing, (3PGCIC’15), pp. 244-247, Krakow, Poland, Nov. 2015
work page 2015
-
[5]
S.N. Shahrouzi and D.G. Perera, "Optimized Hardware Accelerators for Data Mining Applications on Embedded Platform: Case Study Principal Component Analysis," Elsevier Journal on Microprocessor and Microsystems (MICPRO), vol. 65, pp. 79-96, March 2019
work page 2019
-
[6]
Optimizing Density-Based Ant Colony Stream Clustering Using FPGA-Based Hardware Accelerator
J.R. Graf and D.G. Perera, “Optimizing Density-Based Ant Colony Stream Clustering Using FPGA-Based Hardware Accelerator”, in Proc. Of IEEE Int. Symp. on Circuits and Systems (ISCAS’23), 5-page manuscript, Monterey, California, May 2023
work page 2023
-
[7]
A.K. Madsen and D.G. Perera, “Towards Composing Efficient FPGA-Based Hardware Accelerators for Physics-Based Model Predictive Control Smart Sensor for HEV Battery Cell Management”, IEEE ACCESS, (Open Access Journal in IEEE), pp. 106141-106171, 25th September 2023
work page 2023
-
[8]
A.K. Madsen and D.G. Perera, “Composing Optimized Embedded Software Architectures for Physics-Based EKF-MPC Smart Sensor for Li-Ion Battery Cell Management”, Sensors, MDPI open access journal, Intelligent Sensors Section, 21-page manuscript, vol. 22, no. 17, 26th August 2022
work page 2022
Show all 25 references
-
[9]
Efficient FPGA-Based Reconfigurable Accelerators for SIMON Cryptographic Algorithm on Embedded Platforms
A. Alkamil and D.G. Perera, “Efficient FPGA-Based Reconfigurable Accelerators for SIMON Cryptographic Algorithm on Embedded Platforms”, in Proceedings of the IEEE International Conferences on Reconfigurable Computing and FPGAs, (ReConFig’19), 8-page manuscript, Cancun, Mexico,...
2019
-
[10]
Towards Dynamic and Partial Reconfigurable Hardware Architectures for Cryptographic Algorithms on Embedded Devices
A. Alkamil and D.G. Perera, “Towards Dynamic and Partial Reconfigurable Hardware Architectures for Cryptographic Algorithms on Embedded Devices”, IEEE Access, Open Access Journal in IEEE, vol. 8, pp: 221720 – 221742, 10th December 2020. 19 | P a g e
2020
-
[11]
An FPGA-Based Hardware Accelerator for K-Nearest Neighbor Classification for Machine Learning on Mobile Devices
M.A. Mohsin and D.G. Perera, “An FPGA-Based Hardware Accelerator for K-Nearest Neighbor Classification for Machine Learning on Mobile Devices”, in Proceedings of the IEEE/ACM International Symposium on Highly Efficient Accelerators and Reconfigurable Technologies, (HEART’18), ...
2018
-
[12]
FPGA-based Hardware Accelerator for Bottleneck Residual Blocks of MobileNetV2 Convolutional Neural Networks
Jordi P. Miró, Mokhles A. Mohsin, Arkan Alkamil and Darshika G. Perera, “FPGA-based Hardware Accelerator for Bottleneck Residual Blocks of MobileNetV2 Convolutional Neural Networks”, in Proceedings of the IEEE Mid-West Symposium on Circuits and Systems (MWCAS’25), 5-page manus...
2025
-
[13]
Intelligent Cognitive Radio Architecture Applying Machine Learning and Reconfigurability
J. Nurmi and D.G. Perera, “Intelligent Cognitive Radio Architecture Applying Machine Learning and Reconfigurability” in Proc. of IEEE Nordic Circuits and Systems (NorCAS'21) Conf., 6 -page manuscript, Oslo, Norway, October 2021
2021
-
[14]
High-Level Synthesis Based FPGA Accelerator for GPS Signal Image Feature Extraction
Kevin Young and Darsika G. Perera, “High-Level Synthesis Based FPGA Accelerator for GPS Signal Image Feature Extraction”, in Proceedings of the IEEE Mid-West Symposium on Circuits and Systems (MWCAS’25), 5-page manuscript, Lansing MI, August 2025
2025
-
[15]
Reconfigurable Architectures for Data Analytics on Next-Generation Edge-Computing Platforms
D.G. Perera, “Reconfigurable Architectures for Data Analytics on Next-Generation Edge-Computing Platforms”, Featured Article, IEEE Canadian Review, vol. 33, no. 1, Spring 2021. DOI: 10.1109/MICR.2021.3057144
2021
-
[16]
Composing Efficient Computational Models for Real -Time Processing on Next-Generation Edge-Computing Platforms
M.A. Mohsin, S.N. Shahrouzi, and D.G. Perera, “Composing Efficient Computational Models for Real -Time Processing on Next-Generation Edge-Computing Platforms” IEEE ACCESS, (Open Access Journal in IEEE), 30-page manuscript, 13th February 2024
2024
-
[17]
FPGA-Based Hardware Architecture for Sequence Alignment by Genetic Algorithm
Laura H. Garcia, Arkan Alkamil, Mokhles A. Mohsin, Johannes Menzel and Darshika G. Perera, “FPGA-Based Hardware Architecture for Sequence Alignment by Genetic Algorithm”, in Proceedings of the IEEE International Symposium on Circuits and Systems (ISCAS’25), 5-page manuscript, ...
2025
-
[18]
Neuromorphic Sentiment Analysis Using Spiking Neural Networks
R.K. Chunduri and D.G. Perera, “Neuromorphic Sentiment Analysis Using Spiking Neural Networks”, Sensors, MDPI open access journal, Sensing and Imaging Section, 24-page manuscript, vol. 23, no. 7701, 6th September 2023
2023
- [19]
-
[20]
A Systolic Array Architecture for SVM Classifier for Machine Learning on Embedded Devices
S. Ramadurgam and D.G. Perera, “A Systolic Array Architecture for SVM Classifier for Machine Learning on Embedded Devices”, in Proc. of IEEE Int. Symp. on Circuits and Systems (ISCAS’23), 5 -page manuscript, Monterey, California, May 2023
2023
-
[21]
Parallel Computation of Similarity Measures Using an FPGA-Based Processor Array,
D.G. Perera and Kin F. Li, “Parallel Computation of Similarity Measures Using an FPGA-Based Processor Array,” in Proceedings of 22nd IEEE International Conference on Advanced Information Networking and Applications, (AINA’08), pp. 955-962, Okinawa, Japan, March 2008
2008
-
[22]
Discrepancy in Execution Time: Static Vs. Dynamic Reconfigurable Hardware
D.G. Perera and K.F. Li, “Discrepancy in Execution Time: Static Vs. Dynamic Reconfigurable Hardware”, IEEE Pacific Rim Int. Conf. on Communications, Computers, and Signal Processing, (PacRim’24), 6 -page manuscript, Victoria, BC, Canada, August 2024
2024
-
[23]
FPGA-Based Reconfigurable Hardware for Compute Intensive Data Mining Applications
D.G. Perera and Kin F. Li, “FPGA-Based Reconfigurable Hardware for Compute Intensive Data Mining Applications”, in Proc. of 6th IEEE Int. Conf. on P2P, Parallel, Grid, Cloud, and Internet Computing, (3PGCIC’11), pp. 100-108, Barcelona, Spain, October 2011
2011
-
[24]
HDL Code Variation: Impact on FPGA Performance Metrics and CAD Tools
I.D. Atwell and D.G. Perera, “HDL Code Variation: Impact on FPGA Performance Metrics and CAD Tools”, IEEE Pacific Rim Int. Conf. on Communications, Computers, and Signal Processing, (PacRim’24), 6 -page manuscript, Victoria, BC, Canada, August 2024
2024
-
[25]
Towards Composing Optimized Bi-Directional Multi-Ported Memories for Next-Generation FPGAs
S.N. Shahrouzi, A. Alkamil, and D.G. Perera, “Towards Composing Optimized Bi-Directional Multi-Ported Memories for Next-Generation FPGAs”, IEEE Access, Open Access Journal in IEEE, vol. 8, no. 1, pp. 91531- 91545, 14th May 2020
2020
Reviewed August 5, 2026 · model on record in the stance chip above.
Discussion (0). Continue with ORCID to comment.