Received-SPF: pass (google.com: domain of linux-kernel+bounces-135579-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
Date: Mon, 8 Apr 2024 18:11:48 +0300
From: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
To: Guanbing Huang <albanhuang@outlook.com>
Cc: gregkh@linuxfoundation.org, rafael.j.wysocki@intel.com,
	linux-acpi@vger.kernel.org, tony@atomide.com,
	john.ogness@linutronix.de, yangyicong@hisilicon.com,
	jirislaby@kernel.org, linux-kernel@vger.kernel.org,
	linux-serial@vger.kernel.org, lvjianmin@loongson.cn,
	albanhuang@tencent.com, tombinfan@tencent.com
Subject: Re: [PATCH v5 0/3] serial: 8250_pnp: Support configurable reg shift
 property
Message-ID: <ZhQJNMM4J2KAAycn@smile.fi.intel.com>
References: <PSAPR06MB4952B86F64A6CEB853114EA4C9002@PSAPR06MB4952.apcprd06.prod.outlook.com>
 <ZhQI1RgQtQuctUuL@smile.fi.intel.com>
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <ZhQI1RgQtQuctUuL@smile.fi.intel.com>
Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo

On Mon, Apr 08, 2024 at 06:10:13PM +0300, Andy Shevchenko wrote:
> On Mon, Apr 08, 2024 at 11:36:21AM +0800, Guanbing Huang wrote:
> > From: Guanbing Huang <albanhuang@tencent.com>
> > 
> > The 16550a serial port based on the ACPI table requires obtaining the
> > reg-shift attribute. In the ACPI scenario, If the reg-shift property
> > is not configured like in DTS, the 16550a serial driver cannot read or
> > write controller registers properly during initialization.
> > 
> > To address the issue of configuring the reg-shift property, the 
> > __uart_read_properties() universal interface is called to implement it.
> > Adaptation of PNP devices is done in the __uart_read_properties() function.
> 
> Seems okay to me now,
> Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>

Note, you have an issue with your email server. The cover letter is not chained
with the patch series. You really want to fix that if you wish to contribute
to Linux kernel project via respective mailing lists.

-- 
With Best Regards,
Andy Shevchenko