Re: [PATCH 1/2] staging: mt7621-dts: get rid of nodes with no in-tree driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Greg,

On Sun, Oct 17, 2021 at 9:06 AM Sergio Paracuellos
<sergio.paracuellos@xxxxxxxxx> wrote:
>
> Nodes for 'i2s' and 'nand' have no driver present inside the linux tree.
> The normal approach for a dts file to be mainlined is start with those stuff
> which is already mainlined and get rid of the other stuff. If needed it will
> be properly added afterwards together with the suitable device driver. Hence,
> remove both nodes from the device tree include file.
>
> Signed-off-by: Sergio Paracuellos <sergio.paracuellos@xxxxxxxxx>
> ---
>  drivers/staging/mt7621-dts/mt7621.dtsi | 36 --------------------------
>  1 file changed, 36 deletions(-)

I noticed PATCH 2/2 of this series was added but not this one which is
the first one in the series. Since I did not see any mail or something
complaining about it... is it possible you missed it?

Thanks,
     Sergio Paracuellos
>
> diff --git a/drivers/staging/mt7621-dts/mt7621.dtsi b/drivers/staging/mt7621-dts/mt7621.dtsi
> index 72b99d8b4647..fd4ab4006ed8 100644
> --- a/drivers/staging/mt7621-dts/mt7621.dtsi
> +++ b/drivers/staging/mt7621-dts/mt7621.dtsi
> @@ -102,28 +102,6 @@ i2c: i2c@900 {
>                         pinctrl-0 = <&i2c_pins>;
>                 };
>
> -               i2s: i2s@a00 {
> -                       compatible = "mediatek,mt7621-i2s";
> -                       reg = <0xa00 0x100>;
> -
> -                       clocks = <&sysc MT7621_CLK_I2S>;
> -                       clock-names = "i2s";
> -                       resets = <&sysc MT7621_RST_I2S>;
> -                       reset-names = "i2s";
> -
> -                       interrupt-parent = <&gic>;
> -                       interrupts = <GIC_SHARED 16 IRQ_TYPE_LEVEL_HIGH>;
> -
> -                       txdma-req = <2>;
> -                       rxdma-req = <3>;
> -
> -                       dmas = <&gdma 4>,
> -                               <&gdma 6>;
> -                       dma-names = "tx", "rx";
> -
> -                       status = "disabled";
> -               };
> -
>                 memc: syscon@5000 {
>                         compatible = "mediatek,mt7621-memc", "syscon";
>                         reg = <0x5000 0x1000>;
> @@ -355,20 +333,6 @@ cdmm: cdmm@1fbf8000 {
>                 reg = <0x1fbf8000 0x8000>;
>         };
>
> -       nand: nand@1e003000 {
> -               status = "disabled";
> -
> -               compatible = "mediatek,mt7621-nand";
> -               bank-width = <2>;
> -               reg = <0x1e003000 0x800
> -                       0x1e003800 0x800>;
> -               #address-cells = <1>;
> -               #size-cells = <1>;
> -
> -               clocks = <&sysc MT7621_CLK_NAND>;
> -               clock-names = "nand";
> -       };
> -
>         ethernet: ethernet@1e100000 {
>                 compatible = "mediatek,mt7621-eth";
>                 reg = <0x1e100000 0x10000>;
> --
> 2.33.0
>




[Index of Archives]     [Linux Driver Development]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux