mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Wouter de Waal <wrm@ccii.co.za>
To: linux-kernel@vger.kernel.org
Subject: FDDI on Linux kernel 2.6
Date: Tue, 24 Oct 2006 13:19:59 +0200	[thread overview]
Message-ID: <5.0.0.25.2.20061024131939.05e4de70@alpha.ccii.co.za> (raw)

2006-10-24

Hi

We here at CCII Systems are probably the only company in the world still
actively involved with FDDI. The Linux Syskonnect FDDI driver needs a
patch. I have no idea who to speak to, but the guys over at osdl said I
must ask here.

We recently had two separate customers ask us why the FDDI driver in
the Linux kernel version 2.6.x does not work properly.

The first customer eventually found that the driver does not work when
the kernel is configured to use 64 bit memory addressing. This would
obviously have to do with the pointer sizes used by the driver structure.

They did not need 64 bit addressing, and solved their problem by
reconfiguring their kernel for 32 bit addressing.

Is it possible to configure things so that this issue is highlighted
when the kernel is built?

The second customer's problem could not be solved by the above. They
eventually found that kernel 2.6 (specifically, 2.6.8) uses memory
mapping, and that a check in the driver caused an exception because
of the length of the PCI region being 2048 and not 16384 (0x4000).


>//2.6.8 version from skfddi.c
>
>#ifdef MEM_MAPPED_IO
>
>if (!(pci_resource_flags(pdev, 0) & IORESOURCE_MEM)) {
>   printk(KERN_ERR "skfp: region is not an MMIO resource\n");
>   err = -EIO;
>   goto err_out1;
>}
>
>port = pci_resource_start(pdev, 0);
>len = pci_resource_len(pdev, 0);
>if (len < 0x4000) {
>   printk(KERN_ERR "skfp: Invalid PCI region size: %lu\n", len);
>   err = -EIO;
>   goto err_out1;
>}
>
>#else

The customer reports that changing the compare to

>if (len < 2048) {

fixed the problem for them.

Can this patch be applied to the kernel please?

Regards

Wouter

-- 

Wouter de Waal                              Cell  : +27 82 893 8042
Development Manager - Board-Level Products  Phone : +27 21 683 5490
CCII Systems, Kenilworth, South Africa      Fax   : +27 21 683 5435


             reply	other threads:[~2006-10-24 11:20 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-10-24 11:19 Wouter de Waal [this message]
2006-10-24 14:02 ` Alan Cox
2006-10-24 15:20 ` Lee Revell
2006-10-24 15:30   ` Maciej W. Rozycki

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5.0.0.25.2.20061024131939.05e4de70@alpha.ccii.co.za \
    --to=wrm@ccii.co.za \
    --cc=linux-kernel@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®