From: Kalle Valo <kvalo@codeaurora.org>
To: Ajay Singh <ajay.kathat@microchip.com>
Cc: <devel@driverdev.osuosl.org>, <gregkh@linuxfoundation.org>,
        <linux-wireless@vger.kernel.org>, <ganesh.krishna@microchip.com>,
        <aditya.shankar@microchip.com>, <nicolas.ferre@microchip.com>,
        <claudiu.beznea@microchip.com>, <adham.abozaeid@microchip.com>
Subject: Re: feedback on mainlining wilc1000 staging driver
References: <20180816015243.545cd009@ajaysk-VirtualBox>
Date: Thu, 16 Aug 2018 13:47:40 +0300
In-Reply-To: <20180816015243.545cd009@ajaysk-VirtualBox> (Ajay Singh's message
        of "Thu, 16 Aug 2018 01:52:43 +0530")
Message-ID: <87sh3etwoj.fsf@purkki.adurom.net> (sfid-20180816_124752_510786_52C0D8B7)
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Sender: linux-wireless-owner@vger.kernel.org

Ajay Singh <ajay.kathat@microchip.com> writes:

> Hi Greg,
>
> We all are working on submitting and reviewing patches for wilc1000 in
> staging driver for quite some time. 
>
> We would like to have feedback on the next steps to bring wilc1000
> driver closer to move into the wireless subsystem tree. 
>
> In summary, the following major things from TODO have been addressed in
> staging:
> -remove the defined feature as kernel versions
> -remove OS wrapper functions
> -remove custom debug and tracing functions
> -rework comments and function headers(also coding style)
> -remove build warnings
> -replace all semaphores with mutexes or completions
> -make spi and sdio components coexist in one build
> -turn compile-time platform configuration (BEAGLE_BOARD, PANDA_BOARD,
> PLAT_WMS8304, PLAT_RKXXXX, CUSTOMER_PLATFORM, ...) into run-time
> options that are read from DT
> -replace SIOCDEVPRIVATE commands with generic API functions
> -use wext-core handling instead of private SIOCSIWPRIV implementation

>From wireless point of view: if I see wext mentioned anywhere in the
driver I stop right there. cfg80211 is a hard requirement for us
nowadays.

-- 
Kalle Valo