From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Miller Subject: Re: [PATCH net] net: dsa: microchip: initialize mutex before use Date: Wed, 31 Oct 2018 12:52:54 -0700 (PDT) Message-ID: <20181031.125254.1993037104948479061.davem@davemloft.net> References: <1540943149-26832-1-git-send-email-Tristram.Ha@microchip.com> Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Cc: andrew@lunn.ch, f.fainelli@gmail.com, pavel@ucw.cz, UNGLinuxDriver@microchip.com, netdev@vger.kernel.org To: Tristram.Ha@microchip.com Return-path: Received: from shards.monkeyblade.net ([23.128.96.9]:53140 "EHLO shards.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726289AbeKAEw1 (ORCPT ); Thu, 1 Nov 2018 00:52:27 -0400 In-Reply-To: <1540943149-26832-1-git-send-email-Tristram.Ha@microchip.com> Sender: netdev-owner@vger.kernel.org List-ID: From: Date: Tue, 30 Oct 2018 16:45:49 -0700 > @@ -1206,6 +1201,12 @@ int ksz_switch_register(struct ksz_device *dev) > if (dev->pdata) > dev->chip_id = dev->pdata->chip_id; > > + /* mutex is used in next function call. */ > + mutex_init(&dev->reg_mutex); > + mutex_init(&dev->stats_mutex); > + mutex_init(&dev->alu_mutex); > + mutex_init(&dev->vlan_mutex); > + Please remove this comment, as per Andrew Lunn's feedback.