summaryrefslogtreecommitdiffstats
path: root/pcilib
diff options
context:
space:
mode:
authorSuren A. Chilingaryan <csa@suren.me>2015-10-22 17:08:11 +0200
committerSuren A. Chilingaryan <csa@suren.me>2015-10-22 17:08:11 +0200
commitad838ab9f70ff9712c256f072e260e03a8d7a7da (patch)
tree4f82d7a5274ebcfe6451e2c6c0b8ad36d14f86e6 /pcilib
parenta0fe5cdea15bca52d3b101a88c11cb160ef08b2f (diff)
downloadpcitool-ad838ab9f70ff9712c256f072e260e03a8d7a7da.tar.gz
pcitool-ad838ab9f70ff9712c256f072e260e03a8d7a7da.tar.bz2
pcitool-ad838ab9f70ff9712c256f072e260e03a8d7a7da.tar.xz
pcitool-ad838ab9f70ff9712c256f072e260e03a8d7a7da.zip
Fix access to the property-based registers
Diffstat (limited to 'pcilib')
-rw-r--r--pcilib/property.c6
-rw-r--r--pcilib/register.c40
2 files changed, 3 insertions, 43 deletions
diff --git a/pcilib/property.c b/pcilib/property.c
index a39594c..7239601 100644
--- a/pcilib/property.c
+++ b/pcilib/property.c
@@ -34,9 +34,9 @@ int pcilib_add_registers_from_properties(pcilib_t *ctx, size_t n, pcilib_view_co
return err;
}
}
-
- access = ctx->banks[bank].access;
-
+
+ access = ctx->banks[bank].access / 8;
+
for (i = 0; i < n; i++) {
if ((v[i]->flags&PCILIB_VIEW_FLAG_REGISTER) == 0) continue;
diff --git a/pcilib/register.c b/pcilib/register.c
index 1f1cf9e..a11bdac 100644
--- a/pcilib/register.c
+++ b/pcilib/register.c
@@ -185,26 +185,6 @@ static int pcilib_read_register_space_internal(pcilib_t *ctx, pcilib_register_ba
return PCILIB_ERROR_OUTOFRANGE;
}
- if (b->protocol == PCILIB_REGISTER_PROTOCOL_PROPERTY) {
- for (i = 0; i < (bits?(n+1):n); i++) {
- if ((ctx->views[i]->flags&PCILIB_VIEW_FLAG_REGISTER) == 0) {
- pcilib_error("Accessing invalid register %u (associated view does not provide register functionality)", addr + i);
- return PCILIB_ERROR_INVALID_REQUEST;
- }
-
- if ((ctx->views[i]->mode&PCILIB_ACCESS_R) == 0) {
- pcilib_error("Read access is not allowed to register %u", addr + i);
- return PCILIB_ERROR_NOTPERMITED;
- }
- }
- }
-
- //err = pcilib_init_register_banks(ctx);
- //if (err) return err;
-
- //n += bits / b->access;
- //bits %= b->access;
-
for (i = 0; i < n; i++) {
err = bapi->read(ctx, bctx, addr + i * access, buf + i);
if (err) break;
@@ -315,26 +295,6 @@ static int pcilib_write_register_space_internal(pcilib_t *ctx, pcilib_register_b
return PCILIB_ERROR_OUTOFRANGE;
}
- if (b->protocol == PCILIB_REGISTER_PROTOCOL_PROPERTY) {
- for (i = 0; i < (bits?(n+1):n); i++) {
- if ((ctx->views[i]->flags&PCILIB_VIEW_FLAG_REGISTER) == 0) {
- pcilib_error("Accessing invalid register %u (associated view does not provide register functionality)", addr + i);
- return PCILIB_ERROR_INVALID_REQUEST;
- }
-
- if ((ctx->views[i]->mode&PCILIB_ACCESS_W) == 0) {
- pcilib_error("Write access is not allowed to register %u", addr + i);
- return PCILIB_ERROR_NOTPERMITED;
- }
- }
- }
-
- //err = pcilib_init_register_banks(ctx);
- //if (err) return err;
-
- //n += bits / b->access;
- //bits %= b->access;
-
for (i = 0; i < n; i++) {
err = bapi->write(ctx, bctx, addr + i * access, buf[i]);
if (err) break;