Re: [PATCH 2/8] v3 Add new phys_index properties

From: KAMEZAWA Hiroyuki
Date: Tue Jul 20 2010 - 03:02:36 EST


On Mon, 19 Jul 2010 22:52:50 -0500
Nathan Fontenot <nfont@xxxxxxxxxxxxxx> wrote:

> Update the 'phys_index' properties of a memory block to include a
> 'start_phys_index' which is the same as the current 'phys_index' property.
> This also adds an 'end_phys_index' property to indicate the id of the
> last section in th memory block.
>
> Signed-off-by: Nathan Fontenot <nfont@xxxxxxxxxxxxxx>

No, please remain "phys_index" as it is. please don't rename it.
IMHO, just adding end_phys_index is better.
please avoid interface change AFAP.

Do you have a problem if phys_index means start_phys_index ?

Thanks,
-Kame

--
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/