Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754214AbdGUPCh (ORCPT ); Fri, 21 Jul 2017 11:02:37 -0400 Received: from mail.linuxfoundation.org ([140.211.169.12]:39218 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752003AbdGUPBl (ORCPT ); Fri, 21 Jul 2017 11:01:41 -0400 Date: Fri, 21 Jul 2017 17:01:33 +0200 From: Greg KH To: Quentin Schulz Cc: ulf.hansson@linaro.org, linus.walleij@linaro.org, shawn.lin@rock-chips.com, adrian.hunter@intel.com, baolin.wang@linaro.org, hdegoede@redhat.com, maxime.ripard@free-electrons.com, thomas.petazzoni@free-electrons.com, linux-kernel@vger.kernel.org, linux-mmc@vger.kernel.org, devel@driverdev.osuosl.org, icenowy@aosc.xyz, wens@csie.org Subject: Re: [PATCH 1/2] staging: net: wireless: add ESP8089 WiFi driver Message-ID: <20170721150133.GA18198@kroah.com> References: <20170721143502.1991-1-quentin.schulz@free-electrons.com> <20170721143502.1991-2-quentin.schulz@free-electrons.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170721143502.1991-2-quentin.schulz@free-electrons.com> User-Agent: Mutt/1.8.3 (2017-05-23) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 945 Lines: 26 On Fri, Jul 21, 2017 at 04:35:01PM +0200, Quentin Schulz wrote: > The Espressif ESP8089 WiFi chips can be often found in cheap tablets. > There is one in A23 Polaroid tablets for example. > > The chip is often embedded as an eMMC SDIO device. > > The code was taken from an out-of-tree repository and has seen a first > pass in the cleanup process. > > At the moment, there is no publicly available datasheet for this chip. > > Cc: Hans de Goede > Cc: Icenowy Zheng > Signed-off-by: Quentin Schulz Staging drivers need a TODO file that lists what has to be done to the code to get it out of staging. Why not just take a day or so and fix up the remaining issues and get it into the "real" part of the kernel correctly? Also, staging drivers have to be "stand-alone", I can't take stuff that requires core changes only for one staging driver. thanks, greg k-h