[PATCH] mtd: add a new macro about the subpage write
Marek Vasut
marex at denx.de
Fri Jul 13 12:28:31 EDT 2012
Dear Huang Shijie,
> On Fri, Jul 13, 2012 at 11:40 AM, Marek Vasut <marex at denx.de> wrote:
> > Dear Huang Shijie,
> >
> >> On Fri, Jul 13, 2012 at 6:35 AM, Marek Vasut <marex at denx.de> wrote:
> >> > Dear Huang Shijie,
> >> >
> >> >> Hi Brian:
> >> >> > Wood [1]; I don't see a good reason not to just kill the
> >> >> > NAND_CHIPOPTIONS_MSK instead of adding more flags. As long as we
> >> >> > perform a few sanity tests, I think it'd be safe.
> >> >>
> >> >> I think it's more clear in logic to add this new macro:
> >> >> The NAND_NO_SUBPAGE_WRITE can be used only by the MLC nands which
> >> >> do
> >> >>
> >> >> no support the subpage write;
> >> >>
> >> >> The NAND_CONTROLLER_NO_SUBPAGE_WRITE only used by the nand
> >> >>
> >> >> controller such as gpmi nand.
> >> >
> >> > It's not clearer at all. It's just more error-prone.
> >>
> >> ok, thanks. But I do not how to fix it now. I hope some one could give a
> >> patch.
> >
> > Why not remove the mask?
>
> I do not understand why this line was added here, was it added on purpose?
> so I am not sure whether we can just remove this line.
Investigate, you're the author and you have an unresolved bug sitting there for
a while ;-)
Check where the masked values are used and if they has to be masked at all, if
it can't be done in some easier way.
> Best Regards
> Huang Shijie
Best regards,
Marek Vasut
More information about the linux-mtd
mailing list