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


Reply via email to