[bug] generate_registers.py generates an invalid uhal address table for the BLASTER RAM

[jsturdy]

The uhal generated address table has a typo:

<node id="GBT" address="0x0" permission="rw" mode="block"size="3312"/>

which should be (space between the " closing the mode and the next key size):

<node id="GBT" address="0x0" permission="rw" mode="block" size="3312"/>

[jsturdy]

@evka85, as mentioned in person, the problem is here:

GEM_AMC/scripts/generate_registers.py

Lines 634 to 641 in 4fe3966

	if node.permission is not None:
	file.write('permission="%s" ' % node.permission)
	if node.mask is not None:
	file.write('mask="%s"' % hex(node.mask))
	if node.mode is not None:
	file.write('mode="%s"' % node.mode)
	if node.size is not None:
	file.write('size="%s"' % node.size)

where there are no leading/trailing spaces for mask/mode/size

[jsturdy]

@evka85 this needs to be fixed before the next FW release

[evka85]

fixed in v3.8.4

[jsturdy]

Not fixed... see cms-gem-daq-project/cmsgemos#292 (or incorrect address tables attached to release)