* [PATCH] staging: mt7621-dts: get rid of nodes with no in-tree driver @ 2021-10-18 15:35 Sergio Paracuellos 2021-10-18 15:35 ` Sergio Paracuellos 2021-10-18 15:48 ` Greg KH 0 siblings, 2 replies; 4+ messages in thread From: Sergio Paracuellos @ 2021-10-18 15:35 UTC (permalink / raw) To: linux-staging; +Cc: gregkh, neil 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@gmail.com> --- drivers/staging/mt7621-dts/mt7621.dtsi | 36 -------------------------- 1 file changed, 36 deletions(-) diff --git a/drivers/staging/mt7621-dts/mt7621.dtsi b/drivers/staging/mt7621-dts/mt7621.dtsi index dd7d06c5f234..787220d4d3b4 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 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCH] staging: mt7621-dts: get rid of nodes with no in-tree driver 2021-10-18 15:35 [PATCH] staging: mt7621-dts: get rid of nodes with no in-tree driver Sergio Paracuellos @ 2021-10-18 15:35 ` Sergio Paracuellos 2021-10-18 15:48 ` Greg KH 1 sibling, 0 replies; 4+ messages in thread From: Sergio Paracuellos @ 2021-10-18 15:35 UTC (permalink / raw) To: linux-staging; +Cc: gregkh, neil 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@gmail.com> --- drivers/staging/mt7621-dts/mt7621.dtsi | 36 -------------------------- 1 file changed, 36 deletions(-) diff --git a/drivers/staging/mt7621-dts/mt7621.dtsi b/drivers/staging/mt7621-dts/mt7621.dtsi index dd7d06c5f234..787220d4d3b4 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 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] staging: mt7621-dts: get rid of nodes with no in-tree driver 2021-10-18 15:35 [PATCH] staging: mt7621-dts: get rid of nodes with no in-tree driver Sergio Paracuellos 2021-10-18 15:35 ` Sergio Paracuellos @ 2021-10-18 15:48 ` Greg KH 2021-10-18 17:02 ` Sergio Paracuellos 1 sibling, 1 reply; 4+ messages in thread From: Greg KH @ 2021-10-18 15:48 UTC (permalink / raw) To: Sergio Paracuellos; +Cc: linux-staging, neil On Mon, Oct 18, 2021 at 05:35:53PM +0200, Sergio Paracuellos 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@gmail.com> > --- > drivers/staging/mt7621-dts/mt7621.dtsi | 36 -------------------------- > 1 file changed, 36 deletions(-) > > diff --git a/drivers/staging/mt7621-dts/mt7621.dtsi b/drivers/staging/mt7621-dts/mt7621.dtsi > index dd7d06c5f234..787220d4d3b4 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 When trying to apply this I get: checking file drivers/staging/mt7621-dts/mt7621.dtsi Reversed (or previously applied) patch detected! Assume -R? [n] Apply anyway? [n] Skipping patch. 2 out of 2 hunks ignored Are you sure you rebased? Hm, ah, I can handle this with fuzz maybe: patching file drivers/staging/mt7621-dts/mt7621.dtsi Reversed (or previously applied) patch detected! Assume -R? [n] n Apply anyway? [n] y Hunk #1 FAILED at 102. Hunk #2 succeeded at 358 (offset 3 lines). 1 out of 2 hunks FAILED -- saving rejects to file drivers/staging/mt7621-dts/mt7621.dtsi.rej Nope, that didn't work. Again, are you sure you rebased? Against what tree? thanks, greg k-h ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] staging: mt7621-dts: get rid of nodes with no in-tree driver 2021-10-18 15:48 ` Greg KH @ 2021-10-18 17:02 ` Sergio Paracuellos 0 siblings, 0 replies; 4+ messages in thread From: Sergio Paracuellos @ 2021-10-18 17:02 UTC (permalink / raw) To: Greg KH; +Cc: linux-staging, NeilBrown Hi Greg, On Mon, Oct 18, 2021 at 5:48 PM Greg KH <gregkh@linuxfoundation.org> wrote: > > On Mon, Oct 18, 2021 at 05:35:53PM +0200, Sergio Paracuellos 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@gmail.com> > > --- > > drivers/staging/mt7621-dts/mt7621.dtsi | 36 -------------------------- > > 1 file changed, 36 deletions(-) > > > > diff --git a/drivers/staging/mt7621-dts/mt7621.dtsi b/drivers/staging/mt7621-dts/mt7621.dtsi > > index dd7d06c5f234..787220d4d3b4 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 > > When trying to apply this I get: > > checking file drivers/staging/mt7621-dts/mt7621.dtsi > Reversed (or previously applied) patch detected! Assume -R? [n] > Apply anyway? [n] > Skipping patch. > 2 out of 2 hunks ignored > > Are you sure you rebased? > > Hm, ah, I can handle this with fuzz maybe: > patching file drivers/staging/mt7621-dts/mt7621.dtsi > Reversed (or previously applied) patch detected! Assume -R? [n] n > Apply anyway? [n] y > Hunk #1 FAILED at 102. > Hunk #2 succeeded at 358 (offset 3 lines). > 1 out of 2 hunks FAILED -- saving rejects to file drivers/staging/mt7621-dts/mt7621.dtsi.rej > > Nope, that didn't work. > > Again, are you sure you rebased? Against what tree? You are right, I had another staging-testing branch in a different tree and had one commit in between there. I hopefully fixed it now and sent v2. Thanks and sorry for the inconvenience. Best regards, Sergio Paracuellos > > thanks, > > greg k-h ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2021-10-18 17:02 UTC | newest] Thread overview: 4+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2021-10-18 15:35 [PATCH] staging: mt7621-dts: get rid of nodes with no in-tree driver Sergio Paracuellos 2021-10-18 15:35 ` Sergio Paracuellos 2021-10-18 15:48 ` Greg KH 2021-10-18 17:02 ` Sergio Paracuellos
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox