On Mon, 2018-04-30 at 14:03 +0530, Krishna Chaitanya wrote: > On Mon, Apr 30, 2018 at 8:10 AM, Pkshih <pkshih@xxxxxxxxxxx> wrote: > > > > > > > -----Original Message----- > > > From: Barry Day [mailto:briselec@xxxxxxxxx] > > > Sent: Saturday, April 28, 2018 6:42 AM > > > To: Pkshih > > > Cc: Kalle Valo; Larry.Finger@xxxxxxxxxxxx; linux-wireless@xxxxxxxxxxxxxxx > > > Subject: Re: [PATCH v3 00/19] rtlwifi: halmac: Add new module halmac > > > > > > On Fri, Apr 27, 2018 at 05:44:16AM +0000, Pkshih wrote: > > > > > > > > The registers reside in driver causes error frequently, because MAC register > > > > is maintained by Realtek's MAC team so they create this module to avoid mistakes. > > > > Another benefit is to make it possible to become a thin driver, because many > > > > common functions are provided, so duplicate code will be reduced. > > > > > > How is it possible to create a thin driver by adding lots more code and layers > > > of indirection ??? and writing it in a way that it won't compile without the > > > code for every type of bus interface even though most modules only use one ? > > > > > As I mentioned in first paragraph "(I use 'driver' in this mail indicates part of > > rtlwifi excluded from this module.)". If this module was seen as a 'lib', rtl8822be > > would be a "thin driver". For bus interface code, I need to add a way to compile > > type of bus interface according to selected chip. > > > > > It's a horrible pile of garbage slapped together by an inexperienced > > > programmer. Its a major deterrent for anyone looking at working on one of > > > the latest realtek drivers. > > > > > This module is designed to support multiple OS including Windows and Linux, and > > many products have used this module and worked well. We hope Linux user can also > > use Realtek's WiFi without additional installation if driver was built. > > In order to submit this module to kernel upstream, we take a lot of effort > > to fit Linux coding conventions (e.g. coding style), and explicit > > suggestions will be helpful for us to continuously improve this module. > > IMHO, this is a common use case for most organizations. I understand > that Linux cannot > accommodate other OSes requirements but is there an approved/recommended way > to upstream an OS agnostic driver? Agnostic drivers are generally > bulkier compared to > Linux-only drivers and also code organization is also different (to > handle other OSes). > Hi Kalle, The state of this patchset was changed to RFC in patchwork, and I look at RFC's meaning in wireless wiki. Do you expect that I will send v4? If so, what do I need to fix in v4? Or, you need more description about this module, please let me know. Thanks PK
