From:   =?ISO-8859-1?Q?J=E9r=F4me?= Pouiller <jerome.pouiller@silabs.com>
To:     Kalle Valo <kvalo@kernel.org>
Cc:     linux-wireless@vger.kernel.org, netdev@vger.kernel.org,
        devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        "David S . Miller" <davem@davemloft.net>,
        devicetree@vger.kernel.org, Rob Herring <robh+dt@kernel.org>,
        linux-mmc@vger.kernel.org,
        Pali =?ISO-8859-1?Q?Roh=E1r?= <pali@kernel.org>,
        Ulf Hansson <ulf.hansson@linaro.org>
Subject: Re: [PATCH v9 23/24] wfx: remove from the staging area
Date:   Wed, 12 Jan 2022 10:32:41 +0100
Message-ID: <65681266.04G08nq4u0@pc-42>
Organization: Silicon Labs
In-Reply-To: <874k69jsv1.fsf@tynnyri.adurom.net>
References: <20220111171424.862764-1-Jerome.Pouiller@silabs.com> <20220111171424.862764-24-Jerome.Pouiller@silabs.com> <874k69jsv1.fsf@tynnyri.adurom.net>
Content-Transfer-Encoding: quoted-printable
Content-Type: text/plain; charset="iso-8859-1"
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?iso-8859-1?Q?g88XobbKW64fWCE9AQsJ8bYt5H+c3tgi1uUbcso84jvdsHbpSSWxPe/7mT?=
 =?iso-8859-1?Q?MFxf+RJ+usFlZcGQqR2rWxALwpjz8IyKGx8BkQRbF0q7XAFDWFG5tVpVsK?=
 =?iso-8859-1?Q?JiWwpINlRGeJQVcl++fmgtmcYv+hT70Y25m27OSy+INkkARftqHKxrIzT6?=
 =?iso-8859-1?Q?NqBA3s0W3cot85z/AV8l4RMlYd8CXopVDc/TdA0xHqiqOgbytQSnDvS+GZ?=
 =?iso-8859-1?Q?gV2K6c9+mtclJIyScsuSUbxp72WV8ly4FOzl0BnV5r2yIC5cgiIHHXGdc4?=
 =?iso-8859-1?Q?eTLZZVeiyYcg1E3eyXjtx54M4GJ1AOBY2iHt0JTD05Hd3bgY9xLgBOqUFD?=
 =?iso-8859-1?Q?oghSO9UdZt7U9NsK9o6ig+7c6RXYKggk2GdRrfpD8ZtCzZl2kCenwmsdVX?=
 =?iso-8859-1?Q?GfQRRZ+QbuBgmztyqv+6HpDhtHmVpdm2rELtwydoDOxdI1vgUXsIsJ+iMA?=
 =?iso-8859-1?Q?gKeQyIXImsblHixra8Juvz2qwKoGGsJB37zE4Ft+R55FP/fg9a1c3Dohy6?=
 =?iso-8859-1?Q?FacvU2BqbehkikvMGJFkgt/HGYrAbm+Fs//uxNhGDMZmutrGhk+ZPMUWOP?=
 =?iso-8859-1?Q?dg+YHTlk43LzlJZXI5/x0gTIt5R59OTGrSOwRwAGWnipwauNgSMKWO+snA?=
 =?iso-8859-1?Q?Gute8UXEU2+ugdOdMhE2ea6EDw3X8qoxk0Wo+3UB5Cv3oQWiAOp5ofqSfF?=
 =?iso-8859-1?Q?4kDXirscoXNFrCc/Rtb2Flf1wuRc11/2pmzaAL9miwWGHuHfxmiA+1hFjH?=
 =?iso-8859-1?Q?xXbGFPTD8NZMyk4B01dI0v5TkGOiF/FePTIws607vhltXG9L4yv1BIkjjj?=
 =?iso-8859-1?Q?4QlhNjCFWk7OoNz1euY9amalu7k6yfA6bKkE25XQz6u8GanufmfZ7ouKV+?=
 =?iso-8859-1?Q?AWdMCKr1Qqhygz+yybyPxxMiasEipBsZvvUo7kAMm0K12hyGQjGJQfzrIb?=
 =?iso-8859-1?Q?W12JP1EkcOgyypl6GIw4HWyPTCLYq0jRL31EV79gngq18L9NvF0Echv3it?=
 =?iso-8859-1?Q?EnOxiknumFJFdqJ34qpEhJZq7+VNlxRh/kQL9e2b8eFCPxDIsaCvT6Q3D5?=
 =?iso-8859-1?Q?2AVZ4AQJCg19TTxBNuwon237ydlbvZG0JvbNnqtA9aIGepmEiEoZbsoN55?=
 =?iso-8859-1?Q?jQD1T11WpHcs3f7SOta3eK2DlXvF7a2hdOzGcmyPdCCDG6MjnMv1s8Fp6K?=
 =?iso-8859-1?Q?rjvJKBs1JCZV6vcSdvbXX368VpFaejbAdUOcH8H5GdTnodVuMxeySR2OdL?=
 =?iso-8859-1?Q?kGJREb396zP8K6uTJh2obKk2dZJWYtbrmk923YSNT6chUSLX0g7FxP2Kl5?=
 =?iso-8859-1?Q?+F8IXIJGrLAnNafe4rFLUHi9GMBFSE62SDnf+6Gij23gIefTpZoqFeL01A?=
 =?iso-8859-1?Q?Wtc3tc9quELA9e8JtmUPV3T5PxMa1TC1gp9YVdCbqjrBJYLdPnzM1R1QNy?=
 =?iso-8859-1?Q?KfvPWt+lHi0h7qtCJz7qEKHKmWPjpTEBQxfT6CQIWb8QcfDICgDlmH/XsC?=
 =?iso-8859-1?Q?54EyQkUn3syAr0tBrolggIkzAt/VORtCsRCxcjmOETRjtO+eQ96M1Iip2l?=
 =?iso-8859-1?Q?gKYbnVNTEh4X4bNfcUUruTDMnnXfy/PKECudGtNHM3NAfJq36dR+yUgsNr?=
 =?iso-8859-1?Q?ltTKq0l2rzYK9pWXjHTRTbObRQ9K+70RDoWkG/pgw/PBE3V1jPhpyHIjM8?=
 =?iso-8859-1?Q?kBYixo3YJUoPPJDL0r/x2pg3sVe7w5RAlZ9e3KEPKUR779M8phfdjWAy/2?=
 =?iso-8859-1?Q?184dAqMCXtYQdGhozLDYyneok=3D?=
X-OriginatorOrg: silabs.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 568646f1-afa6-4286-5d6b-08d9d5ae8110
X-MS-Exchange-CrossTenant-AuthSource: PH0PR11MB5657.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 12 Jan 2022 09:32:52.0226
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 54dbd822-5231-4b20-944d-6f4abcd541fb
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: 9ASOYozi2ldoQuKKnja+CARYpZLmgtt7iOjOQz+8KfDgoAb9a3R/IxurTr6kJO3zihHA10So6Lj24jQj3SIOMw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH0PR11MB5577
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Wednesday 12 January 2022 08:49:54 CET Kalle Valo wrote:
> Jerome Pouiller <Jerome.Pouiller@silabs.com> writes:
>=20
> > From: J=E9r=F4me Pouiller <jerome.pouiller@silabs.com>
> >
> > Signed-off-by: J=E9r=F4me Pouiller <jerome.pouiller@silabs.com>
> > ---
> >  .../bindings/net/wireless/silabs,wfx.yaml     | 125 ---
> >  drivers/staging/wfx/Kconfig                   |   8 -
> >  drivers/staging/wfx/Makefile                  |  25 -
> >  drivers/staging/wfx/bh.c                      | 330 -------
> >  drivers/staging/wfx/bh.h                      |  33 -
> >  drivers/staging/wfx/bus.h                     |  38 -
> >  drivers/staging/wfx/bus_sdio.c                | 272 ------
> >  drivers/staging/wfx/bus_spi.c                 | 271 ------
> >  drivers/staging/wfx/data_rx.c                 |  94 --
> >  drivers/staging/wfx/data_rx.h                 |  18 -
> >  drivers/staging/wfx/data_tx.c                 | 596 -------------
> >  drivers/staging/wfx/data_tx.h                 |  68 --
> >  drivers/staging/wfx/debug.c                   | 365 --------
> >  drivers/staging/wfx/debug.h                   |  19 -
> >  drivers/staging/wfx/fwio.c                    | 405 ---------
> >  drivers/staging/wfx/fwio.h                    |  15 -
> >  drivers/staging/wfx/hif_api_cmd.h             | 555 ------------
> >  drivers/staging/wfx/hif_api_general.h         | 262 ------
> >  drivers/staging/wfx/hif_api_mib.h             | 346 --------
> >  drivers/staging/wfx/hif_rx.c                  | 416 ---------
> >  drivers/staging/wfx/hif_rx.h                  |  17 -
> >  drivers/staging/wfx/hif_tx.c                  | 513 -----------
> >  drivers/staging/wfx/hif_tx.h                  |  60 --
> >  drivers/staging/wfx/hif_tx_mib.c              | 324 -------
> >  drivers/staging/wfx/hif_tx_mib.h              |  49 --
> >  drivers/staging/wfx/hwio.c                    | 352 --------
> >  drivers/staging/wfx/hwio.h                    |  75 --
> >  drivers/staging/wfx/key.c                     | 241 -----
> >  drivers/staging/wfx/key.h                     |  20 -
> >  drivers/staging/wfx/main.c                    | 506 -----------
> >  drivers/staging/wfx/main.h                    |  43 -
> >  drivers/staging/wfx/queue.c                   | 307 -------
> >  drivers/staging/wfx/queue.h                   |  45 -
> >  drivers/staging/wfx/scan.c                    | 149 ----
> >  drivers/staging/wfx/scan.h                    |  22 -
> >  drivers/staging/wfx/sta.c                     | 833=20
------------------
> >  drivers/staging/wfx/sta.h                     |  73 --
> >  drivers/staging/wfx/traces.h                  | 501 -----------
> >  drivers/staging/wfx/wfx.h                     | 164 ----
> >  39 files changed, 8555 deletions(-)
> >  delete mode 100644 drivers/staging/wfx/Documentation/devicetree/
bindings/net/wireless/silabs,wfx.yaml
> >  delete mode 100644 drivers/staging/wfx/Kconfig
> >  delete mode 100644 drivers/staging/wfx/Makefile
> >  delete mode 100644 drivers/staging/wfx/bh.c
> >  delete mode 100644 drivers/staging/wfx/bh.h
> >  delete mode 100644 drivers/staging/wfx/bus.h
> >  delete mode 100644 drivers/staging/wfx/bus_sdio.c
> >  delete mode 100644 drivers/staging/wfx/bus_spi.c
> >  delete mode 100644 drivers/staging/wfx/data_rx.c
> >  delete mode 100644 drivers/staging/wfx/data_rx.h
> >  delete mode 100644 drivers/staging/wfx/data_tx.c
> >  delete mode 100644 drivers/staging/wfx/data_tx.h
> >  delete mode 100644 drivers/staging/wfx/debug.c
> >  delete mode 100644 drivers/staging/wfx/debug.h
> >  delete mode 100644 drivers/staging/wfx/fwio.c
> >  delete mode 100644 drivers/staging/wfx/fwio.h
> >  delete mode 100644 drivers/staging/wfx/hif_api_cmd.h
> >  delete mode 100644 drivers/staging/wfx/hif_api_general.h
> >  delete mode 100644 drivers/staging/wfx/hif_api_mib.h
> >  delete mode 100644 drivers/staging/wfx/hif_rx.c
> >  delete mode 100644 drivers/staging/wfx/hif_rx.h
> >  delete mode 100644 drivers/staging/wfx/hif_tx.c
> >  delete mode 100644 drivers/staging/wfx/hif_tx.h
> >  delete mode 100644 drivers/staging/wfx/hif_tx_mib.c
> >  delete mode 100644 drivers/staging/wfx/hif_tx_mib.h
> >  delete mode 100644 drivers/staging/wfx/hwio.c
> >  delete mode 100644 drivers/staging/wfx/hwio.h
> >  delete mode 100644 drivers/staging/wfx/key.c
> >  delete mode 100644 drivers/staging/wfx/key.h
> >  delete mode 100644 drivers/staging/wfx/main.c
> >  delete mode 100644 drivers/staging/wfx/main.h
> >  delete mode 100644 drivers/staging/wfx/queue.c
> >  delete mode 100644 drivers/staging/wfx/queue.h
> >  delete mode 100644 drivers/staging/wfx/scan.c
> >  delete mode 100644 drivers/staging/wfx/scan.h
> >  delete mode 100644 drivers/staging/wfx/sta.c
> >  delete mode 100644 drivers/staging/wfx/sta.h
> >  delete mode 100644 drivers/staging/wfx/traces.h
> >  delete mode 100644 drivers/staging/wfx/wfx.h
>=20
> I'm not sure what's your plan here, but with staging wireless drivers
> there's usually a simple simple move (git mv) of the driver from
> drivers/staging to drivers/net/wireless. An example here:
>=20
> https://git.kernel.org/linus/5625f965d764
>=20
> What you seem to do here is that you add a new driver to
> drivers/net/wireless and then remove the old driver from
> drivers/staging. And I'm guessing these two drivers are not identical
> and have differences?

Until v7, I have more or less kept in sync this PR and the staging tree.=20
I have been a bit lazy from the v8.

However, I still have the patches in my local tree. I am going to
clean-up them and send them to staging.

--=20
J=E9r=F4me Pouiller