Hi Heiko, On Fri, Jun 7, 2019 at 11:15 AM Heiko Schocher <h...@denx.de> wrote: > > Hello Mario, > > Am 07.06.2019 um 11:10 schrieb Mario Six: > > 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? > > If you can do it, that would be great! > Sure, one patch coming up...
> Many thanks! > > bye, > Heiko > Best regards, Mario _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot