| Message ID | 1560942714-13330-3-git-send-email-Ashish.Kumar@nxp.com |
|---|---|
| State | Not Applicable, archived |
| Headers | show |
| Series | [v3,1/2] dt-bindings: spi: spi-fsl-qspi: Add ls2080a compatibility string to bindings | expand |
| Context | Check | Description |
|---|---|---|
| robh/checkpatch | warning | "total: 0 errors, 1 warnings, 8 lines checked" |
On Wed, 19 Jun 2019 16:41:54 +0530, Ashish Kumar wrote: > Signed-off-by: Kuldeep Singh <kuldeep.singh@nxp.com> > Signed-off-by: Ashish Kumar <ashish.kumar@nxp.com> > --- > v3: > Rebase to top > v2: > Convert to patch series and rebasing done on top of tree > > Documentation/devicetree/bindings/spi/spi-fsl-qspi.txt | 2 ++ > 1 file changed, 2 insertions(+) > Reviewed-by: Rob Herring <robh@kernel.org>
> -----Original Message----- > From: Rob Herring <robh@kernel.org> > Sent: Wednesday, July 10, 2019 1:39 AM > To: Ashish Kumar <ashish.kumar@nxp.com> > Cc: devicetree@vger.kernel.org; bbrezillon@kernel.org; broonie@kernel.org; > linux-arm-kernel@lists.infradead.org; linux-mtd@lists.infradead.org; Ashish > Kumar <ashish.kumar@nxp.com>; Kuldeep Singh <kuldeep.singh@nxp.com>; > Ashish Kumar <ashish.kumar@nxp.com> > Subject: [EXT] Re: [Patch v3 2/2] dt-bindings: spi: spi-fsl-qspi: Add bindings of > ls1088a and ls1012a > > Caution: EXT Email > > On Wed, 19 Jun 2019 16:41:54 +0530, Ashish Kumar wrote: > > Signed-off-by: Kuldeep Singh <kuldeep.singh@nxp.com> > > Signed-off-by: Ashish Kumar <ashish.kumar@nxp.com> > > --- > > v3: > > Rebase to top > > v2: > > Convert to patch series and rebasing done on top of tree > > > > Documentation/devicetree/bindings/spi/spi-fsl-qspi.txt | 2 ++ > > 1 file changed, 2 insertions(+) > > > > Reviewed-by: Rob Herring <robh@kernel.org> Hi Leo, I think Rob, is waiting for you ack. Regards Ashish
> -----Original Message----- > From: Ashish Kumar > Sent: Monday, July 29, 2019 4:09 AM > To: Rob Herring <robh@kernel.org>; Leo Li <leoyang.li@nxp.com> > Cc: devicetree@vger.kernel.org; bbrezillon@kernel.org; broonie@kernel.org; > linux-arm-kernel@lists.infradead.org; linux-mtd@lists.infradead.org; > Kuldeep Singh <kuldeep.singh@nxp.com> > Subject: RE: [EXT] Re: [Patch v3 2/2] dt-bindings: spi: spi-fsl-qspi: Add > bindings of ls1088a and ls1012a > > > > > -----Original Message----- > > From: Rob Herring <robh@kernel.org> > > Sent: Wednesday, July 10, 2019 1:39 AM > > To: Ashish Kumar <ashish.kumar@nxp.com> > > Cc: devicetree@vger.kernel.org; bbrezillon@kernel.org; > > broonie@kernel.org; linux-arm-kernel@lists.infradead.org; > > linux-mtd@lists.infradead.org; Ashish Kumar <ashish.kumar@nxp.com>; > > Kuldeep Singh <kuldeep.singh@nxp.com>; Ashish Kumar > > <ashish.kumar@nxp.com> > > Subject: [EXT] Re: [Patch v3 2/2] dt-bindings: spi: spi-fsl-qspi: Add > > bindings of ls1088a and ls1012a > > > > Caution: EXT Email > > > > On Wed, 19 Jun 2019 16:41:54 +0530, Ashish Kumar wrote: > > > Signed-off-by: Kuldeep Singh <kuldeep.singh@nxp.com> > > > Signed-off-by: Ashish Kumar <ashish.kumar@nxp.com> > > > --- > > > v3: > > > Rebase to top > > > v2: > > > Convert to patch series and rebasing done on top of tree > > > > > > Documentation/devicetree/bindings/spi/spi-fsl-qspi.txt | 2 ++ > > > 1 file changed, 2 insertions(+) > > > > > > > Reviewed-by: Rob Herring <robh@kernel.org> > > Hi Leo, > > I think Rob, is waiting for you ack. Binding patches usually go through subsystem tree. So I think this actually need ack from qspi maintainer Han Xu and be picked up by SPI maintainer. Regards, Leo
> -----Original Message----- > From: Leo Li > Sent: Thursday, August 1, 2019 4:59 AM > To: Ashish Kumar <ashish.kumar@nxp.com>; Rob Herring > <robh@kernel.org>; Han Xu <han.xu@nxp.com> > Cc: devicetree@vger.kernel.org; bbrezillon@kernel.org; broonie@kernel.org; > linux-arm-kernel@lists.infradead.org; linux-mtd@lists.infradead.org; Kuldeep > Singh <kuldeep.singh@nxp.com> > Subject: RE: [EXT] Re: [Patch v3 2/2] dt-bindings: spi: spi-fsl-qspi: Add > bindings of ls1088a and ls1012a > > > > > -----Original Message----- > > From: Ashish Kumar > > Sent: Monday, July 29, 2019 4:09 AM > > To: Rob Herring <robh@kernel.org>; Leo Li <leoyang.li@nxp.com> > > Cc: devicetree@vger.kernel.org; bbrezillon@kernel.org; > > broonie@kernel.org; linux-arm-kernel@lists.infradead.org; > > linux-mtd@lists.infradead.org; Kuldeep Singh <kuldeep.singh@nxp.com> > > Subject: RE: [EXT] Re: [Patch v3 2/2] dt-bindings: spi: spi-fsl-qspi: > > Add bindings of ls1088a and ls1012a > > > > > > > > > -----Original Message----- > > > From: Rob Herring <robh@kernel.org> > > > Sent: Wednesday, July 10, 2019 1:39 AM > > > To: Ashish Kumar <ashish.kumar@nxp.com> > > > Cc: devicetree@vger.kernel.org; bbrezillon@kernel.org; > > > broonie@kernel.org; linux-arm-kernel@lists.infradead.org; > > > linux-mtd@lists.infradead.org; Ashish Kumar <ashish.kumar@nxp.com>; > > > Kuldeep Singh <kuldeep.singh@nxp.com>; Ashish Kumar > > > <ashish.kumar@nxp.com> > > > Subject: [EXT] Re: [Patch v3 2/2] dt-bindings: spi: spi-fsl-qspi: > > > Add bindings of ls1088a and ls1012a > > > > > > Caution: EXT Email > > > > > > On Wed, 19 Jun 2019 16:41:54 +0530, Ashish Kumar wrote: > > > > Signed-off-by: Kuldeep Singh <kuldeep.singh@nxp.com> > > > > Signed-off-by: Ashish Kumar <ashish.kumar@nxp.com> > > > > --- > > > > v3: > > > > Rebase to top > > > > v2: > > > > Convert to patch series and rebasing done on top of tree > > > > > > > > Documentation/devicetree/bindings/spi/spi-fsl-qspi.txt | 2 ++ > > > > 1 file changed, 2 insertions(+) > > > > > > > > > > Reviewed-by: Rob Herring <robh@kernel.org> > > > > Hi Leo, > > > > I think Rob, is waiting for you ack. > > Binding patches usually go through subsystem tree. So I think this actually > need ack from qspi maintainer Han Xu and be picked up by SPI maintainer. Ok. Hello Han Xu, Could you please ack this few of my dts patches are dependent on this, waiting to be pushed from Shawn's tree. Regards Ashish > > Regards, > Leo
On Tue, Jul 9, 2019 at 3:09 PM Rob Herring <robh@kernel.org> wrote: > > On Wed, 19 Jun 2019 16:41:54 +0530, Ashish Kumar wrote: > > Signed-off-by: Kuldeep Singh <kuldeep.singh@nxp.com> > > Signed-off-by: Ashish Kumar <ashish.kumar@nxp.com> > > --- > > v3: > > Rebase to top > > v2: > > Convert to patch series and rebasing done on top of tree > > > > Documentation/devicetree/bindings/spi/spi-fsl-qspi.txt | 2 ++ > > 1 file changed, 2 insertions(+) > > > > Reviewed-by: Rob Herring <robh@kernel.org> Acked-by: Han Xu <han.xu@nxp.com> > > ______________________________________________________ > Linux MTD discussion mailing list > http://lists.infradead.org/mailman/listinfo/linux-mtd/
> -----Original Message----- > From: Han Xu <xhnjupt@gmail.com> > Sent: Thursday, August 1, 2019 8:54 PM > To: Rob Herring <robh@kernel.org> > Cc: Ashish Kumar <ashish.kumar@nxp.com>; devicetree@vger.kernel.org; > bbrezillon@kernel.org; Kuldeep Singh <kuldeep.singh@nxp.com>; > broonie@kernel.org; linux-mtd@lists.infradead.org; linux-arm- > kernel@lists.infradead.org > Subject: [EXT] Re: [Patch v3 2/2] dt-bindings: spi: spi-fsl-qspi: Add bindings of > ls1088a and ls1012a > > Caution: EXT Email > > On Tue, Jul 9, 2019 at 3:09 PM Rob Herring <robh@kernel.org> wrote: > > > > On Wed, 19 Jun 2019 16:41:54 +0530, Ashish Kumar wrote: > > > Signed-off-by: Kuldeep Singh <kuldeep.singh@nxp.com> > > > Signed-off-by: Ashish Kumar <ashish.kumar@nxp.com> > > > --- > > > v3: > > > Rebase to top > > > v2: > > > Convert to patch series and rebasing done on top of tree > > > > > > Documentation/devicetree/bindings/spi/spi-fsl-qspi.txt | 2 ++ > > > 1 file changed, 2 insertions(+) > > > > > > > Reviewed-by: Rob Herring <robh@kernel.org> > > Acked-by: Han Xu <han.xu@nxp.com> Hello Mark, Could you please send this patch[1] from your spi tree, It applies seamlessly on https://git.kernel.org/pub/scm/linux/kernel/git/broonie/spi.git/ [1]: http://patchwork.ozlabs.org/patch/1118637/ Regards Ashish > > > > > ______________________________________________________ > > Linux MTD discussion mailing list > > > https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2Flists.inf > radead.org%2Fmailman%2Flistinfo%2Flinux- > mtd%2F&data=02%7C01%7CAshish.Kumar%40nxp.com%7C243dc21894 > ad4c2a651808d716945016%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7 > C1%7C637002698562132108&sdata=6YZyZ6D6DywgLYlp%2BWZfkvyaDr > TKvR56JkOxy0XSD9k%3D&reserved=0 > > > > -- > Sincerely, > > Han XU
On Mon, Aug 05, 2019 at 09:07:47AM +0000, Ashish Kumar wrote: > Could you please send this patch[1] from your spi tree, It applies seamlessly on > https://git.kernel.org/pub/scm/linux/kernel/git/broonie/spi.git/ > > [1]: http://patchwork.ozlabs.org/patch/1118637/ Please don't send content free pings and please allow a reasonable time for review. People get busy, go on holiday, attend conferences and so on so unless there is some reason for urgency (like critical bug fixes) please allow at least a couple of weeks for review. If there have been review comments then people may be waiting for those to be addressed. Sending content free pings adds to the mail volume (if they are seen at all) which is often the problem and since they can't be reviewed directly if something has gone wrong you'll have to resend the patches anyway, so sending again is generally a better approach though there are some other maintainers who like them - if in doubt look at how patches for the subsystem are normally handled.
diff --git a/Documentation/devicetree/bindings/spi/spi-fsl-qspi.txt b/Documentation/devicetree/bindings/spi/spi-fsl-qspi.txt index 6d7c9ec..2c2a3e9 100644 --- a/Documentation/devicetree/bindings/spi/spi-fsl-qspi.txt +++ b/Documentation/devicetree/bindings/spi/spi-fsl-qspi.txt @@ -6,6 +6,8 @@ Required properties: "fsl,ls1021a-qspi", "fsl,ls2080a-qspi" or "fsl,ls1043a-qspi" followed by "fsl,ls1021a-qspi" + "fsl,ls1012a-qspi" followed by "fsl,ls1021a-qspi" + "fsl,ls1088a-qspi" followed by "fsl,ls2080a-qspi" - reg : the first contains the register location and length, the second contains the memory mapping address and length - reg-names: Should contain the reg names "QuadSPI" and "QuadSPI-memory"