From mboxrd@z Thu Jan 1 00:00:00 1970 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Subject: [net-next,11/19] net: usb: aqc111: Add support for changing MTU From: Andrew Lunn Message-Id: <20181006165603.GA6990@lunn.ch> Date: Sat, 6 Oct 2018 18:56:03 +0200 To: Igor Russkikh Cc: "David S . Miller" , "linux-usb@vger.kernel.org" , "netdev@vger.kernel.org" , Dmitry Bezrukov List-ID: PiArc3RhdGljIGludCBhcWMxMTFfY2hhbmdlX210dShzdHJ1Y3QgbmV0X2RldmljZSAqbmV0LCBp bnQgbmV3X210dSkKPiArewo+ICsJc3RydWN0IHVzYm5ldCAqZGV2ID0gbmV0ZGV2X3ByaXYobmV0 KTsKPiArCXUxNiByZWcxNiA9IDA7Cj4gKwl1OCBidWZbNV07Cj4gKwo+ICsJaWYgKG5ld19tdHUg PD0gMCB8fCBuZXdfbXR1ID4gMTYzMzQpIHsKPiArCQluZXRkZXZfaW5mbyhuZXQsICJJbnZhbGlk IE1UVSAlZCByZXF1ZXN0ZWQsIGh3IG1heCAxNjMzNCIsCj4gKwkJCSAgICBuZXdfbXR1KTsKPiAr CQlyZXR1cm4gLUVJTlZBTDsKPiArCX0KClBsZWFzZSBzZXQgbmV0LT5taW5fbXR1LCBhbmQgbmV0 LT5tYXhfbXR1LCBhbmQgdGhlIGNvcmUgc2hvdWxkIGRvIHRoaXMKY2hlY2tpbmcgZm9yIHlvdS4g U2VlIGRldl9zZXRfbXR1X2V4dCgpLgoKCSBBbmRyZXcK From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Lunn Subject: Re: [PATCH net-next 11/19] net: usb: aqc111: Add support for changing MTU Date: Sat, 6 Oct 2018 18:56:03 +0200 Message-ID: <20181006165603.GA6990@lunn.ch> References: <4217a1b53f61c5da8f6ba69326c7b759174817e7.1538734658.git.igor.russkikh@aquantia.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: "David S . Miller" , "linux-usb@vger.kernel.org" , "netdev@vger.kernel.org" , Dmitry Bezrukov To: Igor Russkikh Return-path: Received: from vps0.lunn.ch ([185.16.172.187]:55771 "EHLO vps0.lunn.ch" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726256AbeJGAAM (ORCPT ); Sat, 6 Oct 2018 20:00:12 -0400 Content-Disposition: inline In-Reply-To: <4217a1b53f61c5da8f6ba69326c7b759174817e7.1538734658.git.igor.russkikh@aquantia.com> Sender: netdev-owner@vger.kernel.org List-ID: > +static int aqc111_change_mtu(struct net_device *net, int new_mtu) > +{ > + struct usbnet *dev = netdev_priv(net); > + u16 reg16 = 0; > + u8 buf[5]; > + > + if (new_mtu <= 0 || new_mtu > 16334) { > + netdev_info(net, "Invalid MTU %d requested, hw max 16334", > + new_mtu); > + return -EINVAL; > + } Please set net->min_mtu, and net->max_mtu, and the core should do this checking for you. See dev_set_mtu_ext(). Andrew