Date: Mon, 25 Oct 2021 21:17:11 +0000 (UTC) From: "Bjoern A. Zeeb" <bz@FreeBSD.org> To: Ian Lepore <ian@freebsd.org> Cc: src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org Subject: Re: git: 548ada00e54a - main - LinuxKPI: add bcd.h Message-ID: <alpine.BSF.2.00.2110252106440.68830@ai.fobar.qr> In-Reply-To: <832a2403091df094d6f7ab471aef0694adebb939.camel@freebsd.org> References: <202110252023.19PKNSRO031788@gitrepo.freebsd.org> <832a2403091df094d6f7ab471aef0694adebb939.camel@freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
This message is in MIME format. The first part should be readable text, while the remaining parts are likely unreadable without MIME-aware tools. --0-1355922163-1635196632=:68830 Content-Type: TEXT/PLAIN; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8BIT On Mon, 25 Oct 2021, Ian Lepore wrote: > On Mon, 2021-10-25 at 20:23 +0000, Bjoern A. Zeeb wrote: >> The branch main has been updated by bz: >> >> URL: >> https://cgit.FreeBSD.org/src/commit/?id=548ada00e54a9e7745d041b1ec7f68f3bd493365 >> >> commit 548ada00e54a9e7745d041b1ec7f68f3bd493365 >> Author: Bjoern A. Zeeb <bz@FreeBSD.org> >> AuthorDate: 2021-10-25 18:14:08 +0000 >> Commit: Bjoern A. Zeeb <bz@FreeBSD.org> >> CommitDate: 2021-10-25 20:20:53 +0000 >> >> LinuxKPI: add bcd.h >> >> Add bcd2bin() as linuxkpi_bcd2bin(). Libkern does provide a >> bcd2bin() >> >> [...] >> + * We could use libkern, but we need the argument truncating. >> + * > > > What does that mean, "we need the argument truncating"? This one takes an uint8_t as argument. If my memory serves me correctly the Linux driver code through macros passes in larger values 0xabcdxx which are truncated by the argument type. We take an int and then have a KASSERT() which doesn't work so well for these larger values. /bz -- Bjoern A. Zeeb r15:7 --0-1355922163-1635196632=:68830--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?alpine.BSF.2.00.2110252106440.68830>