Module Name: src
Committed By: christos
Date: Wed Nov 12 03:14:00 UTC 2014
Modified Files:
src/sys/external/bsd/drm2/drm: drm_module.c drm_sysctl.c
src/sys/external/bsd/drm2/i915drm: i915_module.c
src/sys/external/bsd/drm2/include/drm: drm_sysctl.h
src/sys/external/bsd/drm2/nouveau: nouveau_module.c
src/sys/external/bsd/drm2/radeon: radeon_module.c
Log Message:
prettify and add to all the modules that have it.
To generate a diff of this commit:
cvs rdiff -u -r1.8 -r1.9 src/sys/external/bsd/drm2/drm/drm_module.c
cvs rdiff -u -r1.1 -r1.2 src/sys/external/bsd/drm2/drm/drm_sysctl.c
cvs rdiff -u -r1.4 -r1.5 src/sys/external/bsd/drm2/i915drm/i915_module.c
cvs rdiff -u -r1.1 -r1.2 src/sys/external/bsd/drm2/include/drm/drm_sysctl.h
cvs rdiff -u -r1.2 -r1.3 src/sys/external/bsd/drm2/nouveau/nouveau_module.c
cvs rdiff -u -r1.2 -r1.3 src/sys/external/bsd/drm2/radeon/radeon_module.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/external/bsd/drm2/drm/drm_module.c
diff -u src/sys/external/bsd/drm2/drm/drm_module.c:1.8 src/sys/external/bsd/drm2/drm/drm_module.c:1.9
--- src/sys/external/bsd/drm2/drm/drm_module.c:1.8 Sun Sep 14 16:08:21 2014
+++ src/sys/external/bsd/drm2/drm/drm_module.c Tue Nov 11 22:14:00 2014
@@ -1,4 +1,4 @@
-/* $NetBSD: drm_module.c,v 1.8 2014/09/14 20:08:21 riastradh Exp $ */
+/* $NetBSD: drm_module.c,v 1.9 2014/11/12 03:14:00 christos Exp $ */
/*-
* Copyright (c) 2013 The NetBSD Foundation, Inc.
@@ -30,7 +30,7 @@
*/
#include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: drm_module.c,v 1.8 2014/09/14 20:08:21 riastradh Exp $");
+__KERNEL_RCSID(0, "$NetBSD: drm_module.c,v 1.9 2014/11/12 03:14:00 christos Exp $");
#include <sys/types.h>
#include <sys/conf.h>
@@ -45,6 +45,7 @@ __KERNEL_RCSID(0, "$NetBSD: drm_module.c
#include <linux/mutex.h>
#include <drm/drmP.h>
+#include <drm/drm_sysctl.h>
/*
* XXX I2C stuff should be moved to a separate drmkms_i2c module.
@@ -53,6 +54,8 @@ MODULE(MODULE_CLASS_DRIVER, drmkms, "iic
struct mutex drm_global_mutex;
+struct drm_sysctl_def drm_def = DRM_SYSCTL_INIT();
+
static int
drm_init(void)
{
@@ -71,6 +74,7 @@ drm_init(void)
linux_mutex_init(&drm_global_mutex);
drm_connector_ida_init();
drm_global_init();
+ drm_sysctl_init(&drm_def);
return 0;
}
@@ -91,7 +95,7 @@ drm_guarantee_initialized(void)
static void
drm_fini(void)
{
-
+ drm_sysctl_fini(&drm_def);
drm_global_release();
drm_connector_ida_destroy();
linux_mutex_destroy(&drm_global_mutex);
Index: src/sys/external/bsd/drm2/drm/drm_sysctl.c
diff -u src/sys/external/bsd/drm2/drm/drm_sysctl.c:1.1 src/sys/external/bsd/drm2/drm/drm_sysctl.c:1.2
--- src/sys/external/bsd/drm2/drm/drm_sysctl.c:1.1 Tue Nov 11 21:24:40 2014
+++ src/sys/external/bsd/drm2/drm/drm_sysctl.c Tue Nov 11 22:14:00 2014
@@ -27,7 +27,7 @@
* POSSIBILITY OF SUCH DAMAGE.
*/
#include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: drm_sysctl.c,v 1.1 2014/11/12 02:24:40 christos Exp $");
+__KERNEL_RCSID(0, "$NetBSD: drm_sysctl.c,v 1.2 2014/11/12 03:14:00 christos Exp $");
#include <sys/param.h>
#include <sys/types.h>
@@ -41,9 +41,10 @@ __KERNEL_RCSID(0, "$NetBSD: drm_sysctl.c
#include <drm/drm_sysctl.h>
static const char *
-drm_sysctl_get_description(const struct linux_module_param_info *p, const void **v)
+drm_sysctl_get_description(const struct linux_module_param_info *p,
+ const struct drm_sysctl_def *def)
{
- const void * const *b = v[0], * const *e = v[1];
+ const void * const *b = def->bd, * const *e = def->ed;
for (; b < e; b++) {
const struct linux_module_param_desc *d = *b;
@@ -112,14 +113,14 @@ drm_sysctl_node(const char *name, const
void
-drm_sysctl_init(const void **v, struct sysctllog **log)
+drm_sysctl_init(struct drm_sysctl_def *def)
{
- const void * const *b = v[0], * const *e = v[1];
+ const void * const *b = def->bp, * const *e = def->ep;
const struct sysctlnode *rnode = NULL, *cnode;
const char *name = "drm2";
int error;
- if ((error = sysctl_createv(log, 0, NULL, &rnode,
+ if ((error = sysctl_createv(&def->log, 0, NULL, &rnode,
CTLFLAG_PERMANENT, CTLTYPE_NODE, name,
SYSCTL_DESCR("DRM driver parameters"),
NULL, 0, NULL, 0, CTL_HW, CTL_CREATE, CTL_EOL)) != 0) {
@@ -135,17 +136,18 @@ drm_sysctl_init(const void **v, struct s
cnode = rnode;
for (n = copy; (nn = strchr(n, '.')) != NULL; n = nn) {
*nn++ = '\0';
- if ((error = drm_sysctl_node(n, &cnode, log)) != 0) {
+ if ((error = drm_sysctl_node(n, &cnode, &def->log))
+ != 0) {
aprint_error("sysctl_createv returned %d, "
"for %s ignoring\n", error, n);
continue;
}
}
- if ((error = sysctl_createv(log, 0, &cnode,
+ if ((error = sysctl_createv(&def->log, 0, &cnode,
&cnode, p->mode == 0600 ? CTLFLAG_READWRITE : 0,
drm_sysctl_get_type(p), n,
- SYSCTL_DESCR(drm_sysctl_get_description(p, v + 2)),
+ SYSCTL_DESCR(drm_sysctl_get_description(p, def)),
NULL, 0, p->ptr, 0, CTL_CREATE, CTL_EOL)) != 0)
aprint_error("sysctl_createv returned %d, "
"for %s ignoring\n", error, n);
@@ -153,7 +155,7 @@ drm_sysctl_init(const void **v, struct s
}
void
-drm_sysctl_fini(struct sysctllog **log)
+drm_sysctl_fini(struct drm_sysctl_def *def)
{
- sysctl_teardown(log);
+ sysctl_teardown(&def->log);
}
Index: src/sys/external/bsd/drm2/i915drm/i915_module.c
diff -u src/sys/external/bsd/drm2/i915drm/i915_module.c:1.4 src/sys/external/bsd/drm2/i915drm/i915_module.c:1.5
--- src/sys/external/bsd/drm2/i915drm/i915_module.c:1.4 Tue Nov 11 21:24:40 2014
+++ src/sys/external/bsd/drm2/i915drm/i915_module.c Tue Nov 11 22:14:00 2014
@@ -1,4 +1,4 @@
-/* $NetBSD: i915_module.c,v 1.4 2014/11/12 02:24:40 christos Exp $ */
+/* $NetBSD: i915_module.c,v 1.5 2014/11/12 03:14:00 christos Exp $ */
/*-
* Copyright (c) 2013 The NetBSD Foundation, Inc.
@@ -30,7 +30,7 @@
*/
#include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: i915_module.c,v 1.4 2014/11/12 02:24:40 christos Exp $");
+__KERNEL_RCSID(0, "$NetBSD: i915_module.c,v 1.5 2014/11/12 03:14:00 christos Exp $");
#include <sys/types.h>
#include <sys/module.h>
@@ -55,9 +55,7 @@ extern struct drm_driver *const i915_drm
extern const struct pci_device_id *const i915_device_ids;
extern const size_t i915_n_device_ids;
-static struct sysctllog *i915_sysctllog;
-__link_set_decl(linux_module_param_info, struct linux_module_param_info);
-__link_set_decl(linux_module_param_desc, struct linux_module_param_desc);
+struct drm_sysctl_def i915_def = DRM_SYSCTL_INIT();
static int
i915drmkms_init(void)
@@ -79,13 +77,7 @@ i915drmkms_init(void)
error);
return error;
}
- const void *v[] = {
- __link_set_start(linux_module_param_info),
- __link_set_end(linux_module_param_info),
- __link_set_start(linux_module_param_desc),
- __link_set_end(linux_module_param_desc),
- };
- drm_sysctl_init(v, &i915_sysctllog);
+ drm_sysctl_init(&i915_def);
return 0;
}
@@ -108,7 +100,7 @@ i915drmkms_fini(void)
{
drm_pci_exit(i915_drm_driver, NULL);
- drm_sysctl_fini(&i915_sysctllog);
+ drm_sysctl_fini(&i915_def);
}
static int
Index: src/sys/external/bsd/drm2/include/drm/drm_sysctl.h
diff -u src/sys/external/bsd/drm2/include/drm/drm_sysctl.h:1.1 src/sys/external/bsd/drm2/include/drm/drm_sysctl.h:1.2
--- src/sys/external/bsd/drm2/include/drm/drm_sysctl.h:1.1 Tue Nov 11 21:24:40 2014
+++ src/sys/external/bsd/drm2/include/drm/drm_sysctl.h Tue Nov 11 22:14:00 2014
@@ -27,5 +27,22 @@
* POSSIBILITY OF SUCH DAMAGE.
*/
struct sysctllog;
-extern void drm_sysctl_init(const void **, struct sysctllog **);
-extern void drm_sysctl_fini(struct sysctllog **);
+
+struct drm_sysctl_def {
+ struct sysctllog *log;
+ const void *bp, *ep, *bd, *ed;
+};
+
+void drm_sysctl_init(struct drm_sysctl_def *);
+void drm_sysctl_fini(struct drm_sysctl_def *);
+
+#define DRM_SYSCTL_INIT() { \
+ NULL, \
+ __link_set_start(linux_module_param_info), \
+ __link_set_end(linux_module_param_info), \
+ __link_set_start(linux_module_param_desc), \
+ __link_set_end(linux_module_param_desc), \
+};
+
+__link_set_decl(linux_module_param_info, struct linux_module_param_info);
+__link_set_decl(linux_module_param_desc, struct linux_module_param_desc);
Index: src/sys/external/bsd/drm2/nouveau/nouveau_module.c
diff -u src/sys/external/bsd/drm2/nouveau/nouveau_module.c:1.2 src/sys/external/bsd/drm2/nouveau/nouveau_module.c:1.3
--- src/sys/external/bsd/drm2/nouveau/nouveau_module.c:1.2 Sat Aug 23 04:03:34 2014
+++ src/sys/external/bsd/drm2/nouveau/nouveau_module.c Tue Nov 11 22:14:00 2014
@@ -1,4 +1,4 @@
-/* $NetBSD: nouveau_module.c,v 1.2 2014/08/23 08:03:34 riastradh Exp $ */
+/* $NetBSD: nouveau_module.c,v 1.3 2014/11/12 03:14:00 christos Exp $ */
/*-
* Copyright (c) 2014 The NetBSD Foundation, Inc.
@@ -30,7 +30,7 @@
*/
#include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: nouveau_module.c,v 1.2 2014/08/23 08:03:34 riastradh Exp $");
+__KERNEL_RCSID(0, "$NetBSD: nouveau_module.c,v 1.3 2014/11/12 03:14:00 christos Exp $");
#include <sys/types.h>
#include <sys/module.h>
@@ -40,6 +40,7 @@ __KERNEL_RCSID(0, "$NetBSD: nouveau_modu
#include <sys/systm.h>
#include <drm/drmP.h>
+#include <drm/drm_sysctl.h>
#include <core/object.h>
#include <engine/device.h>
@@ -50,6 +51,8 @@ MODULE(MODULE_CLASS_DRIVER, nouveau, "dr
#include "ioconf.c"
#endif
+struct drm_sysctl_def nouveau_def = DRM_SYSCTL_INIT();
+
extern struct drm_driver *const nouveau_drm_driver; /* XXX */
static int
@@ -73,6 +76,7 @@ nouveau_init(void)
#if 0 /* XXX nouveau acpi */
nouveau_register_dsm_handler();
#endif
+ drm_sysctl_init(&nouveau_def);
return 0;
}
@@ -94,6 +98,7 @@ static void
nouveau_fini(void)
{
+ drm_sysctl_fini(&nouveau_def);
#if 0 /* XXX nouveau acpi */
nouveau_unregister_dsm_handler();
#endif
Index: src/sys/external/bsd/drm2/radeon/radeon_module.c
diff -u src/sys/external/bsd/drm2/radeon/radeon_module.c:1.2 src/sys/external/bsd/drm2/radeon/radeon_module.c:1.3
--- src/sys/external/bsd/drm2/radeon/radeon_module.c:1.2 Sat Jul 26 17:12:43 2014
+++ src/sys/external/bsd/drm2/radeon/radeon_module.c Tue Nov 11 22:14:00 2014
@@ -1,4 +1,4 @@
-/* $NetBSD: radeon_module.c,v 1.2 2014/07/26 21:12:43 riastradh Exp $ */
+/* $NetBSD: radeon_module.c,v 1.3 2014/11/12 03:14:00 christos Exp $ */
/*-
* Copyright (c) 2014 The NetBSD Foundation, Inc.
@@ -30,7 +30,7 @@
*/
#include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: radeon_module.c,v 1.2 2014/07/26 21:12:43 riastradh Exp $");
+__KERNEL_RCSID(0, "$NetBSD: radeon_module.c,v 1.3 2014/11/12 03:14:00 christos Exp $");
#include <sys/types.h>
#include <sys/module.h>
@@ -40,6 +40,7 @@ __KERNEL_RCSID(0, "$NetBSD: radeon_modul
#include <sys/systm.h>
#include <drm/drmP.h>
+#include <drm/drm_sysctl.h>
#include "radeon_drv.h"
@@ -53,6 +54,8 @@ MODULE(MODULE_CLASS_DRIVER, radeon, "drm
extern struct drm_driver *const radeon_drm_driver;
extern int radeon_max_kms_ioctl;
+struct drm_sysctl_def radeon_def = DRM_SYSCTL_INIT();
+
static int
radeon_init(void)
{
@@ -72,6 +75,7 @@ radeon_init(void)
error);
return error;
}
+ drm_sysctl_init(&radeon_def);
return 0;
}
@@ -92,7 +96,7 @@ radeon_guarantee_initialized(void)
static void
radeon_fini(void)
{
-
+ drm_sysctl_fini(&radeon_def);
drm_pci_exit(radeon_drm_driver, NULL);
}