[U-Boot] [PATCH v2 2/2] mtd: nand: mxc_nand: Fix is_16bit_nand()

Fabio Estevam festevam at gmail.com
Tue Feb 26 20:11:12 CET 2013


On Tue, Feb 26, 2013 at 3:47 PM, Benoît Thébaudeau
<benoit.thebaudeau at advansee.com> wrote:

> That's correct.
>
> is_16bit_nand() is only used to conditionally set NAND_BUSWIDTH_16 in
> this->options, then this flag is tested everywhere else, so we could also
> completely drop is_16bit_nand() and just do:
> #ifdef CONFIG_SYS_NAND_BUSWIDTH_16BIT
>         this->options |= NAND_BUSWIDTH_16;
> #endif

Yes, this could be done.

>
> What do you think is better? I'm fine with your implementation.

My main goal with this patch series was to fix a bug, and I think I
can do as you suggest, and get rid of is_16bit_nand() in a future
patch.

> Should I rebase my series on that, keeping both series separate, or should I
> integrate these 2 patches as is at the beginning of my series?

I would say that you could send this series as is at the beginning of
your next version in order to avoid conflicts.

Thanks,

Fabio Estevam


More information about the U-Boot mailing list