On Mon, Jan 28, 2019 at 12:37:31PM -0800, Ajay Gupta wrote: > + /***************************************************************** > + * CCG firmware image (.cyacd) file line format > + * > + * :00rrrrllll[dd....]cc/r/n > + * > + * :00 header > + * rrrr is row number to flash (4 char) > + * llll is data len to flash (4 char) > + * dd is a data field represents one byte of data (512 char) > + * cc is checksum (2 char) > + * \r\n newline > + * > + * Total length: 3 + 4 + 4 + 512 + 2 + 2 = 527 > + * > + *****************************************************************/ Any reason you can't just use ihex for this so that you don't have to parse it all and you can just use the in-kernel functions for it? Why make a custom firmware image type? thanks, greg k-h