From mboxrd@z Thu Jan 1 00:00:00 1970 From: Yann E. MORIN Date: Tue, 29 Jun 2021 21:44:40 +0200 Subject: [Buildroot] [PATCH] package/pdbg: Bump version to v3.3 In-Reply-To: <20210628071630.226385-1-joel@jms.id.au> References: <20210628071630.226385-1-joel@jms.id.au> Message-ID: <20210629194440.GN2829@scaer> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: buildroot@busybox.net Joel, All, On 2021-06-28 16:46 +0930, Joel Stanley spake thusly: > Remove patches that now are upstream. > > Upstream changes: > - Implement p10 thread controls > - Update p10 sbefifo chip-ops > - For p10 switch default backend to sbefifo > - Separate sbe api into new header libpdbg_sbe.h > > Signed-off-by: Joel Stanley Applied to master, thanks. Regards, Yann E. MORIN. > --- > .../0001-build-Fix-build-with-DNDEBUG.patch | 78 ------------------- > .../0002-build-Fix-build-with-DNDEBUG.patch | 71 ----------------- > package/pdbg/pdbg.hash | 2 +- > package/pdbg/pdbg.mk | 2 +- > 4 files changed, 2 insertions(+), 151 deletions(-) > delete mode 100644 package/pdbg/0001-build-Fix-build-with-DNDEBUG.patch > delete mode 100644 package/pdbg/0002-build-Fix-build-with-DNDEBUG.patch > > diff --git a/package/pdbg/0001-build-Fix-build-with-DNDEBUG.patch b/package/pdbg/0001-build-Fix-build-with-DNDEBUG.patch > deleted file mode 100644 > index fba5d3869383..000000000000 > --- a/package/pdbg/0001-build-Fix-build-with-DNDEBUG.patch > +++ /dev/null > @@ -1,78 +0,0 @@ > -From 9b4276d33c8637ca912911e720d6c760812a769b Mon Sep 17 00:00:00 2001 > -From: Fabrice Fontaine > -Date: Wed, 9 Jun 2021 21:15:31 +0200 > -Subject: [PATCH] build: Fix build with -DNDEBUG > - > -Fix the following build failures with -DNDEBUG: > - > -libpdbg/adu.c: In function 'blog2': > -libpdbg/adu.c:112:1: error: control reaches end of non-void function [-Werror=return-type] > - 112 | } > - | ^ > - CC libpdbg/libpdbg_la-chip.lo > - CC libpdbg/libpdbg_la-cronus.lo > -libpdbg/cfam.c: In function 'cfam_hmfsi_probe': > -libpdbg/cfam.c:325:13: error: 'port' is used uninitialized in this function [-Werror=uninitialized] > - 325 | value |= 1 << (31 - port); > - | ~~^~~~~~~~~~~~~~ > - > -Fixes: > - - http://autobuild.buildroot.org/results/9d90ede1ff7425cbb25b95aed3bf8d27ced865a4 > - > -Signed-off-by: Fabrice Fontaine > -Reviewed-by: Amitay Isaacs > -[Retrieved from: > -https://github.com/open-power/pdbg/commit/9b4276d33c8637ca912911e720d6c760812a769b] > ---- > - libpdbg/adu.c | 3 ++- > - libpdbg/device.c | 2 +- > - src/mem.c | 2 +- > - 3 files changed, 4 insertions(+), 3 deletions(-) > - > -diff --git a/libpdbg/adu.c b/libpdbg/adu.c > -index 6358e7a3f..7ee15c350 100644 > ---- a/libpdbg/adu.c > -+++ b/libpdbg/adu.c > -@@ -15,6 +15,7 @@ > - */ > - #include > - #include > -+#include > - #include > - #include > - > -@@ -107,7 +108,7 @@ uint8_t blog2(uint8_t x) > - case 64: > - return 6; > - default: > -- assert(0); > -+ abort(); > - } > - } > - > -diff --git a/libpdbg/device.c b/libpdbg/device.c > -index 8884baa45..502ca38ab 100644 > ---- a/libpdbg/device.c > -+++ b/libpdbg/device.c > -@@ -501,7 +501,7 @@ static enum pdbg_target_status str_to_status(const char *status) > - else if (!strcmp(status, "unknown")) > - return PDBG_TARGET_UNKNOWN; > - else > -- assert(0); > -+ abort(); > - } > - > - static int dt_expand_node(struct pdbg_target *node, void *fdt, int fdt_node) > -diff --git a/src/mem.c b/src/mem.c > -index 99a673f36..878ac029a 100644 > ---- a/src/mem.c > -+++ b/src/mem.c > -@@ -166,7 +166,7 @@ OPTCMD_DEFINE_CMD_WITH_FLAGS(getmemio, getmemio, (ADDRESS, DATA, BLOCK_SIZE), > - static int _putmem(const char *mem_prefix, uint64_t addr, uint8_t block_size, bool ci) > - { > - uint8_t *buf; > -- size_t buflen; > -+ size_t buflen = 0; > - int rc, count = 0; > - struct pdbg_target *target; > - > diff --git a/package/pdbg/0002-build-Fix-build-with-DNDEBUG.patch b/package/pdbg/0002-build-Fix-build-with-DNDEBUG.patch > deleted file mode 100644 > index 323a87d7798e..000000000000 > --- a/package/pdbg/0002-build-Fix-build-with-DNDEBUG.patch > +++ /dev/null > @@ -1,71 +0,0 @@ > -From cd1a44bac413e8a6bc2572720153cc5e703762e1 Mon Sep 17 00:00:00 2001 > -From: Amitay Isaacs > -Date: Fri, 11 Jun 2021 15:27:29 +1000 > -Subject: [PATCH] build: Fix build with -DNDEBUG > - > -Signed-off-by: Amitay Isaacs > -[Retrieved from: > -https://github.com/open-power/pdbg/commit/cd1a44bac413e8a6bc2572720153cc5e703762e1] > ---- > - libpdbg/cfam.c | 6 +++++- > - libpdbg/device.c | 9 ++++++--- > - 2 files changed, 11 insertions(+), 4 deletions(-) > - > -diff --git a/libpdbg/cfam.c b/libpdbg/cfam.c > -index ffe69296e..976ce5a6d 100644 > ---- a/libpdbg/cfam.c > -+++ b/libpdbg/cfam.c > -@@ -17,6 +17,7 @@ > - */ > - #include > - #include > -+#include > - #include > - > - #include "hwunit.h" > -@@ -320,7 +321,10 @@ static int cfam_hmfsi_probe(struct pdbg_target *target) > - int rc; > - > - /* Enable the port in the upstream control register */ > -- assert(!(pdbg_target_u32_property(target, "port", &port))); > -+ if (pdbg_target_u32_property(target, "port", &port)) { > -+ PR_ERROR("Attribute port not defined for %s\n", pdbg_target_path(target)); > -+ abort(); > -+ } > - fsi_read(fsi_parent, 0x3404, &value); > - value |= 1 << (31 - port); > - if ((rc = fsi_write(fsi_parent, 0x3404, value))) { > -diff --git a/libpdbg/device.c b/libpdbg/device.c > -index 502ca38ab..ab821d387 100644 > ---- a/libpdbg/device.c > -+++ b/libpdbg/device.c > -@@ -417,7 +417,7 @@ static const void *dt_require_property(struct pdbg_target *node, > - > - prerror("DT: Missing required property %s/%s\n", > - path, name); > -- assert(false); > -+ abort(); > - } > - if (wanted_len >= 0 && len != wanted_len) { > - const char *path = dt_get_path(node); > -@@ -426,7 +426,7 @@ static const void *dt_require_property(struct pdbg_target *node, > - path, name); > - prerror("DT: Expected len: %d got len: %zu\n", > - wanted_len, len); > -- assert(false); > -+ abort(); > - } > - > - if (prop_len) { > -@@ -604,7 +604,10 @@ uint64_t pdbg_target_address(struct pdbg_target *target, uint64_t *out_size) > - > - p = dt_require_property(target, "reg", -1, &len); > - n = (na + ns) * sizeof(u32); > -- assert(n <= len); > -+ if (n > len) { > -+ PR_ERROR("Invalid reg value for %s\n", pdbg_target_path(target)); > -+ abort(); > -+ } > - if (out_size) > - *out_size = dt_get_number(p + na * sizeof(u32), ns); > - return dt_get_number(p, na); > diff --git a/package/pdbg/pdbg.hash b/package/pdbg/pdbg.hash > index bc9f99b149b6..9a58fa842c6a 100644 > --- a/package/pdbg/pdbg.hash > +++ b/package/pdbg/pdbg.hash > @@ -1,3 +1,3 @@ > # Locally calculated > -sha256 c47adcac1c404266217be8c4acb408d9eba59562b094c941e96e4f77f3ced7d2 pdbg-3.2.tar.gz > +sha256 87fe68977f72885e46d321da806d30f33a901de817461eb44fa727cd81d1d109 pdbg-3.3.tar.gz > sha256 cfc7749b96f63bd31c3c42b5c471bf756814053e847c10f3eb003417bc523d30 COPYING > diff --git a/package/pdbg/pdbg.mk b/package/pdbg/pdbg.mk > index e8bb36c3272e..7f6b48e6eb57 100644 > --- a/package/pdbg/pdbg.mk > +++ b/package/pdbg/pdbg.mk > @@ -4,7 +4,7 @@ > # > ################################################################################ > > -PDBG_VERSION = 3.2 > +PDBG_VERSION = 3.3 > PDBG_SITE = $(call github,open-power,pdbg,v$(PDBG_VERSION)) > PDBG_LICENSE = Apache-2.0 > PDBG_LICENSE_FILES = COPYING > -- > 2.32.0 > > _______________________________________________ > buildroot mailing list > buildroot at busybox.net > http://lists.busybox.net/mailman/listinfo/buildroot -- .-----------------.--------------------.------------------.--------------------. | Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: | | +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ | | +33 561 099 427 `------------.-------: X AGAINST | \e/ There is no | | http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. | '------------------------------^-------^------------------^--------------------'