From mboxrd@z Thu Jan 1 00:00:00 1970 From: Maxime Ripard Subject: Re: [PATCH 02/11] pinctrl: sunxi: add IRQCHIP_SKIP_SET_WAKE flag for pinctrl irq chip Date: Wed, 28 May 2014 12:29:12 +0200 Message-ID: <20140528102912.GP4730@lukather> References: <1401090486-4414-1-git-send-email-hdegoede@redhat.com> <1401090486-4414-3-git-send-email-hdegoede@redhat.com> <20140527080746.GB4730@lukather> <5384B9E7.1070405@gmail.com> Reply-To: linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="WoqaC9TUMqqIOlla" Return-path: Content-Disposition: inline In-Reply-To: <5384B9E7.1070405-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> List-Post: , List-Help: , List-Archive: Sender: linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org List-Subscribe: , List-Unsubscribe: , To: Tomasz Figa Cc: Hans de Goede , devicetree , Ulf Hansson , linux-wireless-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Linus Walleij , linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org, linux-mmc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Chris Ball , "John W. Linville" , Chen-Yu Tsai , Arend van Spriel , linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org List-Id: linux-mmc@vger.kernel.org --WoqaC9TUMqqIOlla Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Tue, May 27, 2014 at 06:14:31PM +0200, Tomasz Figa wrote: > On 27.05.2014 10:07, Maxime Ripard wrote: > > On Mon, May 26, 2014 at 09:47:57AM +0200, Hans de Goede wrote: > >> From: Chen-Yu Tsai > >>=20 > >> The sunxi pinctrl irq chip driver does not support wakeup at the=20 > >> moment. Adding IRQCHIP_SKIP_SET_WAKE lets the irqs work with > >> drivers using wakeup. > >>=20 > >> Also add a name to the irq chip. > >>=20 > >> Signed-off-by: Chen-Yu Tsai Signed-off-by: Hans > >> de Goede ---=20 > >> drivers/pinctrl/sunxi/pinctrl-sunxi.c | 2 ++ 1 file changed, 2 > >> insertions(+) > >>=20 > >> diff --git a/drivers/pinctrl/sunxi/pinctrl-sunxi.c > >> b/drivers/pinctrl/sunxi/pinctrl-sunxi.c index db9ccd6..ec60c2e > >> 100644 --- a/drivers/pinctrl/sunxi/pinctrl-sunxi.c +++ > >> b/drivers/pinctrl/sunxi/pinctrl-sunxi.c @@ -646,6 +646,8 @@ > >> static struct irq_chip sunxi_pinctrl_irq_chip =3D { .irq_mask_ack =3D > >> sunxi_pinctrl_irq_mask_ack, .irq_unmask =3D > >> sunxi_pinctrl_irq_unmask, .irq_set_type =3D > >> sunxi_pinctrl_irq_set_type, + .name =3D "sunxi-pio", + .flags =3D > >> IRQCHIP_SKIP_SET_WAKE, > >=20 > > I'd rather see the name set to dev_name() or something like that. > > This will not work that great with multiple pin controller > > supporting interrupts. >=20 > Correct me if I'm wrong, but I've always thought that struct irq_chip > describes a class of IRQ controllers, not particular instances. It's not said as such in , but maybe.. Maxime --=20 Maxime Ripard, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com --WoqaC9TUMqqIOlla Content-Type: application/pgp-signature; name="signature.asc" Content-Description: Digital signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iQIcBAEBAgAGBQJThbp4AAoJEBx+YmzsjxAgO+YP/1tgXqWtrqhq+Q/xQU21eIZL IVHDJk7YAKxmsccXfq8Wvk0OLQP1vSWwbzOlMBvhIkf2Qk4VtzCu3NPVeEN9JkuT AOqgaplWRvPyv0QGuWMkr2JGwFaV/7ZkUqZKVV7oYxpV+85Uu5IrgV7+s9HeO7HA cErjaNfLQPKv2hibwPWarQATiUteYFjhKhl9Waa/LIF/jP1zrP4ZCeRqDUj4JAEM 3Tv2cwkkVuq46GNAMcNQYflIKXXhyY3fLgzAvjqupcBB3sTGpLEr9MaOdCJdSNPc HbW7TOlw/rvbD57AOn1AKbGAW94sY6Kq9ttcVfgf6C70I/XfCAN7jlwwhBgl6+zk 0TZilmTuIa29TRjNVRwyhkCPKibErMnw9eRVEP+dXDZ5kIdAYhn/fO08Ecfj7PFh 3jLIQI+Cnv+JOxV5w9AskPRrcnvqL8WZGB2W0zZMREuhaBRsJ4SarPp1po9hbFPu 5g4e5RNzH52wKu2FltKSzIMhoX2cEYpbLaOebW1tUHZ6/KP8cA2ZPKmdN0R/7mtS XiQyTfJBMxhxMqLf5jGwmLxupwbqWW44SiSLNyCmmX8ejIh3MLQqBu6ahKzgO8Dr oVw1XTmSfDz7VCmzq3JQELwDJju7BIvL4GUKlMAEUwU1fI1GvhH2OQhNhvAS8lEt WjDO3hfKWu6GuQHV/Yit =9RTz -----END PGP SIGNATURE----- --WoqaC9TUMqqIOlla--