|A diagram can only tell so much.|
Thomas if it only provides IDE it is missing a lot of what Gayle did.(this isn't meant negative)
Let's keep it to west side or amiga compatible IDE controller.
Unless it also supports PCMCIA cards and other goodies. ;)
Okay i'll list what should be improved if the diagram should be revised.
OPA, I can deduct something like operational amplifier out that others might not.
1. add a small list with a short explanation for yourself and other developers, what each block does this can also prevent and solve missunderstandings like this, legenda to the colour code. (IMHO, Documentation is a very important part, but please do not deviate from the project because i am a document fanatic)
2. don't leave room for asumptions, especialy when you expect that third party developers might interface to that specific part.
things you did right in the diagram.
1. you noted the number of pins for the header, specified connectors. (although, LAN does not specify the RJ45 connector, PCI 5V or 3.3V ?)
2. clear layout, this means the overal overview is great.
3. colour coded.
I cannot stress the point enough how important good documentation is.