Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753321AbbHMOUS (ORCPT ); Thu, 13 Aug 2015 10:20:18 -0400 Received: from smtprelay.synopsys.com ([198.182.47.9]:42686 "EHLO smtprelay.synopsys.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752022AbbHMOUO convert rfc822-to-8bit (ORCPT ); Thu, 13 Aug 2015 10:20:14 -0400 From: Vineet Gupta To: Noam Camus , Peter Hurley CC: "linux-kernel@vger.kernel.org" , "linux-serial@vger.kernel.org" , "gregkh@linuxfoundation.org" , "jslaby@suse.com" , "robh+dt@kernel.org" , "pawel.moll@arm.com" , "mark.rutland@arm.com" , "ijc+devicetree@hellion.org.uk" , "@codeaurora.orggalak@codeaurora.org" , "fransklaver@gmail.com" , "Alexey.Brodkin@synopsys.com" , "vgupta@synopsys.com" Subject: Re: [v3 3/3] serial: 8250_dw: Add UPF_SKIP_TEST to flags depend on device tree Thread-Topic: [v3 3/3] serial: 8250_dw: Add UPF_SKIP_TEST to flags depend on device tree Thread-Index: AQHQ1PD5bQY+yH/oFUGyDCpdzCo+FA== Date: Thu, 13 Aug 2015 14:20:05 +0000 Message-ID: References: <1437886478-29273-1-git-send-email-noamc@ezchip.com> <1439378312-6463-1-git-send-email-noamc@ezchip.com> <1439378312-6463-2-git-send-email-noamc@ezchip.com> <1439378312-6463-3-git-send-email-noamc@ezchip.com> <1439378312-6463-4-git-send-email-noamc@ezchip.com> <55CB4740.7070703@hurleysoftware.com> Accept-Language: en-US, en-IN Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.12.197.191] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1850 Lines: 42 Hi Peter, Greg On Wednesday 12 August 2015 09:21 PM, Noam Camus wrote: >> From: Peter Hurley [mailto:peter@hurleysoftware.com] >> Sent: Wednesday, August 12, 2015 4:17 PM >> I think Greg's question about the simulator still applies: why upstream this? >> The simulator is not even identified so how is someone supposed to know this workaround applies? >> The fact there are no in-tree DT users of this workaround argues against its acceptance. > I am using UART peripheral for Synopsys simulator same as one used by arch/arc/plat-sim The osci virtual platform uses nSIM with SystemC based peripheral models. The issue is in that model and not in nsim per-se. > I know this platform do not use CONFIG_SERIAL_8250_DW due to some problem I suspect it is relate to the loop test. Indeed. If you look at git log of osci platform, there was a commit which switched DT from dw uart driver to stock 8250. Unfortunately the changelog doesn't describe in detail what the root cause was. 2013-05-16 6eda477b3c54 ARC: [nsimosci] Change .dts to use generic 8250 UART > Maybe Vineet Gupta or Alexey Brodkin from Synopsys which are CC here can comment. > > So It just happen for me to be a pioneer with this. > > More than that "no-loopback-test" is an option already exist for core 8250, and since DW is only an extension for this driver it should also benefit from this option. It seems Noam has made some changes to model today and we might need this patch after all. Noam ? > If all this is yet not enough, should I re-send this "patch set" again without this specific patch? > > Regards, > Noam Camus > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/