Date: Thu, 27 Jul 2023 21:04:15 -0600 From: Warner Losh <imp@bsdimp.com> To: Renato Botelho <garga@freebsd.org> Cc: Warner Losh <imp@freebsd.org>, src-committers@freebsd.org, dev-commits-src-all@freebsd.org, dev-commits-src-main@freebsd.org Subject: Re: git: 971bac5ace7a - main - kbd: consolidate kb interfaces (phase one) Message-ID: <CANCZdfoDXaO21ZfW6xvW%2BadyBWvHjs2FL_XGdG7uD23igwXKuA@mail.gmail.com> In-Reply-To: <65b0e348-b27d-f641-1004-6f76ff3ebfbd@FreeBSD.org> References: <202307070510.3675AWZS051152@gitrepo.freebsd.org> <65b0e348-b27d-f641-1004-6f76ff3ebfbd@FreeBSD.org>
index | next in thread | previous in thread | raw e-mail
[-- Attachment #1 --] On Thu, Jul 27, 2023 at 5:48 AM Renato Botelho <garga@freebsd.org> wrote: > On 07/07/23 02:10, Warner Losh wrote: > > The branch main has been updated by imp: > > > > URL: > https://cgit.FreeBSD.org/src/commit/?id=971bac5ace7a3c2262349fa2808188a11a5ffeed > > > > commit 971bac5ace7a3c2262349fa2808188a11a5ffeed > > Author: Michael <git@paepcke.de> > > AuthorDate: 2023-07-07 02:32:45 +0000 > > Commit: Warner Losh <imp@FreeBSD.org> > > CommitDate: 2023-07-07 05:10:18 +0000 > > > > kbd: consolidate kb interfaces (phase one) > > > > Refactor to eliminate duplicated rate and delay tables, with minor > style > > tweaks for changed lines. Remove an obsolete comment about needing > to > > convert from microseconds to ticks (that's done elsewhere). Remove > > traiing whitespace in kbdcontrol.c. > > > > Except for the new warning, no change in behavior > > > > Sponsored by: DSS GmbH > > Reviewed by: imp [minor style tweaks as well] > > Pull Request: https://github.com/freebsd/pull/683 > > Differential Revision: https://reviews.freebsd.org/D38818 > > --- > > sys/dev/adb/adb_kbd.c | 12 ++------- > > sys/dev/atkbdc/atkbd.c | 18 +++++--------- > > sys/dev/gpio/gpiokeys.c | 10 ++------ > > sys/dev/hid/hkbd.c | 10 ++------ > > sys/dev/kbdmux/kbdmux.c | 23 +++++------------ > > sys/dev/usb/input/ukbd.c | 10 ++------ > > sys/dev/vkbd/vkbd.c | 18 +++++--------- > > sys/sys/kbio.h | 7 ++++++ > > usr.sbin/kbdcontrol/kbdcontrol.c | 53 > +++++++++++++++++++++------------------- > > 9 files changed, 61 insertions(+), 100 deletions(-) > > After this change I started seeing following error building kbdcontrol: > > Building /usr/obj/usr/src/amd64.amd64/usr.sbin/kbdcontrol/kbdcontrol.o > /usr/src/usr.sbin/kbdcontrol/kbdcontrol.c:97:35: error: use of > undeclared identifier 'kbdelays'; did you mean 'ndelays'? > static const int ndelays = nitems(kbdelays); > ^~~~~~~~ > ndelays > /usr/include/sys/param.h:324:28: note: expanded from macro 'nitems' > #define nitems(x) (sizeof((x)) / sizeof((x)[0])) > ^ > /usr/src/usr.sbin/kbdcontrol/kbdcontrol.c:97:18: note: 'ndelays' > declared here > static const int ndelays = nitems(kbdelays); > ^ > /usr/src/usr.sbin/kbdcontrol/kbdcontrol.c:97:35: error: use of > undeclared identifier 'kbdelays'; did you mean 'ndelays'? > static const int ndelays = nitems(kbdelays); > ^~~~~~~~ > ndelays > /usr/include/sys/param.h:324:42: note: expanded from macro 'nitems' > #define nitems(x) (sizeof((x)) / sizeof((x)[0])) > ^ > /usr/src/usr.sbin/kbdcontrol/kbdcontrol.c:97:18: note: 'ndelays' > declared here > static const int ndelays = nitems(kbdelays); > ^ > /usr/src/usr.sbin/kbdcontrol/kbdcontrol.c:97:28: error: subscripted > value is not an array, pointer, or vector > static const int ndelays = nitems(kbdelays); > ^~~~~~~~~~~~~~~~ > /usr/include/sys/param.h:324:44: note: expanded from macro 'nitems' > #define nitems(x) (sizeof((x)) / sizeof((x)[0])) > ~~~^~ > /usr/src/usr.sbin/kbdcontrol/kbdcontrol.c:98:36: error: use of > undeclared identifier 'kbrates' > static const int nrepeats = nitems(kbrates); > ^ > /usr/src/usr.sbin/kbdcontrol/kbdcontrol.c:98:36: error: use of > undeclared identifier 'kbrates' > /usr/src/usr.sbin/kbdcontrol/kbdcontrol.c:1072:17: error: use of > undeclared identifier 'kbdelays'; did you mean 'ndelays'? > if (delay <= kbdelays[n]) > ^~~~~~~~ > ndelays > /usr/src/usr.sbin/kbdcontrol/kbdcontrol.c:97:18: note: 'ndelays' > declared here > static const int ndelays = nitems(kbdelays); > ^ > /usr/src/usr.sbin/kbdcontrol/kbdcontrol.c:1072:25: error: subscripted > value is not an array, pointer, or vector > if (delay <= kbdelays[n]) > ~~~~~~~~^~ > /usr/src/usr.sbin/kbdcontrol/kbdcontrol.c:1076:18: error: use of > undeclared identifier 'kbrates' > if (repeat <= kbrates[n]) > ^ > 8 errors generated. > *** Error code 1 > I haven't seen this. How do I recreate this? Does this happen on a clean build? Warner [-- Attachment #2 --] <div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Jul 27, 2023 at 5:48 AM Renato Botelho <<a href="mailto:garga@freebsd.org">garga@freebsd.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 07/07/23 02:10, Warner Losh wrote:<br> > The branch main has been updated by imp:<br> > <br> > URL: <a href="https://cgit.FreeBSD.org/src/commit/?id=971bac5ace7a3c2262349fa2808188a11a5ffeed" rel="noreferrer" target="_blank">https://cgit.FreeBSD.org/src/commit/?id=971bac5ace7a3c2262349fa2808188a11a5ffeed</a><br>; > <br> > commit 971bac5ace7a3c2262349fa2808188a11a5ffeed<br> > Author: Michael <<a href="mailto:git@paepcke.de" target="_blank">git@paepcke.de</a>><br> > AuthorDate: 2023-07-07 02:32:45 +0000<br> > Commit: Warner Losh <imp@FreeBSD.org><br> > CommitDate: 2023-07-07 05:10:18 +0000<br> > <br> > kbd: consolidate kb interfaces (phase one)<br> > <br> > Refactor to eliminate duplicated rate and delay tables, with minor style<br> > tweaks for changed lines. Remove an obsolete comment about needing to<br> > convert from microseconds to ticks (that's done elsewhere). Remove<br> > traiing whitespace in kbdcontrol.c.<br> > <br> > Except for the new warning, no change in behavior<br> > <br> > Sponsored by: DSS GmbH<br> > Reviewed by: imp [minor style tweaks as well]<br> > Pull Request: <a href="https://github.com/freebsd/pull/683" rel="noreferrer" target="_blank">https://github.com/freebsd/pull/683</a><br>; > Differential Revision: <a href="https://reviews.freebsd.org/D38818" rel="noreferrer" target="_blank">https://reviews.freebsd.org/D38818</a><br>; > ---<br> > sys/dev/adb/adb_kbd.c | 12 ++-------<br> > sys/dev/atkbdc/atkbd.c | 18 +++++---------<br> > sys/dev/gpio/gpiokeys.c | 10 ++------<br> > sys/dev/hid/hkbd.c | 10 ++------<br> > sys/dev/kbdmux/kbdmux.c | 23 +++++------------<br> > sys/dev/usb/input/ukbd.c | 10 ++------<br> > sys/dev/vkbd/vkbd.c | 18 +++++---------<br> > sys/sys/kbio.h | 7 ++++++<br> > usr.sbin/kbdcontrol/kbdcontrol.c | 53 +++++++++++++++++++++-------------------<br> > 9 files changed, 61 insertions(+), 100 deletions(-)<br> <br> After this change I started seeing following error building kbdcontrol:<br> <br> Building /usr/obj/usr/src/amd64.amd64/usr.sbin/kbdcontrol/kbdcontrol.o<br> /usr/src/usr.sbin/kbdcontrol/kbdcontrol.c:97:35: error: use of <br> undeclared identifier 'kbdelays'; did you mean 'ndelays'?<br> static const int ndelays = nitems(kbdelays);<br> ^~~~~~~~<br> ndelays<br> /usr/include/sys/param.h:324:28: note: expanded from macro 'nitems'<br> #define nitems(x) (sizeof((x)) / sizeof((x)[0]))<br> ^<br> /usr/src/usr.sbin/kbdcontrol/kbdcontrol.c:97:18: note: 'ndelays' <br> declared here<br> static const int ndelays = nitems(kbdelays);<br> ^<br> /usr/src/usr.sbin/kbdcontrol/kbdcontrol.c:97:35: error: use of <br> undeclared identifier 'kbdelays'; did you mean 'ndelays'?<br> static const int ndelays = nitems(kbdelays);<br> ^~~~~~~~<br> ndelays<br> /usr/include/sys/param.h:324:42: note: expanded from macro 'nitems'<br> #define nitems(x) (sizeof((x)) / sizeof((x)[0]))<br> ^<br> /usr/src/usr.sbin/kbdcontrol/kbdcontrol.c:97:18: note: 'ndelays' <br> declared here<br> static const int ndelays = nitems(kbdelays);<br> ^<br> /usr/src/usr.sbin/kbdcontrol/kbdcontrol.c:97:28: error: subscripted <br> value is not an array, pointer, or vector<br> static const int ndelays = nitems(kbdelays);<br> ^~~~~~~~~~~~~~~~<br> /usr/include/sys/param.h:324:44: note: expanded from macro 'nitems'<br> #define nitems(x) (sizeof((x)) / sizeof((x)[0]))<br> ~~~^~<br> /usr/src/usr.sbin/kbdcontrol/kbdcontrol.c:98:36: error: use of <br> undeclared identifier 'kbrates'<br> static const int nrepeats = nitems(kbrates);<br> ^<br> /usr/src/usr.sbin/kbdcontrol/kbdcontrol.c:98:36: error: use of <br> undeclared identifier 'kbrates'<br> /usr/src/usr.sbin/kbdcontrol/kbdcontrol.c:1072:17: error: use of <br> undeclared identifier 'kbdelays'; did you mean 'ndelays'?<br> if (delay <= kbdelays[n])<br> ^~~~~~~~<br> ndelays<br> /usr/src/usr.sbin/kbdcontrol/kbdcontrol.c:97:18: note: 'ndelays' <br> declared here<br> static const int ndelays = nitems(kbdelays);<br> ^<br> /usr/src/usr.sbin/kbdcontrol/kbdcontrol.c:1072:25: error: subscripted <br> value is not an array, pointer, or vector<br> if (delay <= kbdelays[n])<br> ~~~~~~~~^~<br> /usr/src/usr.sbin/kbdcontrol/kbdcontrol.c:1076:18: error: use of <br> undeclared identifier 'kbrates'<br> if (repeat <= kbrates[n])<br> ^<br> 8 errors generated.<br> *** Error code 1<br></blockquote><div><br></div><div>I haven't seen this. How do I recreate this? Does this happen on a clean build?<br></div><div><br></div><div>Warner <br></div></div></div>help
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CANCZdfoDXaO21ZfW6xvW%2BadyBWvHjs2FL_XGdG7uD23igwXKuA>