<feed xmlns='http://www.w3.org/2005/Atom'>
<title>bcm63xx/atf/plat/mediatek/mt8173/include/platform_def.h, branch master</title>
<subtitle>Broadcom-s Trusted Firmware A</subtitle>
<id>https://git.openwrt.org/project/bcm63xx/atf/atom?h=master</id>
<link rel='self' href='https://git.openwrt.org/project/bcm63xx/atf/atom?h=master'/>
<link rel='alternate' type='text/html' href='https://git.openwrt.org/project/bcm63xx/atf/'/>
<updated>2019-01-04T10:43:17Z</updated>
<entry>
<title>Sanitise includes across codebase</title>
<updated>2019-01-04T10:43:17Z</updated>
<author>
<name>Antonio Nino Diaz</name>
</author>
<published>2018-12-14T00:18:21Z</published>
<link rel='alternate' type='text/html' href='https://git.openwrt.org/project/bcm63xx/atf/commit/?id=09d40e0e08283a249e7dce0e106c07c5141f9b7e'/>
<id>urn:sha1:09d40e0e08283a249e7dce0e106c07c5141f9b7e</id>
<content type='text'>
Enforce full include path for includes. Deprecate old paths.

The following folders inside include/lib have been left unchanged:

- include/lib/cpus/${ARCH}
- include/lib/el3_runtime/${ARCH}

The reason for this change is that having a global namespace for
includes isn't a good idea. It defeats one of the advantages of having
folders and it introduces problems that are sometimes subtle (because
you may not know the header you are actually including if there are two
of them).

For example, this patch had to be created because two headers were
called the same way: e0ea0928d5b7 ("Fix gpio includes of mt8173 platform
to avoid collision."). More recently, this patch has had similar
problems: 46f9b2c3a282 ("drivers: add tzc380 support").

This problem was introduced in commit 4ecca33988b9 ("Move include and
source files to logical locations"). At that time, there weren't too
many headers so it wasn't a real issue. However, time has shown that
this creates problems.

Platforms that want to preserve the way they include headers may add the
removed paths to PLAT_INCLUDES, but this is discouraged.

Change-Id: I39dc53ed98f9e297a5966e723d1936d6ccf2fc8f
Signed-off-by: Antonio Nino Diaz &lt;antonio.ninodiaz@arm.com&gt;
</content>
</entry>
<entry>
<title>mediatek: Migrate to new interfaces</title>
<updated>2018-09-28T14:34:17Z</updated>
<author>
<name>Antonio Nino Diaz</name>
</author>
<published>2018-09-24T16:15:54Z</published>
<link rel='alternate' type='text/html' href='https://git.openwrt.org/project/bcm63xx/atf/commit/?id=b8424642bcbe38433eea0c332aaedadd78fd45d4'/>
<id>urn:sha1:b8424642bcbe38433eea0c332aaedadd78fd45d4</id>
<content type='text'>
- mt6795: Migrate to new GIC interfaces.
- Remove support for PSCI platform compatibility layer.
- Migrate to bl31_early_platform_setup2().
- Migrate from cm_init_context() to cm_init_my_context().
- Use PLAT_VIRT_ADDR_SPACE_SIZE and PLAT_PHY_ADDR_SPACE_SIZE.
- Update Makefile paths.
- Use private definition of bl31_params_t.

This is an incomplete migration, mt6795 doesn't currently compile.

Change-Id: Icf9307637066cd6f2166524715e4f117f5ce2350
Signed-off-by: Antonio Nino Diaz &lt;antonio.ninodiaz@arm.com&gt;
</content>
</entry>
<entry>
<title>PSCI: Fix types of definitions</title>
<updated>2018-07-20T12:49:22Z</updated>
<author>
<name>Antonio Nino Diaz</name>
</author>
<published>2018-07-20T08:17:26Z</published>
<link rel='alternate' type='text/html' href='https://git.openwrt.org/project/bcm63xx/atf/commit/?id=1083b2b315cd71f714eb0d0bca20e54ef7be02ad'/>
<id>urn:sha1:1083b2b315cd71f714eb0d0bca20e54ef7be02ad</id>
<content type='text'>
Also change header guards to fix defects of MISRA C-2012 Rule 21.1.

Change-Id: Ied0d4b0e557ef6119ab669d106d2ac5d99620c57
Acked-by: Sumit Garg &lt;sumit.garg@linaro.org&gt;
Acked-by: Anson Huang &lt;Anson.Huang@nxp.com&gt;
Signed-off-by: Antonio Nino Diaz &lt;antonio.ninodiaz@arm.com&gt;
</content>
</entry>
<entry>
<title>Update ULL() macro and instances of ull to comply with MISRA</title>
<updated>2018-02-27T17:05:51Z</updated>
<author>
<name>David Cunado</name>
</author>
<published>2018-02-16T21:12:58Z</published>
<link rel='alternate' type='text/html' href='https://git.openwrt.org/project/bcm63xx/atf/commit/?id=5724481fdda01067d3bf4386a3dd07a541d16044'/>
<id>urn:sha1:5724481fdda01067d3bf4386a3dd07a541d16044</id>
<content type='text'>
MISRA C-2012 Rule 7.3 violation: lowercase l shall not be used as literal suffixes.

This patch resolves this for the ULL() macro by using ULL suffix instead
of the ull suffix.

Change-Id: Ia8183c399e74677e676956e8653e82375d0e0a01
Signed-off-by: David Cunado &lt;david.cunado@arm.com&gt;
</content>
</entry>
<entry>
<title>mt8173: Migrate to using interrupt properties</title>
<updated>2017-10-17T13:55:17Z</updated>
<author>
<name>Jeenu Viswambharan</name>
</author>
<published>2017-09-29T10:14:51Z</published>
<link rel='alternate' type='text/html' href='https://git.openwrt.org/project/bcm63xx/atf/commit/?id=831b37520ac7d2ff22dc09f453ed6fb0b19970cb'/>
<id>urn:sha1:831b37520ac7d2ff22dc09f453ed6fb0b19970cb</id>
<content type='text'>
Change-Id: I1463a4f9b74d74d59ac1d37b7b9c8e53416ab904
Signed-off-by: Jeenu Viswambharan &lt;jeenu.viswambharan@arm.com&gt;
</content>
</entry>
<entry>
<title>Use SPDX license identifiers</title>
<updated>2017-05-03T08:39:28Z</updated>
<author>
<name>dp-arm</name>
</author>
<published>2017-05-03T08:38:09Z</published>
<link rel='alternate' type='text/html' href='https://git.openwrt.org/project/bcm63xx/atf/commit/?id=82cb2c1ad9897473743f08437d0a3995bed561b9'/>
<id>urn:sha1:82cb2c1ad9897473743f08437d0a3995bed561b9</id>
<content type='text'>
To make software license auditing simpler, use SPDX[0] license
identifiers instead of duplicating the license text in every file.

NOTE: Files that have been imported by FreeBSD have not been modified.

[0]: https://spdx.org/

Change-Id: I80a00e1f641b8cc075ca5a95b10607ed9ed8761a
Signed-off-by: dp-arm &lt;dimitris.papastamos@arm.com&gt;
</content>
</entry>
<entry>
<title>Remove use of all deprecated APIs</title>
<updated>2017-01-24T02:50:14Z</updated>
<author>
<name>Koan-Sin Tan</name>
</author>
<published>2016-04-18T09:20:05Z</published>
<link rel='alternate' type='text/html' href='https://git.openwrt.org/project/bcm63xx/atf/commit/?id=201d535ff65ceefcb35840a53f8048992dcffdaf'/>
<id>urn:sha1:201d535ff65ceefcb35840a53f8048992dcffdaf</id>
<content type='text'>
Now it's possbile to build BL31 for MT8173 with ERROR_DEPRECATED=1.

Signed-off-by: Koan-Sin Tan &lt;koansin.tan@gmail.com&gt;
</content>
</entry>
<entry>
<title>Get rid of use of old GIC APIs</title>
<updated>2017-01-24T02:50:14Z</updated>
<author>
<name>Koan-Sin Tan</name>
</author>
<published>2016-04-18T07:17:57Z</published>
<link rel='alternate' type='text/html' href='https://git.openwrt.org/project/bcm63xx/atf/commit/?id=8bc20038afc4ac87a35b507bfdf1e5d2301bcfcf'/>
<id>urn:sha1:8bc20038afc4ac87a35b507bfdf1e5d2301bcfcf</id>
<content type='text'>
Signed-off-by: Koan-Sin Tan &lt;koansin.tan@gmail.com&gt;
</content>
</entry>
<entry>
<title>Get rid of use of compatibility API</title>
<updated>2017-01-24T02:50:14Z</updated>
<author>
<name>Koan-Sin Tan</name>
</author>
<published>2017-01-19T08:43:49Z</published>
<link rel='alternate' type='text/html' href='https://git.openwrt.org/project/bcm63xx/atf/commit/?id=3fc26aa0938a838686644c146ee84c562d963c34'/>
<id>urn:sha1:3fc26aa0938a838686644c146ee84c562d963c34</id>
<content type='text'>
make 'make ARCH=aarch64 CROSS_COMPILE=aarch64-linux-gnu- PLAT=mt8173
ENABLE_PLAT_COMPAT=0' work.

Change-Id: I13f35d8aef23dfa0e65883fa0be43f1513c9fef5
Signed-off-by: Koan-Sin Tan &lt;koansin.tan@gmail.com&gt;
</content>
</entry>
<entry>
<title>Use #ifdef for IMAGE_BL* instead of #if</title>
<updated>2017-01-23T16:01:21Z</updated>
<author>
<name>Masahiro Yamada</name>
</author>
<published>2016-12-25T14:36:24Z</published>
<link rel='alternate' type='text/html' href='https://git.openwrt.org/project/bcm63xx/atf/commit/?id=3d8256b2a1ef1195aed86bef7378e83d0a61a91b'/>
<id>urn:sha1:3d8256b2a1ef1195aed86bef7378e83d0a61a91b</id>
<content type='text'>
One nasty part of ATF is some of boolean macros are always defined
as 1 or 0, and the rest of them are only defined under certain
conditions.

For the former group, "#if FOO" or "#if !FOO" must be used because
"#ifdef FOO" is always true.  (Options passed by $(call add_define,)
are the cases.)

For the latter, "#ifdef FOO" or "#ifndef FOO" should be used because
checking the value of an undefined macro is strange.

Here, IMAGE_BL* is handled by make_helpers/build_macro.mk like
follows:

  $(eval IMAGE := IMAGE_BL$(call uppercase,$(3)))

  $(OBJ): $(2)
          @echo "  CC      $$&lt;"
          $$(Q)$$(CC) $$(TF_CFLAGS) $$(CFLAGS) -D$(IMAGE) -c $$&lt; -o $$@

This means, IMAGE_BL* is defined when building the corresponding
image, but *undefined* for the other images.

So, IMAGE_BL* belongs to the latter group where we should use #ifdef
or #ifndef.

Signed-off-by: Masahiro Yamada &lt;yamada.masahiro@socionext.com&gt;
</content>
</entry>
</feed>
