Messages in this thread | | | Date | Fri, 7 Jan 2022 16:23:49 -0500 | From | "Gabriel L. Somlo" <> | Subject | Re: [PATCH v7 3/3] mmc: Add driver for LiteX's LiteSDCard interface |
| |
On Fri, Jan 07, 2022 at 10:50:02PM +0200, Andy Shevchenko wrote: > > > + .of_match_table = of_match_ptr(litex_match), > > > > You said "Wrong usage of of_match_ptr()" here, and all I have to go by > > is a bunch of other `drivers/mmc/host/*.c` files that use it in a > > similar way, so can you please clarify and/or provide an example of how > > to do it properly? > > First of all, you have a dependency to OF, try to remove it and > compile with OF=n and you will immediately see the issue. You may also > go for `git log --no-merges --grep of_match_ptr` and analyze the > result.
Ah, grepping for "of_match_ptr" was *never* going to show me the "right way of using of_match_ptr()" :)
OTOH, grepping for "of_match_table" (and of course the commits returned by your command line) suggests that I simply should *not* use it at all, and assign its argument directly to .of_match_table.
Got it, fixed it, thanks!
Now, on to the probe ordering issue... :)
Thanks, --G
| |