2012-05-20 11:42:32

by Joel Reardon

[permalink] [raw]
Subject: [patch] UBI: add in hex the value for UBI_INTERNAL_VOL_START to comment

Explicitly provide the first internal volume ID value in the comment for
UBI_INTERNAL_VOL_START. This allows developers who, when adding features
related to volume ids and observe unexpected very large volume ids, to grep
for the observed value in the source code and find out immediately that it is
expected behaviour.

Signed-off-by: Joel Reardon <[email protected]>
---
drivers/mtd/ubi/ubi-media.h | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/mtd/ubi/ubi-media.h b/drivers/mtd/ubi/ubi-media.h
index 6fb8ec2..9dab25d 100644
--- a/drivers/mtd/ubi/ubi-media.h
+++ b/drivers/mtd/ubi/ubi-media.h
@@ -298,8 +298,8 @@ struct ubi_vid_hdr {
#define UBI_INT_VOL_COUNT 1

/*
- * Starting ID of internal volumes. There is reserved room for 4096 internal
- * volumes.
+ * Starting ID of internal volumes: 0x7fffefff.
+ * There is reserved room for 4096 internal volumes.
*/
#define UBI_INTERNAL_VOL_START (0x7FFFFFFF - 4096)

--
1.7.5.4


2012-05-20 18:32:56

by Artem Bityutskiy

[permalink] [raw]
Subject: Re: [patch] UBI: add in hex the value for UBI_INTERNAL_VOL_START to comment

On Sun, 2012-05-20 at 13:42 +0200, Joel Reardon wrote:
> Explicitly provide the first internal volume ID value in the comment for
> UBI_INTERNAL_VOL_START. This allows developers who, when adding features
> related to volume ids and observe unexpected very large volume ids, to grep
> for the observed value in the source code and find out immediately that it is
> expected behaviour.
>
> Signed-off-by: Joel Reardon <[email protected]>

Pushed to linux-ubi.git, it is also in your branch, thanks!

--
Best Regards,
Artem Bityutskiy


Attachments:
signature.asc (836.00 B)
This is a digitally signed message part