From mboxrd@z Thu Jan 1 00:00:00 1970 From: Noam Camus Subject: Re: [PATCH] arc: use little endian accesses Date: Fri, 11 Mar 2016 05:07:09 +0000 Message-ID: References: <1457544064-16167-1-git-send-email-ltrimas@synopsys.com> <1457595846.2868.10.camel@synopsys.com> , <1457636268.457.34.camel@synopsys.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1457636268.457.34.camel@synopsys.com> Content-Language: en-US List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-snps-arc" Errors-To: linux-snps-arc-bounces+gla-linux-snps-arc=m.gmane.org@lists.infradead.org To: Vineet Gupta , Lada Trimasova Cc: "linux-arch@vger.kernel.org" , "linux-snps-arc@lists.infradead.org" , Alexey Brodkin , "linux-kernel@vger.kernel.org" List-Id: linux-arch.vger.kernel.org > From: Lada Trimasova > Sent: Thursday, March 10, 2016 8:57 PM > And for example, let's see what happens when we use "readl" in function "serial8250_early_in" in driver/tty/serial/8250. Is your DTS entry includes for serial node following entries: reg-io-width = <4> native-endian; if answer is yes you should expect (in case of big endian compilation) that: port->iotype would be equal to UPIO_MEM32BE, hence no readl() use for this case. This is how it works for me here, it sounds like your case is similar. Regards, Noam From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-db5eur01on0054.outbound.protection.outlook.com ([104.47.2.54]:15569 "EHLO EUR01-DB5-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1750727AbcCKFWL convert rfc822-to-8bit (ORCPT ); Fri, 11 Mar 2016 00:22:11 -0500 From: Noam Camus Subject: Re: [PATCH] arc: use little endian accesses Date: Fri, 11 Mar 2016 05:07:09 +0000 Message-ID: References: <1457544064-16167-1-git-send-email-ltrimas@synopsys.com> <1457595846.2868.10.camel@synopsys.com> ,<1457636268.457.34.camel@synopsys.com> In-Reply-To: <1457636268.457.34.camel@synopsys.com> Content-Language: en-US Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7BIT MIME-Version: 1.0 Sender: linux-arch-owner@vger.kernel.org List-ID: To: Vineet Gupta , Lada Trimasova Cc: "linux-kernel@vger.kernel.org" , "linux-arch@vger.kernel.org" , Alexey Brodkin , "linux-snps-arc@lists.infradead.org" Message-ID: <20160311050709.Lq0mAOAVoV015SlXV-Wc5ZkGcz5AUSdDpuvtyWJCLXc@z> > From: Lada Trimasova > Sent: Thursday, March 10, 2016 8:57 PM > And for example, let's see what happens when we use "readl" in function "serial8250_early_in" in driver/tty/serial/8250. Is your DTS entry includes for serial node following entries: reg-io-width = <4> native-endian; if answer is yes you should expect (in case of big endian compilation) that: port->iotype would be equal to UPIO_MEM32BE, hence no readl() use for this case. This is how it works for me here, it sounds like your case is similar. Regards, Noam