diff mbox

[RFC,11/13] gpio-switch is for some reason neccessary for camera to work.

Message ID 20170214134019.GA8631@amd (mailing list archive)
State New, archived
Headers show

Commit Message

Pavel Machek Feb. 14, 2017, 1:40 p.m. UTC
Probably something fun happening in userspace.
---
 arch/arm/mach-omap2/Makefile                 |  1 +
 arch/arm/mach-omap2/board-rx51-peripherals.c | 51 ++++++++++++++++++++++++++++
 2 files changed, 52 insertions(+)
 create mode 100644 arch/arm/mach-omap2/board-rx51-peripherals.c

Comments

Sakari Ailus July 17, 2017, 10:20 p.m. UTC | #1
Hi Pavel,

On Tue, Feb 14, 2017 at 02:40:19PM +0100, Pavel Machek wrote:
> Probably something fun happening in userspace.

What's the status of this one?

I don't think it has a chance to be merged in the foreseeable future. Why
is it needed?

> ---
>  arch/arm/mach-omap2/Makefile                 |  1 +
>  arch/arm/mach-omap2/board-rx51-peripherals.c | 51 ++++++++++++++++++++++++++++
>  2 files changed, 52 insertions(+)
>  create mode 100644 arch/arm/mach-omap2/board-rx51-peripherals.c
> 
> diff --git a/arch/arm/mach-omap2/Makefile b/arch/arm/mach-omap2/Makefile
> index 4698940..d536b1a 100644
> --- a/arch/arm/mach-omap2/Makefile
> +++ b/arch/arm/mach-omap2/Makefile
> @@ -229,6 +229,7 @@ obj-$(CONFIG_SOC_OMAP2420)		+= msdi.o
>  # Specific board support
>  obj-$(CONFIG_MACH_OMAP_GENERIC)		+= board-generic.o pdata-quirks.o
>  obj-$(CONFIG_MACH_NOKIA_N8X0)		+= board-n8x0.o
> +obj-y					+= board-rx51-peripherals.o
>  
>  # Platform specific device init code
>  
> diff --git a/arch/arm/mach-omap2/board-rx51-peripherals.c b/arch/arm/mach-omap2/board-rx51-peripherals.c
> new file mode 100644
> index 0000000..641c2be
> --- /dev/null
> +++ b/arch/arm/mach-omap2/board-rx51-peripherals.c
> @@ -0,0 +1,51 @@
> +/*
> + * linux/arch/arm/mach-omap2/board-rx51-peripherals.c
> + *
> + * Copyright (C) 2008-2009 Nokia
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License version 2 as
> + * published by the Free Software Foundation.
> + */
> +
> +#include <linux/kernel.h>
> +#include <linux/init.h>
> +#include <linux/platform_device.h>
> +#include <linux/gpio.h>
> +#include <linux/gpio_keys.h>
> +#include <linux/gpio/machine.h>
> +#include <linux/module.h>
> +#include <linux/platform_device.h>
> +#include <linux/timer.h>
> +
> +static struct platform_driver gpio_sw_driver = {
> +	.driver		= {
> +		.name	= "gpio-switch",
> +	},
> +};
> +
> +static int __init gpio_sw_init(void)
> +{
> +	int r;
> +
> +	printk(KERN_INFO "OMAP GPIO switch handler initializing\n");
> +
> +	r = platform_driver_register(&gpio_sw_driver);
> +	if (r)
> +		return r;
> +
> +	platform_device_register_simple("gpio-switch",
> +							       -1, NULL, 0);
> +	return 0;
> +}
> +
> +static void __exit gpio_sw_exit(void)
> +{
> +}
> +
> +#ifndef MODULE
> +late_initcall(gpio_sw_init);
> +#else
> +module_init(gpio_sw_init);
> +#endif
> +module_exit(gpio_sw_exit);
> -- 
> 2.1.4
Pavel Machek July 18, 2017, 9:27 a.m. UTC | #2
Hi!

> On Tue, Feb 14, 2017 at 02:40:19PM +0100, Pavel Machek wrote:
> > Probably something fun happening in userspace.
> 
> What's the status of this one?
> 
> I don't think it has a chance to be merged in the foreseeable future. Why
> is it needed?

Good question. And agreed that this one is not for merging. I suspect
something in my userspace depends on gpio-switch device being
present.

I'll try to debug what is going on there once more.

									Pavel
Pavel Machek July 18, 2017, 10:02 a.m. UTC | #3
On Tue 2017-07-18 11:27:53, Pavel Machek wrote:
> Hi!
> 
> > On Tue, Feb 14, 2017 at 02:40:19PM +0100, Pavel Machek wrote:
> > > Probably something fun happening in userspace.
> > 
> > What's the status of this one?
> > 
> > I don't think it has a chance to be merged in the foreseeable future. Why
> > is it needed?
> 
> Good question. And agreed that this one is not for merging. I suspect
> something in my userspace depends on gpio-switch device being
> present.
> 
> I'll try to debug what is going on there once more.

I tried with current userspace, and to my surprise, both sdlcam and
fcam-dev seem to work now. We can forget about this one.

Best regards,
									Pavel
diff mbox

Patch

diff --git a/arch/arm/mach-omap2/Makefile b/arch/arm/mach-omap2/Makefile
index 4698940..d536b1a 100644
--- a/arch/arm/mach-omap2/Makefile
+++ b/arch/arm/mach-omap2/Makefile
@@ -229,6 +229,7 @@  obj-$(CONFIG_SOC_OMAP2420)		+= msdi.o
 # Specific board support
 obj-$(CONFIG_MACH_OMAP_GENERIC)		+= board-generic.o pdata-quirks.o
 obj-$(CONFIG_MACH_NOKIA_N8X0)		+= board-n8x0.o
+obj-y					+= board-rx51-peripherals.o
 
 # Platform specific device init code
 
diff --git a/arch/arm/mach-omap2/board-rx51-peripherals.c b/arch/arm/mach-omap2/board-rx51-peripherals.c
new file mode 100644
index 0000000..641c2be
--- /dev/null
+++ b/arch/arm/mach-omap2/board-rx51-peripherals.c
@@ -0,0 +1,51 @@ 
+/*
+ * linux/arch/arm/mach-omap2/board-rx51-peripherals.c
+ *
+ * Copyright (C) 2008-2009 Nokia
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2 as
+ * published by the Free Software Foundation.
+ */
+
+#include <linux/kernel.h>
+#include <linux/init.h>
+#include <linux/platform_device.h>
+#include <linux/gpio.h>
+#include <linux/gpio_keys.h>
+#include <linux/gpio/machine.h>
+#include <linux/module.h>
+#include <linux/platform_device.h>
+#include <linux/timer.h>
+
+static struct platform_driver gpio_sw_driver = {
+	.driver		= {
+		.name	= "gpio-switch",
+	},
+};
+
+static int __init gpio_sw_init(void)
+{
+	int r;
+
+	printk(KERN_INFO "OMAP GPIO switch handler initializing\n");
+
+	r = platform_driver_register(&gpio_sw_driver);
+	if (r)
+		return r;
+
+	platform_device_register_simple("gpio-switch",
+							       -1, NULL, 0);
+	return 0;
+}
+
+static void __exit gpio_sw_exit(void)
+{
+}
+
+#ifndef MODULE
+late_initcall(gpio_sw_init);
+#else
+module_init(gpio_sw_init);
+#endif
+module_exit(gpio_sw_exit);