Hi,
> > { 'struct' : 'FirmwareMappingFlash',
> > - 'data' : { 'executable' : 'FirmwareFlashFile',
> > - 'nvram-template' : 'FirmwareFlashFile' } }
> > + 'data' : { 'mode': 'FirmwareFlashMode',
> > + 'executable' : 'FirmwareFlashFile',
> > + '*nvram-template' : 'FirmwareFlashFile' } }I think for backward compatibility reasons we want 'mode' be optional, with 'mode' = 'split' being assumed in case it is not present. otherwise looks sane to me. take care, Gerd
