<feed xmlns='http://www.w3.org/2005/Atom'>
<title>talos-skiboot/external/common, branch master</title>
<subtitle>Talos™ II skiboot sources</subtitle>
<id>https://git.raptorcs.com/git/talos-skiboot/atom?h=master</id>
<link rel='self' href='https://git.raptorcs.com/git/talos-skiboot/atom?h=master'/>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/talos-skiboot/'/>
<updated>2017-08-01T03:56:54+00:00</updated>
<entry>
<title>Makefile.check: Fix `make check`</title>
<updated>2017-08-01T03:56:54+00:00</updated>
<author>
<name>Cyril Bur</name>
<email>cyril.bur@au1.ibm.com</email>
</author>
<published>2017-07-28T06:46:19+00:00</published>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/talos-skiboot/commit/?id=5c2ed41022baaac49be3620f375af85a4a852c2b'/>
<id>urn:sha1:5c2ed41022baaac49be3620f375af85a4a852c2b</id>
<content type='text'>
Recent reworks were tested on the travis-ci system. Unfortunately, there
are configurations of running `make check` which the travis-ci doesn't
do. On some systems extra problems crop up.

Removing the stack size check is only done for the host compiler as the
check is only critical for skiboot its self where stack space is
contained.

Signed-off-by: Cyril Bur &lt;cyril.bur@au1.ibm.com&gt;
Reviewed-by: Samuel Mendoza-Jonas &lt;sam@mendozajonas.com&gt;
Signed-off-by: Stewart Smith &lt;stewart@linux.vnet.ibm.com&gt;
</content>
</entry>
<entry>
<title>pflash: build with skiboot HOSTCFLAGS warnings</title>
<updated>2017-07-19T05:49:39+00:00</updated>
<author>
<name>Stewart Smith</name>
<email>stewart@linux.vnet.ibm.com</email>
</author>
<published>2017-07-17T07:23:39+00:00</published>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/talos-skiboot/commit/?id=9880e148c2ea2d7a6f24a025799e59ac1a18cf92'/>
<id>urn:sha1:9880e148c2ea2d7a6f24a025799e59ac1a18cf92</id>
<content type='text'>
mostly missing prototypes and unused parameters.

Reviewed-by: Cyril Bur &lt;cyril.bur@au1.ibm.com&gt;
Signed-off-by: Stewart Smith &lt;stewart@linux.vnet.ibm.com&gt;
</content>
</entry>
<entry>
<title>external/gard: Fix make dist target</title>
<updated>2017-02-23T02:05:11+00:00</updated>
<author>
<name>Cyril Bur</name>
<email>cyril.bur@au1.ibm.com</email>
</author>
<published>2017-01-25T05:38:13+00:00</published>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/talos-skiboot/commit/?id=8ab0caf26de9d9cc12223c4b945052b50508bf78'/>
<id>urn:sha1:8ab0caf26de9d9cc12223c4b945052b50508bf78</id>
<content type='text'>
Several problems:
Firstly, it could never have worked, it was using the wrong variable.

Secondly, if it was using GARD_VERSION it produced a broken tarball that
still looked into the skiboot source for files despite them having been
copied into the tarball.

Lastly (and not really a make dist issue) the current way of symlinking
make_version.sh was racey. Get around the issue by refering to it in its
actual location (if we know it will be there) or by looking at .version
if building from tarball.

Signed-off-by: Cyril Bur &lt;cyril.bur@au1.ibm.com&gt;
Signed-off-by: Stewart Smith &lt;stewart@linux.vnet.ibm.com&gt;
</content>
</entry>
<entry>
<title>arch_flash_arm: Don't assume mtd labels are short</title>
<updated>2016-12-21T05:41:50+00:00</updated>
<author>
<name>Joel Stanley</name>
<email>joel@jms.id.au</email>
</author>
<published>2016-11-16T03:15:35+00:00</published>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/talos-skiboot/commit/?id=7d6e73810dec029678a0d14a3f47485d4025520e'/>
<id>urn:sha1:7d6e73810dec029678a0d14a3f47485d4025520e</id>
<content type='text'>
pflash relies on arch_flash_arm parsing /proc/mtd to discover the pnor
partition. It helpfully uses strcasestr so it can handle the string
changing, which is what has happened as we moved to upstream compliant
mtd device tree bindings.

We currently have a string like this:

dev:    size   erasesize  name
mtd0: 00060000 00001000 "u-boot"
mtd1: 00020000 00001000 "u-boot-env"
mtd2: 00280000 00001000 "kernel"
mtd3: 001c0000 00001000 "initramfs"
mtd4: 01740000 00001000 "rofs"
mtd5: 00400000 00001000 "rwfs"
mtd6: 02000000 00001000 "1e620000.flash-controller:flash@1"
mtd7: 08000000 00001000 "1e630000.flash-controller:pnor@0"

Unfortunately arch_flash_arm assumes the string will be at most 50
characters. That's right before the label we're looking for starts so
we ignore that line and keep searching.

Fix it by allowing for a 255 character line.

Fixes: 48ab7ce09504 (external/pflash: Add --mtd)
Signed-off-by: Joel Stanley &lt;joel@jms.id.au&gt;
Reviewed-by: Cyril Bur &lt;cyril.bur@au1.ibm.com&gt;
Signed-off-by: Stewart Smith &lt;stewart@linux.vnet.ibm.com&gt;
</content>
</entry>
<entry>
<title>external/common: Add default erase chip implementation</title>
<updated>2016-11-08T06:36:58+00:00</updated>
<author>
<name>Cyril Bur</name>
<email>cyril.bur@au1.ibm.com</email>
</author>
<published>2016-11-08T06:08:40+00:00</published>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/talos-skiboot/commit/?id=71f77880462e72b9888237ae8f1fbdf36da2ef0e'/>
<id>urn:sha1:71f77880462e72b9888237ae8f1fbdf36da2ef0e</id>
<content type='text'>
Just blocklevel_erase() from zero to sizeof.

Signed-off-by: Cyril Bur &lt;cyril.bur@au1.ibm.com&gt;
Reviewed-by: Joel Stanley &lt;joel@jms.id.au&gt;
Signed-off-by: Stewart Smith &lt;stewart@linux.vnet.ibm.com&gt;
</content>
</entry>
<entry>
<title>external/common: Teach ARM code to erase 'mtd chips'</title>
<updated>2016-11-08T06:36:49+00:00</updated>
<author>
<name>Cyril Bur</name>
<email>cyril.bur@au1.ibm.com</email>
</author>
<published>2016-11-08T06:08:39+00:00</published>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/talos-skiboot/commit/?id=b36b3d1edd6654d7e4c0e2798b32ff6e4f6cd1fe'/>
<id>urn:sha1:b36b3d1edd6654d7e4c0e2798b32ff6e4f6cd1fe</id>
<content type='text'>
Currently the arch flash code for all architectures can only perform
chip erases if there is a real flash driver attached.

With increasing use of MTD on both host and BMC this code needs to know
how to behave of the backend of blocklevel is MTD.

This patch teaches the ARM specific code to pass an erase for the full
size of the chip down the stack. This can be optimised into a chip erase
within the kernel.

Signed-off-by: Cyril Bur &lt;cyril.bur@au1.ibm.com&gt;
Reviewed-by: Joel Stanley &lt;joel@jms.id.au&gt;
Signed-off-by: Stewart Smith &lt;stewart@linux.vnet.ibm.com&gt;
</content>
</entry>
<entry>
<title>external/common: Rename arch_flash_bmc() to arch_flash_access()</title>
<updated>2016-10-25T08:52:29+00:00</updated>
<author>
<name>Cyril Bur</name>
<email>cyril.bur@au1.ibm.com</email>
</author>
<published>2016-10-24T04:36:48+00:00</published>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/talos-skiboot/commit/?id=39fea04c47ab614ddd2c943f5308cc9d873e67a0'/>
<id>urn:sha1:39fea04c47ab614ddd2c943f5308cc9d873e67a0</id>
<content type='text'>
Honestly the first name was terrible. Initially this was intended only
to serve for BMC/ARM flash access, however, it should be more generic as
it is in the external/common arch code it should play nice on all
architectures.

This change also paves the way to change the default flash access
methods in pflash.

Signed-off-by: Cyril Bur &lt;cyril.bur@au1.ibm.com&gt;
Acked-by: Joel Stanley &lt;joel@jms.id.au&gt;
[stewart@linux.vnet.ibm.com: preserve Joel's ack]
Signed-off-by: Stewart Smith &lt;stewart@linux.vnet.ibm.com&gt;
</content>
</entry>
<entry>
<title>external/common: arm: Don't error trying to wrprotect with MTD access</title>
<updated>2016-10-25T08:52:02+00:00</updated>
<author>
<name>Cyril Bur</name>
<email>cyril.bur@au1.ibm.com</email>
</author>
<published>2016-10-24T04:36:47+00:00</published>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/talos-skiboot/commit/?id=9c3b35a15f3a2762288e1e460843c9c1246e1acd'/>
<id>urn:sha1:9c3b35a15f3a2762288e1e460843c9c1246e1acd</id>
<content type='text'>
If the access method to the flash is PNOR_MTD or BMC_MTD then the
actual access is being done for us by the kernel. This means we don't
need to worry about wrprotecting. The arch level shouldn't be returning
an error, it should be fall though.

Signed-off-by: Cyril Bur &lt;cyril.bur@au1.ibm.com&gt;
Acked-by: Joel Stanley &lt;joel@jms.id.au&gt;
[stewart@linux.vnet.ibm.com: preserve Joel's ack]
Signed-off-by: Stewart Smith &lt;stewart@linux.vnet.ibm.com&gt;
</content>
</entry>
<entry>
<title>external: Fix ARM build failure with parallel make</title>
<updated>2016-06-27T06:19:11+00:00</updated>
<author>
<name>Brad Bishop</name>
<email>bradleyb@fuzziesquirrel.com</email>
</author>
<published>2016-06-21T23:37:48+00:00</published>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/talos-skiboot/commit/?id=5660d300fb23f299c4a306be4a213eb608158b6c'/>
<id>urn:sha1:5660d300fb23f299c4a306be4a213eb608158b6c</id>
<content type='text'>
Arch headers need to be linked in before compiling.

Signed-off-by: Brad Bishop &lt;bradleyb@fuzziesquirrel.com&gt;
Signed-off-by: Stewart Smith &lt;stewart@linux.vnet.ibm.com&gt;
</content>
</entry>
<entry>
<title>external: Update gitignore for building opal-prd</title>
<updated>2016-04-18T20:32:21+00:00</updated>
<author>
<name>Michael Neuling</name>
<email>mikey@neuling.org</email>
</author>
<published>2016-04-15T07:06:26+00:00</published>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/talos-skiboot/commit/?id=7478d2da61879f0f6a24b9ad9583810451f31801'/>
<id>urn:sha1:7478d2da61879f0f6a24b9ad9583810451f31801</id>
<content type='text'>
Building opal-prd currently leaves the git tree dirty.  This fixes it.

Signed-off-by: Michael Neuling &lt;mikey@neuling.org&gt;
Signed-off-by: Stewart Smith &lt;stewart@linux.vnet.ibm.com&gt;
</content>
</entry>
</feed>
