[U-Boot] [PATCH 00/39] dm: sata: Complete driver-model support for SATA

[Simon Glass]

Hi Jaehoon,

On 30 July 2017 at 20:46, Jaehoon Chung <jh80.chung at samsung.com> wrote:
> Hi Simon,
>
> On 07/28/2017 11:40 PM, Simon Glass wrote:
>> Hi Jaehoon,
>>
>> On 28 July 2017 at 06:25, Jaehoon Chung <jh80.chung at samsung.com> wrote:
>>> Dear Simon,
>>>
>>> On 07/27/2017 11:43 AM, Jaehoon Chung wrote:
>>>> On 07/23/2017 02:15 AM, Simon Glass wrote:
>>>>> (pruning the cc list a bit)
>>>>>
>>>>> Hi Jaehoon,
>>>>>
>>>>> On 4 July 2017 at 13:48, Simon Glass <sjg at chromium.org> wrote:
>>>>>> This series completes the conversion of SATA to driver model.
>>>>>>
>>>>>> Previous work converted SCSI including the SCSI-based SATA driver. This
>>>>>> series converts the AHCI uclass itself, adding operations and adjusting
>>>>>> the 'sata' command to work correctly.
>>>>>>
>>>>>> An existing board (Compulab Utilite) is used for this work. It uses SATA
>>>>>> without the SCSI layer. Unfortunately this board does not currently use
>>>>>> driver model for MMC or USB, nor does it include a device tree. Therefore
>>>>>> additional patches are included in this series to convert this over. Note
>>>>>> that USB is rendered inoperative since the relevant device-tree nodes are
>>>>>> disabled. Further work is needed here (e.g. by the board maintainer) but
>>>>>> it is well beyond the objective of this series.
>>>>>>
>>>>>> A long-standing niggle with MMC has been the CONFIG_DM_MMC_OPS option.
>>>>>> This was introduced to allow driver model to handle MMC operations rather
>>>>>> than having them in struct mmc. This was needed since the original uclass
>>>>>> did not have support for operations.
>>>>>>
>>>>>> At present i.MX is the only SoC that defines CONFIG_DM_MMC without
>>>>>> CONFIG_DM_MMC_OPS. With this driver converted, the option is no longer
>>>>>> necessary so this series takes the opportunity to drop that option.
>>>>>
>>>>> Are you planning to review / apply this series please? It is assigned
>>>>> to you in patchwork. I'd like to resolve this because I have two other
>>>>> series depending on it
>>>>
>>>> Sorry..I had holidays..will check this week.
>>>
>>> Apologized for late..I'm checking this patchset and testing with my board.
>>> (But there is some conflict...so i needs to check more...what i missed.)
>>
>> If you like I could rebase it and resend. The series is at
>> u-boot-dm/ata2-working.
>
> Thanks for resending v2!

Do you think this can go in soon? Are there any problems / comments to address?

We are already after rc1 so I am getting a bit nervous!

Regards,
Simon