Received: by 2002:a05:6a10:22f:0:0:0:0 with SMTP id 15csp216154pxk; Thu, 17 Sep 2020 00:40:08 -0700 (PDT) X-Google-Smtp-Source: ABdhPJzmsogP60P1YhUMymSuOaVIwjCkaEk5xc5QUJ9CcJs6ZQivSrRDP7ydVKnPzomLn10eT5vx X-Received: by 2002:a50:e80a:: with SMTP id e10mr31858801edn.4.1600328408051; Thu, 17 Sep 2020 00:40:08 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1600328408; cv=none; d=google.com; s=arc-20160816; b=rVqvWTdWwFssLKsqQr4ts0oUYcFSCMduUNlAOaYvptUJ5EeEstk68fItP6RInqAh74 RSWFRlDmZuyrD6+Sp5LvYsMoBTmpI/FONSS9yWYpz1W0IC5jFsHvsJQeqahedmtZ076C NkAxiwHb1/U7B1L1+hxFPR5npxRQPO/ddj4gxtAJ/bRG32OgeF6Fj6PSmaPZWZ4EERPS YbEmQdDo8jgkSI01H61x+gr3EsN7S1B+jJ7KzPj+8E4dfJVBNY3e5mNtzWaLyz7sbgT0 Nlq/SiluOUdHIKEFT60BH12GJG3iag9q7iHyUGp7PRQJvYSt6WZfiYseiTDqgTt+l7xl Oraw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:date:cc:to:from:subject:message-id :dkim-signature; bh=cSlCNcl11u7qHTtN7BxhpCgzqViFmJKikDj8COuqZE8=; b=as91TndUgdhznprdDPTEPOHoICOFW/7p9MUVKiITpEjZPZHkBNFvJ41ITI4bstSR78 aMJf2SoFgvWU5ZqMiaIJa7W/Civl+EeyF2Vty65u5qNAoOyRJtmmSF5ZlsO0E4K8EUBw 3kFWmTeW4D31wXruG2hYJWkof3XKUs9EdkQvOGQKng3iVxi/3INYz2jvH8aIVfFM5MqU L/2B3rsJ6bW507VdewrIKhnH3vcI3nxl4cIB7x1UnyopMPt5JIgAXI/vJpKTxOXOZp/V fC533SwIR6jgLvrDDjgHxCG3T+ziiQt+gK32BdJ+f3VecCboPLyBiefrgt0rgR4pR4CI Jryw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@mediatek.com header.s=dk header.b=OOGU1T9k; 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=NONE dis=NONE) header.from=mediatek.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id v21si13413329edl.133.2020.09.17.00.39.45; Thu, 17 Sep 2020 00:40: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=@mediatek.com header.s=dk header.b=OOGU1T9k; 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=NONE dis=NONE) header.from=mediatek.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726290AbgIQHi2 (ORCPT + 99 others); Thu, 17 Sep 2020 03:38:28 -0400 Received: from mailgw01.mediatek.com ([216.200.240.184]:22414 "EHLO mailgw01.mediatek.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726236AbgIQHiR (ORCPT ); Thu, 17 Sep 2020 03:38:17 -0400 X-Greylist: delayed 311 seconds by postgrey-1.27 at vger.kernel.org; Thu, 17 Sep 2020 03:38:16 EDT X-UUID: 007f8bb6d6fe4cce89c2335e6e602d91-20200916 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=mediatek.com; s=dk; h=Content-Transfer-Encoding:MIME-Version:Content-Type:References:In-Reply-To:Date:CC:To:From:Subject:Message-ID; bh=cSlCNcl11u7qHTtN7BxhpCgzqViFmJKikDj8COuqZE8=; b=OOGU1T9kbZHyfUTyYSEZ4xcpLiixKJpx9xdV16W1ThP3YNzvqyAdScINoP2cRaA3tasXqbxUK/92B+RC6PDnijMnrH/w1vWlKY9i12d1eCW3bDTP2inY/+9SJEs4iM2eUuU8iy1iYPpYHIBYP7NWG6NbyOtuPcqQ/uK7AoJrLGU=; X-UUID: 007f8bb6d6fe4cce89c2335e6e602d91-20200916 Received: from mtkcas67.mediatek.inc [(172.29.193.45)] by mailgw01.mediatek.com (envelope-from ) (musrelay.mediatek.com ESMTP with TLSv1.2 ECDHE-RSA-AES256-SHA384 256/256) with ESMTP id 74245434; Wed, 16 Sep 2020 23:33:01 -0800 Received: from mtkcas08.mediatek.inc (172.21.101.126) by MTKMBS62N2.mediatek.inc (172.29.193.42) with Microsoft SMTP Server (TLS) id 15.0.1497.2; Thu, 17 Sep 2020 00:32:58 -0700 Received: from [172.21.84.99] (172.21.84.99) by mtkcas08.mediatek.inc (172.21.101.73) with Microsoft SMTP Server id 15.0.1497.2 via Frontend Transport; Thu, 17 Sep 2020 15:32:58 +0800 Message-ID: <1600327978.11746.22.camel@mtksdccf07> Subject: Re: [PATCH] net: dsa: mt7530: Add some return-value checks From: Landen Chao To: Alex Dewar CC: Sean Wang , Andrew Lunn , "Vivien Didelot" , Florian Fainelli , "David S. Miller" , "Jakub Kicinski" , Matthias Brugger , Russell King , "netdev@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "linux-mediatek@lists.infradead.org" , "linux-kernel@vger.kernel.org" Date: Thu, 17 Sep 2020 15:32:58 +0800 In-Reply-To: <20200916195017.34057-1-alex.dewar90@gmail.com> References: <20200916195017.34057-1-alex.dewar90@gmail.com> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.2.3-0ubuntu6 MIME-Version: 1.0 X-MTK: N Content-Transfer-Encoding: base64 Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org SGkgQWxleCwNCg0KVGhhbmtzIGZvciB5b3VyIHJldmlldyBhbmQgZml4aW5nLg0KT24gVGh1LCAy MDIwLTA5LTE3IGF0IDAzOjUwICswODAwLCBBbGV4IERld2FyIHdyb3RlOg0KWy4uXQ0KPiANCj4g SWYgaXQgaXMgbm90IGV4cGVjdGVkIHRoYXQgdGhlc2UgZnVuY3Rpb25zIHdpbGwgdGhyb3cgZXJy b3JzIChpLmUuDQo+IGJlY2F1c2UgdGhlIHBhcmFtZXRlcnMgcGFzc2VkIHdpbGwgYWx3YXlzIGJl IGNvcnJlY3QpLCB3ZSBjb3VsZCBkaXNwZW5zZQ0KPiB3aXRoIHRoZSB1c2Ugb2YgRUlOVkFMIGVy cm9ycyBhbmQganVzdCB1c2UgQlVHKigpIG1hY3JvcyBpbnN0ZWFkLiBMZXQgbWUNCj4ga25vdyBp ZiB5b3UnZCByYXRoZXIgSSBmaXggdGhpbmdzIHVwIGluIHRoYXQgd2F5Lg0KVGhlIGNwdSBwb3J0 IHNldHRpbmcgaXMgcGFzc2VkIGJ5IGR0cy4gVXNlIEVJTlZBTCB0byBjYXRjaCB1bmV4cGVjdGVk DQpzZXR0aW5nIGlzIGZpbmUuDQo+IA0KPiBCZXN0LA0KPiBBbGV4DQo+IA0KPiAgZHJpdmVycy9u ZXQvZHNhL210NzUzMC5jIHwgMTYgKysrKysrKysrKysrLS0tLQ0KPiAgMSBmaWxlIGNoYW5nZWQs IDEyIGluc2VydGlvbnMoKyksIDQgZGVsZXRpb25zKC0pDQo+IA0KPiBkaWZmIC0tZ2l0IGEvZHJp dmVycy9uZXQvZHNhL210NzUzMC5jIGIvZHJpdmVycy9uZXQvZHNhL210NzUzMC5jDQo+IGluZGV4 IDYxMzg4OTQ1ZDMxNi4uMTU3ZDBhMDFmYWFlIDEwMDY0NA0KPiAtLS0gYS9kcml2ZXJzL25ldC9k c2EvbXQ3NTMwLmMNCj4gKysrIGIvZHJpdmVycy9uZXQvZHNhL210NzUzMC5jDQo+IEBAIC05NDUs MTAgKzk0NSwxNCBAQCBzdGF0aWMgaW50DQo+ICBtdDc1M3hfY3B1X3BvcnRfZW5hYmxlKHN0cnVj dCBkc2Ffc3dpdGNoICpkcywgaW50IHBvcnQpDQo+ICB7DQo+ICAJc3RydWN0IG10NzUzMF9wcml2 ICpwcml2ID0gZHMtPnByaXY7DQo+ICsJaW50IHJldDsNCj4gIA0KPiAgCS8qIFNldHVwIG1heCBj YXBhYmlsaXR5IG9mIENQVSBwb3J0IGF0IGZpcnN0ICovDQo+IC0JaWYgKHByaXYtPmluZm8tPmNw dV9wb3J0X2NvbmZpZykNCj4gLQkJcHJpdi0+aW5mby0+Y3B1X3BvcnRfY29uZmlnKGRzLCBwb3J0 KTsNCj4gKwlpZiAocHJpdi0+aW5mby0+Y3B1X3BvcnRfY29uZmlnKSB7DQo+ICsJCXJldCA9IHBy aXYtPmluZm8tPmNwdV9wb3J0X2NvbmZpZyhkcywgcG9ydCk7DQo+ICsJCWlmIChyZXQpDQo+ICsJ CQlyZXR1cm4gcmV0Ow0KPiArCX0NCkhvdyBhYm91dCBjaGVjayByZXR1cm4gdmFsdWUgaW4gY2Fs bGVyIGZ1bmN0aW9uLCBtdDc1MzBfc2V0dXAoKSBhbmQNCm10NzUzMV9zZXR1cCgpLCB0b28/DQog ICAgICAgICAgICAgICAgaWYgKGRzYV9pc19jcHVfcG9ydChkcywgaSkpIHsNCiAgICAgICAgICAg ICAgICAgICAgICAgIHJldCA9IG10NzUzeF9jcHVfcG9ydF9lbmFibGUoZHMsIGkpOw0KCQkJaWYg KHJldCkNCgkJCQlyZXR1cm4gcmV0Ow0KCQl9IGVsc2Ugew0KWy4uXQ0KcmVnYXJkcywNCmxhbmRl bg0KDQo=