Hi Heiko, On Fri, Jun 7, 2019 at 10:56 AM Heiko Schocher <h...@denx.de> wrote: > > Hello Mario, > > Am 07.06.2019 um 09:51 schrieb Mario Six: > > (adding Heiko, since he's the maintainer of the board in question) > > > > On Fri, May 24, 2019 at 9:34 PM Jagan Teki <ja...@amarulasolutions.com> > > wrote: > > Huch, missed this Email ... > > >> > >> Hi Mario, > >> > >> On Tue, May 21, 2019 at 4:42 PM Tom Rini <tr...@konsulko.com> wrote: > >>> > >>> On Mon, May 20, 2019 at 10:48:23PM +0530, Jagan Teki wrote: > >>> > >>>> Hi Tom, > >>>> > >>>> Please pull this PR. > >>>> > >>>> Summary: > >>>> - mpc8xxx spi driver fixes (Mario) > >>>> - mpc8xxx spi dm conversion (Mario, Jagan) > >>>> - SPI DM Migration update (Jagan) > >>>> > >>>> The following changes since commit > >>>> 748198cb8d32d41bc35e6f492bac9948f339bece: > >>>> > >>>> Revert "Makefile: Prioritize external dtb if defined" (2019-05-19 > >>>> 18:09:35 -0400) > >>>> > >>>> are available in the Git repository at: > >>>> > >>>> git://git.denx.de/u-boot-spi.git master > >>>> > >>>> for you to fetch changes up to 93842e580cee40c9d3e70de625e4bd826a749e65: > >>>> > >>>> dm: MIGRATION: Update migration status for SPI (2019-05-20 22:18:54 > >>>> +0530) > >>>> > >>> > >>> NAK: > >>> powerpc: + ids8313 > >>> +(ids8313) cmd/built-in.o: In function `do_spi_xfer': > >>> +(ids8313) build/../cmd/spi.c:53: undefined reference to `spi_setup_slave' > >>> +(ids8313) build/../cmd/spi.c:60: undefined reference to `spi_claim_bus' > >>> +(ids8313) build/../cmd/spi.c:63: undefined reference to `spi_xfer' > >>> +(ids8313) build/../cmd/spi.c:80: undefined reference to `spi_release_bus' > >>> +(ids8313) build/../cmd/spi.c:82: undefined reference to `spi_free_slave' > >>> +(ids8313) common/built-in.o: In function `jumptable_init': > >>> +(ids8313) build/../include/_exports.h:56: undefined reference to > >>> `spi_setup_slave' > >>> +(ids8313) build/../include/_exports.h:58: undefined reference to > >>> `spi_free_slave' > >>> +(ids8313) build/../include/_exports.h:65: undefined reference to > >>> `spi_claim_bus' > >>> +(ids8313) build/../include/_exports.h:66: undefined reference to > >>> `spi_release_bus' > >>> +(ids8313) build/../include/_exports.h:67: undefined reference to > >>> `spi_xfer' > >>> +(ids8313) make[1]: *** [Makefile:1552: u-boot] Error 1 > >>> +(ids8313) make: *** [Makefile:148: sub-make] Error 2 > >>> > >> > >> Any help? > > > > @Heiko: I'll post a minimal patch that starts DM conversion on ids8313; I > > used > > the mpc8313erdb tree from the Linux sources for now. No guarantees for > > functionality, but it does compile. > > Just saw this patch .. it is really a fast shot :-P > > I just wrote an EMail to the customer, if we should remove the board > from mainline (or convert it). > > > @Jagan: Could you rebase this series on the current master to pull in my > > MPC83xx DM updates, and then apply the previously mentioned ids8313 patch > > after > > Heiko gave his OK? > > As it blocks spi patches ... Isn,t it better we remove SPI functionality > from the ids8313 board instead of adding a DTS which does not really fit? > That would be the alternative, of course. If you do want to go forward with removing the board, then I'm all for disabling the SPI support until we know more, of course.
Do you want to prepare a patch for the SPI removal, or should I do it? > bye, > Heiko > Best regards, Mario _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
