* [PATCH -next] dss:use devm_platform_ioremap_resource_byname
@ 2020-09-16 11:13 Qilong Zhang
2020-10-16 21:55 ` Sam Ravnborg
0 siblings, 1 reply; 2+ messages in thread
From: Qilong Zhang @ 2020-09-16 11:13 UTC (permalink / raw)
To: b.zolnierkie; +Cc: linux-omap, linux-fbdev, dri-devel, linux-kernel
Use the devm_platform_ioremap_resource_byname() helper instead of
calling platform_get_resource_byname() and devm_ioremap_resource()
separately.
Signed-off-by: Zhang Qilong <zhangqilong3@huawei.com>
---
.../video/fbdev/omap2/omapfb/dss/hdmi4_core.c | 10 +--------
.../video/fbdev/omap2/omapfb/dss/hdmi5_core.c | 10 +--------
.../video/fbdev/omap2/omapfb/dss/hdmi_phy.c | 10 +--------
.../video/fbdev/omap2/omapfb/dss/hdmi_pll.c | 9 +-------
.../video/fbdev/omap2/omapfb/dss/video-pll.c | 21 +++----------------
5 files changed, 7 insertions(+), 53 deletions(-)
diff --git a/drivers/video/fbdev/omap2/omapfb/dss/hdmi4_core.c b/drivers/video/fbdev/omap2/omapfb/dss/hdmi4_core.c
index 7ca1803bf161..726c190862d4 100644
--- a/drivers/video/fbdev/omap2/omapfb/dss/hdmi4_core.c
+++ b/drivers/video/fbdev/omap2/omapfb/dss/hdmi4_core.c
@@ -875,15 +875,7 @@ void hdmi4_audio_stop(struct hdmi_core_data *core, struct hdmi_wp_data *wp)
int hdmi4_core_init(struct platform_device *pdev, struct hdmi_core_data *core)
{
- struct resource *res;
-
- res = platform_get_resource_byname(pdev, IORESOURCE_MEM, "core");
- if (!res) {
- DSSERR("can't get CORE mem resource\n");
- return -EINVAL;
- }
-
- core->base = devm_ioremap_resource(&pdev->dev, res);
+ core->base = devm_platform_ioremap_resource_byname(pdev, "core");
if (IS_ERR(core->base)) {
DSSERR("can't ioremap CORE\n");
return PTR_ERR(core->base);
diff --git a/drivers/video/fbdev/omap2/omapfb/dss/hdmi5_core.c b/drivers/video/fbdev/omap2/omapfb/dss/hdmi5_core.c
index 2f6ff14a48d9..eda29d3032e1 100644
--- a/drivers/video/fbdev/omap2/omapfb/dss/hdmi5_core.c
+++ b/drivers/video/fbdev/omap2/omapfb/dss/hdmi5_core.c
@@ -887,15 +887,7 @@ int hdmi5_audio_config(struct hdmi_core_data *core, struct hdmi_wp_data *wp,
int hdmi5_core_init(struct platform_device *pdev, struct hdmi_core_data *core)
{
- struct resource *res;
-
- res = platform_get_resource_byname(pdev, IORESOURCE_MEM, "core");
- if (!res) {
- DSSERR("can't get CORE IORESOURCE_MEM HDMI\n");
- return -EINVAL;
- }
-
- core->base = devm_ioremap_resource(&pdev->dev, res);
+ core->base = devm_platform_ioremap_resource_byname(pdev, "core");
if (IS_ERR(core->base)) {
DSSERR("can't ioremap HDMI core\n");
return PTR_ERR(core->base);
diff --git a/drivers/video/fbdev/omap2/omapfb/dss/hdmi_phy.c b/drivers/video/fbdev/omap2/omapfb/dss/hdmi_phy.c
index 9c645adba9e2..6fbfeb01b315 100644
--- a/drivers/video/fbdev/omap2/omapfb/dss/hdmi_phy.c
+++ b/drivers/video/fbdev/omap2/omapfb/dss/hdmi_phy.c
@@ -207,19 +207,11 @@ static const struct hdmi_phy_features *hdmi_phy_get_features(void)
int hdmi_phy_init(struct platform_device *pdev, struct hdmi_phy_data *phy)
{
- struct resource *res;
-
phy_feat = hdmi_phy_get_features();
if (!phy_feat)
return -ENODEV;
- res = platform_get_resource_byname(pdev, IORESOURCE_MEM, "phy");
- if (!res) {
- DSSERR("can't get PHY mem resource\n");
- return -EINVAL;
- }
-
- phy->base = devm_ioremap_resource(&pdev->dev, res);
+ phy->base = devm_platform_ioremap_resource_byname(pdev, "phy");
if (IS_ERR(phy->base)) {
DSSERR("can't ioremap TX PHY\n");
return PTR_ERR(phy->base);
diff --git a/drivers/video/fbdev/omap2/omapfb/dss/hdmi_pll.c b/drivers/video/fbdev/omap2/omapfb/dss/hdmi_pll.c
index 4991be031b0b..eb984d9999fe 100644
--- a/drivers/video/fbdev/omap2/omapfb/dss/hdmi_pll.c
+++ b/drivers/video/fbdev/omap2/omapfb/dss/hdmi_pll.c
@@ -220,17 +220,10 @@ int hdmi_pll_init(struct platform_device *pdev, struct hdmi_pll_data *pll,
struct hdmi_wp_data *wp)
{
int r;
- struct resource *res;
pll->wp = wp;
- res = platform_get_resource_byname(pdev, IORESOURCE_MEM, "pll");
- if (!res) {
- DSSERR("can't get PLL mem resource\n");
- return -EINVAL;
- }
-
- pll->base = devm_ioremap_resource(&pdev->dev, res);
+ pll->base = devm_platform_ioremap_resource_byname(pdev, "pll");
if (IS_ERR(pll->base)) {
DSSERR("can't ioremap PLLCTRL\n");
return PTR_ERR(pll->base);
diff --git a/drivers/video/fbdev/omap2/omapfb/dss/video-pll.c b/drivers/video/fbdev/omap2/omapfb/dss/video-pll.c
index f45fe60b9e7d..bff03d920722 100644
--- a/drivers/video/fbdev/omap2/omapfb/dss/video-pll.c
+++ b/drivers/video/fbdev/omap2/omapfb/dss/video-pll.c
@@ -129,7 +129,6 @@ struct dss_pll *dss_video_pll_init(struct platform_device *pdev, int id,
const char * const clkctrl_name[] = { "pll1_clkctrl", "pll2_clkctrl" };
const char * const clkin_name[] = { "video1_clk", "video2_clk" };
- struct resource *res;
struct dss_video_pll *vpll;
void __iomem *pll_base, *clkctrl_base;
struct clk *clk;
@@ -138,14 +137,7 @@ struct dss_pll *dss_video_pll_init(struct platform_device *pdev, int id,
/* PLL CONTROL */
- res = platform_get_resource_byname(pdev, IORESOURCE_MEM, reg_name[id]);
- if (!res) {
- dev_err(&pdev->dev,
- "missing platform resource data for pll%d\n", id);
- return ERR_PTR(-ENODEV);
- }
-
- pll_base = devm_ioremap_resource(&pdev->dev, res);
+ pll_base = devm_platform_ioremap_resource_byname(pdev, reg_name[id]);
if (IS_ERR(pll_base)) {
dev_err(&pdev->dev, "failed to ioremap pll%d reg_name\n", id);
return ERR_CAST(pll_base);
@@ -153,15 +145,8 @@ struct dss_pll *dss_video_pll_init(struct platform_device *pdev, int id,
/* CLOCK CONTROL */
- res = platform_get_resource_byname(pdev, IORESOURCE_MEM,
- clkctrl_name[id]);
- if (!res) {
- dev_err(&pdev->dev,
- "missing platform resource data for pll%d\n", id);
- return ERR_PTR(-ENODEV);
- }
-
- clkctrl_base = devm_ioremap_resource(&pdev->dev, res);
+ clkctrl_base = devm_platform_ioremap_resource_byname(pdev,
+ clkctrl_name[id]);
if (IS_ERR(clkctrl_base)) {
dev_err(&pdev->dev, "failed to ioremap pll%d clkctrl\n", id);
return ERR_CAST(clkctrl_base);
--
2.17.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH -next] dss:use devm_platform_ioremap_resource_byname
2020-09-16 11:13 [PATCH -next] dss:use devm_platform_ioremap_resource_byname Qilong Zhang
@ 2020-10-16 21:55 ` Sam Ravnborg
0 siblings, 0 replies; 2+ messages in thread
From: Sam Ravnborg @ 2020-10-16 21:55 UTC (permalink / raw)
To: Qilong Zhang
Cc: b.zolnierkie, linux-fbdev, linux-omap, linux-kernel, dri-devel
Hi Zhang Qilong
On Wed, Sep 16, 2020 at 07:13:53PM +0800, Qilong Zhang wrote:
> Use the devm_platform_ioremap_resource_byname() helper instead of
> calling platform_get_resource_byname() and devm_ioremap_resource()
> separately.
>
> Signed-off-by: Zhang Qilong <zhangqilong3@huawei.com>
Thanks, applied to drm-misc-next. The patch will appear in -next in a
few weeks.
While applying I had to update a few things:
- checkpatch warning due to indent
- warning for unused variable
- subject, should start with "omapfb:" like other patches touching the
same driver.
I also got a warning because you name differs in your mail and your
s-o-b. "Zhang Qilong" is not the same as "Qilong Zhang".
It would be nice if you for next submission have the same name in both
places.
The patch itself was fine - nice simplifications.
Sam
> ---
> .../video/fbdev/omap2/omapfb/dss/hdmi4_core.c | 10 +--------
> .../video/fbdev/omap2/omapfb/dss/hdmi5_core.c | 10 +--------
> .../video/fbdev/omap2/omapfb/dss/hdmi_phy.c | 10 +--------
> .../video/fbdev/omap2/omapfb/dss/hdmi_pll.c | 9 +-------
> .../video/fbdev/omap2/omapfb/dss/video-pll.c | 21 +++----------------
> 5 files changed, 7 insertions(+), 53 deletions(-)
>
> diff --git a/drivers/video/fbdev/omap2/omapfb/dss/hdmi4_core.c b/drivers/video/fbdev/omap2/omapfb/dss/hdmi4_core.c
> index 7ca1803bf161..726c190862d4 100644
> --- a/drivers/video/fbdev/omap2/omapfb/dss/hdmi4_core.c
> +++ b/drivers/video/fbdev/omap2/omapfb/dss/hdmi4_core.c
> @@ -875,15 +875,7 @@ void hdmi4_audio_stop(struct hdmi_core_data *core, struct hdmi_wp_data *wp)
>
> int hdmi4_core_init(struct platform_device *pdev, struct hdmi_core_data *core)
> {
> - struct resource *res;
> -
> - res = platform_get_resource_byname(pdev, IORESOURCE_MEM, "core");
> - if (!res) {
> - DSSERR("can't get CORE mem resource\n");
> - return -EINVAL;
> - }
> -
> - core->base = devm_ioremap_resource(&pdev->dev, res);
> + core->base = devm_platform_ioremap_resource_byname(pdev, "core");
> if (IS_ERR(core->base)) {
> DSSERR("can't ioremap CORE\n");
> return PTR_ERR(core->base);
> diff --git a/drivers/video/fbdev/omap2/omapfb/dss/hdmi5_core.c b/drivers/video/fbdev/omap2/omapfb/dss/hdmi5_core.c
> index 2f6ff14a48d9..eda29d3032e1 100644
> --- a/drivers/video/fbdev/omap2/omapfb/dss/hdmi5_core.c
> +++ b/drivers/video/fbdev/omap2/omapfb/dss/hdmi5_core.c
> @@ -887,15 +887,7 @@ int hdmi5_audio_config(struct hdmi_core_data *core, struct hdmi_wp_data *wp,
>
> int hdmi5_core_init(struct platform_device *pdev, struct hdmi_core_data *core)
> {
> - struct resource *res;
> -
> - res = platform_get_resource_byname(pdev, IORESOURCE_MEM, "core");
> - if (!res) {
> - DSSERR("can't get CORE IORESOURCE_MEM HDMI\n");
> - return -EINVAL;
> - }
> -
> - core->base = devm_ioremap_resource(&pdev->dev, res);
> + core->base = devm_platform_ioremap_resource_byname(pdev, "core");
> if (IS_ERR(core->base)) {
> DSSERR("can't ioremap HDMI core\n");
> return PTR_ERR(core->base);
> diff --git a/drivers/video/fbdev/omap2/omapfb/dss/hdmi_phy.c b/drivers/video/fbdev/omap2/omapfb/dss/hdmi_phy.c
> index 9c645adba9e2..6fbfeb01b315 100644
> --- a/drivers/video/fbdev/omap2/omapfb/dss/hdmi_phy.c
> +++ b/drivers/video/fbdev/omap2/omapfb/dss/hdmi_phy.c
> @@ -207,19 +207,11 @@ static const struct hdmi_phy_features *hdmi_phy_get_features(void)
>
> int hdmi_phy_init(struct platform_device *pdev, struct hdmi_phy_data *phy)
> {
> - struct resource *res;
> -
> phy_feat = hdmi_phy_get_features();
> if (!phy_feat)
> return -ENODEV;
>
> - res = platform_get_resource_byname(pdev, IORESOURCE_MEM, "phy");
> - if (!res) {
> - DSSERR("can't get PHY mem resource\n");
> - return -EINVAL;
> - }
> -
> - phy->base = devm_ioremap_resource(&pdev->dev, res);
> + phy->base = devm_platform_ioremap_resource_byname(pdev, "phy");
> if (IS_ERR(phy->base)) {
> DSSERR("can't ioremap TX PHY\n");
> return PTR_ERR(phy->base);
> diff --git a/drivers/video/fbdev/omap2/omapfb/dss/hdmi_pll.c b/drivers/video/fbdev/omap2/omapfb/dss/hdmi_pll.c
> index 4991be031b0b..eb984d9999fe 100644
> --- a/drivers/video/fbdev/omap2/omapfb/dss/hdmi_pll.c
> +++ b/drivers/video/fbdev/omap2/omapfb/dss/hdmi_pll.c
> @@ -220,17 +220,10 @@ int hdmi_pll_init(struct platform_device *pdev, struct hdmi_pll_data *pll,
> struct hdmi_wp_data *wp)
> {
> int r;
> - struct resource *res;
>
> pll->wp = wp;
>
> - res = platform_get_resource_byname(pdev, IORESOURCE_MEM, "pll");
> - if (!res) {
> - DSSERR("can't get PLL mem resource\n");
> - return -EINVAL;
> - }
> -
> - pll->base = devm_ioremap_resource(&pdev->dev, res);
> + pll->base = devm_platform_ioremap_resource_byname(pdev, "pll");
> if (IS_ERR(pll->base)) {
> DSSERR("can't ioremap PLLCTRL\n");
> return PTR_ERR(pll->base);
> diff --git a/drivers/video/fbdev/omap2/omapfb/dss/video-pll.c b/drivers/video/fbdev/omap2/omapfb/dss/video-pll.c
> index f45fe60b9e7d..bff03d920722 100644
> --- a/drivers/video/fbdev/omap2/omapfb/dss/video-pll.c
> +++ b/drivers/video/fbdev/omap2/omapfb/dss/video-pll.c
> @@ -129,7 +129,6 @@ struct dss_pll *dss_video_pll_init(struct platform_device *pdev, int id,
> const char * const clkctrl_name[] = { "pll1_clkctrl", "pll2_clkctrl" };
> const char * const clkin_name[] = { "video1_clk", "video2_clk" };
>
> - struct resource *res;
> struct dss_video_pll *vpll;
> void __iomem *pll_base, *clkctrl_base;
> struct clk *clk;
> @@ -138,14 +137,7 @@ struct dss_pll *dss_video_pll_init(struct platform_device *pdev, int id,
>
> /* PLL CONTROL */
>
> - res = platform_get_resource_byname(pdev, IORESOURCE_MEM, reg_name[id]);
> - if (!res) {
> - dev_err(&pdev->dev,
> - "missing platform resource data for pll%d\n", id);
> - return ERR_PTR(-ENODEV);
> - }
> -
> - pll_base = devm_ioremap_resource(&pdev->dev, res);
> + pll_base = devm_platform_ioremap_resource_byname(pdev, reg_name[id]);
> if (IS_ERR(pll_base)) {
> dev_err(&pdev->dev, "failed to ioremap pll%d reg_name\n", id);
> return ERR_CAST(pll_base);
> @@ -153,15 +145,8 @@ struct dss_pll *dss_video_pll_init(struct platform_device *pdev, int id,
>
> /* CLOCK CONTROL */
>
> - res = platform_get_resource_byname(pdev, IORESOURCE_MEM,
> - clkctrl_name[id]);
> - if (!res) {
> - dev_err(&pdev->dev,
> - "missing platform resource data for pll%d\n", id);
> - return ERR_PTR(-ENODEV);
> - }
> -
> - clkctrl_base = devm_ioremap_resource(&pdev->dev, res);
> + clkctrl_base = devm_platform_ioremap_resource_byname(pdev,
> + clkctrl_name[id]);
> if (IS_ERR(clkctrl_base)) {
> dev_err(&pdev->dev, "failed to ioremap pll%d clkctrl\n", id);
> return ERR_CAST(clkctrl_base);
> --
> 2.17.1
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2020-10-16 21:55 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-09-16 11:13 [PATCH -next] dss:use devm_platform_ioremap_resource_byname Qilong Zhang
2020-10-16 21:55 ` Sam Ravnborg
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).