[Buildroot] [PATCH 1/1] package/mmc-utils: bump to version dfc3b6ecda84d21418fb4408b39c5c71db4c6458

[Sébastien Szymanski]

Hi Guilio,

On 10/12/22 17:38, Giulio Benetti wrote:
> Hi Sébastien,
> 
> On 12/10/22 16:43, Sébastien Szymanski wrote:
>> Drop first patch, upstreamed:
>> https://git.kernel.org/pub/scm/utils/mmc/mmc-utils.git/commit/?id=520f54eb94845e597837ae908a03b91ae8c8758f
>>
>> Fixes: 
>> http://autobuild.buildroot.net/results/01f78081a50743d1faa50a090c00b7e90a17fa39
> 
> With this bump my patch becomes useless since my patch has been 
> upstreamed too:
> https://git.kernel.org/pub/scm/utils/mmc/mmc-utils.git/commit/?id=5086e7c0de4d0094f8674368a88d931b27589d53
> and so we can drop it:
> https://patchwork.ozlabs.org/project/buildroot/patch/20220918163147.1133842-1-giulio.benetti@benettiengineering.com/
> 

Your patch has never been merged into Buildroot. (Otherwise there would 
be no build failure ;))

> But there is an error arising while building with sourcery-arm using
> $ buildroot/utils/test-pkg -p mmc-utils
> ```
> mmc_cmds.c: In function 'erase':
> mmc_cmds.c:2633:31: error: invalid application of 'sizeof' to incomplete 
> type 'struct mmc_ioc_multi_c
>    multi_cmd = calloc(1, sizeof(struct mmc_ioc_multi_cmd) +
>                                 ^
> mmc_cmds.c:2640:11: error: dereferencing pointer to incomplete type
>    multi_cmd->num_of_cmds = 3;
>             ^
> mmc_cmds.c:2642:11: error: dereferencing pointer to incomplete type
>    multi_cmd->cmds[0].opcode = MMC_ERASE_GROUP_START;
>             ^
> mmc_cmds.c:2643:11: error: dereferencing pointer to incomplete type
>    multi_cmd->cmds[0].arg = start;
>             ^
> mmc_cmds.c:2644:11: error: dereferencing pointer to incomplete type
>    multi_cmd->cmds[0].flags = MMC_RSP_SPI_R1 | MMC_RSP_R1 | MMC_CMD_AC;
>             ^
> ...
> ```
> 
> Have you given a try with that script? Could you please provide a patch
> to fix that bug too? Or I can provide it later.
I did now (and just found a bug in it ;))

The fix is that we need to have mmc-utils depend on the right 
linux-headers (>= 4.4), isn't it ?

Best regards,

> 
> Thank you
> 
> Best regards

-- 
Sébastien Szymanski, Armadeus Systems
Software engineer