Re: [PATCH] Staging: silabs si4455 serial driver
From: 'Greg Kroah-Hartman'
Date: Wed Dec 09 2020 - 07:02:02 EST
On Wed, Dec 09, 2020 at 12:09:58PM +0100, Info wrote:
> This is a serial port driver for
> Silicon Labs Si4455 Sub-GHz transciver.
>
> Signed-off-by: József Horváth <info@xxxxxxxxxxx>
Note, your From: line does not match this line, so I can't take this.
But:
> ---
> .../bindings/staging/serial/silabs,si4455.txt | 39 +
staging drivers need to be self-contained, so this should be here. It
needs to be reviewed by the DT maintainers when moving out of staging.
> index 000000000000..aee5c7613b31
> --- /dev/null
> +++ b/drivers/staging/si4455/TODO
> @@ -0,0 +1,3 @@
> +TODO:
> + - add variable packet length support
> + - add firmware patching support in case of Si4455-C2A
Why are these a requirement to get it out of staging? Why go into
staging at all? Why not go into the 'real' part of the kernel directly?
What is keeping that from happening today?
These look like new features that you can add later, and shouldn't be a
requirement for acceptance into the normal part of the kernel for this
driver. Why have you not tried doing that first?
thanks,
greg k-h