Dominik Brodowski | 1 Aug 2010 14:59

[PATCH 01/11] pcmcia: use pcmica_{read,write}_config_byte

Use pcmcia_read_config_byte and pcmcia_write_config_byte instead
of pcmcia_access_configuration_register.

CC: netdev <at> vger.kernel.org
CC: linux-wireless <at> vger.kernel.org
CC: linux-serial <at> vger.kernel.org
CC: Michael Buesch <mb <at> bu3sch.de>
Signed-off-by: Dominik Brodowski <linux <at> dominikbrodowski.net>
---
 drivers/net/pcmcia/axnet_cs.c              |    3 +-
 drivers/net/pcmcia/nmclan_cs.c             |   21 ++-----
 drivers/net/pcmcia/xirc2ps_cs.c            |   16 ++---
 drivers/net/wireless/hostap/hostap_cs.c    |   91 ++++++++--------------------
 drivers/net/wireless/orinoco/spectrum_cs.c |   32 +++-------
 drivers/pcmcia/cistpl.c                    |    7 +-
 drivers/pcmcia/cs_internal.h               |    4 +-
 drivers/pcmcia/pcmcia_resource.c           |   72 +++++++++++++---------
 drivers/serial/serial_cs.c                 |    8 +--
 drivers/ssb/pcmcia.c                       |   14 +----
 include/pcmcia/cs.h                        |   12 ----
 include/pcmcia/ds.h                        |    4 +-
 12 files changed, 105 insertions(+), 179 deletions(-)

diff --git a/drivers/net/pcmcia/axnet_cs.c b/drivers/net/pcmcia/axnet_cs.c
index 467fd4b..ee0a6d0 100644
--- a/drivers/net/pcmcia/axnet_cs.c
+++ b/drivers/net/pcmcia/axnet_cs.c
 <at>  <at>  -378,8 +378,7  <at>  <at>  static int axnet_config(struct pcmcia_device *link)
     /* Maybe PHY is in power down mode. (PPD_SET = 1) 
        Bit 2 of CCSR is active low. */ 
(Continue reading)

Marcel Holtmann | 1 Aug 2010 20:04

Re: [PATCH 05/11] pcmcia: do not use io_req_t when calling pcmcia_request_io()

Hi Dominik,

> Instead of io_req_t, drivers are now requested to fill out
> struct pcmcia_device *p_dev->resource[0,1] for up to two ioport
> ranges. After a call to pcmcia_request_io(), the ports found there
> are reserved, after calling pcmcia_request_configuration(), they may
> be used.
> 
> CC: netdev <at> vger.kernel.org
> CC: linux-wireless <at> vger.kernel.org
> CC: linux-ide <at> vger.kernel.org
> CC: linux-usb <at> vger.kernel.org
> CC: laforge <at> gnumonks.org
> CC: linux-mtd <at> lists.infradead.org
> CC: linux-bluetooth <at> vger.kernel.org
> CC: alsa-devel <at> alsa-project.org
> CC: linux-serial <at> vger.kernel.org
> CC: Michael Buesch <mb <at> bu3sch.de>
> Signed-off-by: Dominik Brodowski <linux <at> dominikbrodowski.net>

for the drivers/bluetooth/ pieces.

Acked-by: Marcel Holtmann <marcel <at> holtmann.org>

Regards

Marcel

--
To unsubscribe from this list: send the line "unsubscribe linux-serial" in
(Continue reading)

Dominik Brodowski | 2 Aug 2010 21:52

Re: [PATCH 01/11] pcmcia: use pcmica_{read,write}_config_byte

Hey,

On Mon, Aug 02, 2010 at 08:59:26PM +0900, Komuro wrote:
> >--- a/drivers/net/pcmcia/xirc2ps_cs.c
> >+++ b/drivers/net/pcmcia/xirc2ps_cs.c
> 
> 
> >+	if (err)
> > 	    goto config_error;
> >-	reg.Action = CS_WRITE;
> >-	reg.Offset = CISREG_IOBASE_1;
> >-	reg.Value = (link->io.BasePort2 >> 8) & 0xff;
> >-	if ((err = pcmcia_access_configuration_register(link, &reg)))
> >+
> >+	err = pcmcia_write_config_byte(link, CISREG_IOBASE_1,
> >+				link->io.BasePort2 & 0xff);
> 
> It should be
> 
> 	err = pcmcia_write_config_byte(link, CISREG_IOBASE_1,
> 				(link->io.BasePort2 >> 8) & 0xff);
> 

Fixed, thanks.

Best,
	Dominik
--
To unsubscribe from this list: send the line "unsubscribe linux-serial" in
the body of a message to majordomo <at> vger.kernel.org
(Continue reading)

Pavel Machek | 4 Aug 2010 14:49
Favicon

Driver for kontron PMC-6L interface card

Hi!

Here's driver for PMC-6L interface card. It is against 2.6.27
(sorry). It seems to mostly work. It is quite an interesting card, for
example it timestamps incoming characters.

Problems remaining:

1) Due to ringbuffer design, I don't know how to handle input buffer
overrun.

2) stop_rx is currently a nop. I'm not sure if it is required? I guess
I could set RX buffer size to zero....?

I hope I did get the locking right.

Signed-off-by: Pavel Machek <pavel <at> ucw.cz>

diff -ur clean-27//drivers/serial/Kconfig linux-27//drivers/serial/Kconfig
--- clean-27//drivers/serial/Kconfig	2008-10-10 00:13:53.000000000 +0200
+++ linux-27//drivers/serial/Kconfig	2010-08-04 14:21:53.000000000 +0200
 <at>  <at>  -1343,6 +1343,14  <at>  <at> 
 	  If you have enabled the serial port on the Hilscher NetX SoC
 	  you can make it the console by answering Y to this option.

+config SERIAL_PMC6L
+        tristate "Serial ports on PMC-6L interface board"
+	select SERIAL_CORE
+	depends on PCI
+	help
(Continue reading)

Darius Augulis | 5 Aug 2010 22:10
Picon

[PATCH] serial: samsung: fix device name

Swap device and driver names in serial/samsung.c

Signed-off-by: Darius Augulis <augulis.darius <at> gmail.com>
---
 drivers/serial/samsung.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/serial/samsung.c b/drivers/serial/samsung.c
index a9d6c56..9d63f24 100644
--- a/drivers/serial/samsung.c
+++ b/drivers/serial/samsung.c
 <at>  <at>  -878,10 +878,10  <at>  <at>  static struct uart_ops s3c24xx_serial_ops = {

 static struct uart_driver s3c24xx_uart_drv = {
 	.owner		= THIS_MODULE,
-	.dev_name	= "s3c2410_serial",
+	.driver_name    = "s3c2410_serial",
 	.nr		= CONFIG_SERIAL_SAMSUNG_UARTS,
 	.cons		= S3C24XX_SERIAL_CONSOLE,
-	.driver_name	= S3C24XX_SERIAL_NAME,
+	.dev_name       = S3C24XX_SERIAL_NAME,
 	.major		= S3C24XX_SERIAL_MAJOR,
 	.minor		= S3C24XX_SERIAL_MINOR,
 };

--
To unsubscribe from this list: send the line "unsubscribe linux-serial" in
the body of a message to majordomo <at> vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

(Continue reading)

Dominik Brodowski | 6 Aug 2010 00:06

[PATCH 07/18] pcmcia: use pcmica_{read,write}_config_byte

Use pcmcia_read_config_byte and pcmcia_write_config_byte instead
of pcmcia_access_configuration_register.

CC: netdev <at> vger.kernel.org
CC: linux-wireless <at> vger.kernel.org
CC: linux-serial <at> vger.kernel.org
CC: Michael Buesch <mb <at> bu3sch.de>
Signed-off-by: Dominik Brodowski <linux <at> dominikbrodowski.net>
---
 drivers/net/pcmcia/axnet_cs.c              |    3 +-
 drivers/net/pcmcia/nmclan_cs.c             |   21 ++-----
 drivers/net/pcmcia/xirc2ps_cs.c            |   16 ++---
 drivers/net/wireless/hostap/hostap_cs.c    |   91 ++++++++--------------------
 drivers/net/wireless/orinoco/spectrum_cs.c |   32 +++-------
 drivers/pcmcia/cistpl.c                    |    7 +-
 drivers/pcmcia/cs_internal.h               |    4 +-
 drivers/pcmcia/pcmcia_resource.c           |   72 +++++++++++++---------
 drivers/serial/serial_cs.c                 |    8 +--
 drivers/ssb/pcmcia.c                       |   14 +----
 include/pcmcia/cs.h                        |   12 ----
 include/pcmcia/ds.h                        |    4 +-
 12 files changed, 105 insertions(+), 179 deletions(-)

diff --git a/drivers/net/pcmcia/axnet_cs.c b/drivers/net/pcmcia/axnet_cs.c
index 467fd4b..ee0a6d0 100644
--- a/drivers/net/pcmcia/axnet_cs.c
+++ b/drivers/net/pcmcia/axnet_cs.c
 <at>  <at>  -378,8 +378,7  <at>  <at>  static int axnet_config(struct pcmcia_device *link)
     /* Maybe PHY is in power down mode. (PPD_SET = 1) 
        Bit 2 of CCSR is active low. */ 
(Continue reading)

Kyungmin Park | 6 Aug 2010 02:04
Favicon

Re: [PATCH] serial: samsung: fix device name

Hi,

It's already posted by Mr. Shim
http://marc.info/?l=linux-arm-kernel&m=127410047306149&w=2

As discussed with Ben. even though it's right patch, there's some
users to use previous interface.
I also think it will be fixed. but need to coordinate with existing users.

Ben, how to you think?

On Fri, Aug 6, 2010 at 5:10 AM, Darius Augulis <augulis.darius <at> gmail.com> wrote:
> Swap device and driver names in serial/samsung.c
>
> Signed-off-by: Darius Augulis <augulis.darius <at> gmail.com>
> ---
>  drivers/serial/samsung.c |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/serial/samsung.c b/drivers/serial/samsung.c
> index a9d6c56..9d63f24 100644
> --- a/drivers/serial/samsung.c
> +++ b/drivers/serial/samsung.c
>  <at>  <at>  -878,10 +878,10  <at>  <at>  static struct uart_ops s3c24xx_serial_ops = {
>
>  static struct uart_driver s3c24xx_uart_drv = {
>        .owner          = THIS_MODULE,
> -       .dev_name       = "s3c2410_serial",
> +       .driver_name    = "s3c2410_serial",
>        .nr             = CONFIG_SERIAL_SAMSUNG_UARTS,
(Continue reading)

Kyungmin Park | 6 Aug 2010 08:26
Favicon

Re: [PATCH] serial: samsung: fix device name

On Fri, Aug 6, 2010 at 3:13 PM, Darius Augulis <augulis.darius <at> gmail.com> wrote:
> On 08/06/2010 03:04 AM, Kyungmin Park wrote:
>>
>> Hi,
>>
>> It's already posted by Mr. Shim
>> http://marc.info/?l=linux-arm-kernel&m=127410047306149&w=2
>>
>> As discussed with Ben. even though it's right patch, there's some
>> users to use previous interface.
>> I also think it will be fixed. but need to coordinate with existing users.
>>
>
> IMHO, it must be fixed. It's very annoying bug and I spend few hours trying
> to understand why kernel with devtmpfs auto-mount option doesn't provide
> ttySAC0, because when grep'ing 'grep -R ttySAC0 drivers/serial/*
> it's found in samsung driver and I thought everything should be o.k. Who
> could imagine that it's driver name, not device?...
> Regarding old users, they should not use such strange device name in their
> configs, they have to sync with main line.

Absolutely agree.

>
>> Ben, how to you think?
>>
>> On Fri, Aug 6, 2010 at 5:10 AM, Darius Augulis<augulis.darius <at> gmail.com>
>>  wrote:
>>>
>>> Swap device and driver names in serial/samsung.c
(Continue reading)

Joonyoung Shim | 6 Aug 2010 09:06

Re: [PATCH] serial: samsung: fix device name

On 8/6/2010 3:47 PM, Paulius Zaleckas wrote:
> On 08/06/2010 03:04 AM, Kyungmin Park wrote:
>> Hi,
>>
>> It's already posted by Mr. Shim
>> http://marc.info/?l=linux-arm-kernel&m=127410047306149&w=2
> 
> Darius patch is a little bit cleaner.
> 

OK, it's better.

Acked-by: Joonyoung Shim <jy0922.shim <at> samsung.com>

>> As discussed with Ben. even though it's right patch, there's some
>> users to use previous interface.
>> I also think it will be fixed. but need to coordinate with existing
>> users.
> 
> IMO this is obvious bug fix.
> 
> Acked-by: Paulius Zaleckas <paulius.zaleckas <at> gmail.com>
> 
>> Ben, how to you think?
>>
>> On Fri, Aug 6, 2010 at 5:10 AM, Darius
>> Augulis<augulis.darius <at> gmail.com>  wrote:
>>> Swap device and driver names in serial/samsung.c
>>>
>>> Signed-off-by: Darius Augulis<augulis.darius <at> gmail.com>
(Continue reading)

Kukjin Kim | 6 Aug 2010 09:23

RE: [PATCH] serial: samsung: fix device name

Joonyoung Shim wrote:
> 
> On 8/6/2010 3:47 PM, Paulius Zaleckas wrote:
> > On 08/06/2010 03:04 AM, Kyungmin Park wrote:
> >> Hi,
> >>
> >> It's already posted by Mr. Shim
> >> http://marc.info/?l=linux-arm-kernel&m=127410047306149&w=2
> >
> > Darius patch is a little bit cleaner.
> >
> 
> OK, it's better.
> 
> Acked-by: Joonyoung Shim <jy0922.shim <at> samsung.com>
> 

Hi all,

I think, this should be handled by Ben Dooks....
As Kyungmin Park said, there was his comment about this.

This is just for your information....

===

On Tue, May 18, 2010 at 12:52:13PM +0900, Joonyoung Shim wrote:
> On 5/18/2010 11:48 AM, Ben Dooks wrote:
> > On Mon, May 17, 2010 at 09:46:50PM +0900, Joonyoung Shim wrote:
> >> Thd dev_name and driver_name should be switched each other.
(Continue reading)


Gmane