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;
Date:   Mon, 3 Apr 2023 07:59:42 -0700
From:   Ira Weiny <ira.weiny@intel.com>
To:     Khadija Kamran <kamrankhadijadj@gmail.com>,
        Ira Weiny <ira.weiny@intel.com>
CC:     <outreachy@lists.linux.dev>, <hvaibhav.linux@gmail.com>,
        <johan@kernel.org>, <elder@kernel.org>,
        <gregkh@linuxfoundation.org>, <greybus-dev@lists.linaro.org>,
        <linux-staging@lists.linux.dev>, <linux-kernel@vger.kernel.org>,
        Alison Schofield <alison.schofield@intel.com>
Subject: Re: [PATCH 1/2] staging: greybus: add a single exit path to
 arche_platform_wd_irq()
Message-ID: <642ae9de5ad28_39845e29467@iweiny-mobl.notmuch>
References: <cover.1680185025.git.kamrankhadijadj@gmail.com>
 <e670baa9bde47a3bdb02e59ec37a438a62c52dd1.1680185025.git.kamrankhadijadj@gmail.com>
 <642a1fdcb2648_394c3829469@iweiny-mobl.notmuch>
 <ZCoqMSy1Ary0sAp9@khadija-virtual-machine>
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <ZCoqMSy1Ary0sAp9@khadija-virtual-machine>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?sWTxEgHcq4G90OgHad8rbN3GvoS0XdZJdF/XPGrgMFKd/h3EUNmSRuOYHGjQ?=
 =?us-ascii?Q?0ky5LoYCuMdmtG4axAnhRfLiNFPcgKsnKyP5ciZzOmkPbzMR9OOXMVLjjzQH?=
 =?us-ascii?Q?ywey7dG2pgwwhgiR6eg9aJjjWyl/L3Mik2tzrFVSfnOW4oiHUAEbhD7LGOEl?=
 =?us-ascii?Q?b3sNjBj5HE8Mvl5mNFLF6eC9HnLGX6Rj6U1DCj+wFsmbtw2rYbSZD8EQ38E1?=
 =?us-ascii?Q?cR3ttMMgKen8Uc5YTA3R1FyxPgz9VNm43bgDfcFtyc0XfYaVftyfrdIgc6fs?=
 =?us-ascii?Q?PMsi3pEORs20ebYU74sWkUk8ehwh9LBiea1FyopXb/VaIovjMXQQ1nZ4m6Rt?=
 =?us-ascii?Q?umrKJIzYJsdORCLpjxqHey6iSIePu6ldvUL6Bn4mz/pNtlj1ECNndjz1zgMA?=
 =?us-ascii?Q?Jy1Bs3EHkTk2/lHZN6ROXjCsu8md70STzZnSnnJFagKBOkguHxrFwZd7b7Zv?=
 =?us-ascii?Q?ka24OzaUgRsRJFvgDCPzNxq0TVGzpJZERc6flwzXdCxGNsd+dq/VAQzLninx?=
 =?us-ascii?Q?hgcia0CUSg2cqDymNFg2ggBMUHx+E/JE1XR+9FCE+n+kpleEuTxt6aR4NjJ7?=
 =?us-ascii?Q?C/R/06AkcnjqXxNmxMUfwh3WLOxxsNtrV/sRfN7T9n8dt1KVJmx33gvMauUz?=
 =?us-ascii?Q?7Tjv89e4SucrTP3IVBXu/wASd030WMtR6o9RQ7eHTMsISQ+CcoVnRw7803FI?=
 =?us-ascii?Q?3AuVG628iDsHse8V5qGhLiEoLCCLLS4etVAhmbTqpmlVNkW4vhNMtD0PvET/?=
 =?us-ascii?Q?aFIRINtHsNbX6rmy5gwNzTopqFzoOWSeK8XHMpwud63rQSlIz54eZ8RAEYt1?=
 =?us-ascii?Q?ulbDDtkvJvHtiHjGtSp4LCF974xa2VptcrSjf1Vp3BctQRmLwgmZZGb/OkaM?=
 =?us-ascii?Q?XQTjUzCElVyO+39A+eETFjzRW4gB7OM4q5DHgP3sZ+SahKE8PByhOe1QTIpy?=
 =?us-ascii?Q?Ugw1QqeqjRJdAkq5Ipqc7z/Lm+7U0DJIUlsol4RWhSNFSiTr8AoIURQcwQTe?=
 =?us-ascii?Q?e9bOP0RRnBMs2F5QjfuwAilu5ZRNX4Au6QkyWmerxmMolg7+p5aG2JVGRvXF?=
 =?us-ascii?Q?jyikUp1+v68lBkKLpmqPz4NgWoqcDxJuLG/k+GndDM1YNDQLa9XtjEU9z2c4?=
 =?us-ascii?Q?u6Zl84BAIPp4GlNFJmuQItmuUTISFTym5xDW2CWugimxfLJJsVzKxS1RcxKK?=
 =?us-ascii?Q?yYU69zAQq+ioFDEcL4Fga4Xo0tq+h/cFlOEZMYO8UkhT/RdDKXlbwM9ieOEi?=
 =?us-ascii?Q?b3nQjhsZqRP62mTOLh5mzlImPm2gO1mZkhW6g9w9ziq/BQfm3kVGPmmrtorK?=
 =?us-ascii?Q?g/KQp9AzaJq3GMgRFP183+nwIAP77YxQdt9OEzvqiPZy/HKtRf083IYsqyRb?=
 =?us-ascii?Q?CrlQ2JFNPPj60lZYQvKTG6vQ1Hpcs0lzWQIRWiwVJTtJl93g0Cb3O5Tuc2zc?=
 =?us-ascii?Q?bLJm8MrMgblLWNm9/SfVxaLllHz0siF8uE8v+wncWNRdThuIDzQvBjwfaZYR?=
 =?us-ascii?Q?RS6uV23xsUil3whSEjWgBeHQmgXp7916Qws5Ew/wFvA6QG+DLeQiyOUeTRyo?=
 =?us-ascii?Q?PD5Kbx3ed0nSF7wd3aSewOi4YkHIMy/cmYbF8Gex?=
X-MS-Exchange-CrossTenant-Network-Message-Id: de09060d-7d83-4083-3eac-08db34541098
X-MS-Exchange-CrossTenant-AuthSource: SA1PR11MB6733.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 03 Apr 2023 14:59:46.5955
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: Sz6wKaEIAKT7My1zy9dtkqd1uI2pSXDC8LAjm85HUZeWIk0OleRlU8cedI0rwBquPsuYBRH7KxZFMUja/5rYBQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MW3PR11MB4729
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-0.2 required=5.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,SPF_HELO_NONE,SPF_NONE
        autolearn=unavailable autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Khadija Kamran wrote:
> On Sun, Apr 02, 2023 at 05:37:48PM -0700, Ira Weiny wrote:
> > Khadija Kamran wrote:
> > > arche_platform_wd_irq() function has two exit paths. To make the code
> > > more readable, use only one exit path.
> > > 
> > > Suggested-by: Alison Schofield <alison.schofield@intel.com>
> > 
> > Reviewed-by: Ira Weiny <ira.weiny@intel.com>
> >
> 
> Okay, noted.
> 
> Also, would it be okay to send a patch revision with the changes or
> should I wait for the feedback on Dan's comment. Here is a link to it:
> https://lore.kernel.org/outreachy/6ce8aa34-e600-4d6a-adad-ead8255342e5@kili.mountain/

In this case you are going to need to make a v2 of the _series_.  In the
next revision you send both patches again as V2.  (If you are using b4[*]
it will help to track the series versions.)

I am saying this patch looks good but there are still issues with patch
2/2.  If there are no other comments on 1/2 and you don't make any changes
then you include it with my review tag as v2.

When I do this I make a note of picking up tags like so:

---
Changes from V1:
	Add review tags

If you get other comments and make changes then my review is no longer
valid because you have changed the patch.  In that case make those changes
and make a note.  Then I can see why my review was dropped and, more
importantly, I know why I should look at the patch again.

Then in patch 2/2 make changes as needed.

Ira

[*] https://b4.docs.kernel.org/en/latest/