Hello Larry, On Sat, Mar 11, 2023 at 3:16 AM Larry Finger <Larry.Finger@xxxxxxxxxxxx> wrote: [...] > I am not qualified to review the code, but I am integrating this version into my > rtw88 repo at GitHub.com. > > It is essential that a successful build is possible after every patch is applied > so that an arbitrary bisection will not fail to build. This patch series fails > after #2 is committed. File mac.c needs symbol SDIO_LOCAL_OFFSET, which was > moved from mac.h to sdio.h. I resolved this be including sdio.h in mac.c. This > breaks #3, where you add the include to mac.c. It needs to happen one patch earlier. Thank you for spotting and reporting this issue! You are right with this, I'll add the sdio.h include to mac.c with patch 2 to resolve this issue as you suggested. > The other problem for my repo is that it cannot modify > include/linux/mmc/sdio_ids.h, thus I have to create a local sdio_ids.h to > contain the new definitions. Once your patches are in the kernel, I will be able > to eliminate this work around. You can also modify the three SDIO driver files (rtw8821cs.c, rtw8822bs.c, rtw8822cs.c) and use the literal IDs there if you have to patch those files anyways. > I do not have any rtw88 SDIO devices, thus I will not be able to test, but I > will pass any information that I get from my users. That sounds great - thank you! Best regards, Martin
