2005-01-09 01:59:12

by Alan

[permalink] [raw]
Subject: ERROR: [PATCH] moxa: Update status of Moxa Smartio driver

This changeset is incorrect. The "1.8" version is for the Moxa mxser
driver. Moxa haven't released any 2.6 driver for the ancient "smartio"
hardware. (Checked this while fixing mxser).

In the mxser case that change and update has now been done.

Please revert this changeset.

Alan (with tty layer hat on)


> ChangeSet 1.2371, 2005/01/08 14:09:24-08:00, [email protected]

> +***NOTE*** - The driver included in the kernel is not maintained by Moxa. They
> +have a version 1.8 driver available from:
> +
> +http://www.moxa.com
> +
> +that works with 2.6 kernels. Currently, Moxa has no plans to have their updated
> +driver merged into the kernel.
> +
> +James Nelson <[email protected]> - 12-12-2004
> +



2005-01-09 06:56:38

by James Nelson

[permalink] [raw]
Subject: Re: ERROR: [PATCH] moxa: Update status of Moxa Smartio driver

Alan Cox wrote:
> This changeset is incorrect. The "1.8" version is for the Moxa mxser
> driver. Moxa haven't released any 2.6 driver for the ancient "smartio"
> hardware. (Checked this while fixing mxser).
>
> In the mxser case that change and update has now been done.
>
> Please revert this changeset.
>
> Alan (with tty layer hat on)
>
>

Actually, per

http://web4.moxa.com/support/download.asp#mxser.tgz

the mxser driver supports the smartio boards - and from the 1.8 mxser package
readme.txt:

The Smartio/Industio/UPCI family Linux driver supports following multiport
boards.


- 2 ports multiport board
CP-102U, CP-102UL
CP-132U-I, CP-132UL,
CP-132, CP-132I, CP132S, CP-132IS,
CI-132, CI-132I, CI-132IS,
(C102H, C102HI, C102HIS, C102P, CP-102, CP-102S)


- 4 ports multiport board
CP-104UL, CP-104JU,
CP-134U, CP-134U-I
C104H/PCI, C104HS/PCI,
CP-114, CP-114I, CP-114S, CP-114IS,
C104H, C104HS,
CI-104J, CI-104JS
CI-134, CI-134I, CI-134IS,
(C114HI, CT-114I, C104P)


- 8 ports multiport board
CP-118U
CP-168U,
C168H/PCI,
C168H, C168HS,
(C168P)

and Documentation/moxa-smartio says:

The Smartio family Linux driver, Ver. 1.1, supports following multiport
boards.

-C104P/H/HS, C104H/PCI, C104HS/PCI, CI-104J 4 port multiport board.
-C168P/H/HS, C168H/PCI 8 port multiport board.

Granted, the mxser driver supports many more boards - but it also supports the
boards mentioned in the smartio driver.

This just means that the smartio driver needs to be marked "obsolete" if mxser is
merged. Not that I'm complaining - it's a bit of a mess.

Jim

>
>>ChangeSet 1.2371, 2005/01/08 14:09:24-08:00, [email protected]
>
>
>>+***NOTE*** - The driver included in the kernel is not maintained by Moxa. They
>>+have a version 1.8 driver available from:
>>+
>>+http://www.moxa.com
>>+
>>+that works with 2.6 kernels. Currently, Moxa has no plans to have their updated
>>+driver merged into the kernel.
>>+
>>+James Nelson <[email protected]> - 12-12-2004
>>+
>
>
>
> -
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to [email protected]
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
>

2005-01-10 00:10:26

by Alan

[permalink] [raw]
Subject: Re: ERROR: [PATCH] moxa: Update status of Moxa Smartio driver

On Sul, 2005-01-09 at 06:56, Jim Nelson wrote:
> the mxser driver supports the smartio boards - and from the 1.8 mxser package
> readme.txt:

Read the code 8)

> (C102H, C102HI, C102HIS, C102P, CP-102, CP-102S)
> (C114HI, CT-114I, C104P)
> (C168P)

Note the brackets - they are not supported. I'm not sure what Moxa were
trying to indicate but I have actually pulled all their latest drivers
(and merged mxser). Their last moxa driver is for old old kernels.

Alan

2005-01-10 21:31:57

by James Nelson

[permalink] [raw]
Subject: Re: ERROR: [PATCH] moxa: Update status of Moxa Smartio driver

Alan Cox wrote:
> On Sul, 2005-01-09 at 06:56, Jim Nelson wrote:
>
>>the mxser driver supports the smartio boards - and from the 1.8 mxser package
>>readme.txt:
>
>
> Read the code 8)
>
>

</foot-in-mouth> I was going off of what their website said was the proper driver.
Oops.

Please revert the patch, Linus.

Jim