Skip to content

Commit ad43f23

Browse files
author
Veijo Pesonen
committed
SFDP: lists unknown headers on Q/SPIF debug prints
1 parent 80f87e3 commit ad43f23

File tree

2 files changed

+8
-8
lines changed

2 files changed

+8
-8
lines changed

components/storage/blockdevice/COMPONENT_QSPIF/QSPIFBlockDevice.cpp

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -676,16 +676,16 @@ int QSPIFBlockDevice::_sfdp_parse_sfdp_headers(uint32_t &basic_table_addr, size_
676676
}
677677

678678
if ((param_header[0] == 0) && (param_header[7] == 0xFF)) {
679-
// Found Basic Params Table: LSB=0x00, MSB=0xFF
680-
tr_debug("Found Basic Param Table at Table: %d", i_ind + 1);
679+
tr_debug("Parameter Header %d: Basic Parameter Header", i_ind);
681680
basic_table_addr = ((param_header[6] << 16) | (param_header[5] << 8) | (param_header[4]));
682681
// Supporting up to 64 Bytes Table (16 DWORDS)
683682
basic_table_size = ((param_header[3] * 4) < SFDP_DEFAULT_BASIC_PARAMS_TABLE_SIZE_BYTES) ? (param_header[3] * 4) : 64;
684683
} else if ((param_header[0] == 0x81) && (param_header[7] == 0xFF)) {
685-
// Found Sector Map Table: LSB=0x81, MSB=0xFF
686-
tr_debug("Found Sector Map Table at Table: %d", i_ind + 1);
684+
tr_debug("Parameter Header %d: Sector Map Parameter Header", i_ind);
687685
sector_map_table_addr = ((param_header[6] << 16) | (param_header[5] << 8) | (param_header[4]));
688686
sector_map_table_size = param_header[3] * 4;
687+
} else {
688+
tr_debug("Parameter Header %d: Vendor specific or unknown header", i_ind);
689689
}
690690
addr += QSPIF_PARAM_HEADER_SIZE;
691691
}

components/storage/blockdevice/COMPONENT_SPIF/SPIFBlockDevice.cpp

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -771,18 +771,18 @@ int SPIFBlockDevice::_sfdp_parse_sfdp_headers(uint32_t &basic_table_addr, size_t
771771
}
772772

773773
if ((param_header[0] == 0) && (param_header[7] == 0xFF)) {
774-
// Found Basic Params Table: LSB=0x00, MSB=0xFF
775-
tr_debug("Found Basic Param Table at Table: %d", i_ind + 1);
774+
tr_debug("Parameter Header %d: Basic Parameter Header", i_ind);
776775
basic_table_addr = ((param_header[6] << 16) | (param_header[5] << 8) | (param_header[4]));
777776
// Supporting up to 64 Bytes Table (16 DWORDS)
778777
basic_table_size = ((param_header[3] * 4) < SFDP_DEFAULT_BASIC_PARAMS_TABLE_SIZE_BYTES) ? (param_header[3] * 4) : 64;
779778

780779
} else if ((param_header[0] == 0x81) && (param_header[7] == 0xFF)) {
781-
// Found Sector Map Table: LSB=0x81, MSB=0xFF
782-
tr_debug("Found Sector Map Table at Table: %d", i_ind + 1);
780+
tr_debug("Parameter Header %d: Sector Map Parameter Header", i_ind);
783781
sector_map_table_addr = ((param_header[6] << 16) | (param_header[5] << 8) | (param_header[4]));
784782
sector_map_table_size = param_header[3] * 4;
785783

784+
} else {
785+
tr_debug("Parameter Header %d: Vendor specific or unknown header", i_ind);
786786
}
787787
addr += SPIF_PARAM_HEADER_SIZE;
788788

0 commit comments

Comments
 (0)