From patchwork Thu Feb 27 20:51:19 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Stephen Warren X-Patchwork-Id: 324946 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 6482A2C008F for ; Fri, 28 Feb 2014 07:51:32 +1100 (EST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751495AbaB0Uva (ORCPT ); Thu, 27 Feb 2014 15:51:30 -0500 Received: from avon.wwwdotorg.org ([70.85.31.133]:42291 "EHLO avon.wwwdotorg.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751150AbaB0Uv3 (ORCPT ); Thu, 27 Feb 2014 15:51:29 -0500 Received: from severn.wwwdotorg.org (unknown [192.168.65.5]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by avon.wwwdotorg.org (Postfix) with ESMTPS id 44E076339; Thu, 27 Feb 2014 13:51:29 -0700 (MST) Received: from swarren-lx1.nvidia.com (localhost [127.0.0.1]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by severn.wwwdotorg.org (Postfix) with ESMTPSA id 4DC3FE40F4; Thu, 27 Feb 2014 13:51:08 -0700 (MST) From: Stephen Warren To: Samuel Ortiz , Lee Jones Cc: devicetree@vger.kernel.org, Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , linux-arm-kernel@lists.infradead.org, linux-tegra@vger.kernel.org, J Keerthy , Ian Lartey , Stephen Warren Subject: [PATCH V2 1/3] dt: palmas: support IRQ inversion at the board level Date: Thu, 27 Feb 2014 13:51:19 -0700 Message-Id: <1393534281-30759-1-git-send-email-swarren@wwwdotorg.org> X-Mailer: git-send-email 1.8.1.5 X-NVConfidentiality: public X-Virus-Scanned: clamav-milter 0.97.8 at avon.wwwdotorg.org X-Virus-Status: Clean Sender: linux-tegra-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-tegra@vger.kernel.org From: Stephen Warren Some boards or SoCs have an inverter between the PMIC IRQ output pin and the IRQ controller input signal. The IRQ specifier in DT is meant to represent the IRQ flags at the input to the IRQ controller. The Palmas HW's IRQ output has configurable polarity. Software needs to know which polarity to choose for the IRQ output. Software may be tempted to extract the IRQ polarity from the IRQ specifier in order to make this choice. That approach works fine if the IRQ signal is routed directly from the PMIC to the IRQ controller with no intervening logic. However, if the signal is inverted between the two, this approach gets the wrong answer. Add an additional optional DT property which indicates that such an inversion occurs. This allows DT to give complete information about the desired IRQ output polarity to software. An alternative would have been to add a new non-optional DT parameter to indicate the exact desired output polarity. However, this would have been an incompatible change to the DT binding. Signed-off-by: Stephen Warren Acked-by: Laxman Dewangan Acked-by: Lee Jones --- v2: Split V1's patch 1/2 into separate patches 1/3 and 2/3. --- Documentation/devicetree/bindings/mfd/palmas.txt | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/Documentation/devicetree/bindings/mfd/palmas.txt b/Documentation/devicetree/bindings/mfd/palmas.txt index e5f0f8303461..76ec509d5f87 100644 --- a/Documentation/devicetree/bindings/mfd/palmas.txt +++ b/Documentation/devicetree/bindings/mfd/palmas.txt @@ -18,6 +18,12 @@ Required properties: ti,tps659038 and also the generic series names ti,palmas +- interrupts : Should contain a single entry for the IRQ output. +- ti,irq-externally-inverted : If missing, the polarity of the Palmas IRQ + output should be set to the opposite of the polarity indicated by the IRQ + specifier in the interrupts property. If absent, the polarity should be + configured to match. This allows the representation of an inverter between + the Palmas IRQ output and the interrupt parent's IRQ input. - interrupt-controller : palmas has its own internal IRQs - #interrupt-cells : should be set to 2 for IRQ number and flags The first cell is the IRQ number.