Received: by 2002:a25:683:0:0:0:0:0 with SMTP id 125csp1536195ybg; Tue, 2 Jun 2020 12:41:08 -0700 (PDT) X-Google-Smtp-Source: ABdhPJwIG0+4jbQMPlFTLYi+j/yIMwT03Yu3Upn8YOkV0YRUgedJSz2K8sa4wfz9kIG1UMVjdCrZ X-Received: by 2002:a17:906:19c7:: with SMTP id h7mr10011370ejd.403.1591126868075; Tue, 02 Jun 2020 12:41:08 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1591126868; cv=none; d=google.com; s=arc-20160816; b=NsnncEULI9w23dr9sK7GCf9hoAvPLhM7XPCzpY05fqc0Qm/acaMEVkr7sm607CjdLh ASroKjofqBX8SrZz1NG3ul23tglE94MG4262EYW+eV2jNw1+GVqR03QwlDLhsnMSPs/D yl5LrrpTuL3bwkROWDCGyenTg8vIyI9CQbDEU5oKgtWDD6/06N+LOnf1ELryzFadFXL1 iCIq+JdZVPfH+XGGq40Wa14d9tJ0sAnN4xMxoekBd2KgtwLgUtuS/4a1SjKOUfds8tW8 1BhXH8owITpI29uzPSIpJwcjXazYL7Z6CbT8OgeEYQ6HurDqsroP4W7jhTCqrYKvXdkT 5MSA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:dkim-signature; bh=ZaeDmvwutMFDkhGXN+aSrIZ/aqZNZQqgehXmm4OU8WQ=; b=wzVy0Q9id2KZ+aStRZtZXH0WNrQLGhpYj/MfR+SqFme8LT/NKsFtbOocZxZ6J6PnTX e9jGjlQ5cRW32LDY3pH57Mxk9moYPNbfKOe/Hz/+ZypH/ffQUQrh7rlgXWBoU0N/x5ie lzYnfFSWoMbEmj3kBxeWS+brvH854n+0wzGjZNpxxm3Vb1fkkJJSjhl2CtJI8ieQIKKI hISsZjYvaS1JTlVQbIUd0h9AfrL5GSaHlE41Ed+ajtq/WcToPn5xt8RGG10RcTIt/nuM xWikYZiSqnJn/VP9d8s+eEJUVmcdl1JKQodV5cMpT9BxJXodU8a2+WBeQjAHJ6+Pdjo5 sVnQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=saHbz83D; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id 5si2084067edl.268.2020.06.02.12.40.44; Tue, 02 Jun 2020 12:41:08 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=saHbz83D; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728180AbgFBTf6 (ORCPT + 99 others); Tue, 2 Jun 2020 15:35:58 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:50320 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726420AbgFBTf5 (ORCPT ); Tue, 2 Jun 2020 15:35:57 -0400 Received: from mail-il1-x141.google.com (mail-il1-x141.google.com [IPv6:2607:f8b0:4864:20::141]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 7C70DC08C5C0; Tue, 2 Jun 2020 12:35:57 -0700 (PDT) Received: by mail-il1-x141.google.com with SMTP id z2so74824ilq.0; Tue, 02 Jun 2020 12:35:57 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=ZaeDmvwutMFDkhGXN+aSrIZ/aqZNZQqgehXmm4OU8WQ=; b=saHbz83DdIX3HLPmbOrEnGy96gXDa2LqsBe3j+qnBclnC0O60/LkTiWXVukocvudeF qu6gwtDugAsZ5QnsvW8SIPWsXhv2lLTcmo5RzkoWhfclb7ZFhaSwvAoT7AF6DI01TJ8t oHWcKllVoMI6sGMUbiwW6IPkt6/p8SwyWr2B1p788jWcu1HkoL3ja8ofpv7VKd3ovALh BV7/3tjW62EE0k8wUTOpwpe4jI/5ryrpSpKFN5CbQHYZaYKR2HRyR/xCR065a7WjIVUQ xXvBlIiY0+b900oIjjP7MiObfTXF1O7e3D9p2LgFFSsikadCqryBWtLGWwvJYg409SPu DVJQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=ZaeDmvwutMFDkhGXN+aSrIZ/aqZNZQqgehXmm4OU8WQ=; b=tniaLj3pgfTy9hGlLRNJuF2qbUw90wuvtghLopRe7umNMdAggU3ftOTdg5UnGvd8IG Fh5V60A9FkvOIaLIHQ4zXqT7rA4qg3JWp+2IMTM+x5tRYbFueELRsV8NRuRNl0S+9K5p Mw0ArXt7dbqDyfZ7pEDvO8MeDE9axskXjYBqFBOzDwFPM5eJN0jt7iA+yD3BdvTqiWUr A1OMVpTQjqB3xaiDQXSa9NWPk3yPcNfzB8vPYPGLF61/1pimC9FqBHVqxmENrOOjcpzj tXDYqMXU/ph1sne+R9d06YdANtdXTG28gdqoOEO3WXbYQTstuOi8A0HsLmVJV07ocTUG uNww== X-Gm-Message-State: AOAM53225w9tKZ3aSJtpDuVXsLU35xAvasICx0AeW/gsZ17IuyoRCs5j 3mqx4gKCV2P0n0StmkIVjW+lqZMr9B5TXmNsKMk= X-Received: by 2002:a05:6e02:13f4:: with SMTP id w20mr865871ilj.294.1591126556726; Tue, 02 Jun 2020 12:35:56 -0700 (PDT) MIME-Version: 1.0 References: <26028f50-3fb8-eb08-3c9f-08ada018bf9e@web.de> <20200602094947.GA5684@sirena.org.uk> <1c13e0ec-e50f-9eea-5704-052d2d682727@web.de> <20200602141306.GH5684@sirena.org.uk> <20200602183644.GI5684@sirena.org.uk> In-Reply-To: <20200602183644.GI5684@sirena.org.uk> From: Navid Emamdoost Date: Tue, 2 Jun 2020 14:35:45 -0500 Message-ID: Subject: Re: spi: spi-ti-qspi: call pm_runtime_put on pm_runtime_get failure To: Mark Brown Cc: Markus Elfring , linux-spi@vger.kernel.org, Navid Emamdoost , Kangjie Lu , Stephen McCamant , Qiushi Wu , Dinghao Liu , LKML , kernel-janitors@vger.kernel.org Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Jun 2, 2020 at 1:36 PM Mark Brown wrote: > > On Tue, Jun 02, 2020 at 05:05:18PM +0200, Markus Elfring wrote: > > >> I find this commit message improvable also according to Linux software > > >> development documentation. > > > > Causing people to send out new versions of things for tweaks to the > > > commit log consumes time for them and everyone they're sending changes to. > > > Improving patches (besides source code adjustments) is an usual software > > development activity, isn't it? > > Your updates were not improvements. The formatting was worse and to my > native speaker eyes the grammar was worse. With this sort of stylistic > thing it's especially important that any review aligns with the needs > and practices of the subsystem, there is opinion in there and multiple > opinions just makes things harder for submitters. Thanks Mark for your constructive opinion, In most cases, such stylistic comments become confusing and discouraging to those who are trying to chip in. Personally I think as long as the patch does not contain typo and is not ambiguous from the maintainer's perspective, it should be fine to let it go forward. -- Navid.