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

[Jaehoon Chung]

Hi Simon,

On 08/17/2017 11:44 PM, Simon Glass wrote:
> Hi Jaehoon,
> 
> On 17 August 2017 at 01:40, Jaehoon Chung <jh80.chung at samsung.com> wrote:
>>
>> Hi Simon,
>>
>> On 07/30/2017 02:34 AM, Simon Glass wrote:
>>> Note: This series is rebased to u-boot-mmc/master with a fix-up patch
>>> for a broken patch there.
>>>
>>> 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.
>>
>> Sorry for late. I applied these patches to u-boot-mmc after fixing some conflicts.
>> Now i'm testing buildman..There is no problem. i will send PR to Tom.
>>
>> Sorry again.
>>
> 
> That's great, thanks for getting through this!
> 
> Let me know if any issues and I can take a look.

When i tested the buildman, there is no issue..but i might be missed something..
So after merging u-boot/master, need the double checking..

Best Regards,
Jaehoon Chung

> 
> Regards,
> Simon
> 
> 
>