From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <stable-owner@vger.kernel.org>
Received: from ozlabs.org ([203.11.71.1]:37777 "EHLO ozlabs.org"
        rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP
        id S2392416AbeLVRBE (ORCPT <rfc822;stable@vger.kernel.org>);
        Sat, 22 Dec 2018 12:01:04 -0500
In-Reply-To: <154445509294.867383.14905585855902294361.stgit@bahia.lan>
To: Greg Kurz <groug@kaod.org>, linuxppc-dev@lists.ozlabs.org
From: Michael Ellerman <patch-notifications@ellerman.id.au>
Cc: Christophe Lombard <clombard@linux.vnet.ibm.com>,
        Vaibhav Jain <vaibhav@linux.vnet.ibm.com>,
        stable@vger.kernel.org,
        Frederic Barrat <fbarrat@linux.vnet.ibm.com>,
        Andrew Donnellan <andrew.donnellan@au1.ibm.com>
Subject: Re: ocxl: Clarify error path in setup_xsl_irq()
Message-Id: <43MLSc5tbHz9sNM@ozlabs.org>
Date: Sat, 22 Dec 2018 20:54:40 +1100 (AEDT)
Sender: stable-owner@vger.kernel.org
List-ID: <stable.vger.kernel.org>

On Mon, 2018-12-10 at 15:18:13 UTC, Greg Kurz wrote:
> Implementing rollback with goto and labels is a common practice that
> leads to prettier and more maintainable code. FWIW, this design pattern
> is already being used in alloc_link() a few lines below in this file.
> 
> Do the same in setup_xsl_irq().
> 
> Signed-off-by: Greg Kurz <groug@kaod.org>
> Acked-by: Frederic Barrat <fbarrat@linux.ibm.com>
> Acked-by: Andrew Donnellan <andrew.donnellan@au1.ibm.com>

Applied to powerpc next, thanks.

https://git.kernel.org/powerpc/c/759bc01586535700fbb1aaf40dd727

cheers