public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Tom Rini <trini@konsulko.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [U-Boot, PATCHv3, 2/4] dm: MIGRATION: Add migration plan for DM_USB
Date: Wed, 5 Dec 2018 07:23:08 -0500	[thread overview]
Message-ID: <20181205122308.GG32109@bill-the-cat> (raw)
In-Reply-To: <ea2bff56-c573-456e-7b2b-8e65919afbef@denx.de>

On Wed, Dec 05, 2018 at 08:14:14AM +0100, Stefan Roese wrote:
> Hi Tom,
> 
> On 04.12.18 05:49, Tom Rini wrote:
> >On Thu, Nov 29, 2018 at 06:21:12PM -0500, Tom Rini wrote:
> >
> >>As much of the USB system has been migrated to DM now, formalize a
> >>deadline for migration.
> >>
> >>Reviewed-by: Marek Vasut <marex@denx.de>
> >>Reviewed-by: Simon Glass <sjg@chromium.org>
> >>Signed-off-by: Tom Rini <trini@konsulko.com>
> >
> >Applied to u-boot/master, thanks!
> 
> Just a quick note on this, since I just tested compiling current
> mainline on MVEBU "theadorable_debug". Here I get:
> 
> ===================== WARNING ======================
> This board does not use CONFIG_DM_USB. Please update
> the board to use CONFIG_DM_USB before the v2019.07 release.
> Failure to update by the deadline may result in board removal.
> See doc/driver-model/MIGRATION.txt for more info.
> ====================================================
> ===================== WARNING ======================
> This board does not use CONFIG_DM_SCSI. Please update
> the storage controller to use CONFIG_DM_SCSI before the v2019.07 release.
> Failure to update by the deadline may result in board removal.
> See doc/driver-model/MIGRATION.txt for more info.
> ====================================================
> 
> The DM_SCSI warning seems to be okay, because of the not converted
> ATA driver sata_mv.c (the controller is unfortunately not AHCI
> compatible). I hope to find some time shortly to work on this.
> 
> But the board definitely uses DM_USB:
> 
> $ gg DM_USB configs/theadorable_debug_defconfig
> configs/theadorable_debug_defconfig:CONFIG_DM_USB=y
> 
> So the DM_USB warning seems to be wrong here.

I think you're "stuck" here because you can't turn on CONFIG_BLK to
silence the CONFIG_DM_USB warning without then breaking sata_mv.c.  The
CONFIG_DM_USB migration isn't complete until you're also using
CONFIG_BLK.

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

  reply	other threads:[~2018-12-05 12:23 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-11-29 23:21 [U-Boot] [PATCHv3 1/4] dm: MIGRATION: Add migration plan for DM_MMC Tom Rini
2018-11-29 23:21 ` [U-Boot] [PATCHv3 2/4] dm: MIGRATION: Add migration plan for DM_USB Tom Rini
2018-12-04  4:49   ` [U-Boot] [U-Boot, PATCHv3, " Tom Rini
2018-12-05  7:14     ` Stefan Roese
2018-12-05 12:23       ` Tom Rini [this message]
2018-12-07  0:05         ` Chris Packham
2018-12-07  1:19           ` Tom Rini
2018-11-29 23:21 ` [U-Boot] [PATCHv3 3/4] dm: MIGRATION: Add migration plan for CONFIG_SATA Tom Rini
2018-12-04  4:49   ` [U-Boot] [U-Boot, PATCHv3, " Tom Rini
2018-11-29 23:21 ` [U-Boot] [PATCHv3 4/4] dm: MIGRATION: Update migration plan for BLK Tom Rini
2018-12-04  4:49   ` [U-Boot] [U-Boot, PATCHv3, " Tom Rini
2018-11-30  8:10 ` [U-Boot] [PATCHv3 1/4] dm: MIGRATION: Add migration plan for DM_MMC Simon Goldschmidt
2018-11-30  8:50   ` Philipp Tomsich
2018-12-04  4:49 ` [U-Boot] [U-Boot, PATCHv3, " Tom Rini
2018-12-13  9:45   ` Simon Goldschmidt
2018-12-14  3:10     ` Simon Glass
2018-12-14  5:14       ` Simon Goldschmidt
2018-12-20 21:17         ` Simon Glass
2018-12-20 21:20           ` Simon Goldschmidt

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20181205122308.GG32109@bill-the-cat \
    --to=trini@konsulko.com \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox