Received: by 2002:ac0:a5a7:0:0:0:0:0 with SMTP id m36-v6csp2786874imm; Mon, 16 Jul 2018 14:16:42 -0700 (PDT) X-Google-Smtp-Source: AAOMgpcqbPHT732GqqMxDmtXq3cIkMP2yypotYNoMNaR+JLjxrnFA0/97MCXnif1j0gES4aZbzW0 X-Received: by 2002:a62:678f:: with SMTP id t15-v6mr19190378pfj.85.1531775802507; Mon, 16 Jul 2018 14:16:42 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1531775802; cv=none; d=google.com; s=arc-20160816; b=fYf9zv4TJq7NrzTMEM9ODNKQ0bOOtXaciRYiR3gR/TsqhhaUokjG6LzEhftuOX3WuU xhM2hyltarlDnfhFh43mjEzyaFatQMkoiQwpGP9GQsVIR4XndD1TjlHMWZYg+3I4xr/F 3B3KTvBfxcx2LDDJnjNMeNPDIXkMz1bl7z8ef0xPcitHHBLpopQMKyUYmR4hTwQydmgK zRrwNyHSkjGilS5iJTrw5TIB1lzF74/8LndSX3fNPuQvSD2dv3panvMC0B84kl3/ucWn 74Z6hTuXHtHi/duJ7xipv4WNttxzs2SzL6MZqx6o1Nv5tuvVHOJUJOJPDSF/U2JX6X5r U+XA== 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 :references:in-reply-to:mime-version:dkim-signature :arc-authentication-results; bh=8Pnok3buutn8JR8JlnE4KBdJPSkhyNW/nef+ikpRMko=; b=dy2SBSAmpX8EuC/WyqtTEWk1/I22Un1cr+LH0jMjJam79H9XfxxsYJmZQFCMENb9Qc Q2FYxdnf1y9cvO60aypueTndXrJiogCf/pz85Sg8g9N0xoSYWk4HMvPzDk6uuBZep95O SogrDvWsx4kvurXa8Ml+6xLrQyGG8CDNDD59cqlAHLcOsFi0V1cIxSG8to8UGQsG0THa zb9eDQRohXqgpmzSg7H66tWXiRg+CLEVTO9yEe9kMHpU+4cPKFD0bkgTUPQFjrBuTSNE ZtmYIKJtjLebyToADVQJ3B4W0uqRZN9tbdpZx4xo3TubjAiQsIs2ygVWvP0ReD4fc//L dSSA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=CBzpNsul; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 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. [209.132.180.67]) by mx.google.com with ESMTP id l6-v6si6581756pgl.567.2018.07.16.14.16.27; Mon, 16 Jul 2018 14:16:42 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=CBzpNsul; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 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 S1729791AbeGPVpG (ORCPT + 99 others); Mon, 16 Jul 2018 17:45:06 -0400 Received: from mail-yb0-f177.google.com ([209.85.213.177]:43307 "EHLO mail-yb0-f177.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728413AbeGPVpF (ORCPT ); Mon, 16 Jul 2018 17:45:05 -0400 Received: by mail-yb0-f177.google.com with SMTP id x10-v6so15997961ybl.10 for ; Mon, 16 Jul 2018 14:15:53 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc; bh=8Pnok3buutn8JR8JlnE4KBdJPSkhyNW/nef+ikpRMko=; b=CBzpNsulYTQgVLFIBLsJkAsT6TH3UdnuFbTncbGXDsAzP1HTcyfE5xIGVPNTIixVNE 48+CWLQWFiVTWhjiGaRA1en5/VOfdpo+qd+/dTZAMZ0TnRtGlqa3+NyzI28Xa3J4JgHl PmMeFnBbf/Hnpt1NyarU90sy0OId26UFkJErxHRpaP1BLh/XqKm6fAvjL5kE3qcaGwc9 OuY1JVZ0fQmqw0VLBMlwFAeyLPAmMIu8Hm9O5xEfTVIhbRy/sKWtVNCkiXYnPsb8Ull4 8qTSAS3v1tzzQN6iMEEhhjYxByj5VOVtfsmmaUvNud6kdDtRNJuAFbLnzRd+7atMOTmq eNQA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=8Pnok3buutn8JR8JlnE4KBdJPSkhyNW/nef+ikpRMko=; b=NFcsXukYOmSIZAWNpbmRiUpjbC7EW1EIMdH+dlxzkMetVNScT0NlV73NeQAyStL9/G sBREcLfK6HwH1H2NrxtXOE2Xw3MpLuK8t7F2LjUhIe91t92rIAu2bBM5Z26BBLg/2YlH SzrhrZ03v+pxKdv9ZuRO5DBBz8Xem/y69Vt62yn48d0mzB0BEv+71k22KIq4VUgjy0F0 EgQzTnIuTKWlMHatf7G1zI+fB/FynhFzmsXh5yKTHcM88Iasg86nXY395bMN9jHvH/Rk R+4WnTkvYGS5iCB/wPjAVnylJcCOOJqXf5zVBvT8ZOLEwX0c9LN3idIqYIldEWjI8scb cAWg== X-Gm-Message-State: AOUpUlH/2xv5Qocu0fU+yOnlPFWAdtF5HdNkPFF9xsmVe7OPY+iilTQI i4QOhAEyh+JyBul2/z/eiB9EkiXrix0/H1TbSX8= X-Received: by 2002:a25:2f93:: with SMTP id v141-v6mr9385030ybv.214.1531775752467; Mon, 16 Jul 2018 14:15:52 -0700 (PDT) MIME-Version: 1.0 Received: by 2002:a25:5e42:0:0:0:0:0 with HTTP; Mon, 16 Jul 2018 14:15:51 -0700 (PDT) In-Reply-To: <20180716135335.hnvj5kqvsrrwzp3b@mwanda> References: <20180714055816.223754-1-toddpoynor@gmail.com> <20180714055816.223754-5-toddpoynor@gmail.com> <20180716135335.hnvj5kqvsrrwzp3b@mwanda> From: Todd Poynor Date: Mon, 16 Jul 2018 14:15:51 -0700 Message-ID: Subject: Re: [PATCH 04/18] staging: gasket: device registration error and unregister fixups To: Dan Carpenter Cc: Rob Springer , John Joseph , Ben Chan , Greg Kroah-Hartman , devel@driverdev.osuosl.org, Zhongze Hu , lkml , Simon Que , Guenter Roeck , Todd Poynor , Dmitry Torokhov 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 >> fail1: >> class_destroy(internal->class); >> >> +fail0: > > GW-BASIC sytle label names are an anti-pattern. It's better to name the > labels after what they do, just like function names describes what the > function does. Here it would be "goto clear_desc;" or something. It > doesn't have to be perfect but so long as you *try* to name it something > useful that's better than not trying. > >> + mutex_lock(&g_mutex); >> g_descs[desc_idx].driver_desc = NULL; >> + mutex_unlock(&g_mutex); >> return ret; >> } > > regards, > dan carpenter Thanks, I'll fix up the newly-added one in a respin of this patch and fix up existing labels in a future patch. -- Todd