Thread (9 messages) 9 messages, 5 authors, 2015-06-18

[PATCH v6 0/7] Add Mediatek MMC driver

From: Ulf Hansson <hidden>
Date: 2015-06-18 07:26:56
Also in: linux-devicetree, linux-gpio, linux-mediatek, linux-mmc, lkml

On 17 June 2015 at 23:01, Paul Gortmaker [off-list ref] wrote:
On Tue, Jun 16, 2015 at 4:03 AM, Ulf Hansson [off-list ref] wrote:
quoted
On 15 June 2015 at 13:20, Chaotian Jing [off-list ref] wrote:
quoted
This series enables MMC support on the MT8135/MT8173 platform.
MT8135 has 5 MMC controllers and MT8173 has 4 MMC controllers.

Changes base on Ulf's comments
Make hclk mandatory in the driver code
Change host->hclk to host->src_clk_freq
Add linux/pm.h

Chaotian Jing (3):
  mmc: dt-bindings: add Mediatek MMC bindings
  mmc: mediatek: Add Mediatek MMC driver.
This commit breaks when merged to linux-next since it gets
exposed as an implicit module.h user that does not include
that header:

   drivers/mmc/host/mtk-sd.c:1459:1: note: in expansion of macro
'module_platform_driver'
    module_platform_driver(mt_msdc_driver);
    ^
   include/linux/device.h:1296:1: error: type defaults to 'int' in
declaration of 'module_init' [-Werror=implicit-int]
    module_init(__driver##_init); \
    ^

Since the code doesn't exist in mainline, I can't fix it from
within my header cleanup series, so can you please add
an appropriate include of module.h to the above?

Also, I noticed the Makefile addition from this commit
is whitespace mangled; might as well fix that too if
rebasing to fix the include problem.

Thanks,
Paul.
Paul, appreciate your feedback!
I have re-based my next branch and updated the commit according to
your suggestions.

Kind regards
Uffe
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help