Message ID | 1442304809-9554-1-git-send-email-jteki@openedev.com |
---|---|
State | Deferred |
Delegated to: | Tom Rini |
Headers | show |
Hi Jagan, On 15 September 2015 at 02:13, Jagan Teki <jteki@openedev.com> wrote: > Hi Tom, > > Zynq patches has SPL build dependency with Simon changes [1]. > > Simon, Any plan to send a PR for your changes. > > [1] http://git.denx.de/?p=u-boot/u-boot-dm.git;a=shortlog;h=refs/heads/zynq-working These are for the zynq maintainer I think. They are not assigned to me in patchwork. > > thanks! > Jagan. > > The following changes since commit 850f788709cef8f7d53d571aec3bfb73b14c5531: > > Merge branch 'rmobile' of git://git.denx.de/u-boot-sh (2015-09-13 17:25:16 -0400) > > are available in the git repository at: > > > git://git.denx.de/u-boot-spi.git master > > for you to fetch changes up to a63981e1727139c51f05d3fa1cb1b299d2c00a7d: > > zynq-common: Add SPL SPI offset, size configs (2015-09-15 13:31:44 +0530) > > ---------------------------------------------------------------- > Jagan Teki (23): > spi: Add zynq qspi controller driver > dts: zynq: Add zynq qspi controller nodes > doc: device-tree-bindings: spi: Add zynq qspi info > dts: microzed: Enable zynq qspi controller node > dts: zc702: Enable zynq qspi controller node > dts: zc706: Enable zynq qspi controller node > dts: zc770-xm010: Enable zynq qspi controller node > dts: zed: Enable zynq qspi controller node > configs: Enable legacy SPI flash interface support > zynq-common: Enable zynq qspi controller support > zynq-common: Enable Bank/Extended address register support > configs: zynq: Enable zynq qspi controller > spi: Kconfig: Add Zynq QSPI controller entry > spi: zynq_spi: Add config reg shift named macros > spi: zynq_spi: Rename baudrate divisor mask name > spi: zynq_spi: Store cs value into private data > sf: params: Add IS25LP032 part support > sf: params: Add IS25LP064 part support > sf: params: Add IS25LP128 part support > zynq-common: Enable ISSI SPI-NOR flash support > spi: xilinx_spi: Fix to configure CPOL, CPHA mask > spi: zynq_spi: Fix to configure CPOL, CPHA mask > spi: zynq_qspi: Fix to configure CPOL, CPHA mask > > Mirza Krak (1): > spi: tegra20: Add support for mode selection > > Siva Durga Prasad Paladugu (1): > zynq-common: Add SPL SPI offset, size configs > > arch/arm/dts/zynq-7000.dtsi | 12 + > arch/arm/dts/zynq-microzed.dts | 5 + > arch/arm/dts/zynq-zc702.dts | 5 + > arch/arm/dts/zynq-zc706.dts | 5 + > arch/arm/dts/zynq-zc770-xm010.dts | 7 +- > arch/arm/dts/zynq-zed.dts | 5 + > configs/zynq_microzed_defconfig | 2 + > configs/zynq_zc702_defconfig | 2 + > configs/zynq_zc706_defconfig | 2 + > configs/zynq_zc70x_defconfig | 2 + > configs/zynq_zc770_xm010_defconfig | 1 + > configs/zynq_zed_defconfig | 2 + > doc/device-tree-bindings/spi/spi-zynq-qspi.txt | 26 ++ > drivers/mtd/spi/sf_params.c | 5 + > drivers/spi/Kconfig | 9 + > drivers/spi/Makefile | 1 + > drivers/spi/tegra20_slink.c | 21 + > drivers/spi/xilinx_spi.c | 10 +- > drivers/spi/zynq_qspi.c | 623 +++++++++++++++++++++++++ > drivers/spi/zynq_spi.c | 27 +- > include/configs/zynq-common.h | 15 + > 21 files changed, 771 insertions(+), 16 deletions(-) > create mode 100644 doc/device-tree-bindings/spi/spi-zynq-qspi.txt > create mode 100644 drivers/spi/zynq_qspi.c Regards, Simon
Hi Michal, On 15 September 2015 at 18:58, Simon Glass <sjg@chromium.org> wrote: > Hi Jagan, > > On 15 September 2015 at 02:13, Jagan Teki <jteki@openedev.com> wrote: >> Hi Tom, >> >> Zynq patches has SPL build dependency with Simon changes [1]. >> >> Simon, Any plan to send a PR for your changes. >> >> [1] http://git.denx.de/?p=u-boot/u-boot-dm.git;a=shortlog;h=refs/heads/zynq-working > > These are for the zynq maintainer I think. They are not assigned to me > in patchwork. Seems like this patch[1] resolves the SPL build issue, any plan on pushing this? of-course I will test this and let you know. [1] http://git.denx.de/?p=u-boot/u-boot-dm.git;a=commitdiff;h=7916a8cf902177afd20a09527a9cf307622bb6d2 >> >> The following changes since commit 850f788709cef8f7d53d571aec3bfb73b14c5531: >> >> Merge branch 'rmobile' of git://git.denx.de/u-boot-sh (2015-09-13 17:25:16 -0400) >> >> are available in the git repository at: >> >> >> git://git.denx.de/u-boot-spi.git master >> >> for you to fetch changes up to a63981e1727139c51f05d3fa1cb1b299d2c00a7d: >> >> zynq-common: Add SPL SPI offset, size configs (2015-09-15 13:31:44 +0530) >> >> ---------------------------------------------------------------- >> Jagan Teki (23): >> spi: Add zynq qspi controller driver >> dts: zynq: Add zynq qspi controller nodes >> doc: device-tree-bindings: spi: Add zynq qspi info >> dts: microzed: Enable zynq qspi controller node >> dts: zc702: Enable zynq qspi controller node >> dts: zc706: Enable zynq qspi controller node >> dts: zc770-xm010: Enable zynq qspi controller node >> dts: zed: Enable zynq qspi controller node >> configs: Enable legacy SPI flash interface support >> zynq-common: Enable zynq qspi controller support >> zynq-common: Enable Bank/Extended address register support >> configs: zynq: Enable zynq qspi controller >> spi: Kconfig: Add Zynq QSPI controller entry >> spi: zynq_spi: Add config reg shift named macros >> spi: zynq_spi: Rename baudrate divisor mask name >> spi: zynq_spi: Store cs value into private data >> sf: params: Add IS25LP032 part support >> sf: params: Add IS25LP064 part support >> sf: params: Add IS25LP128 part support >> zynq-common: Enable ISSI SPI-NOR flash support >> spi: xilinx_spi: Fix to configure CPOL, CPHA mask >> spi: zynq_spi: Fix to configure CPOL, CPHA mask >> spi: zynq_qspi: Fix to configure CPOL, CPHA mask >> >> Mirza Krak (1): >> spi: tegra20: Add support for mode selection >> >> Siva Durga Prasad Paladugu (1): >> zynq-common: Add SPL SPI offset, size configs >> >> arch/arm/dts/zynq-7000.dtsi | 12 + >> arch/arm/dts/zynq-microzed.dts | 5 + >> arch/arm/dts/zynq-zc702.dts | 5 + >> arch/arm/dts/zynq-zc706.dts | 5 + >> arch/arm/dts/zynq-zc770-xm010.dts | 7 +- >> arch/arm/dts/zynq-zed.dts | 5 + >> configs/zynq_microzed_defconfig | 2 + >> configs/zynq_zc702_defconfig | 2 + >> configs/zynq_zc706_defconfig | 2 + >> configs/zynq_zc70x_defconfig | 2 + >> configs/zynq_zc770_xm010_defconfig | 1 + >> configs/zynq_zed_defconfig | 2 + >> doc/device-tree-bindings/spi/spi-zynq-qspi.txt | 26 ++ >> drivers/mtd/spi/sf_params.c | 5 + >> drivers/spi/Kconfig | 9 + >> drivers/spi/Makefile | 1 + >> drivers/spi/tegra20_slink.c | 21 + >> drivers/spi/xilinx_spi.c | 10 +- >> drivers/spi/zynq_qspi.c | 623 +++++++++++++++++++++++++ >> drivers/spi/zynq_spi.c | 27 +- >> include/configs/zynq-common.h | 15 + >> 21 files changed, 771 insertions(+), 16 deletions(-) >> create mode 100644 doc/device-tree-bindings/spi/spi-zynq-qspi.txt >> create mode 100644 drivers/spi/zynq_qspi.c thanks!
On 09/16/2015 09:20 AM, Jagan Teki wrote: > Hi Michal, > > On 15 September 2015 at 18:58, Simon Glass <sjg@chromium.org> wrote: >> Hi Jagan, >> >> On 15 September 2015 at 02:13, Jagan Teki <jteki@openedev.com> wrote: >>> Hi Tom, >>> >>> Zynq patches has SPL build dependency with Simon changes [1]. >>> >>> Simon, Any plan to send a PR for your changes. >>> >>> [1] http://git.denx.de/?p=u-boot/u-boot-dm.git;a=shortlog;h=refs/heads/zynq-working >> >> These are for the zynq maintainer I think. They are not assigned to me >> in patchwork. > > Seems like this patch[1] resolves the SPL build issue, any plan on > pushing this? of-course I will test this and let you know. > > > [1] http://git.denx.de/?p=u-boot/u-boot-dm.git;a=commitdiff;h=7916a8cf902177afd20a09527a9cf307622bb6d2 Simon: Are you going to push it? Or do you want me to collect all of these zynq/zynqmp patches? Thanks, Michal
On 09/19/2015 02:55 AM, Michal Simek wrote: > On 09/16/2015 09:20 AM, Jagan Teki wrote: >> Hi Michal, >> >> On 15 September 2015 at 18:58, Simon Glass <sjg@chromium.org> wrote: >>> Hi Jagan, >>> >>> On 15 September 2015 at 02:13, Jagan Teki <jteki@openedev.com> wrote: >>>> Hi Tom, >>>> >>>> Zynq patches has SPL build dependency with Simon changes [1]. >>>> >>>> Simon, Any plan to send a PR for your changes. >>>> >>>> [1] http://git.denx.de/?p=u-boot/u-boot-dm.git;a=shortlog;h=refs/heads/zynq-working >>> >>> These are for the zynq maintainer I think. They are not assigned to me >>> in patchwork. >> >> Seems like this patch[1] resolves the SPL build issue, any plan on >> pushing this? of-course I will test this and let you know. >> >> >> [1] http://git.denx.de/?p=u-boot/u-boot-dm.git;a=commitdiff;h=7916a8cf902177afd20a09527a9cf307622bb6d2 > > Simon: Are you going to push it? Or do you want me to collect all of > these zynq/zynqmp patches? Sorry I have read Simon's reaction after. I will try to collect all these zynq patches and will send pull request. Thanks, Michal
Hi Michal, On 19 September 2015 at 06:38, Michal Simek <michal.simek@xilinx.com> wrote: > On 09/19/2015 02:55 AM, Michal Simek wrote: >> On 09/16/2015 09:20 AM, Jagan Teki wrote: >>> Hi Michal, >>> >>> On 15 September 2015 at 18:58, Simon Glass <sjg@chromium.org> wrote: >>>> Hi Jagan, >>>> >>>> On 15 September 2015 at 02:13, Jagan Teki <jteki@openedev.com> wrote: >>>>> Hi Tom, >>>>> >>>>> Zynq patches has SPL build dependency with Simon changes [1]. >>>>> >>>>> Simon, Any plan to send a PR for your changes. >>>>> >>>>> [1] http://git.denx.de/?p=u-boot/u-boot-dm.git;a=shortlog;h=refs/heads/zynq-working >>>> >>>> These are for the zynq maintainer I think. They are not assigned to me >>>> in patchwork. >>> >>> Seems like this patch[1] resolves the SPL build issue, any plan on >>> pushing this? of-course I will test this and let you know. >>> >>> >>> [1] http://git.denx.de/?p=u-boot/u-boot-dm.git;a=commitdiff;h=7916a8cf902177afd20a09527a9cf307622bb6d2 >> >> Simon: Are you going to push it? Or do you want me to collect all of >> these zynq/zynqmp patches? > > Sorry I have read Simon's reaction after. I will try to collect all > these zynq patches and will send pull request. Please send your PR on top of these changes, and let me know for any inputs. thanks!
On 21 September 2015 at 23:56, Jagan Teki <jteki@openedev.com> wrote: > Hi Michal, > > On 19 September 2015 at 06:38, Michal Simek <michal.simek@xilinx.com> wrote: >> On 09/19/2015 02:55 AM, Michal Simek wrote: >>> On 09/16/2015 09:20 AM, Jagan Teki wrote: >>>> Hi Michal, >>>> >>>> On 15 September 2015 at 18:58, Simon Glass <sjg@chromium.org> wrote: >>>>> Hi Jagan, >>>>> >>>>> On 15 September 2015 at 02:13, Jagan Teki <jteki@openedev.com> wrote: >>>>>> Hi Tom, >>>>>> >>>>>> Zynq patches has SPL build dependency with Simon changes [1]. >>>>>> >>>>>> Simon, Any plan to send a PR for your changes. >>>>>> >>>>>> [1] http://git.denx.de/?p=u-boot/u-boot-dm.git;a=shortlog;h=refs/heads/zynq-working >>>>> >>>>> These are for the zynq maintainer I think. They are not assigned to me >>>>> in patchwork. >>>> >>>> Seems like this patch[1] resolves the SPL build issue, any plan on >>>> pushing this? of-course I will test this and let you know. >>>> >>>> >>>> [1] http://git.denx.de/?p=u-boot/u-boot-dm.git;a=commitdiff;h=7916a8cf902177afd20a09527a9cf307622bb6d2 >>> >>> Simon: Are you going to push it? Or do you want me to collect all of >>> these zynq/zynqmp patches? >> >> Sorry I have read Simon's reaction after. I will try to collect all >> these zynq patches and will send pull request. > > Please send your PR on top of these changes, and let me know for any inputs. Seems like Michal is busy with something, Shall I go with build fix (not sure whether it's working SPI-SPL or not) -- Jagan.
On 09/29/2015 11:13 PM, Jagan Teki wrote: > On 21 September 2015 at 23:56, Jagan Teki <jteki@openedev.com> wrote: >> Hi Michal, >> >> On 19 September 2015 at 06:38, Michal Simek <michal.simek@xilinx.com> wrote: >>> On 09/19/2015 02:55 AM, Michal Simek wrote: >>>> On 09/16/2015 09:20 AM, Jagan Teki wrote: >>>>> Hi Michal, >>>>> >>>>> On 15 September 2015 at 18:58, Simon Glass <sjg@chromium.org> wrote: >>>>>> Hi Jagan, >>>>>> >>>>>> On 15 September 2015 at 02:13, Jagan Teki <jteki@openedev.com> wrote: >>>>>>> Hi Tom, >>>>>>> >>>>>>> Zynq patches has SPL build dependency with Simon changes [1]. >>>>>>> >>>>>>> Simon, Any plan to send a PR for your changes. >>>>>>> >>>>>>> [1] http://git.denx.de/?p=u-boot/u-boot-dm.git;a=shortlog;h=refs/heads/zynq-working >>>>>> >>>>>> These are for the zynq maintainer I think. They are not assigned to me >>>>>> in patchwork. >>>>> >>>>> Seems like this patch[1] resolves the SPL build issue, any plan on >>>>> pushing this? of-course I will test this and let you know. >>>>> >>>>> >>>>> [1] http://git.denx.de/?p=u-boot/u-boot-dm.git;a=commitdiff;h=7916a8cf902177afd20a09527a9cf307622bb6d2 >>>> >>>> Simon: Are you going to push it? Or do you want me to collect all of >>>> these zynq/zynqmp patches? >>> >>> Sorry I have read Simon's reaction after. I will try to collect all >>> these zynq patches and will send pull request. >> >> Please send your PR on top of these changes, and let me know for any inputs. > > Seems like Michal is busy with something, Shall I go with build fix > (not sure whether it's working SPI-SPL or not) Next week I will look at it. Thanks, Michal
Hi Jagan, On 09/30/2015 12:23 AM, Michal Simek wrote: > On 09/29/2015 11:13 PM, Jagan Teki wrote: >> On 21 September 2015 at 23:56, Jagan Teki <jteki@openedev.com> wrote: >>> Hi Michal, >>> >>> On 19 September 2015 at 06:38, Michal Simek <michal.simek@xilinx.com> wrote: >>>> On 09/19/2015 02:55 AM, Michal Simek wrote: >>>>> On 09/16/2015 09:20 AM, Jagan Teki wrote: >>>>>> Hi Michal, >>>>>> >>>>>> On 15 September 2015 at 18:58, Simon Glass <sjg@chromium.org> wrote: >>>>>>> Hi Jagan, >>>>>>> >>>>>>> On 15 September 2015 at 02:13, Jagan Teki <jteki@openedev.com> wrote: >>>>>>>> Hi Tom, >>>>>>>> >>>>>>>> Zynq patches has SPL build dependency with Simon changes [1]. >>>>>>>> >>>>>>>> Simon, Any plan to send a PR for your changes. >>>>>>>> >>>>>>>> [1] http://git.denx.de/?p=u-boot/u-boot-dm.git;a=shortlog;h=refs/heads/zynq-working >>>>>>> >>>>>>> These are for the zynq maintainer I think. They are not assigned to me >>>>>>> in patchwork. >>>>>> >>>>>> Seems like this patch[1] resolves the SPL build issue, any plan on >>>>>> pushing this? of-course I will test this and let you know. >>>>>> >>>>>> >>>>>> [1] http://git.denx.de/?p=u-boot/u-boot-dm.git;a=commitdiff;h=7916a8cf902177afd20a09527a9cf307622bb6d2 >>>>> >>>>> Simon: Are you going to push it? Or do you want me to collect all of >>>>> these zynq/zynqmp patches? >>>> >>>> Sorry I have read Simon's reaction after. I will try to collect all >>>> these zynq patches and will send pull request. >>> >>> Please send your PR on top of these changes, and let me know for any inputs. >> >> Seems like Michal is busy with something, Shall I go with build fix >> (not sure whether it's working SPI-SPL or not) > > Next week I will look at it. All your patches will go on the top of zynq/zynqmp patches and the merge will happen after 2015.10. Is there any particular reason to push these changes before 2015.10? Thanks, Michal
On Fri, Oct 09, 2015 at 03:47:59PM +0200, Michal Simek wrote: > Hi Jagan, > > On 09/30/2015 12:23 AM, Michal Simek wrote: > > On 09/29/2015 11:13 PM, Jagan Teki wrote: > >> On 21 September 2015 at 23:56, Jagan Teki <jteki@openedev.com> wrote: > >>> Hi Michal, > >>> > >>> On 19 September 2015 at 06:38, Michal Simek <michal.simek@xilinx.com> wrote: > >>>> On 09/19/2015 02:55 AM, Michal Simek wrote: > >>>>> On 09/16/2015 09:20 AM, Jagan Teki wrote: > >>>>>> Hi Michal, > >>>>>> > >>>>>> On 15 September 2015 at 18:58, Simon Glass <sjg@chromium.org> wrote: > >>>>>>> Hi Jagan, > >>>>>>> > >>>>>>> On 15 September 2015 at 02:13, Jagan Teki <jteki@openedev.com> wrote: > >>>>>>>> Hi Tom, > >>>>>>>> > >>>>>>>> Zynq patches has SPL build dependency with Simon changes [1]. > >>>>>>>> > >>>>>>>> Simon, Any plan to send a PR for your changes. > >>>>>>>> > >>>>>>>> [1] http://git.denx.de/?p=u-boot/u-boot-dm.git;a=shortlog;h=refs/heads/zynq-working > >>>>>>> > >>>>>>> These are for the zynq maintainer I think. They are not assigned to me > >>>>>>> in patchwork. > >>>>>> > >>>>>> Seems like this patch[1] resolves the SPL build issue, any plan on > >>>>>> pushing this? of-course I will test this and let you know. > >>>>>> > >>>>>> > >>>>>> [1] http://git.denx.de/?p=u-boot/u-boot-dm.git;a=commitdiff;h=7916a8cf902177afd20a09527a9cf307622bb6d2 > >>>>> > >>>>> Simon: Are you going to push it? Or do you want me to collect all of > >>>>> these zynq/zynqmp patches? > >>>> > >>>> Sorry I have read Simon's reaction after. I will try to collect all > >>>> these zynq patches and will send pull request. > >>> > >>> Please send your PR on top of these changes, and let me know for any inputs. > >> > >> Seems like Michal is busy with something, Shall I go with build fix > >> (not sure whether it's working SPI-SPL or not) > > > > Next week I will look at it. > > All your patches will go on the top of zynq/zynqmp patches and the merge > will happen after 2015.10. > Is there any particular reason to push these changes before 2015.10? I had/have been holding this for after the merge window. If there are any release-critical fixes in the SPI tree they need to be put into a separate pull request asap. Thanks!
Hi Tom, On 10/09/2015 03:54 PM, Tom Rini wrote: > On Fri, Oct 09, 2015 at 03:47:59PM +0200, Michal Simek wrote: >> Hi Jagan, >> >> On 09/30/2015 12:23 AM, Michal Simek wrote: >>> On 09/29/2015 11:13 PM, Jagan Teki wrote: >>>> On 21 September 2015 at 23:56, Jagan Teki <jteki@openedev.com> wrote: >>>>> Hi Michal, >>>>> >>>>> On 19 September 2015 at 06:38, Michal Simek <michal.simek@xilinx.com> wrote: >>>>>> On 09/19/2015 02:55 AM, Michal Simek wrote: >>>>>>> On 09/16/2015 09:20 AM, Jagan Teki wrote: >>>>>>>> Hi Michal, >>>>>>>> >>>>>>>> On 15 September 2015 at 18:58, Simon Glass <sjg@chromium.org> wrote: >>>>>>>>> Hi Jagan, >>>>>>>>> >>>>>>>>> On 15 September 2015 at 02:13, Jagan Teki <jteki@openedev.com> wrote: >>>>>>>>>> Hi Tom, >>>>>>>>>> >>>>>>>>>> Zynq patches has SPL build dependency with Simon changes [1]. >>>>>>>>>> >>>>>>>>>> Simon, Any plan to send a PR for your changes. >>>>>>>>>> >>>>>>>>>> [1] http://git.denx.de/?p=u-boot/u-boot-dm.git;a=shortlog;h=refs/heads/zynq-working >>>>>>>>> >>>>>>>>> These are for the zynq maintainer I think. They are not assigned to me >>>>>>>>> in patchwork. >>>>>>>> >>>>>>>> Seems like this patch[1] resolves the SPL build issue, any plan on >>>>>>>> pushing this? of-course I will test this and let you know. >>>>>>>> >>>>>>>> >>>>>>>> [1] http://git.denx.de/?p=u-boot/u-boot-dm.git;a=commitdiff;h=7916a8cf902177afd20a09527a9cf307622bb6d2 >>>>>>> >>>>>>> Simon: Are you going to push it? Or do you want me to collect all of >>>>>>> these zynq/zynqmp patches? >>>>>> >>>>>> Sorry I have read Simon's reaction after. I will try to collect all >>>>>> these zynq patches and will send pull request. >>>>> >>>>> Please send your PR on top of these changes, and let me know for any inputs. >>>> >>>> Seems like Michal is busy with something, Shall I go with build fix >>>> (not sure whether it's working SPI-SPL or not) >>> >>> Next week I will look at it. >> >> All your patches will go on the top of zynq/zynqmp patches and the merge >> will happen after 2015.10. >> Is there any particular reason to push these changes before 2015.10? > > I had/have been holding this for after the merge window. If there are > any release-critical fixes in the SPI tree they need to be put into a > separate pull request asap. Thanks! Good. I was talking to Simon yesterday and retest zynq and zynqmp SPL DM changes which should go first to your tree and then these spi changes should go on the top. Thanks, Michal
On 9 October 2015 at 19:25, Michal Simek <michal.simek@xilinx.com> wrote: > Hi Tom, > > On 10/09/2015 03:54 PM, Tom Rini wrote: >> On Fri, Oct 09, 2015 at 03:47:59PM +0200, Michal Simek wrote: >>> Hi Jagan, >>> >>> On 09/30/2015 12:23 AM, Michal Simek wrote: >>>> On 09/29/2015 11:13 PM, Jagan Teki wrote: >>>>> On 21 September 2015 at 23:56, Jagan Teki <jteki@openedev.com> wrote: >>>>>> Hi Michal, >>>>>> >>>>>> On 19 September 2015 at 06:38, Michal Simek <michal.simek@xilinx.com> wrote: >>>>>>> On 09/19/2015 02:55 AM, Michal Simek wrote: >>>>>>>> On 09/16/2015 09:20 AM, Jagan Teki wrote: >>>>>>>>> Hi Michal, >>>>>>>>> >>>>>>>>> On 15 September 2015 at 18:58, Simon Glass <sjg@chromium.org> wrote: >>>>>>>>>> Hi Jagan, >>>>>>>>>> >>>>>>>>>> On 15 September 2015 at 02:13, Jagan Teki <jteki@openedev.com> wrote: >>>>>>>>>>> Hi Tom, >>>>>>>>>>> >>>>>>>>>>> Zynq patches has SPL build dependency with Simon changes [1]. >>>>>>>>>>> >>>>>>>>>>> Simon, Any plan to send a PR for your changes. >>>>>>>>>>> >>>>>>>>>>> [1] http://git.denx.de/?p=u-boot/u-boot-dm.git;a=shortlog;h=refs/heads/zynq-working >>>>>>>>>> >>>>>>>>>> These are for the zynq maintainer I think. They are not assigned to me >>>>>>>>>> in patchwork. >>>>>>>>> >>>>>>>>> Seems like this patch[1] resolves the SPL build issue, any plan on >>>>>>>>> pushing this? of-course I will test this and let you know. >>>>>>>>> >>>>>>>>> >>>>>>>>> [1] http://git.denx.de/?p=u-boot/u-boot-dm.git;a=commitdiff;h=7916a8cf902177afd20a09527a9cf307622bb6d2 >>>>>>>> >>>>>>>> Simon: Are you going to push it? Or do you want me to collect all of >>>>>>>> these zynq/zynqmp patches? >>>>>>> >>>>>>> Sorry I have read Simon's reaction after. I will try to collect all >>>>>>> these zynq patches and will send pull request. >>>>>> >>>>>> Please send your PR on top of these changes, and let me know for any inputs. >>>>> >>>>> Seems like Michal is busy with something, Shall I go with build fix >>>>> (not sure whether it's working SPI-SPL or not) >>>> >>>> Next week I will look at it. >>> >>> All your patches will go on the top of zynq/zynqmp patches and the merge >>> will happen after 2015.10. >>> Is there any particular reason to push these changes before 2015.10? >> >> I had/have been holding this for after the merge window. If there are >> any release-critical fixes in the SPI tree they need to be put into a >> separate pull request asap. Thanks! > > Good. I was talking to Simon yesterday and retest zynq and zynqmp SPL DM > changes which should go first to your tree and then these spi changes > should go on the top. No hurry with the zynq qspi patches for this release, will send the PR again after release. Meanwhile, I sent one more for this release [1], pls- pull the same. [1] https://patchwork.ozlabs.org/patch/528729/ -- Jagan.