Re: [PATCH v2 0/6] usb: host: Cleanup for ioremap'ing hcd memory
On Sat, 10 May 2014, Vivek Gautam wrote: > Based on 'usb-next' branch of Greg's usb tree. > > devm_ioremap_resource() API is advantageous over devm_ioremap() > and should therefore be preferred to request any ioremap'ed address > for hcd. > > Changes from v1: > - Changed the way returned pointer is checked for error value >as pointed out in the review comment in the mailing list. > > Vivek Gautam (6): > usb: host: ehci-exynos: Use devm_ioremap_resource instead of > devm_ioremap > usb: host: ehci-msm: Use devm_ioremap_resource instead of > devm_ioremap > usb: host: ehci-mv: Use devm_ioremap_resource instead of devm_ioremap > usb: host: ehci-spear: Use devm_ioremap_resource instead of > devm_ioremap > usb: host: ehci-tegra: Use devm_ioremap_resource instead of > devm_ioremap > usb: host: ohci-exynos: Use devm_ioremap_resource instead of > devm_ioremap > > drivers/usb/host/ehci-exynos.c |7 +++ > drivers/usb/host/ehci-msm.c|7 +++ > drivers/usb/host/ehci-mv.c | 16 ++-- > drivers/usb/host/ehci-spear.c | 13 +++-- > drivers/usb/host/ehci-tegra.c |7 +++ > drivers/usb/host/ohci-exynos.c |7 +++ > 6 files changed, 21 insertions(+), 36 deletions(-) For all six patches, Acked-by: Alan Stern -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v2 0/6] usb: host: Cleanup for ioremap'ing hcd memory
Hi, On Sat, May 10, 2014 at 7:55 PM, Alexander Shiyan wrote: > Sat, 10 May 2014 17:30:04 +0530 от Vivek Gautam : >> Based on 'usb-next' branch of Greg's usb tree. >> >> devm_ioremap_resource() API is advantageous over devm_ioremap() >> and should therefore be preferred to request any ioremap'ed address >> for hcd. >> >> Changes from v1: >> - Changed the way returned pointer is checked for error value >>as pointed out in the review comment in the mailing list. > > hcd->rsrc_len field can be removed entirely, since I cannot find any reason > how this filed can be used in the drivers now. > This is of course should be in an another series. This one looks good. True, even i noticed the same, since after grep'ing for rsrc_len. i could not find any mention in the USB layer. Will send that as a separate patch-series. -- Best Regards Vivek Gautam Samsung R&D Institute, Bangalore India -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v2 0/6] usb: host: Cleanup for ioremap'ing hcd memory
Sat, 10 May 2014 17:30:04 +0530 от Vivek Gautam : > Based on 'usb-next' branch of Greg's usb tree. > > devm_ioremap_resource() API is advantageous over devm_ioremap() > and should therefore be preferred to request any ioremap'ed address > for hcd. > > Changes from v1: > - Changed the way returned pointer is checked for error value >as pointed out in the review comment in the mailing list. hcd->rsrc_len field can be removed entirely, since I cannot find any reason how this filed can be used in the drivers now. This is of course should be in an another series. This one looks good. ---
[PATCH v2 0/6] usb: host: Cleanup for ioremap'ing hcd memory
Based on 'usb-next' branch of Greg's usb tree. devm_ioremap_resource() API is advantageous over devm_ioremap() and should therefore be preferred to request any ioremap'ed address for hcd. Changes from v1: - Changed the way returned pointer is checked for error value as pointed out in the review comment in the mailing list. Vivek Gautam (6): usb: host: ehci-exynos: Use devm_ioremap_resource instead of devm_ioremap usb: host: ehci-msm: Use devm_ioremap_resource instead of devm_ioremap usb: host: ehci-mv: Use devm_ioremap_resource instead of devm_ioremap usb: host: ehci-spear: Use devm_ioremap_resource instead of devm_ioremap usb: host: ehci-tegra: Use devm_ioremap_resource instead of devm_ioremap usb: host: ohci-exynos: Use devm_ioremap_resource instead of devm_ioremap drivers/usb/host/ehci-exynos.c |7 +++ drivers/usb/host/ehci-msm.c|7 +++ drivers/usb/host/ehci-mv.c | 16 ++-- drivers/usb/host/ehci-spear.c | 13 +++-- drivers/usb/host/ehci-tegra.c |7 +++ drivers/usb/host/ohci-exynos.c |7 +++ 6 files changed, 21 insertions(+), 36 deletions(-) -- 1.7.10.4 -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html