From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Brown Subject: Re: [PATCH 2/2] serial: msm: Add support for UARTDM cores Date: Wed, 19 Jan 2011 09:31:55 -0800 Message-ID: <8yaaaiwsstw.fsf@huya.qualcomm.com> References: <1295407585-18386-1-git-send-email-stepanm@codeaurora.org> <20110119082534.GA9569@gallagher> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from wolverine01.qualcomm.com ([199.106.114.254]:23707 "EHLO wolverine01.qualcomm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753373Ab1ASRbz (ORCPT ); Wed, 19 Jan 2011 12:31:55 -0500 In-Reply-To: <20110119082534.GA9569@gallagher> (Jamie Iles's message of "Wed, 19 Jan 2011 08:25:34 +0000") Sender: linux-arm-msm-owner@vger.kernel.org List-Id: linux-arm-msm@vger.kernel.org To: Jamie Iles Cc: Stepan Moskovchenko , linux-arm-msm@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, rlove@google.com On Wed, Jan 19 2011, Jamie Iles wrote: >> @@ -38,9 +40,20 @@ struct msm_port { >> struct uart_port uart; >> char name[16]; >> struct clk *clk; >> + struct clk *pclk; >> unsigned int imr; >> + unsigned int *gsbi_base; >> + int is_dm; >> + unsigned int old_snap_state; >> }; > > Out of interest, what does .is_dm mean? Is that obvious to someone who > knows about msm? It would be clear to people rather familiar with the chip, but I agree it's probably not that clear of a name. We have two UART blocks we use in the MSMs, the older one, supported by this driver before this change, and a newer one that was designed to work with our DMA engine (known as the Data Mover, or dm). This newer UART can be used in a somewhat compatible manner when being used for serial-port type operations. I'm not sure what would be a better name for the flag, though. The block is called UARTDM. David -- Sent by an employee of the Qualcomm Innovation Center, Inc. The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum.