Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 25 Oct 2021 20:23:28 GMT
From:      "Bjoern A. Zeeb" <bz@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org
Subject:   git: 548ada00e54a - main - LinuxKPI: add bcd.h
Message-ID:  <202110252023.19PKNSRO031788@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help
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()
    which cannot be used leaving us with a conflict (see comment in file).
    Fortunately this is only seen in one driver so far and it seems easier
    to drop this in and change a single line in the driver than to add this
    inline in the driver.
    
    MFC after:      3 days
    Reviewed by:    hselasky
    Differential Revision: https://reviews.freebsd.org/D32647
---
 sys/compat/linuxkpi/common/include/linux/bcd.h | 53 ++++++++++++++++++++++++++
 1 file changed, 53 insertions(+)

diff --git a/sys/compat/linuxkpi/common/include/linux/bcd.h b/sys/compat/linuxkpi/common/include/linux/bcd.h
new file mode 100644
index 000000000000..57fcb4c6bd98
--- /dev/null
+++ b/sys/compat/linuxkpi/common/include/linux/bcd.h
@@ -0,0 +1,53 @@
+/*-
+ * SPDX-License-Identifier: BSD-2-Clause
+ *
+ * Copyright (c) 2021 Bjoern A. Zeeb
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ * 1. Redistributions of source code must retain the above copyright
+ *    notice, this list of conditions and the following disclaimer.
+ * 2. Redistributions in binary form must reproduce the above copyright
+ *    notice, this list of conditions and the following disclaimer in the
+ *    documentation and/or other materials provided with the distribution.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE AUTHOR AND CONTRIBUTORS ``AS IS'' AND
+ * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
+ * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
+ * ARE DISCLAIMED.  IN NO EVENT SHALL THE AUTHOR OR CONTRIBUTORS BE LIABLE
+ * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
+ * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS
+ * OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
+ * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
+ * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY
+ * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
+ * SUCH DAMAGE.
+ *
+ * $FreeBSD$
+ */
+
+#ifndef	_LINUXKPI_LINUX_BCD_H
+#define	_LINUXKPI_LINUX_BCD_H
+
+#include <sys/types.h>
+
+/*
+ * We could use libkern, but we need the argument truncating.
+ *
+ * This leaves us with a duplicate symbol with conflicting types
+ * so we cannot simply re-define as libkern.h gets included in
+ * too many places directly or indirectly.  This means for now
+ * drivers will have to be adjusted to call linuxkpi_bcd2bin().
+ */
+static inline uint8_t linuxkpi_bcd2bin(uint8_t x)
+{
+
+	return (x & 0x0f) + (x >> 4) * 10;
+}
+
+#if 0
+#define	bcd2bin(_x)	linuxkpi_bcd2bin(_x)
+#endif
+
+#endif	/* _LINUXKPI_LINUX_BCD_H */



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202110252023.19PKNSRO031788>