netdev - Re: [PATCH V6 1/1] net: ethernet: Add TSE PCS support to dwmac-socfpga

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <20160630.083433.705935493078392286.davem@davemloft.net>
Date:	Thu, 30 Jun 2016 08:34:33 -0400 (EDT)
From:	David Miller <davem@...emloft.net>
To:	thloh@...era.com
Cc:	robh+dt@...nel.org, mark.rutland@....com, peppe.cavallaro@...com,
	alexandre.torgue@...com, preid@...ctromag.com.au,
	dinguyen@...nsource.altera.com, netdev@...r.kernel.org,
	devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
	thloh85@...il.com, cnphoon@...era.com
Subject: Re: [PATCH V6 1/1] net: ethernet: Add TSE PCS support to
 dwmac-socfpga

From: <thloh@...era.com>
Date: Wed, 29 Jun 2016 02:10:13 -0700

> +static void auto_nego_timer_callback(unsigned long data)
> +{
> +	u16 val = 0;
> +	u16 speed = 0;
> +	u16 duplex = 0;
> +
> +	struct tse_pcs *pcs = (struct tse_pcs *)data;
> +	void __iomem *tse_pcs_base = pcs->tse_pcs_base;
> +	void __iomem *sgmii_adapter_base = pcs->sgmii_adapter_base;

Please do not break up a set of local variable declarations with
an empty line.

> +	if ((tse_pcs_base) && (sgmii_adapter_base))

Unnecessary parenthesis.

> +	if ((tse_pcs_base) && (sgmii_adapter_base))

Likewise.

> @@ -88,9 +104,13 @@ static int socfpga_dwmac_parse_data(struct socfpga_dwmac *dwmac, struct device *
>  	struct device_node *np = dev->of_node;
>  	struct regmap *sys_mgr_base_addr;
>  	u32 reg_offset, reg_shift;
> -	int ret;
> -	struct device_node *np_splitter;
> +	int ret, index;
> +	struct device_node *np_splitter = NULL;
> +	struct device_node *np_sgmii_adapter = NULL;
> +
>  	struct resource res_splitter;
> +	struct resource res_tse_pcs;
> +	struct resource res_sgmii_adapter;

Again, do not break up local variable declarations with an empty line
like this.