On Wednesday 15 May 2013, dinguyen@xxxxxxxxxx wrote:
> +
> +#define SYSMGR_SDMMCGRP_CTRL_OFFSET 0x108
> +#define DRV_CLK_PHASE_SHIFT_SEL_MASK 0x7
> +#define SYSMGR_SDMMC_CTRL_SET(smplsel, drvsel) \
> + ((((drvsel) << 0) & 0x7) | (((smplsel) << 3) & 0x38))
> +
> +extern void __iomem *sys_manager_base_addr;
This is not acceptable, you cannot just reference external symbols
from one driver in another, without a proper interface.
Please explain what the functionality is that you need here, then
we can help you find the proper interface. My guess is that you
need either the functionality provided by drivers/reset/
or drivers/mfd/syscon.c.
> + if (of_property_read_u32(dev->of_node, "pwr-en", &pwr_en)) {
> + dev_info(dev, "couldn't determine pwr-en, assuming pwr-en = 0\n");
> + pwr_en = 0;
> + }
> +
> + /* Set PWREN bit */
> + mci_writel(host, PWREN, pwr_en);
If you add new properties, you have to document them in
Documentations/devicetree/bindings/*.
What is the requirement for this property? Is there no way to
automatically power the card on/off using the normal
dw_mci_set_ios function?
The rest of this patch looks ok to me.
Arnd
--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
