CVS commit: src/sys/arch/mips/ingenic

[Michael Lorenz]

Module Name:    src
Committed By:   macallan
Date:           Tue Nov 17 16:53:21 UTC 2015

Modified Files:
        src/sys/arch/mips/ingenic: ingenic_rng.c

Log Message:
Long overdue suggestions from Taylor Campbell and a few syntax/style
tweaks from myself.

>From Michael McConville


To generate a diff of this commit:
cvs rdiff -u -r1.2 -r1.3 src/sys/arch/mips/ingenic/ingenic_rng.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.


Modified files:

Index: src/sys/arch/mips/ingenic/ingenic_rng.c
diff -u src/sys/arch/mips/ingenic/ingenic_rng.c:1.2 src/sys/arch/mips/ingenic/ingenic_rng.c:1.3
--- src/sys/arch/mips/ingenic/ingenic_rng.c:1.2	Sun Aug 30 05:09:16 2015
+++ src/sys/arch/mips/ingenic/ingenic_rng.c	Tue Nov 17 16:53:21 2015
@@ -1,4 +1,4 @@
-/*	$NetBSD: ingenic_rng.c,v 1.2 2015/08/30 05:09:16 macallan Exp $ */
+/*	$NetBSD: ingenic_rng.c,v 1.3 2015/11/17 16:53:21 macallan Exp $ */
 
 /*-
  * Copyright (c) 2015 Michael McConville
@@ -27,7 +27,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: ingenic_rng.c,v 1.2 2015/08/30 05:09:16 macallan Exp $");
+__KERNEL_RCSID(0, "$NetBSD: ingenic_rng.c,v 1.3 2015/11/17 16:53:21 macallan Exp $");
 
 /*
  * adapted from Jared McNeill's amlogic_rng.c
@@ -79,32 +79,28 @@ CFATTACH_DECL_NEW(ingenic_rng, sizeof(st
 static int
 ingenic_rng_match(device_t parent, cfdata_t cf, void *aux)
 {
-	struct apbus_attach_args *aa = aux;
+	const struct apbus_attach_args *aa = aux;
 
-	if (strcmp(aa->aa_name, "jzrng") == 0) {
-		return 1;
-	} else {
-		return 0;
-	}
+	return !(strcmp(aa->aa_name, "jzrng"));
 }
 
 static void
 ingenic_rng_attach(device_t parent, device_t self, void *aux)
 {
 	struct ingenic_rng_softc * const sc = device_private(self);
-	struct apbus_attach_args * const aa = aux;
+	const struct apbus_attach_args * const aa = aux;
+	bus_addr_t addr = aa->aa_addr;
 	int error;
 
 	sc->sc_dev = self;
 	sc->sc_bst = aa->aa_bst;
-	if (aa->aa_addr == 0) {
-		aa->aa_addr = JZ_RNG;
-	}
+	if (addr == 0)
+		addr = JZ_RNG;
 
-	error = bus_space_map(aa->aa_bst, aa->aa_addr, 4, 0, &sc->sc_bsh);
+	error = bus_space_map(aa->aa_bst, addr, 4, 0, &sc->sc_bsh);
 	if (error) {
 		aprint_error_dev(self,
-			"can't map registers for %s: %d\n", aa->aa_name, error);
+		    "can't map registers for %s: %d\n", aa->aa_name, error);
 		return;
 	}
 
@@ -135,10 +131,7 @@ ingenic_rng_get(struct ingenic_rng_softc
 
 	mutex_spin_enter(&sc->sc_intr_lock);
 	while (sc->sc_bytes_wanted) {
-		bus_space_read_region_4(sc->sc_bst, sc->sc_bsh, 0, &data, 1);
-#if 0
-		device_printf(sc->sc_dev, "random output: %x\n", data);
-#endif
+		data = bus_space_read_4(sc->sc_bst, sc->sc_bsh, 0);
 		mutex_spin_exit(&sc->sc_intr_lock);
 		mutex_spin_enter(&sc->sc_rnd_lock);
 		rnd_add_data(&sc->sc_rndsource, &data, sizeof(data),
@@ -157,14 +150,14 @@ ingenic_rng_get_cb(size_t bytes_wanted, 
 	struct ingenic_rng_softc * const sc = priv;
 
 	mutex_spin_enter(&sc->sc_intr_lock);
-	if (sc->sc_bytes_wanted == 0) {
+	if (sc->sc_bytes_wanted == 0)
 		softint_schedule(sc->sc_sih);
-	}
-	if (bytes_wanted > (UINT_MAX - sc->sc_bytes_wanted)) {
+
+	if (bytes_wanted > (UINT_MAX - sc->sc_bytes_wanted))
 		sc->sc_bytes_wanted = UINT_MAX;
-	} else {
+	else
 		sc->sc_bytes_wanted += bytes_wanted;
-	}
+
 	mutex_spin_exit(&sc->sc_intr_lock);
 }