Project

General

Profile

Bug #4622

asy.conf interrupt configuration broken

Added by Paul Henson about 6 years ago. Updated about 6 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
-
Category:
-
Start date:
2014-02-24
Due date:
% Done:

0%

Estimated time:
Difficulty:
Medium
Tags:
needs-triage

Description

Configuration explicitly defined in asy.conf should take precedence over autodiscovered configuration, whether from ACPI or defaults.

On a system with a serial port not defined in ACPI, the default irq from isa.c is used rather than the explicitly defined irq configured in asy.conf. With the following config in asy.conf:

name="asy" parent="isa" reg=1,0x3e8,8 interrupts=10;

prtconv shows the serial port as:

asy, instance #2
System properties:
name='interrupts' type=int items=1
value=0000000a
name='reg' type=int items=3
value=00000001.000003e8.00000008
name='instance' type=int items=1
value=00000002
Driver properties:
name='ddi-no-autodetach' type=int items=1 dev=none
value=00000001
Hardware properties:
name='interrupts' type=int items=1
value=00000004
name='reg' type=int items=3
value=00000001.000003e8.00000008
name='model' type=string items=1
value='Standard PC COM port'
name='compatible' type=string items=1
value='PNP0500'

The asy.conf interrupt shows up as a "System property", but the default irq listed as a hardware property is the one the OS actually tries to use.

History

#1

Updated by Paul Henson about 6 years ago

Given the changes in device enumeration since it last worked, it looks like it's no longer possible for values specified in asy.conf to override the defaults or ACPI values. Instead, we will add new eeprom configuration values, tty[a-d]-irq, matching existing tty eeprom config such as tty[a-d]-mode or tty[a-d]-ignore-cd, which will allow explicit configuration of interrupts for serial ports.

#2

Updated by Electric Monk about 6 years ago

git commit 709ee7cac5ff7cd991f4b21face672a23a7409a5

Author: Paul B. Henson <henson@acm.org>

4622 asy.conf interrupt configuration broken
Reviewed by: Garrett D'Amore <garrett@damore.org>
Approved by: Dan McDonald <danmcd@omniti.com>

#3

Updated by Dan McDonald about 6 years ago

  • Status changed from New to Resolved

See above.

Also available in: Atom PDF