Re: [PATCH V4 5/9] mtd: replace the hardcode with the onfi_feature()

From: Huang Shijie
Date: Thu May 02 2013 - 02:46:24 EST


ä 2013å05æ02æ 14:17, Gupta, Pekon åé:
[Pekon]: onfi_feature() is actually not useful unless someone re-scans the
I do not think so. :)

I think the onfi_feature() is useful in the future.
I only add the two feauture for this helper:
[1] 16-bit and
[2] extended parameter page

But in actually, we may add more feature to this helper, such as _synchronous_.
For example, some driver may support the synchronous mode for the ONFI nand.
We can use this onfi_feature() in the driver to check if the onfi nand supports this
synchronous feature.

Add this helper makes the code more readable, though we introduce a little redandancy.


thanks
Huang Shijie



--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/