Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753345AbdCOJxc (ORCPT ); Wed, 15 Mar 2017 05:53:32 -0400 Received: from mail-it0-f48.google.com ([209.85.214.48]:35427 "EHLO mail-it0-f48.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753218AbdCOJxa (ORCPT ); Wed, 15 Mar 2017 05:53:30 -0400 MIME-Version: 1.0 In-Reply-To: References: From: Linus Walleij Date: Wed, 15 Mar 2017 10:53:23 +0100 Message-ID: Subject: Re: [PATCH 02/19] gpio: altera: make use of raw_spinlock variants To: Julia Cartwright Cc: Tien Hock Loh , 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: 668 Lines: 17 On Thu, Mar 9, 2017 at 5:21 PM, Julia Cartwright wrote: > The altera 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. Yours, Linus Walleij