Messages in this thread | | | Date | Wed, 26 Aug 2015 18:10:05 -0700 | From | Stefan Agner <> | Subject | Re: [PATCH v10 1/5] mtd: nand: vf610_nfc: Freescale NFC for VF610, MPC5125 and others |
| |
On 2015-08-25 13:34, Brian Norris wrote: > One more thing... > > On Mon, Aug 03, 2015 at 11:27:26AM +0200, Stefan Agner wrote: >> --- /dev/null >> +++ b/drivers/mtd/nand/vf610_nfc.c >> @@ -0,0 +1,645 @@ > ... >> +struct vf610_nfc { >> + struct mtd_info mtd; >> + struct nand_chip chip; >> + struct device *dev; >> + void __iomem *regs; >> + struct completion cmd_done; >> + uint buf_offset; >> + int page_sz; > > AFAICT (even with the 2nd patch), you never really use this field. You > just set it/increment it, but don't use it for anything. Kill it?
It is used in the write path, I think I meant to use it for subpage writes, when I thought it would just mean to transfer only parts of the page to the controller.
However, as the subpage discussion basically concluded in not using it for now on this controller, we can as well transfer the complete page (page_sz). Or is there another case in which vf610_nfc_write_buf could be called with less than page_sz?
-- Stefan
| |