Hi Fabio, Thanks for you reviewing and comment! On 2017/12/23 20:23, Fabio Estevam wrote: > On Sat, Dec 23, 2017 at 8:58 AM, Yisheng Xie <xieyisheng1@xxxxxxxxxx> wrote: >> Default ioremap is ioremap_nocache, so devm_ioremap has the same >> function with devm_ioremap_nocache, which can just be killed to >> save the size of devres.o >> >> This patch is to use use devm_ioremap instead of devm_ioremap_nocache, >> which should not have any function change but prepare for killing >> devm_ioremap_nocache. >> >> Cc: Bjorn Helgaas <bhelgaas@xxxxxxxxxx> >> Cc: linux-pci@xxxxxxxxxxxxxxx >> Signed-off-by: Yisheng Xie <xieyisheng1@xxxxxxxxxx> >> --- >> drivers/pci/dwc/pci-dra7xx.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/drivers/pci/dwc/pci-dra7xx.c b/drivers/pci/dwc/pci-dra7xx.c >> index e77a4cee..3f3712a 100644 >> --- a/drivers/pci/dwc/pci-dra7xx.c >> +++ b/drivers/pci/dwc/pci-dra7xx.c >> @@ -637,7 +637,7 @@ static int __init dra7xx_pcie_probe(struct platform_device *pdev) >> } >> >> res = platform_get_resource_byname(pdev, IORESOURCE_MEM, "ti_conf"); >> - base = devm_ioremap_nocache(dev, res->start, resource_size(res)); >> + base = devm_ioremap(dev, res->start, resource_size(res)); > > You could also consider using devm_ioremap_resource(), which checks > whether the resource is NULL and it is a bit shorter: > > base = devm_ioremap_resource(dev, res); > if (IS_ERR(base)) > return PTR_ERR(base); Yeah, I will check if all the archs have the same ioremap and ioremap_noache, first. If it is so, I will take your suggestion. Thanks Yisheng > > >> >> -- >> 1.8.3.1 >> > > . >
