From: "pankaj.dubey" <pankaj.dubey@samsung.com>
To: Jisheng Zhang <jszhang@marvell.com>
Cc: linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org, rmk+kernel@armlinux.org.uk,
arnd@arndb.de, horms@verge.net.au, magnus.damm@gmail.com,
geert+renesas@glider.be, vireshk@kernel.org,
shiraz.linux.kernel@gmail.com, krzk@kernel.org,
thomas.ab@samsung.com, Russell King <linux@armlinux.org.uk>,
Dinh Nguyen <dinguyen@opensource.altera.com>,
Patrice Chotard <patrice.chotard@st.com>,
Linus Walleij <linus.walleij@linaro.org>,
Liviu Dudau <liviu.dudau@arm.com>, Ray Jui <rjui@broadcom.com>,
Stephen Warren <swarren@wwwdotorg.org>,
Heiko Stuebner <heiko@sntech.de>, Shawn Guo <shawnguo@kernel.org>,
Michal Simek <michal.simek@xilinx.com>,
Wei Xu <xuwei5@hisilicon.com>, Andrew Lunn <andrew@lunn.ch>,
Jun Nie <jun.nie@linaro.org>, "cpgs ." <cpgs@samsung.com>
Subject: Re: [PATCH 01/16] ARM: scu: Provide support for parsing SCU device node to enable SCU
Date: Mon, 14 Nov 2016 14:10:16 +0530 [thread overview]
Message-ID: <71596b86-7653-5c02-fc46-59409a91243f@samsung.com> (raw)
In-Reply-To: <20161114141251.7ea86e7a@xhacker>
Hi Jisheng,
On Monday 14 November 2016 11:42 AM, Jisheng Zhang wrote:
> Hi Pankaj,
>
> On Mon, 14 Nov 2016 10:31:56 +0530 Pankaj Dubey wrote:
<snip>
>> +
>> + np = of_find_matching_node(NULL, scu_match);
>
> could we check np before calling of_iomap()?
>
of_iomap takes care of that, and will return NULL if np is NULL.
So additional check of np is not required here.
>> + scu_base = of_iomap(np, 0);
>> + of_node_put(np);
>> + if (!scu_base) {
>> + pr_err("%s failed to map scu_base via DT\n", __func__);
>
> For non-ca5, non-ca9 based SoCs, we'll see this error msg. We understand
> what does it mean, but it may confuse normal users. In current version,
> berlin doesn't complain like this for non-ca9 SoCs
>
OK, let me see other reviewer's comment on this. Then we will decide if
this error message is required or can be omitted.
Thanks,
Pankaj Dubey
next prev parent reply other threads:[~2016-11-14 8:37 UTC|newest]
Thread overview: 48+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-11-14 5:01 [PATCH 00/16] Provide support of generic function for SCU enable Pankaj Dubey
2016-11-14 5:01 ` [PATCH 01/16] ARM: scu: Provide support for parsing SCU device node to enable SCU Pankaj Dubey
2016-11-14 6:12 ` Jisheng Zhang
2016-11-14 6:54 ` Jisheng Zhang
2016-11-14 8:23 ` pankaj.dubey
2016-11-14 8:47 ` Jisheng Zhang
2016-11-14 8:40 ` pankaj.dubey [this message]
2016-11-14 12:03 ` Arnd Bergmann
2016-11-14 13:50 ` Russell King - ARM Linux
2016-11-14 14:37 ` Arnd Bergmann
2016-11-14 14:51 ` Russell King - ARM Linux
2016-11-17 4:20 ` pankaj.dubey
2016-11-17 17:03 ` Arnd Bergmann
2016-11-18 3:24 ` pankaj.dubey
2016-11-18 12:14 ` Arnd Bergmann
2016-11-18 12:48 ` Russell King - ARM Linux
2016-11-18 13:32 ` Arnd Bergmann
2016-12-08 15:18 ` Pankaj Dubey
2016-11-14 13:48 ` Russell King - ARM Linux
2016-11-17 2:22 ` pankaj.dubey
2016-11-14 5:01 ` [PATCH 02/16] ARM: EXYNOS: use generic API " Pankaj Dubey
2016-11-15 18:59 ` Krzysztof Kozlowski
2016-11-17 2:15 ` pankaj.dubey
2016-11-14 5:01 ` [PATCH 03/16] ARM: berlin: use generic API for enabling SCU Pankaj Dubey
2016-11-14 8:51 ` Jisheng Zhang
2016-11-14 16:20 ` Pankaj Dubey
2016-11-14 5:01 ` [PATCH 04/16] ARM: realview: " Pankaj Dubey
2016-11-14 11:56 ` Arnd Bergmann
2016-11-14 12:06 ` pankaj.dubey
2016-11-14 14:28 ` Arnd Bergmann
2016-11-14 13:19 ` Pankaj Dubey
2016-11-14 5:02 ` [PATCH 05/16] ARM: socfpga: " Pankaj Dubey
2016-11-14 5:02 ` [PATCH 06/16] ARM: STi: " Pankaj Dubey
2016-11-14 5:02 ` [PATCH 07/16] ARM: ux500: " Pankaj Dubey
2016-11-14 5:02 ` [PATCH 08/16] ARM: vexpress: " Pankaj Dubey
2016-11-16 14:34 ` Sudeep Holla
2016-11-17 2:12 ` pankaj.dubey
2016-11-14 5:02 ` [PATCH 09/16] ARM: BCM: " Pankaj Dubey
2016-11-14 6:10 ` Florian Fainelli
2016-11-14 5:02 ` [PATCH 10/16] ARM: tegra: " Pankaj Dubey
2016-11-14 5:02 ` [PATCH 11/16] ARM: rockchip: " Pankaj Dubey
2016-11-14 5:02 ` [PATCH 12/16] ARM: imx: " Pankaj Dubey
2016-11-14 14:26 ` Shawn Guo
2016-11-17 4:29 ` pankaj.dubey
2016-11-14 5:02 ` [PATCH 13/16] ARM: zynq: " Pankaj Dubey
2016-11-14 5:02 ` [PATCH 14/16] ARM: hisi: " Pankaj Dubey
2016-11-14 5:02 ` [PATCH 15/16] ARM: mvebu: " Pankaj Dubey
2016-11-14 5:02 ` [PATCH 16/16] ARM: zx: " Pankaj Dubey
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=71596b86-7653-5c02-fc46-59409a91243f@samsung.com \
--to=pankaj.dubey@samsung.com \
--cc=andrew@lunn.ch \
--cc=arnd@arndb.de \
--cc=cpgs@samsung.com \
--cc=dinguyen@opensource.altera.com \
--cc=geert+renesas@glider.be \
--cc=heiko@sntech.de \
--cc=horms@verge.net.au \
--cc=jszhang@marvell.com \
--cc=jun.nie@linaro.org \
--cc=krzk@kernel.org \
--cc=linus.walleij@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux@armlinux.org.uk \
--cc=liviu.dudau@arm.com \
--cc=magnus.damm@gmail.com \
--cc=michal.simek@xilinx.com \
--cc=patrice.chotard@st.com \
--cc=rjui@broadcom.com \
--cc=rmk+kernel@armlinux.org.uk \
--cc=shawnguo@kernel.org \
--cc=shiraz.linux.kernel@gmail.com \
--cc=swarren@wwwdotorg.org \
--cc=thomas.ab@samsung.com \
--cc=vireshk@kernel.org \
--cc=xuwei5@hisilicon.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).