Received: by 2002:a05:7412:8d1c:b0:fa:4c10:6cad with SMTP id bj28csp453478rdb; Wed, 17 Jan 2024 07:06:54 -0800 (PST) X-Google-Smtp-Source: AGHT+IHkgkzVDO30GVJk8k7iIOJxyZBYExg7eMxsmqL9qpxAcmNNonApKF8Rqr3zmhQjN5UTdF9p X-Received: by 2002:a17:902:7ed0:b0:1d4:e36f:748f with SMTP id p16-20020a1709027ed000b001d4e36f748fmr4223292plb.41.1705504013717; Wed, 17 Jan 2024 07:06:53 -0800 (PST) ARC-Seal: i=2; a=rsa-sha256; t=1705504013; cv=pass; d=google.com; s=arc-20160816; b=yPb4IdAeCTyPQ086PKlur9fysplV3pQHq3LtfbUEUw4ItzA4aCYFQjzlEjZnmPO1pb qMChp3CRyl1sdHPxdGZkCIKCCDqpYMHlDyOAy6nUvxLTilxUCERpItpm/ubGDywFIlMq 15MU56SvZjAWTcFjue5OtnE0AiS4PqmLklUi2Jj41pCLKs5zCEfu7jYIhuGQ/Ab5RMRu KgxS0KBFsv4g2CsKZTVuZFNfW2nQKdIQs3z8ZlE1ZvTv2oDoMf1YYQcojVr+ekdlmcxv 6UdJAmcWsu3/v/AvEsE/7EaVl6XKfDFhP/7MhmuZsi5ts63g9t0moT2l4TlxdRkmlEp1 XOWA== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=mime-version:list-unsubscribe:list-subscribe:list-id:precedence :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature; bh=8zEDP540sjwJaBrMK5oZNAb5uebPyAwo3ymgl0om22g=; fh=p/fKohjN6Hbg7cdSjgVG4Y/3cD3JFrathgwD24bS0CE=; b=LClChNptoYCr6tWLsDEZ6ALZaFnquFd8j04wwVzW5Y9QONC1vBqEQ7zJt0youGXUsO Y+RZBBKaxCqx0SAbwUrjETag2rjSlDuFmbvTKzFTeLGa7p8Y+mRghDGCh6m2SIrNurBv SpiPdN13oEvYLgpYwo3DSzNL1GAKoU1cA9rc8SmT+Y7nTMWiFh5V+mJKiON/c97sragO jP3WbrtLPF0cLcboT5jhQcpAoNTErPxHEWAlynzeSDJLbb0dAEagMeK3qOebr4sMqAtd g7m+x9MYfwJj076XY5XGXZgL74/4y3uZWz7Lv3tSSG4MeQ3FcfxzlanyL0sZq8D7GsMh seTw== ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@dh-electronics.com header.s=dhelectronicscom header.b=MDJEmvVv; arc=pass (i=1 spf=pass spfdomain=dh-electronics.com dkim=pass dkdomain=dh-electronics.com dmarc=pass fromdomain=dh-electronics.com); spf=pass (google.com: domain of linux-kernel+bounces-29120-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-29120-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=REJECT sp=REJECT dis=NONE) header.from=dh-electronics.com Return-Path: Received: from sv.mirrors.kernel.org (sv.mirrors.kernel.org. [2604:1380:45e3:2400::1]) by mx.google.com with ESMTPS id kn15-20020a170903078f00b001d55aae85a4si507929plb.186.2024.01.17.07.06.53 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 17 Jan 2024 07:06:53 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-29120-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1; Authentication-Results: mx.google.com; dkim=pass header.i=@dh-electronics.com header.s=dhelectronicscom header.b=MDJEmvVv; arc=pass (i=1 spf=pass spfdomain=dh-electronics.com dkim=pass dkdomain=dh-electronics.com dmarc=pass fromdomain=dh-electronics.com); spf=pass (google.com: domain of linux-kernel+bounces-29120-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-29120-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=REJECT sp=REJECT dis=NONE) header.from=dh-electronics.com Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by sv.mirrors.kernel.org (Postfix) with ESMTPS id 80DFC28769F for ; Wed, 17 Jan 2024 15:06:21 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 6473E21352; Wed, 17 Jan 2024 15:06:14 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=dh-electronics.com header.i=@dh-electronics.com header.b="MDJEmvVv" Received: from mx4.securetransport.de (mx4.securetransport.de [178.254.6.145]) by smtp.subspace.kernel.org (Postfix) with ESMTP id AD33B1E485; Wed, 17 Jan 2024 15:06:09 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=178.254.6.145 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1705503973; cv=none; b=dnFpGkhI3LPaR9Ik3sNcIRe0Zw88gSbQN7YsmnBi2McJI7SXjbhWDUpj4sxPPoVjbSm9C0Ck1RDzfg5Q6/+lQVi6b504PQ6Tzry2sKWu0nltOFHE9AK7ZlaiO5vPc2WS4U8wO8673/F+8cMjYBSkdZKDVOBMyONN33o7el/7UdU= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1705503973; c=relaxed/simple; bh=0puuW3TJ0FCRviSMs7e5yTWLZRzhlBbPbMaNHCKi2EQ=; h=DKIM-Signature:From:To:CC:Subject:Date:Message-ID:X-klartext: In-Reply-To:References:MIME-Version:Content-Type; b=LDtqCfPBDa1xaykrWJUG6TWzBVJ+A2GqcFFjAY/DKS/pCwR/MjZlkf+sa40PWDQLzqK3Hwtj1bic+vdMpUcK+H+HWBW8/qz2zEQt1MwgBkP9TZ4KCtFDq6bIN8BQY3hU6/ZF1D3Fk4pBfP5xK9dK57uqTJwRHsfUZW+crQR+ZUg= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=dh-electronics.com; spf=pass smtp.mailfrom=dh-electronics.com; dkim=pass (2048-bit key) header.d=dh-electronics.com header.i=@dh-electronics.com header.b=MDJEmvVv; arc=none smtp.client-ip=178.254.6.145 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=dh-electronics.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=dh-electronics.com DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=dh-electronics.com; s=dhelectronicscom; t=1705503387; bh=8zEDP540sjwJaBrMK5oZNAb5uebPyAwo3ymgl0om22g=; h=From:To:CC:Subject:Date:In-Reply-To:References:From; b=MDJEmvVvjPOMDdAmlvf/L+gbqe/avosXlfx/ll/72v6KZtxWHsyHrUuRGZ2MUempq ehcjuB8jZS+tdZFgS/R1zeFwA5dgJYrLQKKocgtF2bA4jwrvCLPuoRjD23ABWm+OOW Q3rfTgicGG915sTiu4xOkORDo54i3EiOT+XB40P8DlFfseAzy3e2rnBAA2CHzaXu6E mPjVbtdWRs7Hq83gnIdPGILVDVRPtluoQCS+Bw4TWE7t0jokB0GFVp4/WQnytRGjFL LieTe2jaOOHRNAf57WmOJjGk+H8iDnwAkTAa0IfEqp4u3uJD9I827K9Z+0d2VnKj4V nhErBTHFZ72tA== From: Christoph Niedermaier To: CC: , , , , , , , Subject: Re: [PATCH v6 1/2] tty: serial: Cleanup the bit shift with macro Date: Wed, 17 Jan 2024 15:56:23 +0100 Message-ID: <20240117145623.3556-1-cniedermaier@dh-electronics.com> X-klartext: yes In-Reply-To: <20231201071554.258607-2-crescentcy.hsieh@moxa.com> References: <20231201071554.258607-2-crescentcy.hsieh@moxa.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain Hi everyone, > This patch replaces the bit shift code with "_BITUL()" macro inside > "serial_rs485" struct. > > Signed-off-by: Crescent CY Hsieh > --- > include/uapi/linux/serial.h | 17 +++++++++-------- > 1 file changed, 9 insertions(+), 8 deletions(-) > > diff --git a/include/uapi/linux/serial.h b/include/uapi/linux/serial.h > index 53bc1af67..6c75ebdd7 100644 > --- a/include/uapi/linux/serial.h > +++ b/include/uapi/linux/serial.h > @@ -11,6 +11,7 @@ > #ifndef _UAPI_LINUX_SERIAL_H > #define _UAPI_LINUX_SERIAL_H > > +#include > #include > > #include > @@ -140,14 +141,14 @@ struct serial_icounter_struct { > */ > struct serial_rs485 { > __u32 flags; > -#define SER_RS485_ENABLED (1 << 0) > -#define SER_RS485_RTS_ON_SEND (1 << 1) > -#define SER_RS485_RTS_AFTER_SEND (1 << 2) In the old definition (1 << 3) wasn't used. > -#define SER_RS485_RX_DURING_TX (1 << 4) > -#define SER_RS485_TERMINATE_BUS (1 << 5) > -#define SER_RS485_ADDRB (1 << 6) > -#define SER_RS485_ADDR_RECV (1 << 7) > -#define SER_RS485_ADDR_DEST (1 << 8) > +#define SER_RS485_ENABLED _BITUL(0) > +#define SER_RS485_RTS_ON_SEND _BITUL(1) > +#define SER_RS485_RTS_AFTER_SEND _BITUL(2) > +#define SER_RS485_RX_DURING_TX _BITUL(3) Isn't it a break if number 3 isn't skipped here as well? Regards Christoph