vboot_reference: refine ebuild and add new dev tools

vboot_reference: refine ebuild and add new dev tools

New tools are added into vboot_reference to help development.
(ref: http://codereview.chromium.org/3822002
      http://codereview.chromium.org/3772013 )

Since now host and all targets need to access to devkeys,
prompt the /usr/share/vboot/devkeys to be installed by default.

After this change, you can simply change firmware to devkeys by command:
  sudo /usr/share/vboot/bin/make_dev_firmware.sh
and change SSD to devkeys by:
  sudo /usr/share/vboot/bin/make_dev_ssd.sh
or even to disable rootfs verification:
  sudo /usr/share/vboot/bin/make_dev_ssd.sh --remove_rootfs_verification

This CL also re-tab and refined the vboot_reference ebuild file.

BUG=chrome-os-partner:1276
TEST=emerge-x86-generic vboot_reference; the result has:
 /usr/share/vboot/devkeys/ # having dev keys and firmware_bmpfv.bin
 /usr/share/vboot/bin/ # make_dev*.sh, and common/resign_firmwarefd

Change-Id: Ia903ea4112206ca24dce6617fca47ffdb4e68b23

Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=9a8e93b

[Hung-Te, 2010-10-15 16:46:14]

anush: please help to review if this change is OK
rspangler: FYI

[anush, 2010-10-15 17:15:34]

LGTM with a nit

http://codereview.chromium.org/3799003/diff/1/2
File chromeos-base/vboot_reference/vboot_reference-9999.ebuild (right):

http://codereview.chromium.org/3799003/diff/1/2#newcode103
chromeos-base/vboot_reference/vboot_reference-9999.ebuild:103: insinto
"${dst_dir}"
Don't you want to install this just on the host and not the target?

[Hung-Te, 2010-10-15 17:35:43]

http://codereview.chromium.org/3799003/diff/1/2
File chromeos-base/vboot_reference/vboot_reference-9999.ebuild (right):

http://codereview.chromium.org/3799003/diff/1/2#newcode103
chromeos-base/vboot_reference/vboot_reference-9999.ebuild:103: insinto
"${dst_dir}"
It is there because "required by uboot for cross compile + headers"
(http://codereview.chromium.org/3607006), so I think we should not change it.

[Randall, 2010-10-15 17:36:31]

LGTM (assuming you've addressed Anush's concerns)

[anush, 2010-10-15 17:58:29]

LGTM

On Fri, Oct 15, 2010 at 10:35 AM, <hungte@chromium.org> wrote:

>
> http://codereview.chromium.org/3799003/diff/1/2
> File chromeos-base/vboot_reference/vboot_reference-9999.ebuild (right):
>
> http://codereview.chromium.org/3799003/diff/1/2#newcode103
> chromeos-base/vboot_reference/vboot_reference-9999.ebuild:103: insinto
> "${dst_dir}"
> It is there because "required by uboot for cross compile + headers"
> (http://codereview.chromium.org/3607006), so I think we should not
> change it.
>
>
> http://codereview.chromium.org/3799003/show
>