From mboxrd@z Thu Jan 1 00:00:00 1970 From: Florian Fainelli Subject: Re: [PATCH net-next 3/5] net: phy: mscc: remove unneeded parenthesis Date: Sat, 15 Sep 2018 13:52:49 -0700 Message-ID: <9a590892-8d58-8a13-c4f4-504abf849c5a@gmail.com> References: Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Cc: allan.nielsen@microchip.com, linux-kernel@vger.kernel.org, netdev@vger.kernel.org, thomas.petazzoni@bootlin.com To: Quentin Schulz , davem@davemloft.net, andrew@lunn.ch Return-path: In-Reply-To: Content-Language: en-US Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org On 9/14/2018 1:33 AM, Quentin Schulz wrote: > The == operator precedes the || operator, so we can remove the > parenthesis around (a == b) || (c == d). > > The condition is rather explicit and short so removing the parenthesis > definitely does not make it harder to read. > > Signed-off-by: Quentin Schulz Reviewed-by: Florian Fainelli -- Florian