On 08/03/2010 12:01 PM, zeal wrote:
  This patch add the below commands (just for determining stuff).

  1. read_bits for reading Fuse/Lock bits.
  2. read_byte for reading Signature/Calibration byte.
This looks good - however, I have issues with the command names: both names do not represent the function of the commands: - read_bits: it reads & dispays the *fuse* bytes, in byte form - why "bits"? I would have expected "read_fuses" or something along that lines.

 - read_byte: which byte? There are lots of bytes that can be read ;-)
   "read_calibration" would indicate what the command really does.

cu
Michael

_______________________________________________
Openocd-development mailing list
[email protected]
https://lists.berlios.de/mailman/listinfo/openocd-development

Reply via email to