[PATCH v3 2/2] spl: mmc: Pass eMMC HW partition 7 through

[Marek Vasut]

On 4/5/23 00:16, Pali Rohár wrote:
> On Wednesday 05 April 2023 00:11:17 Marek Vasut wrote:
>> On 4/4/23 21:25, Pali Rohár wrote:
>>> On Tuesday 04 April 2023 20:05:15 Marek Vasut wrote:
>>>> The eMMC HW partition 0 and 7 both mean USER HW partition.
>>>
>>> This is not truth!
>>
>> Can you please provide further details to back your claim ?
> 
> Yes, see a patch with explanation which I meanwhile sent:
> https://patchwork.ozlabs.org/project/uboot/patch/20230404202805.8523-1-pali@kernel.org/

This very much adds a comment that looks like the content of the second 
paragraph of my reply, see below.

It is a good thing you mention the aforementioned patch, since exactly 
one line past the changes you implemented is the following piece of code:

https://source.denx.de/u-boot/u-boot/-/blob/master/common/spl/spl_mmc.c#L382

"
int default_spl_mmc_emmc_boot_partition(struct mmc *mmc)
...
part = (mmc->part_config >> 3) & PART_ACCESS_MASK;
if (part == 7)
     part = 0;
"

Which maps 7 to 0 .

>> This kind of screaming feedback with zero additional information is
>> worthless, sorry. Before you proceed with any further replies, please have a
>> look at JEDEC JESD84-B51 section 7.4.69 PARTITION_CONFIG Bit[5:3]
>> BOOT_PARTITION_ENABLE , this is what is being discussed here. Value 1/2 is
>> either BOOT HW partition, value 0/7 is treated as USER HW partition by the
>> boot code.

Please read the paragraph above, I do not see any reply to it and I 
think that might put the conversation back on track.