Received: by 2002:a05:6a10:1a4d:0:0:0:0 with SMTP id nk13csp215238pxb; Mon, 7 Feb 2022 09:34:43 -0800 (PST) X-Google-Smtp-Source: ABdhPJweYda1HLVLD1oG5xQw85ieIjRujb9nWqSZBbrIk/0s2wr4WzRAuQf/LEkfIYOK2QtoIMWf X-Received: by 2002:a63:187:: with SMTP id 129mr380572pgb.500.1644255283430; Mon, 07 Feb 2022 09:34:43 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1644255283; cv=none; d=google.com; s=arc-20160816; b=Lwd3NRgyrWUHisZvMKxgOb0oSPXZf9zU3tJ59Nv1JVwRA0YtC8dEWbIcS0WXHdGRFJ wB8NEcywWVppPFS1CxUCbwbo80fzrUi+emvKXgu2nZLCMkh0WG4/FiktCcQ6toNRnnGk tUJZFcpgJEWVqkiTW4G64+jDN63ez9bzSXYGjnpUtDPK3BCW7Stcg0z9DVPe9zXWAmpV ssXDA+rc131QB+iZ0fn/nn32tQ6xYgR/oKh5vVE1x6Yg+owozA6j9gj2MeeXGXh6AmY2 YSf6cd2P+8sYvdB1FtE9ByJyojwxBt8jVGbJy+VPY4WugEaP24FOUJkI84lpYiemQa2K iAwg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :user-agent:references:in-reply-to:date:cc:to:from:subject :message-id; bh=U878SHv2EaXH5SU43ktpKxLnbnIC0095rzNolbtOjSU=; b=pNdVucUwUWEIZ4RoB/7z/gwvu4aCuCzpLvmXLf5eZR7d8ZYJSRSPvMI+JSyNCGH5ou l6aSethj15O2v9w6KwmmKVluSuRO8NsOiIHpNw44w9JaMDoUg6Sz5xdU+Ys0yXVTk7U2 lFsto8XSyA9er6CjdiaGuAef4f0EqdChaZNge8v+5OzF+EQHkX9OGiVDdorI/ym+KhMj ynkQdqSQtWtXziT3n+SIaEy2yBhZfAO3ufoaLP0KluwhqdT2uE/DdjM01qac6MwlDi/z 9nFkeJxKZdGr1uRgdL/eC/r33pwAARvpRcpmWDo0WsAaO0cnJUJlallq3Bc63edxMuP9 WIkQ== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id a184si10777992pge.253.2022.02.07.09.34.31; Mon, 07 Feb 2022 09:34:43 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1359220AbiBDOCQ (ORCPT + 99 others); Fri, 4 Feb 2022 09:02:16 -0500 Received: from relay3.hostedemail.com ([64.99.140.33]:18512 "EHLO relay3.hostedemail.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1359191AbiBDOCP (ORCPT ); Fri, 4 Feb 2022 09:02:15 -0500 Received: from omf17.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay01.hostedemail.com (Postfix) with ESMTP id CEA06614A8; Fri, 4 Feb 2022 14:02:07 +0000 (UTC) Received: from [HIDDEN] (Authenticated sender: joe@perches.com) by omf17.hostedemail.com (Postfix) with ESMTPA id A72A11B; Fri, 4 Feb 2022 14:01:08 +0000 (UTC) Message-ID: <7cb1ce88cbf977801f2519178c270c1271100ac6.camel@perches.com> Subject: Re: [PATCH] HPE BMC GXP SUPPORT From: Joe Perches To: "Russell King (Oracle)" Cc: nick.hawkins@hpe.com, verdun@hpe.com, David Airlie , Daniel Vetter , Rob Herring , Linus Walleij , Bartosz Golaszewski , Corey Minyard , Krzysztof Kozlowski , Miquel Raynal , Richard Weinberger , Vignesh Raghavendra , "David S. Miller" , Jakub Kicinski , Thierry Reding , Uwe =?ISO-8859-1?Q?Kleine-K=F6nig?= , Lee Jones , Greg Kroah-Hartman , Mark Brown , "Rafael J. Wysocki" , Daniel Lezcano , Amit Kucheria , Zhang Rui , Thomas Gleixner , Wim Van Sebroeck , Guenter Roeck , Arnd Bergmann , Olof Johansson , soc@kernel.org, Shawn Guo , Stanislav Jakubek , Sam Ravnborg , Hao Fang , Mark Rutland , Marc Zyngier , Ard Biesheuvel , Anshuman Khandual , Wang Kefeng , Lukas Bulwahn , Masahiro Yamada , dri-devel@lists.freedesktop.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-gpio@vger.kernel.org, linux-i2c@vger.kernel.org, openipmi-developer@lists.sourceforge.net, linux-mtd@lists.infradead.org, netdev@vger.kernel.org, linux-pwm@vger.kernel.org, linux-serial@vger.kernel.org, linux-spi@vger.kernel.org, linux-pm@vger.kernel.org, linux-usb@vger.kernel.org, linux-watchdog@vger.kernel.org, linux-arm-kernel@lists.infradead.org Date: Fri, 04 Feb 2022 06:01:44 -0800 In-Reply-To: References: <20220202165315.18282-1-nick.hawkins@hpe.com> Content-Type: text/plain; charset="ISO-8859-1" User-Agent: Evolution 3.40.4-1ubuntu2 MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Rspamd-Queue-Id: A72A11B X-Spam-Status: No, score=-2.03 X-Stat-Signature: c3yzg6zcu4cpdcebaczubogb94ck3gqq X-Rspamd-Server: rspamout07 X-Session-Marker: 6A6F6540706572636865732E636F6D X-Session-ID: U2FsdGVkX1+KwoOMBfXWPFnO9qh3WITIVuG9hjhnLgA= X-HE-Tag: 1643983268-605794 Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, 2022-02-04 at 12:31 +0000, Russell King (Oracle) wrote: > On Fri, Feb 04, 2022 at 04:18:24AM -0800, Joe Perches wrote: > > On Fri, 2022-02-04 at 12:05 +0000, Russell King (Oracle) wrote: > > > On Wed, Feb 02, 2022 at 10:52:50AM -0600, nick.hawkins@hpe.com wrote: > > > > + if (readb_relaxed(timer->control) & MASK_TCS_TC) { > > > > + writeb_relaxed(MASK_TCS_TC, timer->control); > > > > + > > > > + event_handler = READ_ONCE(timer->evt.event_handler); > > > > + if (event_handler) > > > > + event_handler(&timer->evt); > > > > + return IRQ_HANDLED; > > > > + } else { > > > > + return IRQ_NONE; > > > > + } > > > > +} > > > > It's also less indented code and perhaps clearer to reverse the test > > > > if (!readb_relaxed(timer->control) & MASK_TCS_TC) > > This will need to be: > > if (!(readb_relaxed(timer->control) & MASK_TCS_TC)) right, thanks.