From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga03.intel.com ([134.134.136.65]) by bombadil.infradead.org with esmtp (Exim 4.80.1 #2 (Red Hat Linux)) id 1Xdg06-0001ws-3m for linux-mtd@lists.infradead.org; Mon, 13 Oct 2014 13:49:26 +0000 Message-ID: <1413208135.7906.24.camel@sauron.fi.intel.com> Subject: Re: [PATCH] nandsim: add id_bytes module parameter From: Artem Bityutskiy Reply-To: artem.bityutskiy@linux.intel.com To: Akinobu Mita Date: Mon, 13 Oct 2014 16:48:55 +0300 In-Reply-To: <1407713341-4446-1-git-send-email-akinobu.mita@gmail.com> References: <1407713341-4446-1-git-send-email-akinobu.mita@gmail.com> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Cc: David Woodhouse , Brian Norris , linux-mtd@lists.infradead.org List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Mon, 2014-08-11 at 08:29 +0900, Akinobu Mita wrote: > nandsim can simulate NAND Flash which returns the ID bytes specified > by first_id_byte, ..., fourth_id_byte module parameters. > > In order to simulate NAND flash which returns more than four ID bytes, > this adds id_bytes module parameter which is specified by the array of > byte like this: > > # modprobe nandsim id_bytes=0x98,0xdc,0x90,0x26,0x76,0x15,0x01,0x08 bch=1 > > This doesn't add fifth_id_byte, ..., seventh_id_byte module parameters, > becuase they are redundant. But the existing first_id_byte, ..., > fourth_id_byte module parameters are preserved. Hi, I missed this patch, sorry. It looks good to me, I'll take it to my tree. -- Best Regards, Artem Bityutskiy