Attention is currently required from: osmith.
Hello Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/dahdi-linux/+/39593?usp=email
to look at the new patch set (#3).
The following approvals got outdated and were removed:
Verified+1 by osmith
Change subject: drivers/dahdi: fix always true cond
......................................................................
drivers/dahdi: fix always true cond
Remove these checks as they are not useful. For example, wc->xb gets
allocated together with wc. With these removed, the code behaves the
same as before.
Fix for these warnings with current linux master:
wcte43x-base.c: In function 't43x_init_one':
wcte43x-base.c:3517:13: warning: the comparison will always evaluate as 'true' for
the address of 'xb' will never be NULL [-Waddress]
3517 | if (&wc->xb)
| ^
wcte43x-base.c:141:21: note: 'xb' declared here
141 | struct wcxb xb;
| ^~
xpp/xbus-core.c: In function 'xbus_check_unique':
xpp/xbus-core.c:116:13: warning: the comparison will always evaluate as 'true' for
the address of 'label' will never be NULL [-Waddress]
116 | if (xbus->label && *(xbus->label)) {
| ^~~~
etc.
Change-Id: I33ee446a32de3bb8d9fe966a2989994bf4bbd806
---
M drivers/dahdi/dahdi-base.c
M drivers/dahdi/dahdi_dynamic_ethmf.c
M drivers/dahdi/wcte13xp-base.c
M drivers/dahdi/wcte43x-base.c
M drivers/dahdi/xpp/xbus-core.c
5 files changed, 7 insertions(+), 15 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/dahdi-linux refs/changes/93/39593/3
--
To view, visit
https://gerrit.osmocom.org/c/dahdi-linux/+/39593?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: dahdi-linux
Gerrit-Branch: master
Gerrit-Change-Id: I33ee446a32de3bb8d9fe966a2989994bf4bbd806
Gerrit-Change-Number: 39593
Gerrit-PatchSet: 3
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-CC: laforge <laforge(a)osmocom.org>
Gerrit-CC: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: osmith <osmith(a)sysmocom.de>