Message ID | 52F89AFF.90402@monstr.eu |
---|---|
State | Accepted |
Delegated to: | Albert ARIBAUD |
Headers | show |
Hi Michal, On Mon, 10 Feb 2014 10:25:19 +0100, Michal Simek <monstr@monstr.eu> wrote: > Hi Albert, > > here are all patches which I have in my queue for Xilinx Zynq. > > clk changes depends on clk command which has been added recently. > (common/cmd_clk.c) that's why this tree is based on 3 days old > Tom's tree. > > Thanks, > Michal This has the current tftpboot patch, affecting only the Zynq config. Does it mean you prefer not to extend the patch to even short env variables in other configs ? Amicalement,
Hi Albert, On 02/10/2014 10:44 AM, Albert ARIBAUD wrote: > Hi Michal, > > On Mon, 10 Feb 2014 10:25:19 +0100, Michal Simek <monstr@monstr.eu> > wrote: > >> Hi Albert, >> >> here are all patches which I have in my queue for Xilinx Zynq. >> >> clk changes depends on clk command which has been added recently. >> (common/cmd_clk.c) that's why this tree is based on 3 days old >> Tom's tree. >> >> Thanks, >> Michal > > This has the current tftpboot patch, affecting only the Zynq config. > Does it mean you prefer not to extend the patch to even short env > variables in other configs ? It affects because in the next patch I am enabling tftpput command. My preference is to have tftp->tftpboot discussion out of my zynq series because it can/should end up in more generic discussion if we should use shortcuts in variables in general in uboot. If you git grep "tftp " you will see that all readmes should be also fixed and probably we can check others commands too. Thanks, Michal
Hi Michal, On Mon, 10 Feb 2014 10:25:19 +0100, Michal Simek <monstr@monstr.eu> wrote: > Hi Albert, > > here are all patches which I have in my queue for Xilinx Zynq. > > clk changes depends on clk command which has been added recently. > (common/cmd_clk.c) that's why this tree is based on 3 days old > Tom's tree. > > Thanks, > Michal > > > The following changes since commit dbf3de2dd26cae37d16b00b348828c883b658cc5: > > include/usb/s3c_udc.h: Add <asm/sizes.h> (2014-02-06 14:26:05 -0500) > > are available in the git repository at: > > git://www.denx.de/git/u-boot-microblaze.git zynq > > for you to fetch changes up to 229f856da6f12ee30d43bcdf8111816f7945ccbe: > > zynq: Add support for U-BOOT SPL (2014-02-10 10:01:13 +0100) Seems like this pulls in a lot more than simply your 13 patches. Can you check this on your side? Amicalement,
On 02/13/2014 04:12 PM, Albert ARIBAUD wrote: > Hi Michal, > > On Mon, 10 Feb 2014 10:25:19 +0100, Michal Simek <monstr@monstr.eu> > wrote: > >> Hi Albert, >> >> here are all patches which I have in my queue for Xilinx Zynq. >> >> clk changes depends on clk command which has been added recently. >> (common/cmd_clk.c) that's why this tree is based on 3 days old >> Tom's tree. >> >> Thanks, >> Michal >> >> >> The following changes since commit dbf3de2dd26cae37d16b00b348828c883b658cc5: >> >> include/usb/s3c_udc.h: Add <asm/sizes.h> (2014-02-06 14:26:05 -0500) >> >> are available in the git repository at: >> >> git://www.denx.de/git/u-boot-microblaze.git zynq >> >> for you to fetch changes up to 229f856da6f12ee30d43bcdf8111816f7945ccbe: >> >> zynq: Add support for U-BOOT SPL (2014-02-10 10:01:13 +0100) > > Seems like this pulls in a lot more than simply your 13 patches. Can > you check this on your side? Where have you taken number 13? patchwork? I can find out the status on that there too. In pull request was 20 and in my branch is also 20 patches which I have collected. I believe all of them have been properly sent for review. Thanks, Michal
Hi Michal, On Thu, 13 Feb 2014 16:30:37 +0100, Michal Simek <monstr@monstr.eu> wrote: > On 02/13/2014 04:12 PM, Albert ARIBAUD wrote: > > Hi Michal, > > > > On Mon, 10 Feb 2014 10:25:19 +0100, Michal Simek <monstr@monstr.eu> > > wrote: > > > >> Hi Albert, > >> > >> here are all patches which I have in my queue for Xilinx Zynq. > >> > >> clk changes depends on clk command which has been added recently. > >> (common/cmd_clk.c) that's why this tree is based on 3 days old > >> Tom's tree. > >> > >> Thanks, > >> Michal > >> > >> > >> The following changes since commit dbf3de2dd26cae37d16b00b348828c883b658cc5: > >> > >> include/usb/s3c_udc.h: Add <asm/sizes.h> (2014-02-06 14:26:05 -0500) > >> > >> are available in the git repository at: > >> > >> git://www.denx.de/git/u-boot-microblaze.git zynq > >> > >> for you to fetch changes up to 229f856da6f12ee30d43bcdf8111816f7945ccbe: > >> > >> zynq: Add support for U-BOOT SPL (2014-02-10 10:01:13 +0100) > > > > Seems like this pulls in a lot more than simply your 13 patches. Can > > you check this on your side? > > Where have you taken number 13? patchwork? > I can find out the status on that there too. (13+7 in fact, but issue discussed on IRC with Michal => I'll be sending a PR to Tom's u-boot/master branch) > In pull request was 20 and in my branch is also 20 patches which > I have collected. I believe all of them have been properly sent for review. > > Thanks, > Michal Amicalement,