The values for ADTS could use some more descriptive names.

[K4LCIFER]

Currently, for the atmega328p they're named as like val_0x00(), val_0x01, val_0x02, etc. Ideas for alternative names could be free_running, analog_comparator, external_interrupt, timer0_compare, timer0_overflow, timer1_compare, timer1_overflow, timer1_capture (See table 24-6 of the atmega328p datasheet).

[Rahix]

Yeah, the val_0x##() names are placeholders used by Atmel. We patch these manually and should probably also do it for this one.

The README describes where patches are added.

Here is one such patch that changes the names of enumerated values:

avr-device/patch/common/adc.yaml

Lines 4 to 14 in 816c800

	ADC:
	ADCSRA:
	ADPS:
	_replace_enum:
	PRESCALER_2: [1, "Prescaler Value 2"]
	PRESCALER_4: [2, "Prescaler Value 4"]
	PRESCALER_8: [3, "Prescaler Value 8"]
	PRESCALER_16: [4, "Prescaler Value 16"]
	PRESCALER_32: [5, "Prescaler Value 32"]
	PRESCALER_64: [6, "Prescaler Value 64"]
	PRESCALER_128: [7, "Prescaler Value 128"]