From: Noam Camus <noamc@ezchip.com>
To: Peter Hurley <peter@hurleysoftware.com>
CC: "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "linux-serial@vger.kernel.org" <linux-serial@vger.kernel.org>,
        "gregkh@linuxfoundation.org" <gregkh@linuxfoundation.org>,
        "jslaby@suse.com" <jslaby@suse.com>,
        "robh+dt@kernel.org" <robh+dt@kernel.org>,
        "pawel.moll@arm.com" <pawel.moll@arm.com>,
        "mark.rutland@arm.com" <mark.rutland@arm.com>,
        "ijc+devicetree@hellion.org.uk" <ijc+devicetree@hellion.org.uk>,
        "@codeaurora.orggalak@codeaurora.org" 
	<IMCEAINVALID-+40codeaurora+2Eorggalak+40codeaurora+2Eorg@eurprd02.prod.outlook.com>,
        "fransklaver@gmail.com" <fransklaver@gmail.com>,
        "Alexey.Brodkin@synopsys.com" <Alexey.Brodkin@synopsys.com>,
        "vgupta@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: AQHQ1PDlm3Sx1nkzPkKuBQ/Nd53nop4IWGcAgAAgOrA=
Date: Wed, 12 Aug 2015 15:51:05 +0000
Message-ID: <DB4PR02MB2722D091C4A40FC95B48070D67E0@DB4PR02MB272.eurprd02.prod.outlook.com>
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>
In-Reply-To: <55CB4740.7070703@hurleysoftware.com>
Accept-Language: en-US
Content-Language: en-US
spamdiagnosticoutput: 1:23
spamdiagnosticmetadata: NSPM
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-originalarrivaltime: 12 Aug 2015 15:51:06.0089
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 0fc16e0a-3cd3-4092-8b2f-0a42cff122c3
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB4PR02MB271
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
Content-Transfer-Encoding: 8bit
X-MIME-Autoconverted: from base64 to 8bit by mail.home.local id t7CFpMTN019660
Content-Length: 1161
Lines: 22

> 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
I know this platform do not use CONFIG_SERIAL_8250_DW due to some problem I suspect it is relate to the loop test.

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.

If all this is yet not enough, should I re-send this "patch set" again without this specific patch?

Regards,
Noam Camus
????{.n?+???????+%?????ݶ??w??{.n?+????{??G?????{ay?ʇڙ?,j??f???h?????????z_??(?階?ݢj"???m??????G????????????&???~???iO???z??v?^?m????????????I?