| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <VI1PR04MB324790118B79E0A04082FE4A91330@VI1PR04MB3247.eurprd04.prod.outlook.com>
Date: Thu, 9 May 2019 02:31:57 +0000
From: Qiang Zhao <qiang.zhao@....com>
To: Rasmus Villemoes <rasmus.villemoes@...vas.dk>,
"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
Leo Li <leoyang.li@....com>
CC: "linuxppc-dev@...ts.ozlabs.org" <linuxppc-dev@...ts.ozlabs.org>,
"linux-arm-kernel@...ts.infradead.org"
<linux-arm-kernel@...ts.infradead.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
Rob Herring <robh+dt@...nel.org>,
Scott Wood <oss@...error.net>,
Christophe Leroy <christophe.leroy@....fr>,
Mark Rutland <mark.rutland@....com>,
Rasmus Villemoes <Rasmus.Villemoes@...vas.se>
Subject: RE: [PATCH v2 3/6] soc/fsl/qe: qe.c: introduce qe_get_device_node
helper
On 2019/5/1 17:29, Rasmus Villemoes <rasmus.villemoes@...vas.dk> wrote:
> -----Original Message-----
> From: Rasmus Villemoes <rasmus.villemoes@...vas.dk>
> Sent: 2019年5月1日 17:29
> To: devicetree@...r.kernel.org; Qiang Zhao <qiang.zhao@....com>; Leo Li
> <leoyang.li@....com>
> Cc: linuxppc-dev@...ts.ozlabs.org; linux-arm-kernel@...ts.infradead.org;
> linux-kernel@...r.kernel.org; Rob Herring <robh+dt@...nel.org>; Scott Wood
> <oss@...error.net>; Christophe Leroy <christophe.leroy@....fr>; Mark
> Rutland <mark.rutland@....com>; Rasmus Villemoes
> <Rasmus.Villemoes@...vas.se>
> Subject: [PATCH v2 3/6] soc/fsl/qe: qe.c: introduce qe_get_device_node
> helper
>
> The 'try of_find_compatible_node(NULL, NULL, "fsl,qe"), fall back to
> of_find_node_by_type(NULL, "qe")' pattern is repeated five times. Factor it
> into a common helper.
>
> Reviewed-by: Christophe Leroy <christophe.leroy@....fr>
> Signed-off-by: Rasmus Villemoes <rasmus.villemoes@...vas.dk>
> ---
Reviewed-by: Qiang Zhao <qiang.zhao@....com>
> drivers/soc/fsl/qe/qe.c | 71 +++++++++++++++++------------------------
> 1 file changed, 29 insertions(+), 42 deletions(-)
>
> diff --git a/drivers/soc/fsl/qe/qe.c b/drivers/soc/fsl/qe/qe.c index
> 303aa29cb27d..0fb8b59f61ad 100644
> --- a/drivers/soc/fsl/qe/qe.c
> +++ b/drivers/soc/fsl/qe/qe.c
> @@ -56,6 +56,20 @@ static unsigned int qe_num_of_snum;
>
> static phys_addr_t qebase = -1;
>
> +static struct device_node *qe_get_device_node(void) {
> + struct device_node *qe;
> +
> + /*
> + * Newer device trees have an "fsl,qe" compatible property for the
> QE
> + * node, but we still need to support older device trees.
> + */
> + qe = of_find_compatible_node(NULL, NULL, "fsl,qe");
> + if (qe)
> + return qe;
> + return of_find_node_by_type(NULL, "qe"); }
> +
> static phys_addr_t get_qe_base(void)
> {
> struct device_node *qe;
> @@ -65,12 +79,9 @@ static phys_addr_t get_qe_base(void)
> if (qebase != -1)
> return qebase;
>
> - qe = of_find_compatible_node(NULL, NULL, "fsl,qe");
> - if (!qe) {
> - qe = of_find_node_by_type(NULL, "qe");
> - if (!qe)
> - return qebase;
> - }
> + qe = qe_get_device_node();
> + if (!qe)
> + return qebase;
>
> ret = of_address_to_resource(qe, 0, &res);
> if (!ret)
> @@ -164,12 +175,9 @@ unsigned int qe_get_brg_clk(void)
> if (brg_clk)
> return brg_clk;
>
> - qe = of_find_compatible_node(NULL, NULL, "fsl,qe");
> - if (!qe) {
> - qe = of_find_node_by_type(NULL, "qe");
> - if (!qe)
> - return brg_clk;
> - }
> + qe = qe_get_device_node();
> + if (!qe)
> + return brg_clk;
>
> prop = of_get_property(qe, "brg-frequency", &size);
> if (prop && size == sizeof(*prop)) @@ -557,16 +565,9 @@ struct
> qe_firmware_info *qe_get_firmware_info(void)
>
> initialized = 1;
>
> - /*
> - * Newer device trees have an "fsl,qe" compatible property for the QE
> - * node, but we still need to support older device trees.
> - */
> - qe = of_find_compatible_node(NULL, NULL, "fsl,qe");
> - if (!qe) {
> - qe = of_find_node_by_type(NULL, "qe");
> - if (!qe)
> - return NULL;
> - }
> + qe = qe_get_device_node();
> + if (!qe)
> + return NULL;
>
> /* Find the 'firmware' child node */
> fw = of_get_child_by_name(qe, "firmware"); @@ -612,16 +613,9
> @@ unsigned int qe_get_num_of_risc(void)
> unsigned int num_of_risc = 0;
> const u32 *prop;
>
> - qe = of_find_compatible_node(NULL, NULL, "fsl,qe");
> - if (!qe) {
> - /* Older devices trees did not have an "fsl,qe"
> - * compatible property, so we need to look for
> - * the QE node by name.
> - */
> - qe = of_find_node_by_type(NULL, "qe");
> - if (!qe)
> - return num_of_risc;
> - }
> + qe = qe_get_device_node();
> + if (!qe)
> + return num_of_risc;
>
> prop = of_get_property(qe, "fsl,qe-num-riscs", &size);
> if (prop && size == sizeof(*prop)) @@ -641,16 +635,9 @@ unsigned
> int qe_get_num_of_snums(void)
> const u32 *prop;
>
> num_of_snums = 28; /* The default number of snum for threads is 28
> */
> - qe = of_find_compatible_node(NULL, NULL, "fsl,qe");
> - if (!qe) {
> - /* Older devices trees did not have an "fsl,qe"
> - * compatible property, so we need to look for
> - * the QE node by name.
> - */
> - qe = of_find_node_by_type(NULL, "qe");
> - if (!qe)
> - return num_of_snums;
> - }
> + qe = qe_get_device_node();
> + if (!qe)
> + return num_of_snums;
>
> prop = of_get_property(qe, "fsl,qe-num-snums", &size);
> if (prop && size == sizeof(*prop)) {
> --
> 2.20.1
Best Regards
Qiang Zhao
Powered by blists - more mailing lists