Hi Arnd, On 05/15/2013 12:11 PM, Arnd Bergmann wrote: > On Wednesday 15 May 2013 11:40:12 Dinh Nguyen wrote: >> On 05/15/2013 08:25 AM, Arnd Bergmann wrote: >>> 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. >> >> Our implementation has the timing controls for the SD/MMC controller in >> another custom IP block(system manager). sys_manager_base_addr was >> mapped in mach-socfpga/socfpga.c. I saw the same approach with >> drivers/clk(clk_mgr_base_addr), so I thought it would be ok with this >> driver. Please advise on another way to do this... > > The clock code is tied more more closely to the platform code, so I > was turning a blind eye on that one, under the assumption that it > was only used there. I think I can use the syscon interface for this. Thanks for the pointer. I kinda thought that these dw_mmc-<platform> is platform specific enough, but I will go the syscon route. Dinh > > 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