[U-Boot] [PATCH 09/15] iMX28: Add GPMI NAND driver

Scott Wood
2011-09-28T22:23:59Z

On 09/28/2011 05:12 PM, Marek Vasut wrote:
> On Wednesday, September 28, 2011 11:26:45 PM Scott Wood wrote:
>> On 09/11/2011 11:06 PM, Marek Vasut wrote:
>>> +static void mxs_nand_return_dma_descs(struct mxs_nand_info *info)
>>> +{
>>> + int i = info->desc_index;
>>> + struct mxs_dma_desc *desc;
>>> +
>>> + for (--i; i >= 0; i--) {
>>
>> This is an awkward construct.
>>
>> Why not just the usual:
>>
>> for (i = 0; i < info->desc_index; i++)
>>
>
> Checking the driver, this modifies the global data so this is ok

I don't follow... I'm assuming by "this" you mean the original code
since you didn't change it, so are you saying that the global data has
to be modified in descending order?

If that's really a requirement, put in a comment saying so (and ideally
why), and you could still just do:

for (i = info->desc_index - 1; i >= 0; i--)

or

while (--i >= 0)

-Scott