Re: [PATCH] staging: octeon: ethernet: remove redundant parentheses in conditionals

[Greg KH]

On Thu, Jul 02, 2026 at 01:35:57PM -0700, Abhinav Srivastava wrote:
> Remove extra parentheses in 3 multiline if conditions and align
> continuation lines under opening parenthesis. No functional change
> intended.
> 
> Signed-off-by: Abhinav Srivastava <atg271@xxxxxxxxx>
> ---
>  drivers/staging/octeon/ethernet.c | 18 +++++++++---------
>  1 file changed, 9 insertions(+), 9 deletions(-)
> 
> diff --git a/drivers/staging/octeon/ethernet.c b/drivers/staging/octeon/ethernet.c
> index f3fa221f4..e28625b47 100644
> --- a/drivers/staging/octeon/ethernet.c
> +++ b/drivers/staging/octeon/ethernet.c
> @@ -247,9 +247,9 @@ static int cvm_oct_common_change_mtu(struct net_device *dev, int new_mtu)
>  
>  	dev->mtu = new_mtu;
>  
> -	if ((interface < 2) &&
> -	    (cvmx_helper_interface_get_mode(interface) !=
> -		CVMX_HELPER_INTERFACE_MODE_SPI)) {
> +	if (interface < 2 &&
> +	    cvmx_helper_interface_get_mode(interface) !=
> +	    CVMX_HELPER_INTERFACE_MODE_SPI) {

You changed the indentation :(

Also, now we have to remember what the precidence order is, which is a
pain.  We write code for people first, compilers second, and this code
is the same either way for the compiler, so leave it alone to make it
easier for people to understand it.

thanks,

greg k-h