len0rd/dtc
1
0
Fork 0
mirror of https://github.com/dgibson/dtc.git synced 2026-04-11 16:37:42 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions

dtc: Remove unused dts_version in dtc-lexer.l
Some checks failed
Build test / build-make (alpine) (push) Has been cancelled
Details
Build test / build-make (archlinux) (push) Has been cancelled
Details
Build test / build-make (fedora) (push) Has been cancelled
Details
Build test / build-make (ubuntu) (push) Has been cancelled
Details
Build test / build-meson (alpine) (push) Has been cancelled
Details
Build test / build-meson (archlinux) (push) Has been cancelled
Details
Build test / build-meson (fedora) (push) Has been cancelled
Details
Build test / build-meson (ubuntu) (push) Has been cancelled
Details
Build test / clang64 (push) Has been cancelled
Details
Build test / mingw32 (push) Has been cancelled
Details
Build test / mingw64 (push) Has been cancelled
Details
Build test / ucrt64 (push) Has been cancelled
Details

Browse source 
A recent strengthening of -Wunused-but-set-variable (enabled with -Wall)
in clang under a new subwarning, -Wunused-but-set-global, points out an
unused static global variable in dtc-lexer.l:

  ../dtc-lexer.l:42:12: error: variable 'dts_version' set but not used [-Werror,-Wunused-but-set-global]
     42 | static int dts_version = 1;
        |            ^

This variable has been unused since commit 4e1a0a0 ("Remove support for
the legacy DTS source file format."). Remove it to clear up the warning.

Signed-off-by: Nathan Chancellor <nathan@kernel.org>
Message-ID: <20260325-dtc-lexer-remove-dts_version-v1-1-0b5d64903bbb@kernel.org>
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
This commit is contained in:
Nathan Chancellor 2026-03-25 17:13:44 -07:00 • committed by David Gibson
parent caf7465c5d
commit 53373d1355
1 changed files with 0 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

3
dtc-lexer.l
View file

@ -39,8 +39,6 @@ extern bool treesource_error;
#define DPRINT(fmt, ...) do { } while (0)
#endif
static int dts_version = 1;
#define BEGIN_DEFAULT() DPRINT("<V1>\n"); \
BEGIN(V1); \
@ -101,7 +99,6 @@ static void PRINTF(1, 2) lexical_error(const char *fmt, ...);
<*>"/dts-v1/" {
DPRINT("Keyword: /dts-v1/\n");
dts_version = 1;
BEGIN_DEFAULT();
return DT_V1;
}