From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 826FCC433F5 for ; Sun, 2 Oct 2022 15:04:15 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:Subject:Cc:To:From:Date:References: In-Reply-To:Message-Id:Mime-Version:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=akgtHWS+lbuUlR8p/4njH1pSKqDRBs6C+5bDPUPhE60=; b=filmYHBd5xjfzL Q4DtZiRQ0NP814/ZBJ92NAFt6M8ADIfI37k6nQHw08qAiQK9FCOT2kYPBNSCv3Lt4jq7mtSU7CQvb qwb0KZ5SfcuQ4MgEH/u9RNGz5Lczd6sPOeDI4+lQY7VwsBtMRyRyiwky4+k+7t2InsMXTxzlie8Jq yhELYUmKil+UV/ekHj8zmRqSmSkfz906RUnxvr99qEn7GIFBsQNbaxJtMPeaLRVkZT33KUGrM4Txu IADJjFj6CX1Rjgw2zdFH+SUrrXbUh+OK9d8P4toA54qVVBmdQH/L5VYtub4VUlqSGxsB714avB5st iSOLLbeIXUT3aLXfH5zw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1of0UJ-001YHS-46; Sun, 02 Oct 2022 15:02:39 +0000 Received: from out4-smtp.messagingengine.com ([66.111.4.28]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1of0UF-001YFO-Ky for linux-arm-kernel@lists.infradead.org; Sun, 02 Oct 2022 15:02:37 +0000 Received: from compute2.internal (compute2.nyi.internal [10.202.2.46]) by mailout.nyi.internal (Postfix) with ESMTP id 7889F5C0088; Sun, 2 Oct 2022 11:02:31 -0400 (EDT) Received: from imap47 ([10.202.2.97]) by compute2.internal (MEProxy); Sun, 02 Oct 2022 11:02:31 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=svenpeter.dev; h=cc:cc:content-type:date:date:from:from:in-reply-to :in-reply-to:message-id:mime-version:references:reply-to:sender :subject:subject:to:to; s=fm2; t=1664722951; x=1664809351; bh=gt bV2ARN10oUA4d7+HFMmKZYuHzsFVOrur3KwfZi+tU=; b=Yl5/d7ln6FEayDKb3a N/sVwgwnhxcXakbRaHrSfWI6CLcsoxFCAgiUHCKlasth4F3x4ZTHBuzFf5wSELBe A1dDE5H6+ghtayghaoKho3WtHpTd5avmwl+XySxppoxzD9ujrxiRrlcTzmLhvTDo JcQRnJ0xziBf4agLrrOG5KmypFsw1x2VCEn1qVMppnz/vOqv11XgQZ8g4hOtVZIb VaSqvQ+JTCJX3zDmnfIoGV4PFc1mZsoJHZY1jTAbEMxriBHCVvEBYGz2EV1gzx0j soKctLR+DH0XrlnK6yhKwby6yaJm3JGkRHXjTx5tsaP5znB8XaQDdUAbQZJJOUNv SShQ== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:cc:content-type:date:date:feedback-id :feedback-id:from:from:in-reply-to:in-reply-to:message-id :mime-version:references:reply-to:sender:subject:subject:to:to :x-me-proxy:x-me-proxy:x-me-sender:x-me-sender:x-sasl-enc; s= fm2; t=1664722951; x=1664809351; bh=gtbV2ARN10oUA4d7+HFMmKZYuHzs FVOrur3KwfZi+tU=; b=zozp5BMwW/askMznzeNXAoZAJN9Zyook2fl4uza7fj1t 10jWNQg8MxsoQeXuLxESlXuxseRyKJ9ct6ExzHj6R6kVv1BBgcL1ZkEea6Eun19b ATmJGhc2GbUQjXbLWjiekoMezy2Fs/0wLALQrbbhrjUbk10PcH4qV/fh6EM5f6nh ZuXg8dMEq5tTplRK3dbDRSKUJbWdR8Us1U5yvHS5xbN6BWI6USUR54pBPXJopgly uBQzT2trDUBkFjHoXjJx6OPlqVKAaQhF/1WxGnXttcupqkPZwjb6VjZRt9nCBO2X rH4DSsIx7G7nwVxflTFKLirVQlS044jSP2i9OvlYLQ== X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedvfedrfeehjedgkeefucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne cujfgurhepofgfggfkjghffffhvfevufgtsehttdertderredtnecuhfhrohhmpedfufhv vghnucfrvghtvghrfdcuoehsvhgvnhesshhvvghnphgvthgvrhdruggvvheqnecuggftrf grthhtvghrnhepleevgfegffehvedtieevhfekheeftedtjeetudevieehveevieelgffh ieevieeunecuvehluhhsthgvrhfuihiivgeptdenucfrrghrrghmpehmrghilhhfrhhomh epshhvvghnsehsvhgvnhhpvghtvghrrdguvghv X-ME-Proxy: Feedback-ID: i51094778:Fastmail Received: by mailuser.nyi.internal (Postfix, from userid 501) id 0A2F8A6007C; Sun, 2 Oct 2022 11:02:28 -0400 (EDT) X-Mailer: MessagingEngine.com Webmail Interface User-Agent: Cyrus-JMAP/3.7.0-alpha0-968-g04df58079d-fm-20220921.001-g04df5807 Mime-Version: 1.0 Message-Id: <5e89999e-5c9d-428c-a6c6-b454fb47aaa3@app.fastmail.com> In-Reply-To: References: Date: Sun, 02 Oct 2022 17:02:07 +0200 From: "Sven Peter" To: "Arminder Singh" , "Wolfram Sang" Cc: "Alyssa Rosenzweig" , asahi@lists.linux.dev, "Benjamin Herrenschmidt" , "Christian Zigotzky" , "Darren Stevens" , linux-arm-kernel@lists.infradead.org, linux-i2c@vger.kernel.org, linux-kernel@vger.kernel.org, linuxppc-dev , "Hector Martin" , "Michael Ellerman" , "Paul Mackerras" Subject: Re: [PATCH v2] i2c-pasemi: PASemi I2C controller IRQ enablement X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20221002_080236_166706_D1B69633 X-CRM114-Status: GOOD ( 16.02 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi, On Sun, Oct 2, 2022, at 16:07, Arminder Singh wrote: > Hi, > >> #define REG_MTXFIFO 0x00 >> #define REG_MRXFIFO 0x04 >> #define REG_SMSTA 0x14 >> +#define REG_IMASK 0x18 > >> This doesn't seem to be aligned correctly, this file seems to use a tab >> to separate the register name and the offset and you used spaces here. > >> @@ -15,7 +16,11 @@ struct pasemi_smbus { >> struct i2c_adapter adapter; >> void __iomem *ioaddr; >> unsigned int clk_div; >> - int hw_rev; >> + int hw_rev; >> + int use_irq; >> + struct completion irq_completion; > >> This doesn't seem to be aligned correctly and the hw_rev line >> doesn't have to be changed. > > I'm sorry for the alignment issues in the patch, I genuinely didn't notice > them as from the perspective of my primary editor (Visual Studio Code) > the entries were aligned. I just saw them when opening the files in > nano. No worries, it's just a small nit and quickly fixed after all! :) > > Does fixing the alignment issues and the commit description justify a v3 > of the patch or should the minor fixes go out as a "resend"? Just not sure > in this particular case as the fixes seem to be very minor ones. I'd send a v3. I've only used resend when e.g. my previous mail provider messed up and silently converted all my outgoing mails to HTML. Best, Sven _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel