On Fri, Aug 10, 2018 at 1:27 PM, Mario Six <mario....@gdsys.cc> wrote: > Hi Christophe, > > On Fri, Aug 10, 2018 at 9:35 AM Christophe LEROY > <christophe.le...@c-s.fr> wrote: >> >> Hello Mario, >> >> Le 26/04/2018 à 10:36, Mario Six a écrit : >> > Hi Joakim, >> > >> > On Thu, Apr 26, 2018 at 10:23 AM, Joakim Tjernlund >> > <joakim.tjernl...@infinera.com> wrote: >> >> On Thu, 2018-04-26 at 11:35 +0530, Jagan Teki wrote: >> >>> CAUTION: This email originated from outside of the organization. Do not >> >>> click links or open attachments unless you recognize the sender and know >> >>> the content is safe. >> >>> >> >>> >> >>> On Thu, Apr 26, 2018 at 11:24 AM, Mario Six <mario....@gdsys.cc> wrote: >> >>>> Hi Jagan, >> >>>> >> >>>> On Thu, Apr 26, 2018 at 7:30 AM, Jagan Teki <jagannadh.t...@gmail.com> >> >>>> wrote: >> >>>>> On Thu, Apr 19, 2018 at 6:06 PM, Mario Six <mario....@gdsys.cc> wrote: >> >>>>>> This is v2 of a patch series that adds support for DM to the MPC8XXX >> >>>>>> SPI >> >>>>>> driver, cleans up the driver code, fixes a few minor problems. >> >>>>>> >> >>>>>> Some TODOs are left over for later, such as proper SPI speed setting, >> >>>>>> and support for SPI mode setting. These would be enhancements to the >> >>>>>> original functionality, and can come later. >> >>>>>> >> >>>>>> The legacy functionality is removed in this version, so old boards in >> >>>>>> the tree might end up with broken SPI functionality. >> >>>>>> >> >>>>>> Mario Six (18): >> >>>>>> spi: mpc8xxx: Use short type names >> >>>>>> spi: mpc8xxx: Fix comments >> >>>>>> spi: mpc8xxx: Rename camel-case variables >> >>>>>> spi: mpc8xxx: Fix space after cast >> >>>>>> spi: mpc8xxx: Fix function names in strings >> >>>>>> spi: mpc8xxx: Replace defines with enums >> >>>>>> spi: mpc8xxx: Use IO accessors >> >>>>>> spi: mpc8xxx: Simplify if >> >>>>>> spi: mpc8xxx: Get rid of is_read >> >>>>>> spi: mpc8xxx: Simplify logic a bit >> >>>>>> spi: mpc8xxx: Reduce scope of loop variables >> >>>>>> spi: mpc8xxx: Make code more readable >> >>>>>> spi: mpc8xxx: Rename variable >> >>>>>> spi: mpc8xxx: Document LEN setting better >> >>>>>> spi: mpc8xxx: Re-order transfer setup >> >>>>>> spi: mpc8xxx: Fix if check >> >>>>>> spi: mpc8xxx: Use get_timer >> >>>>>> spi: mpc8xxx: Convert to DM >> >>>>> >> >>>>> Boards with >> >>>>> - configs/MPC8349EMDS_defconfig >> >>>>> - configs/ids8313_defconfig >> >>>>> >> >>>>> are using this driver, so Kim, Heiko please convert enable DM_SPI for >> >>>>> the same. >> >>>>> >> >>>>> Use below tree for respective changes and update on top of this. >> >>>>> http://git.denx.de/?p=u-boot-spi.git;a=shortlog;h=refs/heads/next >> >>>>> >> >>>> >> >>>> I have a few series in the making that will enable DM on the MPC83xx >> >>>> platform >> >>>> (I'm doing a respin on the first right now). If there is still >> >>>> interests in the >> >>>> boards, I could push it to the MPC83xx repository (but mind that the >> >>>> work >> >>>> required per board is quite extensive). >> >>>> >> >>>> Also, MPC8349EMDS is de facto abandoned, and I don't have access to the >> >>>> hardware, so I can't really maintain it. >> >>> >> >>> It's up to you, look like this board maintained by Kim is not >> >>> available with freescale e-mail (or may be changed) if none can't >> >>> maintain, it better to drop the board. >> >> >> >> we use custom 832x boards so please don't remove 83xx from u-boot. >> >> >> > >> > I'm not planning to do that; on the contrary: I'm trying to update the >> > platform >> > to fully support DM (I hope to get a fully converted board in after the >> > next >> > release). >> > >> > The problem is that we only use MPC8308 SoCs, so I can only vouche for the >> > correctness of that specific SoC. Everything else is a bit up in the air, >> > since >> > I'm changing code blindly pretty much. >> >> I have a MPC8321 board so I may test it on it if it helps. >> > > That would be very much appreciated! Thanks. > >> In the meantime, I was thinking about using your converted driver and >> see if I can adapt it to support MPC8xx as well, instead of converting >> the mpc8xx_spi driver to DM, however I've not been able to find your >> patches in the master tree allthough they are flagged as accepted in >> patchwork. >> >> Are they on another branch somewhere ? >> > Hmm, indeed. I thought they would be in the SPI custodian repository, but > apparently they are not? > > @Jagan: Were those patches forgotten somehow?
http://git.denx.de/?p=u-boot-spi.git;a=shortlog;h=refs/heads/spi-dm-migrate If all the boards which are using this driver enabled DM_SPI, then I will pick the same. _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot