On Wednesday 20 November 2013, Kuninori Morimoto wrote: > const struct mfd_cell *cell = mfd_get_cell(pdev); > struct tmio_mmc_data *pdata; > struct tmio_mmc_host *host; > + struct resource *res; > int ret = -EINVAL, irq; > > if (pdev->num_resources != 2) > @@ -84,6 +85,13 @@ static int tmio_mmc_probe(struct platform_device *pdev) > goto out; > } > > + res = platform_get_resource(pdev, IORESOURCE_MEM, 0); > + if (!res) > + return -EINVAL; > + > + /* SD control register space size is 0x200, 0x400 for bus_shift=1 */ > + pdata->bus_shift = resource_size(res_ctl) >> 10; > + > ret = tmio_mmc_host_probe(&host, pdev, pdata); > if (ret) > goto cell_disable; Unfortunately, this has introduced a build error, since you use an incorrect variable name (res_ctl vs. res) in the last line that was added. Please add a fix on top. 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