Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752730AbdCOJ7M (ORCPT ); Wed, 15 Mar 2017 05:59:12 -0400 Received: from mail-it0-f45.google.com ([209.85.214.45]:36846 "EHLO mail-it0-f45.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751642AbdCOJ7K (ORCPT ); Wed, 15 Mar 2017 05:59:10 -0400 MIME-Version: 1.0 In-Reply-To: <1774d7bad8bbc672372480f540dfc8f3d1592533.1489015238.git.julia@ni.com> References: <1774d7bad8bbc672372480f540dfc8f3d1592533.1489015238.git.julia@ni.com> From: Linus Walleij Date: Wed, 15 Mar 2017 10:59:08 +0100 Message-ID: Subject: Re: [PATCH 06/19] gpio: ath79: make use of raw_spinlock variants To: Julia Cartwright Cc: Alban Bedel , Alexandre Courbot , "linux-kernel@vger.kernel.org" , Thomas Gleixner , "linux-gpio@vger.kernel.org" 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: 684 Lines: 17 On Thu, Mar 9, 2017 at 5:21 PM, Julia Cartwright wrote: > The ath79 gpio driver currently implements an irq_chip for handling > interrupts; due to how irq_chip handling is done, it's necessary for the > irq_chip methods to be invoked from hardirq context, even on a a > real-time kernel. Because the spinlock_t type becomes a "sleeping" > spinlock w/ RT kernels, it is not suitable to be used with irq_chips. > > A quick audit of the operations under the lock reveal that they do only > minimal, bounded work, and are therefore safe to do under a raw spinlock. > > Signed-off-by: Julia Cartwright Patch applied with Alban's ACK. Yours, Linus Walleij