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); }