Received-SPF: pass (google.com: domain of linux-kernel+bounces-29846-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1;
Date: Thu, 18 Jan 2024 16:14:41 +0800
From: Crescent CY Hsieh <crescentcy.hsieh@moxa.com>
To: Greg KH <gregkh@linuxfoundation.org>
Cc: Christoph Niedermaier <cniedermaier@dh-electronics.com>,
	jirislaby@kernel.org, LinoSanfilippo@gmx.de, lukas@wunner.de,
	linux-kernel@vger.kernel.org, linux-serial@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH v6 1/2] tty: serial: Cleanup the bit shift with macro
Message-ID: <Zajd8fd/U3foRyLB@moxa-ThinkCentre-M90t>
References: <20231201071554.258607-2-crescentcy.hsieh@moxa.com>
 <20240117145623.3556-1-cniedermaier@dh-electronics.com>
 <2024011836-glimmer-seventh-f2a7@gregkh>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <2024011836-glimmer-seventh-f2a7@gregkh>
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?us-ascii?Q?opw1W/8hrTO1ujJdhiG7rZJYBv6zreHDIAQimTKmfctIhJYvCXG8o8jp1v6p?=
 =?us-ascii?Q?rrsi5lOvruAO/j9BAL2V2nHqLifeIEFQYaX9VMUhbLv3DFgHbrcY/4Fdvaxl?=
 =?us-ascii?Q?HQFxOPMvYYHyqj6StG0tO4jLmVfQx3Ko3RawKcGsZtNeFy2X7FhWkvaj1eBz?=
 =?us-ascii?Q?PPjBafM0L5icZWSTIn0neKMAkkIXTFQstI9g22CGO1I8K2UbcfHXvnH+S6Xh?=
 =?us-ascii?Q?PXQaJuYGfI4CUkeMpW8wtP6pluGi5ABh5opfF6guNIZ0JibEHdO8DWSR29Kt?=
 =?us-ascii?Q?JX2LFZKSuUdu0NGhuSD1UiFHs9Jgtvyu1+LsqLYGrp6XaV3bpoobq24n6/fy?=
 =?us-ascii?Q?WsIz+/z4XwerAiJV1fJ32UwpO0Tqc9hXZMuXWd46bW+i5glgPgX/Wkfy+L7f?=
 =?us-ascii?Q?fh7XTQ+w3bAB3l+Rcw+sxmUnez3RH1Vz1jHlFdM4Wjm92oFeUYVbTsiwxuwt?=
 =?us-ascii?Q?kJuptFh7jtVEkgXBp9mfvLmAcPmQrD9L/le+uFrbKXV9jNOMDtUlLYIyB1NI?=
 =?us-ascii?Q?wBRIrYsZCp3FBnOLImI4BZkaDJoWdTqhpftiiVn2UivX4/VH4GxmsqKsqVt7?=
 =?us-ascii?Q?qWH59UiB2NOz+h6mUWSzp1V3m5yocnqX+22XjPGbNBtRqPFZ33McAHY8R4HJ?=
 =?us-ascii?Q?SpFMAdjoYCGPpV2necKjB6zqLUoOsuCG1hSUjbKD6TFRMepT26Pa2hidlsn2?=
 =?us-ascii?Q?1WpEvSezjEtaSAUC1wQmjRtkvuGhaa4GLvE3Dobm8wDcZzXW7OkSCUTQ5rPI?=
 =?us-ascii?Q?OEUKOkBO/wA4p5L+VPMe9IebpTM1A7YGq+r1CBT93yZ5Ut7xs+GteXIIZslf?=
 =?us-ascii?Q?TwIT5ytJcqw55SeTbiSAwS5etVmmx9YH26SQHhUjbjuCCW9hQifigwnIb8Ki?=
 =?us-ascii?Q?QiqgFFzlL4v5PARnS0XQ+/LH5ezJ2OdzG3CrTk5fqL61R+HV/AibkuBRf3ya?=
 =?us-ascii?Q?bkOC+ceAlM942KS6lADopeUAGNjiLIdxzBKYDOMxvOjgtqLIuloTz1n7zPpm?=
 =?us-ascii?Q?+Te2itGK40GdLVU5/pikaBuYJNBffCO2165gViQZtQu2PrV9H0bljVfpzsHu?=
 =?us-ascii?Q?DaTuUnE+WG0UN41kqzL+BL6H4qsMsVL7cSU1ETelU8skHWwU9fG42XbMYrNR?=
 =?us-ascii?Q?7YMZ3r94VKsuysZ7/n/SqZeAERot36z3pQCstuoDUBspgSfvEcET/7gKgXF/?=
 =?us-ascii?Q?moL4g1GqIwX9U4+zMNtLOrgGgR+Y3qMRNigCU6tHyTYE4rgRqkA7W0eW45gU?=
 =?us-ascii?Q?FL/OJlMrTHsTT2n1OKmhMe/C8SvkLXTR9SOiLLKUDlrTaradBt+Gnlb5+pyJ?=
 =?us-ascii?Q?BG/lSJ3cLQT6FpI4fRTL8s83daWOnUoBxPYf+9bAn/dfKABBIvKY1Cec+Hf4?=
 =?us-ascii?Q?6W17vrKTUX2uWItSxNvCNbf2fceHEs9c4ty3PDO9DwdoxpSDHKQCwi1FuwKe?=
 =?us-ascii?Q?Fot/LeMIkIiDuxoxhkvVjAWqOwow8zxuKvnqTsfGkjPLTzsj8v8jxyXUyKrM?=
 =?us-ascii?Q?UpqtV8YTvxgWyEX1cxyb6OqzmZdEYTWNIAxqddH9rMgCCNVMWMj49v7JD5+v?=
 =?us-ascii?Q?fovpUE9xjLAK8ouzAqubwtkfE5URYtK+LEc4iRBY2mMJyZ9VGjwKRSxYAjSN?=
 =?us-ascii?Q?oQ=3D=3D?=
X-OriginatorOrg: moxa.com
X-MS-Exchange-CrossTenant-Network-Message-Id: eea8ad19-870c-4f7d-5a15-08dc17fd87a1
X-MS-Exchange-CrossTenant-AuthSource: PUZPR01MB5405.apcprd01.prod.exchangelabs.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 18 Jan 2024 08:14:45.1074
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 5571c7d4-286b-47f6-9dd5-0aa688773c8e
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: C9Bl/PeujIkblzs+4Lk+ENccXboGtLRzzeXhX49m0rlW4tQEs5TDOE0Vi0e2daAqYm79OTi9jHPjVlBrN+5RuBZFG6P4/FccbdvecEamiGs=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SI6PR01MB6423

On Thu, Jan 18, 2024 at 08:01:58AM +0100, Greg KH wrote:
> On Wed, Jan 17, 2024 at 03:56:23PM +0100, Christoph Niedermaier wrote:
> > >  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?

Sorry I might have misunderstood the meaning of "broke userspace".

In this case, does it imply splitting the "cleanup" patch and the "add
feature" patch, or leaving the third bit unused? Or perhaps both?

---
Sincerely,
Crescent Hsieh