1. Is there a SparkFun Logo? [NO]
   comment: Missing logo in sch

2. Is there an OSHW logo? [NO]
   comment: missing oshw logo in sch

3. If net labels are used, are they the correct 'FLAG' type? [NO]
   comment: A4 and A5 nets off the micro pins could use some flag labels

4. Are dashed grey lines ('info' eagle layer 97) used to clearly separate different areas? [NO]
   comment: could be better

5. Do all GND connections use the GND symbol (oriented downward)? [NO]
   comment: nets off of JP1, JP7 and JP6

6. Do all VCC connections use the VCC symbol (oriented upward)? [NO]
   comment: nets off of JP1 and JP6

7. Is the VCC range clearly labeled? [NO]
   comment: Yes, but RAW input range would be useful too

8. Have you verified that the BOM in Sparkle is accurate (also checking for PROD_ID, NAME, VALUE, PREFIX)? [NO]
   comment: no PROD_IDs on anything. value on Q1 should loose the "20MHz", and U1 could use a "ATMEGA328P" (note the "P"), LED1 should be a "D"

9. Do any parts have unnecessary values (Eagle has autoset the package as the value)? [YES]
   comment: JP6, JP7

10. Is there a note for a board thickness other than 0.63 inch? [YES]
    comment: I like the large text, but putting in "0.8mm thickness" is what most people are used to, I think both the large text and the number would be great.

11. Is there a URL for any kits or unique boards? [YES]
    comment: asked at bug, but it might be a good idea to put the landing page for this product on the bottom silk...

I recently designed a system using the Pro Mini 3 and the Pro Mini 5 modules. The mini3 interfaces with the Sparkfun FS-3000 air flow sensor. I've had no issues with this pair. I've attached a block diagram for reference. Also part of this design is a Pro mini 5 module. This module reads the speed data from the Pro mini 3 and holds it until the I2C master requests it from the Mini 5. The master device is a Spektrum AR637T radio receiver used in electric RC planes. The AR637T receiver sends the speed data to the ground transmitter through the telemetry channel. Everything worked but there was one concern I had. When I programmed the Mini 5 with the Arduino IDE. under the tools menu, I expected I'd have to select the Arduino Pro Mini option with the 5V/Mege 328P version selected. However I could not download to the module after compiling. I tried the selection of Arduino Uno Mini and this worked, i.e. I was able to download and the module worked fine in the system.
I accepted this since everything was working, but it was a concern because it wasn't what I expected.
However, I ordered 3 more Mini 5s to build up more systems and these new units would NOT allow downloads unless the Arduino Pro Mini was selected along with th5 5v Mega 328P option. Everything compiled and downloaded fine. This is what I expected. But, none of the the modules would not link up with the master over the I2C bus. I dug a little deeper and observed that it takes over 400ms for the Mini 5 code to start executing after power up reset which makes it too late to respond to the initial polling by the bus master. The radio master initial poll for I2C devices comes after 250 ms from power up and if a slave doesn't respond to this first poll, it is not polled again. I tested this by making sure the Mini5 had power before the radio did and it linked up just fine. But in the final design the Radio master, and my Mini 3 and Mini5 all power up together. I used the Sparkfun FTD module connected to a USB port to download. One for the Mini 3 and another one for the Mini5.
I then saw a post about this and someone suggested to use the ISP method for programming which I set up. Everything downloaded and there was only about a 50 ms delay and the Mini 5 linked up and worked fine.
So my questions are:

1. Why did the first 2 Mini5 modules not have the 400ms delay out of the box. And I used the USB cable for programming.
2. Why did the first 2 Mini 5 modules need the Arduino Uno Mini as the board selection and not the Arduino Pro Mini option.
3. Why did the subsequent 3 Mini 5 modules have a significant start up delay out of the box.
4. What is different between these two orders of Mini 5s?

Thank you for helping me with these questions.
Steve
True Air Speed Sensor for Stall Warning Block Diagram.pdf

After much research, and careful experimentation, I still cannot reconcile the values displayed by AVRdude for the fuse and lock bytes of my Sparkfun Pro Mini against the Atmel datasheet for the ATmega328P microprocessor. I read the values of the fuse and lock bytes using the following command:

...\avrdude.exe" -C"C:\Users\Bob2\Documents\Ardiuno_IDE-1.8.19_Portable\hardware\tools\avr\etc\avrdude.conf" -patmega328p -carduino -PCOM3 -b57600 -u -Ulfuse:r:lfuse.hex:i -Uhfuse:r:hfuse.hex:i -Uefuse:r:efuse.hex:i -Ulock:r:lock.hex:i -v -D

The resulting output for every one of the fuse bytes and the lock byte as well are all zero (0x00). Yet the Atmel documentation indicates that there should be several non-zero bits within these bytes.
So my question is this: am I doing something wrong when reading these bytes, or is there a reason why they are all actually zero?

I believe Pin 10, "PORT" identifier "PCB2" to be incorrect. It should be "PB2".