Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH v2] mdev.conf: Add ttyO devices
@ 2012-01-03  4:27 Frank Hunleth
  2012-02-01 22:19 ` Peter Korsgaard
  0 siblings, 1 reply; 4+ messages in thread
From: Frank Hunleth @ 2012-01-03  4:27 UTC (permalink / raw)
  To: buildroot

This is needed to support the serial ports on OMAP/AM335x devices
when using mdev.

Signed-off-by: Frank Hunleth <fhunleth@troodon-software.com>
---
 package/busybox/mdev.conf |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/package/busybox/mdev.conf b/package/busybox/mdev.conf
index 247c0ed..7fb71bc 100644
--- a/package/busybox/mdev.conf
+++ b/package/busybox/mdev.conf
@@ -20,6 +20,7 @@ tty		root:tty 666
 tty[0-9]*	root:tty 660
 vcsa*[0-9]*	root:tty 660
 ttyS[0-9]*	root:root 660
+ttyO[0-9]*	root:root 660
 
 # alsa sound devices
 pcm.*		root:audio 660 =snd/
-- 
1.7.0.4

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [Buildroot] [PATCH v2] mdev.conf: Add ttyO devices
  2012-01-03  4:27 [Buildroot] [PATCH v2] mdev.conf: Add ttyO devices Frank Hunleth
@ 2012-02-01 22:19 ` Peter Korsgaard
  2012-02-02  3:40   ` Frank Hunleth
  0 siblings, 1 reply; 4+ messages in thread
From: Peter Korsgaard @ 2012-02-01 22:19 UTC (permalink / raw)
  To: buildroot

>>>>> "Frank" == Frank Hunleth <fhunleth@troodon-software.com> writes:

 Frank> This is needed to support the serial ports on OMAP/AM335x devices
 Frank> when using mdev.

 Frank> Signed-off-by: Frank Hunleth <fhunleth@troodon-software.com>
 Frank> ---
 Frank>  package/busybox/mdev.conf |    1 +
 Frank>  1 files changed, 1 insertions(+), 0 deletions(-)

 Frank> diff --git a/package/busybox/mdev.conf b/package/busybox/mdev.conf
 Frank> index 247c0ed..7fb71bc 100644
 Frank> --- a/package/busybox/mdev.conf
 Frank> +++ b/package/busybox/mdev.conf
 Frank> @@ -20,6 +20,7 @@ tty		root:tty 666
 Frank>  tty[0-9]*	root:tty 660
 Frank>  vcsa*[0-9]*	root:tty 660
 Frank>  ttyS[0-9]*	root:root 660
 Frank> +ttyO[0-9]*	root:root 660

What does this exactly do? Isn't root:root / perm 660 default anyway?

-- 
Bye, Peter Korsgaard

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [Buildroot] [PATCH v2] mdev.conf: Add ttyO devices
  2012-02-01 22:19 ` Peter Korsgaard
@ 2012-02-02  3:40   ` Frank Hunleth
  2012-02-02  9:17     ` Peter Korsgaard
  0 siblings, 1 reply; 4+ messages in thread
From: Frank Hunleth @ 2012-02-02  3:40 UTC (permalink / raw)
  To: buildroot

On Wed, Feb 1, 2012 at 5:19 PM, Peter Korsgaard <jacmet@uclibc.org> wrote:
>>>>>> "Frank" == Frank Hunleth <fhunleth@troodon-software.com> writes:
>
> ?Frank> This is needed to support the serial ports on OMAP/AM335x devices
> ?Frank> when using mdev.
>
> ?Frank> Signed-off-by: Frank Hunleth <fhunleth@troodon-software.com>
> ?Frank> ---
> ?Frank> ?package/busybox/mdev.conf | ? ?1 +
> ?Frank> ?1 files changed, 1 insertions(+), 0 deletions(-)
>
> ?Frank> diff --git a/package/busybox/mdev.conf b/package/busybox/mdev.conf
> ?Frank> index 247c0ed..7fb71bc 100644
> ?Frank> --- a/package/busybox/mdev.conf
> ?Frank> +++ b/package/busybox/mdev.conf
> ?Frank> @@ -20,6 +20,7 @@ tty ? ? ? ? ? root:tty 666
> ?Frank> ?tty[0-9]* ? ? ?root:tty 660
> ?Frank> ?vcsa*[0-9]* ? ?root:tty 660
> ?Frank> ?ttyS[0-9]* ? ? root:root 660
> ?Frank> +ttyO[0-9]* ? ? root:root 660
>
> What does this exactly do? Isn't root:root / perm 660 default anyway?

I just removed the line from my configuration, and everything works
fine. At the moment, I do not understand how I had seen the console
not work without the line last month. However, it is clearly not
needed. Please disregard the patch. Sorry about the noise.

Frank

>
> --
> Bye, Peter Korsgaard

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [Buildroot] [PATCH v2] mdev.conf: Add ttyO devices
  2012-02-02  3:40   ` Frank Hunleth
@ 2012-02-02  9:17     ` Peter Korsgaard
  0 siblings, 0 replies; 4+ messages in thread
From: Peter Korsgaard @ 2012-02-02  9:17 UTC (permalink / raw)
  To: buildroot

>>>>> "Frank" == Frank Hunleth <fhunleth@troodon-software.com> writes:

 >> ?Frank> +ttyO[0-9]* ? ? root:root 660
 >> 
 >> What does this exactly do? Isn't root:root / perm 660 default anyway?

 Frank> I just removed the line from my configuration, and everything works
 Frank> fine. At the moment, I do not understand how I had seen the console
 Frank> not work without the line last month. However, it is clearly not
 Frank> needed. Please disregard the patch. Sorry about the noise.

Ok, good and no problem.

-- 
Bye, Peter Korsgaard

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2012-02-02  9:17 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-01-03  4:27 [Buildroot] [PATCH v2] mdev.conf: Add ttyO devices Frank Hunleth
2012-02-01 22:19 ` Peter Korsgaard
2012-02-02  3:40   ` Frank Hunleth
2012-02-02  9:17     ` Peter Korsgaard

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox