From: Marek Vasut <marex@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 1/2] dm: Protect device_unbind() with CONFIG_DM_DEVICE_REMOVE
Date: Thu, 19 Feb 2015 15:34:09 +0100 [thread overview]
Message-ID: <201502191534.09117.marex@denx.de> (raw)
In-Reply-To: <CAPnjgZ25qjNRbQC8HdvEaY5wtj03h5jq4heY3vpgB1PtrnkssQ@mail.gmail.com>
On Thursday, February 19, 2015 at 03:29:52 PM, Simon Glass wrote:
> Hi Marek,
Hi Simon,
> On 19 February 2015 at 01:50, Marek Vasut <marex@denx.de> wrote:
> > On Thursday, February 19, 2015 at 02:06:38 AM, Simon Glass wrote:
> >> On 18 February 2015 at 14:36, Marek Vasut <marex@denx.de> wrote:
> >> > Since device_unbind() is also defined in device-remove.c,
> >> > which is compiled in only in case CONFIG_DM_DEVICE_REMOVE
> >> > is defined, protect the device_unbind() prototype with the
> >> > same CONFIG_DM_DEVICE_REMOVE check.
> >> >
> >> > Signed-off-by: Marek Vasut <marex@denx.de>
> >> > Cc: Simon Glass <sjg@chromium.org>
> >> > Cc: Stefan Roese <sr@denx.de>
> >> > Cc: Tom Rini <trini@ti.com>
> >> > ---
> >> >
> >> > include/dm/device-internal.h | 4 ++++
> >> > 1 file changed, 4 insertions(+)
> >>
> >> Acked-by: Simon Glass <sjg@chromium.org>
> >
> > Tom, can you please pick this one, I need it to repair the socfpga,
> > so I'd like to have it in before I submit new PR for that.
>
> I'll bring it into u-boot-dm in any case.
>
> I didn't see a build error though? What is broken at present?
Try building u-boot-socfpga/master . The patches there enable DT
(CONFIG_OF_CONTROL) on SoCFPGA always. And in case this is enabled,
the Cadence SPI and DW SPI drivers are compiled in. But those two
drivers require CONFIG_DM and CONFIG_DM_SPI ... and spi-uclass.c
contains calls to device_unbind() .
Best regards,
Marek Vasut
next prev parent reply other threads:[~2015-02-19 14:34 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-02-18 21:36 [U-Boot] [PATCH 1/2] dm: Protect device_unbind() with CONFIG_DM_DEVICE_REMOVE Marek Vasut
2015-02-18 21:36 ` [U-Boot] [PATCH 2/2] arm: socfpga: Enable DM for Cadence and DW SPI Marek Vasut
2015-02-19 1:07 ` Simon Glass
2015-02-19 9:44 ` Marek Vasut
2015-02-19 18:28 ` Simon Glass
2015-03-05 21:00 ` Marek Vasut
2015-02-19 1:06 ` [U-Boot] [PATCH 1/2] dm: Protect device_unbind() with CONFIG_DM_DEVICE_REMOVE Simon Glass
2015-02-19 8:50 ` Marek Vasut
2015-02-19 14:29 ` Simon Glass
2015-02-19 14:34 ` Marek Vasut [this message]
2015-02-19 15:24 ` Simon Glass
2015-02-19 18:19 ` Marek Vasut
2015-02-20 19:31 ` Simon Glass
2015-03-05 21:00 ` Marek Vasut
2015-02-19 18:27 ` Simon Glass
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=201502191534.09117.marex@denx.de \
--to=marex@denx.de \
--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