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

[Tom Rini]

On Fri, Aug 04, 2017 at 03:34:10AM -0600, Simon Glass wrote:
> 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!

Yes, I would like to see this come in ASAP, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170809/cdbbc495/attachment.sig>