@@ -143,6 +143,19 @@ static int sandbox_serial_getc(struct udevice *dev)
return result;
}
+static int sandbox_serial_setconfig(struct udevice *dev, uint serial_config)
+{
+ u8 parity = SERIAL_GET_PARITY(serial_config);
+ u8 bits = SERIAL_GET_BITS(serial_config);
+ u8 stop = SERIAL_GET_STOP(serial_config);
+
+ if (bits != SERIAL_8_BITS || stop != SERIAL_ONE_STOP ||
+ parity != SERIAL_PAR_NONE)
+ return -ENOTSUPP; /* not supported in driver*/
+
+ return 0;
+}
+
static const char * const ansi_colour[] = {
"black", "red", "green", "yellow", "blue", "megenta", "cyan",
"white",
@@ -173,6 +186,7 @@ static const struct dm_serial_ops sandbox_serial_ops = {
.putc = sandbox_serial_putc,
.pending = sandbox_serial_pending,
.getc = sandbox_serial_getc,
+ .setconfig = sandbox_serial_setconfig,
};
static const struct udevice_id sandbox_serial_ids[] = {
@@ -359,6 +359,7 @@ void serial_putc_raw(const char);
void serial_puts (const char *);
int serial_getc (void);
int serial_tstc (void);
+int serial_setconfig(uint config);
/* $(CPU)/speed.c */
int get_clocks (void);
@@ -102,6 +102,11 @@ enum serial_stop {
#define SERIAL_GET_STOP(config) \
((config & SERIAL_STOP_MASK) >> SERIAL_STOP_SHIFT)
+#define SERIAL_CONFIG(par, bits, stop) \
+ (par << SERIAL_PAR_SHIFT | \
+ bits << SERIAL_BITS_SHIFT | \
+ stop << SERIAL_STOP_SHIFT)
+
#define SERIAL_DEFAULT_CONFIG SERIAL_PAR_NONE << SERIAL_PAR_SHIFT | \
SERIAL_8_BITS << SERIAL_BITS_SHIFT | \
SERIAL_ONE_STOP << SERIAL_STOP_SHIFT
@@ -44,4 +44,5 @@ obj-$(CONFIG_DM_VIDEO) += video.o
obj-$(CONFIG_ADC) += adc.o
obj-$(CONFIG_SPMI) += spmi.o
obj-$(CONFIG_WDT) += wdt.o
+obj-$(CONFIG_DM_SERIAL) += serial.o
endif
new file mode 100644
@@ -0,0 +1,54 @@
+// SPDX-License-Identifier: GPL-2.0
+/*
+ * Copyright (c) 2018, STMicroelectronics
+ */
+
+#include <common.h>
+#include <serial.h>
+#include <dm.h>
+#include <dm/test.h>
+#include <test/ut.h>
+
+static int dm_test_serial(struct unit_test_state *uts)
+{
+ struct udevice *dev_serial;
+
+ ut_assertok(uclass_get_device_by_name(UCLASS_SERIAL, "serial",
+ &dev_serial));
+
+ ut_assertok(serial_tstc());
+ /*
+ * test with default config which is the only one supported by
+ * sandbox_serial driver
+ */
+ ut_assertok(serial_setconfig(SERIAL_DEFAULT_CONFIG));
+ /*
+ * test with a serial config which is not supported by
+ * sandbox_serial driver: test with wrong parity
+ */
+ ut_asserteq(-ENOTSUPP,
+ setconfig(SERIAL_CONFIG(SERIAL_PAR_ODD,
+ SERIAL_8_BITS,
+ SERIAL_ONE_STOP)));
+ /*
+ * test with a serial config which is not supported by
+ * sandbox_serial driver: test with wrong bits number
+ */
+ ut_asserteq(-ENOTSUPP,
+ serial_setconfig(SERIAL_CONFIG(SERIAL_PAR_NONE,
+ SERIAL_6_BITS,
+ SERIAL_ONE_STOP)));
+
+ /*
+ * test with a serial config which is not supported by
+ * sandbox_serial driver: test with wrong stop bits number
+ */
+ ut_asserteq(-ENOTSUPP,
+ serial_setconfig(SERIAL_CONFIG(SERIAL_PAR_NONE,
+ SERIAL_8_BITS,
+ SERIAL_TWO_STOP)));
+
+ return 0;
+}
+
+DM_TEST(dm_test_serial, DM_TESTF_SCAN_FDT);
Signed-off-by: Patrice Chotard <patrice.chotard@st.com> --- Changes in v3: - Update serial test when wrong serial options are specified - Add SERIAL_CONFIG(par, bits, stop) macro to create serial mask Changes in v2: - Add sandbox serial test drivers/serial/sandbox.c | 14 +++++++++++++ include/common.h | 1 + include/serial.h | 5 +++++ test/dm/Makefile | 1 + test/dm/serial.c | 54 ++++++++++++++++++++++++++++++++++++++++++++++++ 5 files changed, 75 insertions(+) create mode 100644 test/dm/serial.c