From mboxrd@z Thu Jan 1 00:00:00 1970 From: swarren@wwwdotorg.org (Stephen Warren) Date: Tue, 18 Feb 2014 10:42:05 -0700 Subject: [PATCH 1/2] mfd: palmas: support IRQ inversion at the board level In-Reply-To: <20140217092658.GC17875@lee--X1> References: <1392415108-4365-1-git-send-email-swarren@wwwdotorg.org> <20140217092658.GC17875@lee--X1> Message-ID: <53039B6D.8080004@wwwdotorg.org> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 02/17/2014 02:26 AM, Lee Jones wrote: >> 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. The driver >> currently selects the output polarity by querying the input polarity at >> the IRQ controller. This 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 automatic polarity >> selection gets the wrong answer. >> >> Add an additional optional DT and platform data parameter which indicates >> that such an inversion occurs. If this option is enabled, the Palmas >> driver will configure its IRQ output to the opposite polarity of the IRQ >> controller's input. >> >> 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 >> --- >> If this patch could be applied to its own branch (w/ signed tag) in the >> MFD tree, that would great; then I can pull patch 1/2 into the Tegra tree >> so that I can apply patch 2/2 to the Tegra tree. Thanks. >> --- >> Documentation/devicetree/bindings/mfd/palmas.txt | 6 ++++++ >> drivers/mfd/palmas.c | 4 ++++ >> include/linux/mfd/palmas.h | 1 + >> 3 files changed, 11 insertions(+) > > For the core changes: > Acked-by: Lee Jones Thanks. Since you ack'd this and are an MFD maintainer, was that an indication that I should take this patch through the Tegra tree?