From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934312Ab3DGUpK (ORCPT ); Sun, 7 Apr 2013 16:45:10 -0400 Received: from shards.monkeyblade.net ([149.20.54.216]:58207 "EHLO shards.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S934257Ab3DGUpJ (ORCPT ); Sun, 7 Apr 2013 16:45:09 -0400 Date: Sun, 07 Apr 2013 16:45:07 -0400 (EDT) Message-Id: <20130407.164507.795222306786228518.davem@davemloft.net> To: romieu@fr.zoreil.com Cc: hayeswang@realtek.com, netdev@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2 net-next 1/8] r8169: Remove firmware code From: David Miller In-Reply-To: <20130404234229.GA13629@electric-eye.fr.zoreil.com> References: <1364891022-3220-1-git-send-email-hayeswang@realtek.com> <20130404.174717.2232220782916755499.davem@davemloft.net> <20130404234229.GA13629@electric-eye.fr.zoreil.com> X-Mailer: Mew version 6.5 on Emacs 24.1 / Mule 6.0 (HANACHIRUSATO) Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit X-Greylist: Sender succeeded SMTP AUTH, not delayed by milter-greylist-4.2.7 (shards.monkeyblade.net [0.0.0.0]); Sun, 07 Apr 2013 13:45:09 -0700 (PDT) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Francois Romieu Date: Fri, 5 Apr 2013 01:42:29 +0200 > David Miller : > [...] >> If so, should I just apply this series as-is ? > > Yes. > > - the series is imho -stable unfriendly: whoever wants to figure what > should be fed into a -stable branch will have a hard time. :o/ > - the driver could had been more careful about firmware version/magic > checks and firmware opcodes recycling. It's a bit late. It won't > necessarily hurt. > - there is a whole release cycle ahead to find problems - if any - due > to the hw_start flow change. It seems sane. > - the relative amount of binary like cruft is going down. > > I am not overflowed with enthusiasm but the gain should exceed the pain. All applied to net-next, thanks!