From: afaerber@suse.de (Andreas Färber)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v5 2/4] mmc: meson: Add driver for the SD/MMC host found on Amlogic Meson SoCs
Date: Wed, 2 Mar 2016 03:14:50 +0100 [thread overview]
Message-ID: <56D64C9A.3030004@suse.de> (raw)
In-Reply-To: <1456596108-1406-3-git-send-email-carlo@caione.org>
Hi Carlo,
Am 27.02.2016 um 19:01 schrieb Carlo Caione:
> Add a driver for the SD/MMC host found on the Amlogic Meson SoCs. This
> is an MMC controller which provides an interface between the application
> processor and various memory cards. It supports the SD specification
> v2.0 and the eMMC specification v4.41.
>
> Signed-off-by: Carlo Caione <carlo@endlessm.com>
Enabling this driver on top of my v2 arm64 gxbb series, I get:
CC drivers/mmc/host/meson-mmc.o
drivers/mmc/host/meson-mmc.c: In function 'meson_mmc_probe':
drivers/mmc/host/meson-mmc.c:471:12: error: implicit declaration of
function 'devm_pinctrl_get' [-Werror=implicit-function-declaration]
pinctrl = devm_pinctrl_get(&pdev->dev);
^
drivers/mmc/host/meson-mmc.c:471:10: warning: assignment makes pointer
from integer without a cast [-Wint-conversion]
pinctrl = devm_pinctrl_get(&pdev->dev);
^
cc1: some warnings being treated as errors
scripts/Makefile.build:258: recipe for target
'drivers/mmc/host/meson-mmc.o' failed
make[3]: *** [drivers/mmc/host/meson-mmc.o] Error 1
scripts/Makefile.build:407: recipe for target 'drivers/mmc/host' failed
make[2]: *** [drivers/mmc/host] Error 2
scripts/Makefile.build:407: recipe for target 'drivers/mmc' failed
make[1]: *** [drivers/mmc] Error 2
Makefile:959: recipe for target 'drivers' failed
make: *** [drivers] Error 2
Either this is missing a Kconfig dependency on PINCTRL, or should be
using #ifdef CONFIG_PINCTRL or some other guard.
For gxbb we could just ignore this in the hopes that Nicolas will have a
working pinctrl driver soon, but it might reappear for COMPILE_TEST.
Regards,
Andreas
--
SUSE Linux GmbH, Maxfeldstr. 5, 90409 N?rnberg, Germany
GF: Felix Imend?rffer, Jane Smithard, Graham Norton; HRB 21284 (AG N?rnberg)
next prev parent reply other threads:[~2016-03-02 2:14 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-02-27 18:01 [PATCH v5 0/4] Add support for Amlogic Meson MMC driver Carlo Caione
2016-02-27 18:01 ` [PATCH v5 1/4] mmc: meson: Document Amlogic Meson MMC DT bindings Carlo Caione
2016-03-03 22:17 ` Rob Herring
2016-03-04 8:11 ` Carlo Caione
2016-02-27 18:01 ` [PATCH v5 2/4] mmc: meson: Add driver for the SD/MMC host found on Amlogic Meson SoCs Carlo Caione
2016-03-02 2:14 ` Andreas Färber [this message]
2016-03-02 8:11 ` Carlo Caione
2016-04-18 13:47 ` Ulf Hansson
2016-02-27 18:01 ` [PATCH v5 3/4] ARM: dts: meson8b: Add MMC nodes Carlo Caione
2016-05-11 17:57 ` Kevin Hilman
2016-05-11 18:05 ` Carlo Caione
2016-02-27 18:01 ` [PATCH v5 4/4] ARM: multi_v7_defconfig: Add Amlogic Meson MMC support Carlo Caione
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=56D64C9A.3030004@suse.de \
--to=afaerber@suse.de \
--cc=linux-arm-kernel@lists.infradead.org \
/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