Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757189AbcC2OOo (ORCPT ); Tue, 29 Mar 2016 10:14:44 -0400 Received: from mail-io0-f171.google.com ([209.85.223.171]:35324 "EHLO mail-io0-f171.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756902AbcC2OOj (ORCPT ); Tue, 29 Mar 2016 10:14:39 -0400 MIME-Version: 1.0 In-Reply-To: <1459241739.2565.9.camel@pengutronix.de> References: <1459241739.2565.9.camel@pengutronix.de> Date: Tue, 29 Mar 2016 11:14:38 -0300 Message-ID: Subject: Re: [PATCH] i.MX6 PCIe: Fix imx6_pcie_deassert_core_reset() polarity From: Fabio Estevam To: Lucas Stach Cc: Tim Harvey , =?UTF-8?Q?Krzysztof_Ha=C5=82asa?= , =?UTF-8?Q?Petr_=C5=A0tetiar?= , Richard Zhu , Bjorn Helgaas , "linux-pci@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , linux-kernel Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 412 Lines: 11 On Tue, Mar 29, 2016 at 5:55 AM, Lucas Stach wrote: > I would suspect that most boards specify the reset polarity the wrong > way around. Fixing this without breaking DT stability is hard. OTOH we It is not hard if we just revert the buggy commit. > could just argue that the system description in DT is wrong and needs to > be fixed. Sure, this would be a nice thing to do as well.