From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752221AbcBOURG (ORCPT ); Mon, 15 Feb 2016 15:17:06 -0500 Received: from mout.kundenserver.de ([212.227.126.131]:53837 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751529AbcBOURE (ORCPT ); Mon, 15 Feb 2016 15:17:04 -0500 From: Arnd Bergmann To: Lars-Peter Clausen Cc: Linus Walleij , Alexandre Courbot , linux-arm-kernel@lists.infradead.org, Russell King , Bjorn Helgaas , linux-gpio@vger.kernel.org, linux-kernel@vger.kernel.org, Ralf Baechle , Paul Burton , linux-mips@linux-mips.org Subject: Re: [PATCH 1/4] gpio: remove broken irq_to_gpio() interface Date: Mon, 15 Feb 2016 21:16:30 +0100 Message-ID: <4277616.myYDBWrFcU@wuerfel> User-Agent: KMail/4.11.5 (Linux/3.16.0-10-generic; KDE/4.11.5; x86_64; ; ) In-Reply-To: <56C1F645.1070405@metafoo.de> References: <20160202194831.10827.63244.stgit@bhelgaas-glaptop2.roam.corp.google.com> <1455551208-2825510-2-git-send-email-arnd@arndb.de> <56C1F645.1070405@metafoo.de> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" X-Provags-ID: V03:K0:5V3mmFSVQ3ctjraJU486u+T21/fiTK1/YwlOH3oKzdBGO+2g2PB QKGByyMvTG6jmQnAvb1PY99dMQcix9O8YpZo1j7UCF0ZxK+eYkADefJ0vbOxcyj+I1OZ+Z7 UAlHtFfXU/23gE/aSzE/U4M1bC18lCQw/MJKVTYHwj9l4bz5/DGY7FjzlfVLp5KhJnLcZCj yh8PrJzhVskuAf8Ytm3Sg== X-UI-Out-Filterresults: notjunk:1;V01:K0:ylWPSZ/yuxo=:gCC3WqzzCpvefvDvzOFsrf OKlK36GoYf+MRjNXUye8Eu+o2957nZv3uxdDzWn4NBVIIRDozzWZoN6yQdATDnbtkXNMvQrEC NAL74VoJ6i+mX0zkaM+WXjEhoEEPYJBegok2JjffWs2MK/gRJjjA7nwNqX566o/p6DUDdUQWC UWtmgodGPtGN1wE/umIEM7ZDx2hot5JJgo7B/XnEgdIMUHf30G0FSgQXIpsGSzyUxPZU6KUkl 3mt/R5sVWZy2LJtpaNBCMU59yAN8uVbl0+N2AASGhtnsKbyhMledtnsIvdagRfpnsyuGFbg0V DjL+OVet4+ghNvr3bvfPGKQBDH6GOJ5HVWFCsOzOoDWelB11e455zwB7nxK9cr+II3lnlGrLN tZpmtTxtNPG7Ds9foceoSVa06hsY7hRM7IB6lJLVW70fKhXBHRGncNW7gR5C1ca32RlJOmWA/ 5pGNnKg0MkMNeJm0Nvk5UhM/CBNV/V2SeceHgz+JCGJf75M7rwAjVjKumIee9JFjSw3NZrHD1 krzjZ9MMC846zkJSOJi9mn37WRlN+WNy7L3tjkTCEckPZXOE/C9xH74RErthblDXoVEyoOqhR oxJyLEJ0RgontsdFVKjb1YIsOJdvEeS4OzpxhxVEnqdoKak9rnS447iJBfNN6FslbBI8rV5a1 r1qu8qQVKSEr+KZJCF6Ho6aQ9QZ9AwcEsL0EzSfcz/pMaTbGfVKgcbYr10j261Tz6LIWFt1X/ eAgLveRo7tKt3r3U Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Monday 15 February 2016 17:01:09 Lars-Peter Clausen wrote: > On 02/15/2016 04:46 PM, Arnd Bergmann wrote: > > +static inline __deprecated int irq_to_gpio(unsigned int irq) > > +{ > > + /* this has clearly not worked for a long time */ > > + return -EINVAL; > > +} > > + > > #define IRQ_TO_BIT(irq) BIT(irq_to_gpio(irq) & 0x1f) > > The issue seems to be a fallout from commit 832f5dacfa0b ("MIPS: Remove all > the uses of custom gpio.h"). > > The irq_to_gpio() should be replaced with "(irq - JZ4740_IRQ_GPIO(0))". > > Ah, that explain it, so it has not been broken for that long. Arnd