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] [PATCH] sandbox: Remove sandbox_noblk build
Date: Fri, 11 Oct 2019 19:06:22 -0400	[thread overview]
Message-ID: <20191011230622.GD16029@bill-the-cat> (raw)
In-Reply-To: <CAPnjgZ0JNXS9-a+KKAQadJimuEgMBpQRV=ZFezZa4OkUTU3xUg@mail.gmail.com>

On Fri, Oct 11, 2019 at 04:21:56PM -0600, Simon Glass wrote:
> On Fri, 11 Oct 2019 at 14:30, Tom Rini <trini@konsulko.com> wrote:
> >
> > At this point, all drivers that do not use CONFIG_BLK are past their
> > migration deadlines, so remove this config as it's no longer helpful and
> > hinders enhancing block drivers.
> >
> > Cc: Simon Glass <sjg@chromium.org>
> > Cc: Faiz Abbas <faiz_abbas@ti.com>
> > Signed-off-by: Tom Rini <trini@konsulko.com>
> > ---
> > This is required for Faiz's series that adds UFS support to SCSI as that
> > breaks the non-BLK case.  This path seems the most reasonable at this
> > point in time.
> > ---
> >  board/sandbox/MAINTAINERS       |   7 --
> >  configs/sandbox_noblk_defconfig | 178 --------------------------------
> >  doc/arch/sandbox.rst            |   7 +-
> >  3 files changed, 1 insertion(+), 191 deletions(-)
> >  delete mode 100644 configs/sandbox_noblk_defconfig
> 
> Reviewed-by: Simon Glass <sjg@chromium.org>
> 
> I wonder if we can assume that if block drivers are needed in SPL,
> then SPL_BLK will always be enabled? If not, we might have a problem.
> 
> +Marek Vasut

Perhaps?  Unfortunately doing this showed another set of problems on
PowerPC so I'm going to put the UFS aside for this batch of patches as I
think I saw something from Peng that will help PowerPC.

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

      reply	other threads:[~2019-10-11 23:06 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-10-11 20:30 [U-Boot] [PATCH] sandbox: Remove sandbox_noblk build Tom Rini
2019-10-11 22:21 ` Simon Glass
2019-10-11 23:06   ` Tom Rini [this message]

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=20191011230622.GD16029@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