* [Buildroot] [PATCH] package/pdbg: new package
@ 2016-07-25 6:16 Joel Stanley
2016-07-25 6:53 ` [Buildroot] [PATCH v2] " Joel Stanley
0 siblings, 1 reply; 7+ messages in thread
From: Joel Stanley @ 2016-07-25 6:16 UTC (permalink / raw)
To: buildroot
PowerPC FSI Debugger, for low level debugging of a Power8 CPU over FSI.
Signed-off-by: Joel Stanley <joel@jms.id.au>
---
package/pdbg/Config.in | 6 ++++++
package/pdbg/pdbg.mk | 14 ++++++++++++++
2 files changed, 20 insertions(+)
create mode 100644 package/pdbg/Config.in
create mode 100644 package/pdbg/pdbg.mk
diff --git a/package/pdbg/Config.in b/package/pdbg/Config.in
new file mode 100644
index 000000000000..f25867fa2c41
--- /dev/null
+++ b/package/pdbg/Config.in
@@ -0,0 +1,6 @@
+config BR2_PACKAGE_PDBG
+ bool "pdbg"
+ depends on BR2_arm
+ help
+ PowerPC FSI Debugger, for low level debugging of a PowerPC CPU
+ over FSI.
diff --git a/package/pdbg/pdbg.mk b/package/pdbg/pdbg.mk
new file mode 100644
index 000000000000..48444567bee3
--- /dev/null
+++ b/package/pdbg/pdbg.mk
@@ -0,0 +1,14 @@
+################################################################################
+#
+# pdbg
+#
+################################################################################
+
+PDBG_VERSION = 90a7370a11e727f1482dea6ff2bd6aec20c64805
+
+PDBG_SITE = $(call github,open-power,pdbg,$(PDBG_VERSION))
+PDBG_LICENSE = Apache 2.0
+PDBG_LICENSE_FILES = COPYING
+PDBG_AUTORECONF = YES
+
+$(eval $(autotools-package))
--
2.8.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [Buildroot] [PATCH v2] package/pdbg: new package
2016-07-25 6:16 [Buildroot] [PATCH] package/pdbg: new package Joel Stanley
@ 2016-07-25 6:53 ` Joel Stanley
2016-07-25 22:09 ` Thomas Petazzoni
2016-07-26 7:28 ` [Buildroot] [PATCH v3] " Joel Stanley
0 siblings, 2 replies; 7+ messages in thread
From: Joel Stanley @ 2016-07-25 6:53 UTC (permalink / raw)
To: buildroot
PowerPC FSI Debugger, for low level debugging of a Power8 CPU over FSI.
Signed-off-by: Joel Stanley <joel@jms.id.au>
---
v2:
- Fix whitespace in pdbg.mk
- Add URL to help text
package/pdbg/Config.in | 8 ++++++++
package/pdbg/pdbg.mk | 13 +++++++++++++
2 files changed, 21 insertions(+)
create mode 100644 package/pdbg/Config.in
create mode 100644 package/pdbg/pdbg.mk
diff --git a/package/pdbg/Config.in b/package/pdbg/Config.in
new file mode 100644
index 000000000000..cf37eb8aca41
--- /dev/null
+++ b/package/pdbg/Config.in
@@ -0,0 +1,8 @@
+config BR2_PACKAGE_PDBG
+ bool "pdbg"
+ depends on BR2_arm
+ help
+ PowerPC FSI Debugger, for low level debugging of a PowerPC CPU
+ over FSI.
+
+ https://github.com/open-power/pdbg
diff --git a/package/pdbg/pdbg.mk b/package/pdbg/pdbg.mk
new file mode 100644
index 000000000000..9cc05cd728c0
--- /dev/null
+++ b/package/pdbg/pdbg.mk
@@ -0,0 +1,13 @@
+################################################################################
+#
+# pdbg
+#
+################################################################################
+
+PDBG_VERSION = 90a7370a11e727f1482dea6ff2bd6aec20c64805
+PDBG_SITE = $(call github,open-power,pdbg,$(PDBG_VERSION))
+PDBG_LICENSE = Apache 2.0
+PDBG_LICENSE_FILES = COPYING
+PDBG_AUTORECONF = YES
+
+$(eval $(autotools-package))
--
2.8.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [Buildroot] [PATCH v2] package/pdbg: new package
2016-07-25 6:53 ` [Buildroot] [PATCH v2] " Joel Stanley
@ 2016-07-25 22:09 ` Thomas Petazzoni
2016-07-26 2:13 ` Joel Stanley
2016-07-26 7:28 ` [Buildroot] [PATCH v3] " Joel Stanley
1 sibling, 1 reply; 7+ messages in thread
From: Thomas Petazzoni @ 2016-07-25 22:09 UTC (permalink / raw)
To: buildroot
Hello,
On Mon, 25 Jul 2016 16:23:12 +0930, Joel Stanley wrote:
> diff --git a/package/pdbg/Config.in b/package/pdbg/Config.in
> new file mode 100644
> index 000000000000..cf37eb8aca41
> --- /dev/null
> +++ b/package/pdbg/Config.in
> @@ -0,0 +1,8 @@
> +config BR2_PACKAGE_PDBG
> + bool "pdbg"
> + depends on BR2_arm
Anything that makes it ARM-specific?
> + help
> + PowerPC FSI Debugger, for low level debugging of a PowerPC CPU
> + over FSI.
It would have been good to explain somewhere than pdbg runs on the BMC,
which is generally an ARM-powered system monitoring/controlling a
number of PowerPC CPUs. Otherwise, it feels odd to have a
PowerPC-related package available only on the ARM architecture.
> +PDBG_VERSION = 90a7370a11e727f1482dea6ff2bd6aec20c64805
> +PDBG_SITE = $(call github,open-power,pdbg,$(PDBG_VERSION))
> +PDBG_LICENSE = Apache 2.0
> +PDBG_LICENSE_FILES = COPYING
> +PDBG_AUTORECONF = YES
Side note (unrelated to Buildroot packaging): had a look at the code,
and this mapping of /dev/mem to then bing-bang some protocol on GPIOs
from userspace is really, really nasty :-/
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Buildroot] [PATCH v2] package/pdbg: new package
2016-07-25 22:09 ` Thomas Petazzoni
@ 2016-07-26 2:13 ` Joel Stanley
2016-07-26 7:18 ` Thomas Petazzoni
0 siblings, 1 reply; 7+ messages in thread
From: Joel Stanley @ 2016-07-26 2:13 UTC (permalink / raw)
To: buildroot
Hi Thomas,
On Tue, Jul 26, 2016 at 7:39 AM, Thomas Petazzoni
<thomas.petazzoni@free-electrons.com> wrote:
> Hello,
>
> On Mon, 25 Jul 2016 16:23:12 +0930, Joel Stanley wrote:
>
>> diff --git a/package/pdbg/Config.in b/package/pdbg/Config.in
>> new file mode 100644
>> index 000000000000..cf37eb8aca41
>> --- /dev/null
>> +++ b/package/pdbg/Config.in
>> @@ -0,0 +1,8 @@
>> +config BR2_PACKAGE_PDBG
>> + bool "pdbg"
>> + depends on BR2_arm
>
> Anything that makes it ARM-specific?
Upstream has only ever tested the code on ARM. It does build for x86
(and power, and anything else I guess), but as you point out below it
is only useful when run from a processor that is attached to a p8
host.
What's the general rule in this circumstance?
>
>> + help
>> + PowerPC FSI Debugger, for low level debugging of a PowerPC CPU
>> + over FSI.
>
> It would have been good to explain somewhere than pdbg runs on the BMC,
> which is generally an ARM-powered system monitoring/controlling a
> number of PowerPC CPUs. Otherwise, it feels odd to have a
> PowerPC-related package available only on the ARM architecture.
Sure, I will add some more text.
>
>> +PDBG_VERSION = 90a7370a11e727f1482dea6ff2bd6aec20c64805
>> +PDBG_SITE = $(call github,open-power,pdbg,$(PDBG_VERSION))
>> +PDBG_LICENSE = Apache 2.0
>> +PDBG_LICENSE_FILES = COPYING
>> +PDBG_AUTORECONF = YES
>
> Side note (unrelated to Buildroot packaging): had a look at the code,
> and this mapping of /dev/mem to then bing-bang some protocol on GPIOs
> from userspace is really, really nasty :-/
Yep! You and I are the same page here. To date the tool has been used
on a BIOS vendor's BMC firmware, running a very old kernel that lacks
the proper linux APIs for access to hardware. It's not ideal.
I've been slowly upstreaming a from-scratch kernel port. Once we have
that in place pdbg can add support for doing GPIO through the kernel.
In the future near future we will add FSI support in the kernel, and
from there perhaps use the Coldfire processor inside the Aspeed SoC to
offload the bitbanging. Lots of work to do, but we're getting there.
Cheers,
Joel
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Buildroot] [PATCH v2] package/pdbg: new package
2016-07-26 2:13 ` Joel Stanley
@ 2016-07-26 7:18 ` Thomas Petazzoni
0 siblings, 0 replies; 7+ messages in thread
From: Thomas Petazzoni @ 2016-07-26 7:18 UTC (permalink / raw)
To: buildroot
Hello,
On Tue, 26 Jul 2016 11:43:53 +0930, Joel Stanley wrote:
> > Anything that makes it ARM-specific?
>
> Upstream has only ever tested the code on ARM. It does build for x86
> (and power, and anything else I guess), but as you point out below it
> is only useful when run from a processor that is attached to a p8
> host.
>
> What's the general rule in this circumstance?
I believe in such cases we tend to make the package available on all
architectures anyway. There is no inherent limitation in having it
build and run on other architectures.
The only cases where we add architecture dependencies are:
- The package contains architecture-specific code.
- The package contains pre-built binaries for a given architecture
(e.g. OpenGL blobs).
- The package is really relevant only for an architecture, like some
GPU or VPU related stuff, where the GPU/VPU only exists for this
architecture. We could say that pdbg falls into this category though.
> >> +PDBG_VERSION = 90a7370a11e727f1482dea6ff2bd6aec20c64805
> >> +PDBG_SITE = $(call github,open-power,pdbg,$(PDBG_VERSION))
> >> +PDBG_LICENSE = Apache 2.0
> >> +PDBG_LICENSE_FILES = COPYING
> >> +PDBG_AUTORECONF = YES
> >
> > Side note (unrelated to Buildroot packaging): had a look at the code,
> > and this mapping of /dev/mem to then bing-bang some protocol on GPIOs
> > from userspace is really, really nasty :-/
>
> Yep! You and I are the same page here. To date the tool has been used
> on a BIOS vendor's BMC firmware, running a very old kernel that lacks
> the proper linux APIs for access to hardware. It's not ideal.
>
> I've been slowly upstreaming a from-scratch kernel port. Once we have
> that in place pdbg can add support for doing GPIO through the kernel.
>
> In the future near future we will add FSI support in the kernel, and
> from there perhaps use the Coldfire processor inside the Aspeed SoC to
> offload the bitbanging. Lots of work to do, but we're getting there.
OK, sounds good.
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Buildroot] [PATCH v3] package/pdbg: new package
2016-07-25 6:53 ` [Buildroot] [PATCH v2] " Joel Stanley
2016-07-25 22:09 ` Thomas Petazzoni
@ 2016-07-26 7:28 ` Joel Stanley
2016-07-26 19:30 ` Thomas Petazzoni
1 sibling, 1 reply; 7+ messages in thread
From: Joel Stanley @ 2016-07-26 7:28 UTC (permalink / raw)
To: buildroot
PowerPC FSI Debugger, for low level debugging of a Power8 CPU over FSI.
Signed-off-by: Joel Stanley <joel@jms.id.au>
---
v3:
- do not restrict to ARM, as it builds on all CPU architectures
- metion where you might want to run pdbg in the help text
v2:
- Fix whitespace in pdbg.mk
- Add URL to help text
package/pdbg/Config.in | 10 ++++++++++
package/pdbg/pdbg.mk | 13 +++++++++++++
2 files changed, 23 insertions(+)
create mode 100644 package/pdbg/Config.in
create mode 100644 package/pdbg/pdbg.mk
diff --git a/package/pdbg/Config.in b/package/pdbg/Config.in
new file mode 100644
index 000000000000..5a3bcda48ba8
--- /dev/null
+++ b/package/pdbg/Config.in
@@ -0,0 +1,10 @@
+config BR2_PACKAGE_PDBG
+ bool "pdbg"
+ help
+ PowerPC FSI Debugger, for low level debugging of a IBM Power8 CPU
+ over FSI.
+
+ pdbg runs on a BMC, which is generally an ARM-powered system
+ monitoring/controlling a host CPU.
+
+ https://github.com/open-power/pdbg
diff --git a/package/pdbg/pdbg.mk b/package/pdbg/pdbg.mk
new file mode 100644
index 000000000000..9cc05cd728c0
--- /dev/null
+++ b/package/pdbg/pdbg.mk
@@ -0,0 +1,13 @@
+################################################################################
+#
+# pdbg
+#
+################################################################################
+
+PDBG_VERSION = 90a7370a11e727f1482dea6ff2bd6aec20c64805
+PDBG_SITE = $(call github,open-power,pdbg,$(PDBG_VERSION))
+PDBG_LICENSE = Apache 2.0
+PDBG_LICENSE_FILES = COPYING
+PDBG_AUTORECONF = YES
+
+$(eval $(autotools-package))
--
2.8.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [Buildroot] [PATCH v3] package/pdbg: new package
2016-07-26 7:28 ` [Buildroot] [PATCH v3] " Joel Stanley
@ 2016-07-26 19:30 ` Thomas Petazzoni
0 siblings, 0 replies; 7+ messages in thread
From: Thomas Petazzoni @ 2016-07-26 19:30 UTC (permalink / raw)
To: buildroot
Hello,
On Tue, 26 Jul 2016 16:58:24 +0930, Joel Stanley wrote:
> PowerPC FSI Debugger, for low level debugging of a Power8 CPU over FSI.
>
> Signed-off-by: Joel Stanley <joel@jms.id.au>
> ---
> v3:
> - do not restrict to ARM, as it builds on all CPU architectures
> - metion where you might want to run pdbg in the help text
>
> v2:
> - Fix whitespace in pdbg.mk
> - Add URL to help text
>
> package/pdbg/Config.in | 10 ++++++++++
> package/pdbg/pdbg.mk | 13 +++++++++++++
You forgot to add the package to package/Config.in, and to add a hash
file. I've added both and applied. Thanks!
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2016-07-26 19:30 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-07-25 6:16 [Buildroot] [PATCH] package/pdbg: new package Joel Stanley
2016-07-25 6:53 ` [Buildroot] [PATCH v2] " Joel Stanley
2016-07-25 22:09 ` Thomas Petazzoni
2016-07-26 2:13 ` Joel Stanley
2016-07-26 7:18 ` Thomas Petazzoni
2016-07-26 7:28 ` [Buildroot] [PATCH v3] " Joel Stanley
2016-07-26 19:30 ` Thomas Petazzoni
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.