From mboxrd@z Thu Jan  1 00:00:00 1970
From: "Edgar E. Iglesias" <edgar.iglesias@gmail.com>
Subject: Re: [PATCH v2 1/2] xen/arm: Add Cadence UART driver
Date: Mon, 9 Mar 2015 21:59:12 +1000
Message-ID: <20150309115912.GC29038@toto>
References: <1425604081-29730-1-git-send-email-edgar.iglesias@gmail.com>
	<1425604081-29730-2-git-send-email-edgar.iglesias@gmail.com>
	<54FD7A57.7090601@linaro.org>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xen.org>
Content-Disposition: inline
In-Reply-To: <54FD7A57.7090601@linaro.org>
List-Unsubscribe: <http://lists.xen.org/cgi-bin/mailman/options/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xen.org>
List-Help: <mailto:xen-devel-request@lists.xen.org?subject=help>
List-Subscribe: <http://lists.xen.org/cgi-bin/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=subscribe>
Sender: xen-devel-bounces@lists.xen.org
Errors-To: xen-devel-bounces@lists.xen.org
To: Julien Grall <julien.grall@linaro.org>
Cc: tim@xen.org, stefano.stabellini@citrix.com, ian.campbell@citrix.com, xen-devel@lists.xen.org
List-Id: xen-devel@lists.xenproject.org

On Mon, Mar 09, 2015 at 12:47:51PM +0200, Julien Grall wrote:
> Hi Edgar,

Hi Julien,

> 
> On 06/03/2015 03:08, Edgar E. Iglesias wrote:
> >+#define cuart_read(uart, off)           readl((uart)->regs + (off))
> >+#define cuart_write(uart, off,val)      writel((val), (uart)->regs + (off))
> >+
> >+static void cuart_interrupt(int irq, void *data, struct cpu_user_regs *regs)
> >+{
> >+    struct serial_port *port = data;
> >+    struct cuart *uart = port->uart;
> >+    unsigned int status;
> >+
> >+    do {
> >+        status = cuart_read(uart, R_UART_SR);
> >+        /* ACK.  */
> >+        if (status & UART_SR_INTR_RTRIG) {
> 
> 
> [..]
> 
> >+static void __init cuart_init_postirq(struct serial_port *port)
> >+{
> >+    struct cuart *uart = port->uart;
> >+    int rc;
> >+
> >+    if ( uart->irq > 0 ) {
> 
> Can you be consistent with the coding style? Either use Linux or Xen
> one but not a mix of them.

Yes, I'll fix it up and send a v3.

Thanks,
Edgar