diff options
author | np <np@FreeBSD.org> | 2013-04-11 21:15:35 +0000 |
---|---|---|
committer | np <np@FreeBSD.org> | 2013-04-11 21:15:35 +0000 |
commit | ebbc873044d70fc8b8cbbd1b281559c33d3bcf43 (patch) | |
tree | e5dceed51d425ef96897955f7c3499dc67a26844 | |
parent | 5f41a5dc88a9597fadcef346c2433abc00b1f650 (diff) | |
download | FreeBSD-src-ebbc873044d70fc8b8cbbd1b281559c33d3bcf43.zip FreeBSD-src-ebbc873044d70fc8b8cbbd1b281559c33d3bcf43.tar.gz |
There is no need for elaborate queries and error checking when trying to
set FW4MSG_ENCAP.
MFC after: 3 days
-rw-r--r-- | sys/dev/cxgbe/t4_main.c | 22 |
1 files changed, 4 insertions, 18 deletions
diff --git a/sys/dev/cxgbe/t4_main.c b/sys/dev/cxgbe/t4_main.c index 58b2c45..74bfa51 100644 --- a/sys/dev/cxgbe/t4_main.c +++ b/sys/dev/cxgbe/t4_main.c @@ -2480,27 +2480,13 @@ static int set_params__post_init(struct adapter *sc) { uint32_t param, val; - int rc; + /* ask for encapsulated CPLs */ param = FW_PARAM_PFVF(CPLFW4MSG_ENCAP); - rc = -t4_query_params(sc, sc->mbox, sc->pf, 0, 1, ¶m, &val); - if (rc == 0) { - /* ask for encapsulated CPLs */ - param = FW_PARAM_PFVF(CPLFW4MSG_ENCAP); - val = 1; - rc = -t4_set_params(sc, sc->mbox, sc->pf, 0, 1, ¶m, &val); - if (rc != 0) { - device_printf(sc->dev, - "failed to set parameter (post_init): %d.\n", rc); - return (rc); - } - } else if (rc != FW_EINVAL) { - device_printf(sc->dev, - "failed to check for encapsulated CPLs: %d.\n", rc); - } else - rc = 0; /* the firmware doesn't support the param, no worries */ + val = 1; + (void)t4_set_params(sc, sc->mbox, sc->pf, 0, 1, ¶m, &val); - return (rc); + return (0); } #undef FW_PARAM_PFVF |