Run `fx format-code` without changing "#pragma once" to header guards.
Also set the line limit to 100 columns in the Zircon clang-format
configuration.

Test: N/A
Change-Id: I7049fcec3ebe80a0ca910ed2be95bdbc0303a596

This is the correct way to move a value out of an optional.

Tested: runtests
Change-Id: I3c0b24eb85f5be0cc91a3fbebc145860ef1b854a

Test: builds / ftl-test

Change-Id: I6ceb6363b143153eee9ac137d2d601f9aabbbe51

Test: builds / ftl-test

Change-Id: Ibbfcf1778f932772d0e2f25f2cf08a35d2f53ff8

Use metadata to specify the size of a protected memory region, which
will be created using contigous memory. Portions of that memory region
can be handed out to clients as requested. Currently the memory isn't
actually marked as protected using the TEE, so it's no different from
regular memory.

Also convert the query of the current VID and PID to read from the
sysmem metadata, as pdev_get_device_info gets information about the
current device, which always PDEV_VID_GENERIC and PDEV_PID_GENERIC for
sysmem.

Test: vkcube on astro

Change-Id: I681f928ccde4e4d99787d2b0853e19ed9b9eaab0

This design isn't necessarily something that will be kept long term, but
it's useful for bring-up of the port while things are solidifying.

Test: Used addr() in debug output.

Change-Id: I69112ae7e678e23ff4841e053efba212127743d7

volume.cpp currently uses a variable length array of type char
allocated on the stack to pass to block_impl_queue. This is not guaranteed
to be aligned to the required ABI requirements, and is also unsafe.

This patch moves the dynamic memory allocation to the heap
and ensures the allocation is at least aligned to alignof(block_op_t)

Test: Ensure that all automated tests pass on ARM64 and x86_64.
Change-Id: I5ccab789b02d9411b84bcd2b05c44d722852429d

This is a reland of a3b4302f

This CL was reverted due to lack of support in measure.
The measure library support was added in:
https://fuchsia-review.googlesource.com/c/garnet/+/246312

TEST: The garnet-*-perf-* bot was failing before the support was added, but now it is green.

PT-72 #done

Original change's description:
> [tracing] Make TRACE_DURATION use trace duration complete events
>
> ... instead of a start and end events. Saving ~50% in buffer and JSON
> sizes.
>
> There is no noticeable change in performance for NTRACE, tracing
> disabled, category disabled and tracing enabled.
>
> The compiled code size is less than or nearly equal the old version for 4
> or less arguments but it is bigger for 8 arguments and more, issue
> tracked in PT-87.
>
> TEST: fx shell /system/test/sys/trace-test
> TEST: fx shell trace-benchmark
>
> PT-72 #done

Change-Id: I21510f2ffd1b7d79a622927bc999ce8cb9375396

This patch enables the USB peripheral driver to obtain its configuration through board-specific metadata.

Test: On Cleo, validate that this configuration works. Validate that the usb-virtual-bus does NOT set a default configuration.

Change-Id: I328931427a4fe30fc3e34f199aa02cfa1ba9cb08

This reverts commit 0f31d965.

Reason for revert: Breaking the zircon roller

Original change's description:
> [vfs] Plumb a hash table around to track koids of tokens
> 
> Rather than rely on set/get_cookie
> 
> Test: fs unit tests
> Change-Id: I03d6b5de4eae90e8bbcef09958fb983b84e86e64

TBR=kulakowski@google.com,smklein@google.com,abarth@google.com

Change-Id: Id223a343a627e8d91539a909b1ea4b7a2494396d
No-Presubmit: true
No-Tree-Checks: true
No-Try: true

Tested: Paved astro
Change-Id: I5bec4ec8cc5107be72a28c487485421b3a4ba446

TEST: USB works on NUC and in qemu with USB virtual bus

Change-Id: I98aeff7c3c0ae7bbdc6a6620261286da7271d5a1

Tested: runtests -t operation-test
Change-Id: I1b1897b3630998631a9b4c82b4e2de3737c849c2

This test goes through the full bind of a device with a devhost, and the
disconnection of the devhost.

ZX-3286

Test: Ran Fuchsia and /system/test/ddk tests.
Change-Id: Ibfc9bc5a2f041772130d9417cd9e032c75b7b587

Test: docs only

Change-Id: Ib67cf051fcb2f41167c2dbf715e5fb07388d2cad

* Return the status of AttemptBind in BindDevice and BindDriver
* Use ZX_CHANNEL_MAX_MSG_BYTES in HandleDeviceRead

ZX-3286

Test: Ran Fuchsia and /system/test/ddk tests.
Change-Id: I18aa9c488f988edd4d863dcaca8845f7148baefb

Rather than rely on set/get_cookie

Test: fs unit tests
Change-Id: I03d6b5de4eae90e8bbcef09958fb983b84e86e64

These bindings have parity with the remaining ioctl declarations.
For direct comparison, refer to: system/public/zircon/device/block.h

Test: Declarations only; bindings utilized in subsequent patches.

Change-Id: I6cfba4834b3d4688e1be92c04da20bd945df4afc

Currently, we are skipping display initialization and depending on
u-boot. This change introduces a late init function that will be called
the first time apply_config is called.

Test: Tested on Astro hardware and pave was succesfully. Netbooting was
also successfull.

Change-Id: Iebdb15ef7dd6aa1b42121a75926a81582ff64ab2

Tested: paved astro
Change-Id: I920406359b4c63ed3f4a5a6b68fe6ffea2c35b39

Fix a couple of intra-document links in the design doc.

Bug: None
Test: previewed in gitiles
Change-Id: Idc3251c5a01b8458f0de675bcf77e18ae29a2cab

This allows devfs to be initialized in tests, and then shutdown without
aborting on the intrusive list destructor.

ZX-3286

Test: Ran Fuchsia and /system/test/ddk tests.
Change-Id: If4160854844ef14941d66c2e06675f0fc9fd9a8a

Tested: paved astro.
Change-Id: I6855a8c29fde7cf88db2b5282bcd7bf4f81fd30d

Test: Boots on Cleo
Change-Id: Id5fb9db3342c85703fff109ecd3c021e14321514

Specialized operation::Operation and friends for nand protocol
use cases.

Tested: Tested with dependent nandpart CL.
Change-Id: Ica9c242d90a1047b29c13934242b95ca0b54e99c

Tested: runtests -t usb-unittest
Change-Id: If51ad062fbaf5157749635c40e587bc37f962ad7

The contract for fidl_encode and fidl_decode is that all handles must be
closed in case of an error. The big refactor to visitors silently broke
that contract in error cases where the caller supplied an overly large
buffer/handles array. This patch fixes the behavior and adds tests.

TEST: /boot/test/sys/fidl-test
Change-Id: I50e81f3c9168370f08a4043aad2384ce5ac62b6e

Test: Tested on Frank with vim2, confirmed same behavior before and
after changes. All USB peripherals enumerated, touch and OKG working

ZX-3177 #done

Change-Id: I6bee8f66c3e9ce9c93eae67875f64d7c1a9812bf

This brings the object docs in line with the newer syscall docs.

TEST: visual inspection.
Change-Id: Ia4911a48adcff1c7c7ab65777638a45479981722

- Re-write Camera Driver according to
  https://docs.google.com/document/d/113kjqX6IN8wyALSyzBBZtZFrt4g6F3Cdi0dCQtYDZIU/edit?usp=sharing
- Implement phase 2
- Add ISP protocol dummy driver
- Add ISP_IMPL protocol
- Remove Camera Sensor FIDL protocol
- Remove Camera test which are irrelevant now
- Remove IOCTLs

Test: "dm dump" shows all 3 drivers loaded

Change-Id: Ida8c38b93b8325f9190ff8175061bc9c91a236b3

Test: mt8167 ref board boots.
Tets: runtests -t mtk-gpio-test
Change-Id: I8c18380558503040fa84956b73349face7694076

ZX-3294

Block devices in Zircon do not support flushing/fsync operations.

* This patch updates the rpc-server to return an error (rather than success) when a caller tries to perform fsync on a block device to indicat that the operation is not supported.

Test: Verify that fsync returns an error when called on a block device.

Change-Id: Ie23fe7b71eef54fb3e4ea0ee0a50626c9d193d34

Mediatek devices crash in the bootloader when the
decompressed kernel is written to the ramdisk
region. Add an option to package-image.sh that
will prevent a ramdisk from being added to the
boot image, and use this option in the Cleo and
mt8167s_ref flash scripts. The default behavior
has not changed, so other devices shouldn't be
affected.

ZX-3356 #done
Test: mt8167 ref board boots.
Test: Ran package-image.sh and inspected the boot
      image with a hex editor.

Change-Id: If084e162d2bfe9b4ad37339c9b4daacc43791861

Just logs the endpoints.

ZX-3155 #comment

TEST= load firmware and check the driver binds.

Change-Id: I9dd8c3a37be2d2ca34802b750dfc3a16aa07a266

TEST: Visual inspection.
Change-Id: I4e30e64bb3b4a01b409dd9276d15592e4df01c9a

TEST: /boot/test/sys/fidl-test
Change-Id: If753ff134ce82c14e35a1ead320174cca51b4264

This reduces boilerplate callers need to write to invoke
protocol
methods where they send interfaces across.

Also includes some small unrelated fixes.

Tested: Builds.
Change-Id: If615c4f16f14b0dfa708eb8b4f5f50022762ff10

Specialized operation::Operation and friends for block protocol use cases.

Tested: Tested with ramdisk changes in future CL.
Change-Id: I279efbbf0964f14f1c713a9c1aa1c6918f91e5ee

This adds support for using .build-id directories.

Bug: DX-209
Bug: TC-303
Test: 1) Symbolize some dummy markup against a release build
      2) Symbolize result of crasher using symbolize script
      3) Symbolize result of crasher using fx script
      4) Symbolize result of crasher using the tool manually
         to test .build-id stuff.
Change-Id: I26d4431e301a816ad18aa6d6410094ba06a3f313

Bug: ZX-3357 #comment
Test: no functional change; full system build; CQ
Change-Id: Ic1988836197fd296ab80e91788953b890a14b04f