Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754191AbcKPV3V convert rfc822-to-8bit (ORCPT ); Wed, 16 Nov 2016 16:29:21 -0500 Received: from mx1.redhat.com ([209.132.183.28]:34626 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753058AbcKPV3S (ORCPT ); Wed, 16 Nov 2016 16:29:18 -0500 From: Jes Sorensen To: John Heenan Cc: Barry Day , =?utf-8?Q?Rafa=C5=82_Mi=C5=82ecki?= , Kalle Valo , linux-wireless@vger.kernel.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] rtl8xxxu: Fix for agressive power saving by rtl8723bu wireless IC References: <20161031183522.GA13315@cube> Date: Wed, 16 Nov 2016 16:29:16 -0500 In-Reply-To: (John Heenan's message of "Wed, 2 Nov 2016 11:13:34 +1000") Message-ID: User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8BIT X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.32]); Wed, 16 Nov 2016 21:29:17 +0000 (UTC) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2387 Lines: 51 John Heenan writes: > Barry Day has submitted real world reports for the 8192eu and 8192cu. > This needs to be acknowledged. I have submitted real world reports for > the 8723bu. Lets get this a little more clear - first of all, I have asked you to investigate which part resolves the problem. Rather than 'I randomly moved something around and it happens to work for me'. > When it comes down to it, it looks like the kernel code changes are > really going to be very trivial to fix this problem and we need to > take the focus off dramatic outbursts over style issues to a strategy > for getting usable results from real world testing. > > Addressing style issues in a dramatic manner to me looks like a mean > sport for maintainers who line up to easy target first time > contributors. This mean attitude comes from the top with a well known > comment about "publicly making fun of people". The polite comments > over style from Joe Perches and Rafał Miłecki are welcomed. Once bad code is in place, it is way harder to get rid of it again. It is *normal* for maintainers to ask contributors to do things correctly. In addition you have been asked repeatedly by multiple people to respect coding style, but every patch you posted violated it again in a different way, instead of spending the little time it would take for you to get it right. > An effective strategy would be to insert some printk statements to > trace what init steps vendor derived drivers do each time > wpa_supplicant is called and ask real world testers to report their > results. This is a lot more productive and less error prone than > laboriously pouring over vendor source code. Alternative drivers that > use vendor code from Realtek is enormously complicated and a huge pain > to make sense of. > > Joe Sorensen's driver code is far easier to make sense of and it is a > shame Realtek don't come to the party. Joe Sorensens's code take takes > advantage of the excellent work of kernel contributors to the mac80211 > driver. Now you are pissing on my name - do you really want to be taken seriously here? > Previous comments I made about enable_rf, rtl8xxxu_start, > rtl8xxxu_init_device etc should be clarified. I will leave it for the > moment as it currently serves no direct useful purpose. I have made it very clear I want this issue resolved, but I want it done right. Jes