[U-Boot] [PATCH v2 00/18] spi: mpc8xxx: DM conversion

[Christophe LEROY]

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.Tjernlund at 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.six at gdsys.cc> wrote:
>>>> Hi Jagan,
>>>>
>>>> On Thu, Apr 26, 2018 at 7:30 AM, Jagan Teki <jagannadh.teki at gmail.com> wrote:
>>>>> On Thu, Apr 19, 2018 at 6:06 PM, Mario Six <mario.six at 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.

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 ?

Thanks
Christophe

> 
> Best regards,
> Mario
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> https://lists.denx.de/listinfo/u-boot
>