summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorGovindraj Raja <govindraj.raja@arm.com>2023-10-12 17:31:41 -0500
committerGovindraj Raja <govindraj.raja@arm.com>2023-10-27 08:31:54 -0500
commit0f23e7eb1ee9a6a400e8e35bd8403f657b6405ae (patch)
tree0572ee1d32a424186c6af0deb598eb175d6b0346
parentad2dd65871b4411c735271f98a4fa5102abb2a00 (diff)
refactor(partition): convert warn to verbose
Convert all warn messages to verbose messages. As most warning are needed during debug only and and won't increase the binary size by default. Change-Id: Icc5d5157f13507ccbc34675c20357117cad98255 Signed-off-by: Govindraj Raja <govindraj.raja@arm.com>
-rw-r--r--drivers/partition/partition.c62
1 files changed, 31 insertions, 31 deletions
diff --git a/drivers/partition/partition.c b/drivers/partition/partition.c
index cd7ffadc9..eff5c57cd 100644
--- a/drivers/partition/partition.c
+++ b/drivers/partition/partition.c
@@ -56,32 +56,32 @@ static int load_mbr_header(uintptr_t image_handle, mbr_entry_t *mbr_entry)
/* MBR partition table is in LBA0. */
result = io_seek(image_handle, IO_SEEK_SET, MBR_OFFSET);
if (result != 0) {
- WARN("Failed to seek (%i)\n", result);
+ VERBOSE("Failed to seek (%i)\n", result);
return result;
}
result = io_read(image_handle, (uintptr_t)&mbr_sector,
PLAT_PARTITION_BLOCK_SIZE, &bytes_read);
if ((result != 0) || (bytes_read != PLAT_PARTITION_BLOCK_SIZE)) {
- WARN("Failed to read data (%i)\n", result);
+ VERBOSE("Failed to read data (%i)\n", result);
return result;
}
/* Check MBR boot signature. */
if ((mbr_sector[LEGACY_PARTITION_BLOCK_SIZE - 2] != MBR_SIGNATURE_FIRST) ||
(mbr_sector[LEGACY_PARTITION_BLOCK_SIZE - 1] != MBR_SIGNATURE_SECOND)) {
- WARN("MBR boot signature failure\n");
+ VERBOSE("MBR boot signature failure\n");
return -ENOENT;
}
tmp = (mbr_entry_t *)(&mbr_sector[MBR_PRIMARY_ENTRY_OFFSET]);
if (tmp->first_lba != 1) {
- WARN("MBR header may have an invalid first LBA\n");
+ VERBOSE("MBR header may have an invalid first LBA\n");
return -EINVAL;
}
if ((tmp->sector_nums == 0) || (tmp->sector_nums == UINT32_MAX)) {
- WARN("MBR header entry has an invalid number of sectors\n");
+ VERBOSE("MBR header entry has an invalid number of sectors\n");
return -EINVAL;
}
@@ -103,21 +103,21 @@ static int load_gpt_header(uintptr_t image_handle, size_t header_offset,
result = io_seek(image_handle, IO_SEEK_SET, header_offset);
if (result != 0) {
- WARN("Failed to seek into the GPT image at offset (%zu)\n",
- header_offset);
+ VERBOSE("Failed to seek into the GPT image at offset (%zu)\n",
+ header_offset);
return result;
}
result = io_read(image_handle, (uintptr_t)&header,
sizeof(gpt_header_t), &bytes_read);
if ((result != 0) || (sizeof(gpt_header_t) != bytes_read)) {
- WARN("GPT header read error(%i) or read mismatch occurred"
- "expected(%zu) and actual(%zu)\n", result,
- sizeof(gpt_header_t), bytes_read);
+ VERBOSE("GPT header read error(%i) or read mismatch occurred,"
+ "expected(%zu) and actual(%zu)\n", result,
+ sizeof(gpt_header_t), bytes_read);
return result;
}
if (memcmp(header.signature, GPT_SIGNATURE,
sizeof(header.signature)) != 0) {
- WARN("GPT header signature failure\n");
+ VERBOSE("GPT header signature failure\n");
return -EINVAL;
}
@@ -162,20 +162,20 @@ static int load_mbr_entry(uintptr_t image_handle, mbr_entry_t *mbr_entry,
/* MBR partition table is in LBA0. */
result = io_seek(image_handle, IO_SEEK_SET, MBR_OFFSET);
if (result != 0) {
- WARN("Failed to seek (%i)\n", result);
+ VERBOSE("Failed to seek (%i)\n", result);
return result;
}
result = io_read(image_handle, (uintptr_t)&mbr_sector,
PLAT_PARTITION_BLOCK_SIZE, &bytes_read);
if (result != 0) {
- WARN("Failed to read data (%i)\n", result);
+ VERBOSE("Failed to read data (%i)\n", result);
return result;
}
/* Check MBR boot signature. */
if ((mbr_sector[LEGACY_PARTITION_BLOCK_SIZE - 2] != MBR_SIGNATURE_FIRST) ||
(mbr_sector[LEGACY_PARTITION_BLOCK_SIZE - 1] != MBR_SIGNATURE_SECOND)) {
- WARN("MBR Entry boot signature failure\n");
+ VERBOSE("MBR Entry boot signature failure\n");
return -ENOENT;
}
offset = (uintptr_t)&mbr_sector +
@@ -218,9 +218,9 @@ static int load_gpt_entry(uintptr_t image_handle, gpt_entry_t *entry)
result = io_read(image_handle, (uintptr_t)entry, sizeof(gpt_entry_t),
&bytes_read);
if ((result != 0) || (sizeof(gpt_entry_t) != bytes_read)) {
- WARN("GPT Entry read error(%i) or read mismatch occurred"
- "expected(%zu) and actual(%zu)\n", result,
- sizeof(gpt_entry_t), bytes_read);
+ VERBOSE("GPT Entry read error(%i) or read mismatch occurred,"
+ "expected(%zu) and actual(%zu)\n", result,
+ sizeof(gpt_entry_t), bytes_read);
return -EINVAL;
}
@@ -240,16 +240,16 @@ static int load_partition_gpt(uintptr_t image_handle,
result = io_seek(image_handle, IO_SEEK_SET, gpt_entry_offset);
if (result != 0) {
- WARN("Failed to seek (%i), Failed loading GPT partition"
- "table entries\n", result);
+ VERBOSE("Failed to seek (%i), Failed loading GPT partition"
+ "table entries\n", result);
return result;
}
for (i = 0; i < list.entry_count; i++) {
result = load_gpt_entry(image_handle, &entry);
if (result != 0) {
- WARN("Failed to load gpt entry data(%i) error is (%i)\n",
- i, result);
+ VERBOSE("Failed to load gpt entry data(%i) error is (%i)\n",
+ i, result);
return result;
}
@@ -259,7 +259,7 @@ static int load_partition_gpt(uintptr_t image_handle,
}
}
if (i == 0) {
- WARN("No Valid GPT Entries found\n");
+ VERBOSE("No Valid GPT Entries found\n");
return -EINVAL;
}
/*
@@ -287,8 +287,8 @@ static int load_backup_gpt(unsigned int image_id, unsigned int sector_nums)
result = plat_get_image_source(image_id, &dev_handle, &image_spec);
if (result != 0) {
- WARN("Failed to obtain reference to image id=%u (%i)\n",
- image_id, result);
+ VERBOSE("Failed to obtain reference to image id=%u (%i)\n",
+ image_id, result);
return result;
}
@@ -309,7 +309,7 @@ static int load_backup_gpt(unsigned int image_id, unsigned int sector_nums)
result = io_open(dev_handle, image_spec, &image_handle);
if (result != 0) {
- WARN("Failed to access image id (%i)\n", result);
+ VERBOSE("Failed to access image id (%i)\n", result);
return result;
}
@@ -349,8 +349,8 @@ static int load_primary_gpt(uintptr_t image_handle, unsigned int first_lba)
gpt_header_offset = LBA(first_lba);
result = load_gpt_header(image_handle, gpt_header_offset, &part_lba);
if ((result != 0) || (part_lba == 0)) {
- WARN("Failed to retrieve Primary GPT header,"
- "trying to retrieve back-up GPT header\n");
+ VERBOSE("Failed to retrieve Primary GPT header,"
+ "trying to retrieve back-up GPT header\n");
return result;
}
@@ -368,20 +368,20 @@ int load_partition_table(unsigned int image_id)
result = plat_get_image_source(image_id, &dev_handle, &image_spec);
if (result != 0) {
- WARN("Failed to obtain reference to image id=%u (%i)\n",
- image_id, result);
+ VERBOSE("Failed to obtain reference to image id=%u (%i)\n",
+ image_id, result);
return result;
}
result = io_open(dev_handle, image_spec, &image_handle);
if (result != 0) {
- WARN("Failed to access image id=%u (%i)\n", image_id, result);
+ VERBOSE("Failed to access image id=%u (%i)\n", image_id, result);
return result;
}
result = load_mbr_header(image_handle, &mbr_entry);
if (result != 0) {
- WARN("Failed to access image id=%u (%i)\n", image_id, result);
+ VERBOSE("Failed to access image id=%u (%i)\n", image_id, result);
goto out;
}
if (mbr_entry.type == PARTITION_TYPE_GPT) {