Re: [U-Boot] [PATCH] ARM: enable ARMv7 virt support for the Arndale board
Am 01.08.2014 15:29, schrieb Andre Przywara: > On 01/08/14 14:02, Christoffer Dall wrote: >> On 1 August 2014 14:59, Andre Przywara wrote: >>> On 01/08/14 13:53, Christoffer Dall wrote: On 1 August 2014 14:46, Andreas Färber wrote: > Hi, > > Am 01.08.2014 13:35, schrieb Christoffer Dall: >> From: Andre Przywara >> >> To enable hypervisors utilizing the ARMv7 virtualization extension >> on the Arndale board, add the simple SMP pen address writer function >> and add the required configuration variables to switch all cores to >> HYP mode before launching the OS. >> This allows booting KVM and Xen directly from u-boot. >> >> Reviewed-by: Christoffer Dall >> Signed-off-by: Andre Przywara > > You forgot to sign off the patch. > > But nice to see progress on this! > Can you just add when applying the patch? Signed-off-by: Christoffer Dall >>> >>> Andreas, >>> >>> I wonder if you could change (or add) my new address, as my Linaro >>> address does not work any more (just tested). >>> >>> Signed-off-by: Andre Przywara >>> >> So I actually did not know the policy here, given that you did the >> work under your linaro.org address, so I decided to keep this as they >> were and just cc your arm address. If you're fine with using the >> @arm.com address I can respin the patch with the right addresses and >> signed-off-by's. > > I don't know about the policy either - actually I hoped you could help > here ;-) I am perfectly fine with giving Linaro as a sponsor credit for > this, but I am just concerned about potential feedback, which would > bounce and at best annoy people. As long as a post goes to a list also I > catch it, but private only email would be stuck. > This said I am fine both ways - maybe we keep both addresses for > reference in? > > Andreas, > could you just add Christoffer's Signed-off-by and my Signed-off-by > @arm.com without a respin? Guys, I just reacted to this patch because I spotted "virt" and we have a small farm of Arndale boards for OBS. Please take a look at http://www.denx.de/wiki/U-Boot/Custodians, which lists Albert for ARM and Minkyu for Samsung (CC'ed). Regards, Andreas -- SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] ARM: enable ARMv7 virt support for the Arndale board
On 1 August 2014 15:29, Andre Przywara wrote: > > > On 01/08/14 14:02, Christoffer Dall wrote: >> On 1 August 2014 14:59, Andre Przywara wrote: >>> >>> >>> On 01/08/14 13:53, Christoffer Dall wrote: On 1 August 2014 14:46, Andreas Färber wrote: > Hi, > > Am 01.08.2014 13:35, schrieb Christoffer Dall: >> From: Andre Przywara >> >> To enable hypervisors utilizing the ARMv7 virtualization extension >> on the Arndale board, add the simple SMP pen address writer function >> and add the required configuration variables to switch all cores to >> HYP mode before launching the OS. >> This allows booting KVM and Xen directly from u-boot. >> >> Reviewed-by: Christoffer Dall >> Signed-off-by: Andre Przywara > > You forgot to sign off the patch. > > But nice to see progress on this! > Can you just add when applying the patch? Signed-off-by: Christoffer Dall >>> >>> Andreas, >>> >>> I wonder if you could change (or add) my new address, as my Linaro >>> address does not work any more (just tested). >>> >>> Signed-off-by: Andre Przywara >>> >> So I actually did not know the policy here, given that you did the >> work under your linaro.org address, so I decided to keep this as they >> were and just cc your arm address. If you're fine with using the >> @arm.com address I can respin the patch with the right addresses and >> signed-off-by's. > > I don't know about the policy either - actually I hoped you could help > here ;-) I am perfectly fine with giving Linaro as a sponsor credit for > this, but I am just concerned about potential feedback, which would > bounce and at best annoy people. As long as a post goes to a list also I > catch it, but private only email would be stuck. > This said I am fine both ways - maybe we keep both addresses for > reference in? > > Andreas, > could you just add Christoffer's Signed-off-by and my Signed-off-by > @arm.com without a respin? > Andreas pointed out that he's not the maintainer, so let's see what the maintainer says :) all this policy stuff seems to be bikeshedding anyhow. -Christoffer ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] ARM: enable ARMv7 virt support for the Arndale board
On 01/08/14 14:02, Christoffer Dall wrote: > On 1 August 2014 14:59, Andre Przywara wrote: >> >> >> On 01/08/14 13:53, Christoffer Dall wrote: >>> On 1 August 2014 14:46, Andreas Färber wrote: Hi, Am 01.08.2014 13:35, schrieb Christoffer Dall: > From: Andre Przywara > > To enable hypervisors utilizing the ARMv7 virtualization extension > on the Arndale board, add the simple SMP pen address writer function > and add the required configuration variables to switch all cores to > HYP mode before launching the OS. > This allows booting KVM and Xen directly from u-boot. > > Reviewed-by: Christoffer Dall > Signed-off-by: Andre Przywara You forgot to sign off the patch. But nice to see progress on this! >>> Can you just add when applying the patch? >>> >>> Signed-off-by: Christoffer Dall >> >> Andreas, >> >> I wonder if you could change (or add) my new address, as my Linaro >> address does not work any more (just tested). >> >> Signed-off-by: Andre Przywara >> > So I actually did not know the policy here, given that you did the > work under your linaro.org address, so I decided to keep this as they > were and just cc your arm address. If you're fine with using the > @arm.com address I can respin the patch with the right addresses and > signed-off-by's. I don't know about the policy either - actually I hoped you could help here ;-) I am perfectly fine with giving Linaro as a sponsor credit for this, but I am just concerned about potential feedback, which would bounce and at best annoy people. As long as a post goes to a list also I catch it, but private only email would be stuck. This said I am fine both ways - maybe we keep both addresses for reference in? Andreas, could you just add Christoffer's Signed-off-by and my Signed-off-by @arm.com without a respin? Regards, Andre. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] ARM: enable ARMv7 virt support for the Arndale board
On 01/08/14 13:53, Christoffer Dall wrote: > On 1 August 2014 14:46, Andreas Färber wrote: >> Hi, >> >> Am 01.08.2014 13:35, schrieb Christoffer Dall: >>> From: Andre Przywara >>> >>> To enable hypervisors utilizing the ARMv7 virtualization extension >>> on the Arndale board, add the simple SMP pen address writer function >>> and add the required configuration variables to switch all cores to >>> HYP mode before launching the OS. >>> This allows booting KVM and Xen directly from u-boot. >>> >>> Reviewed-by: Christoffer Dall >>> Signed-off-by: Andre Przywara >> >> You forgot to sign off the patch. >> >> But nice to see progress on this! >> > Can you just add when applying the patch? > > Signed-off-by: Christoffer Dall Andreas, I wonder if you could change (or add) my new address, as my Linaro address does not work any more (just tested). Signed-off-by: Andre Przywara Christoffer, thanks for taking care of this! Cheers, Andre. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] ARM: enable ARMv7 virt support for the Arndale board
On 1 August 2014 14:59, Andre Przywara wrote: > > > On 01/08/14 13:53, Christoffer Dall wrote: >> On 1 August 2014 14:46, Andreas Färber wrote: >>> Hi, >>> >>> Am 01.08.2014 13:35, schrieb Christoffer Dall: From: Andre Przywara To enable hypervisors utilizing the ARMv7 virtualization extension on the Arndale board, add the simple SMP pen address writer function and add the required configuration variables to switch all cores to HYP mode before launching the OS. This allows booting KVM and Xen directly from u-boot. Reviewed-by: Christoffer Dall Signed-off-by: Andre Przywara >>> >>> You forgot to sign off the patch. >>> >>> But nice to see progress on this! >>> >> Can you just add when applying the patch? >> >> Signed-off-by: Christoffer Dall > > Andreas, > > I wonder if you could change (or add) my new address, as my Linaro > address does not work any more (just tested). > > Signed-off-by: Andre Przywara > So I actually did not know the policy here, given that you did the work under your linaro.org address, so I decided to keep this as they were and just cc your arm address. If you're fine with using the @arm.com address I can respin the patch with the right addresses and signed-off-by's. -Christoffer ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] ARM: enable ARMv7 virt support for the Arndale board
Hi, Am 01.08.2014 13:35, schrieb Christoffer Dall: > From: Andre Przywara > > To enable hypervisors utilizing the ARMv7 virtualization extension > on the Arndale board, add the simple SMP pen address writer function > and add the required configuration variables to switch all cores to > HYP mode before launching the OS. > This allows booting KVM and Xen directly from u-boot. > > Reviewed-by: Christoffer Dall > Signed-off-by: Andre Przywara You forgot to sign off the patch. But nice to see progress on this! Cheers, Andreas -- SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] ARM: enable ARMv7 virt support for the Arndale board
On 1 August 2014 14:46, Andreas Färber wrote: > Hi, > > Am 01.08.2014 13:35, schrieb Christoffer Dall: >> From: Andre Przywara >> >> To enable hypervisors utilizing the ARMv7 virtualization extension >> on the Arndale board, add the simple SMP pen address writer function >> and add the required configuration variables to switch all cores to >> HYP mode before launching the OS. >> This allows booting KVM and Xen directly from u-boot. >> >> Reviewed-by: Christoffer Dall >> Signed-off-by: Andre Przywara > > You forgot to sign off the patch. > > But nice to see progress on this! > Can you just add when applying the patch? Signed-off-by: Christoffer Dall ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
