2004-04-20  Theodore A. Roth  <address@hidden>

	* configure.in (AM_INIT_AUTOMAKE): Bump version.

	[This change should fix bug #8391]
	* NEWS: Add note about this bug fix.
	* include/avr/boot.h (__BOOT_LOCK_BITS_MASK): Remove.
	(__boot_lock_bits_set): Use the compliment of lock_bits arg so the user
	specifies only which bits are to be set. NOTE that this changes the
	usage but the previous usage was confusing and not well documented.
	(__boot_lock_bits_set_alternate): Ditto.
	(boot_lock_bits_set): Improve documentation.

Index: NEWS
===================================================================
RCS file: /cvsroot/avr-libc/avr-libc/NEWS,v
retrieving revision 1.17.2.13
diff -u -p -p -r1.17.2.13 NEWS
--- NEWS	17 Apr 2004 20:56:33 -0000	1.17.2.13
+++ NEWS	20 Apr 2004 18:40:31 -0000
@@ -22,6 +22,9 @@ See ChangeLog for details.
           existing code, but the OCR1A and OCR1B names where just plain wrong
           so they have been changed and the incorrect names (OCRA1 and OCRB1) 
           are undefined.
+  [#8391] Fuse mask in boot.h wrong?
+          NOTE: boot_lock_bits_set() was broken. Please see the new
+          documentation for examples of how the new version works.
   [#8119] _wdt_write macro in wdt.h doesn't work with ATmega169
 
 *** Changes in avr-libc-1.0.3:
Index: include/avr/boot.h
===================================================================
RCS file: /cvsroot/avr-libc/avr-libc/include/avr/boot.h,v
retrieving revision 1.4.2.6
diff -u -p -p -r1.4.2.6 boot.h
--- include/avr/boot.h	9 Mar 2004 01:14:45 -0000	1.4.2.6
+++ include/avr/boot.h	20 Apr 2004 18:40:31 -0000
@@ -182,9 +182,6 @@
 #define __BOOT_RWW_ENABLE         (_BV(SPMEN) | _BV(__COMMON_ASRE))
 #define __BOOT_LOCK_BITS_SET      (_BV(SPMEN) | _BV(BLBSET))
 
-#define __BOOT_LOCK_BITS_MASK     (_BV(BLB01) | _BV(BLB02) \
-                                   | _BV(BLB11) | _BV(BLB12))
-
 #define __boot_page_fill_normal(address, data)   \
 ({                                               \
     boot_spm_busy_wait();                        \
@@ -378,9 +375,26 @@
     );                                           \
 })
 
+/* From the mega16/mega128 data sheets (maybe others):
+
+     Bits by SPM To set the Boot Loader Lock bits, write the desired data to
+     R0, write "X0001001" to SPMCR and execute SPM within four clock cycles
+     after writing SPMCR. The only accessible Lock bits are the Boot Lock bits
+     that may prevent the Application and Boot Loader section from any
+     software update by the MCU.
+
+     If bits 5..2 in R0 are cleared (zero), the corresponding Boot Lock bit
+     will be programmed if an SPM instruction is executed within four cycles
+     after BLBSET and SPMEN are set in SPMCR. The Z-pointer is don t care
+     during this operation, but for future compatibility it is recommended to
+     load the Z-pointer with $0001 (same as used for reading the Lock
+     bits). For future compatibility It is also recommended to set bits 7, 6,
+     1, and 0 in R0 to 1 when writing the Lock bits. When programming the Lock
+     bits the entire Flash can be read during the operation. */
+
 #define __boot_lock_bits_set(lock_bits)                    \
 ({                                                         \
-    uint8_t value = (uint8_t)(lock_bits | __BOOT_LOCK_BITS_MASK); \
+    uint8_t value = (uint8_t)(~(lock_bits));               \
     boot_spm_busy_wait();                                  \
     eeprom_busy_wait();                                    \
     __asm__ __volatile__                                   \
@@ -399,7 +413,7 @@
 
 #define __boot_lock_bits_set_alternate(lock_bits)          \
 ({                                                         \
-    uint8_t value = (uint8_t)(lock_bits | __BOOT_LOCK_BITS_MASK); \
+    uint8_t value = (uint8_t)(~(lock_bits));               \
     boot_spm_busy_wait();                                  \
     eeprom_busy_wait();                                    \
     __asm__ __volatile__                                   \
@@ -453,7 +467,24 @@
 /** \ingroup avr_boot
     \def boot_lock_bits_set(lock_bits)
 
-    Set the bootloader lock bits. */
+    Set the bootloader lock bits.
+
+    \param lock_bits A mask of which Boot Loader Lock Bits to set.
+
+    \note In this context, a 'set bit' will be written to a zero value.
+
+    For example, to disallow the SPM instruction from writing to the Boot
+    Loader memory section of flash, you would do this macro as such:
+
+    \code
+    boot_lock_bits_set (_BV (BLB12));
+    \endcode
+
+    And to remove any SPM restrictions, you would do this:
+
+    \code
+    boot_lock_bits_set (0);
+    \endcode */
 
 /* Normal versions of the macros use 16-bit addresses.
    Extended versions of the macros use 32-bit addresses.