Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1370734 > unrolled thread
| Started by | Sudip Mukherjee <sudipm.mukherjee@gmail.com> |
|---|---|
| First post | 2016-04-04 18:20 +0200 |
| Last post | 2016-04-06 00:30 +0200 |
| Articles | 2 — 2 participants |
Back to article view | Back to linux.kernel
This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by
below is the oldest one visible, not the original post.
Re: [PATCH][RESEND] parport: parport_pc: PCI SIO access should also depend on SIO option Sudip Mukherjee <sudipm.mukherjee@gmail.com> - 2016-04-04 18:20 +0200
Re: [PATCH][RESEND] parport: parport_pc: PCI SIO access should also depend on SIO option "Maciej S. Szmigiero" <mail@maciej.szmigiero.name> - 2016-04-06 00:30 +0200
| From | Sudip Mukherjee <sudipm.mukherjee@gmail.com> |
|---|---|
| Date | 2016-04-04 18:20 +0200 |
| Subject | Re: [PATCH][RESEND] parport: parport_pc: PCI SIO access should also depend on SIO option |
| Message-ID | <rkfom-5Jt-11@gated-at.bofh.it> |
On Tue, Mar 29, 2016 at 12:00:56AM +0200, Maciej S. Szmigiero wrote: > CONFIG_PARPORT_PC_SUPERIO toggles Super IO chip support in parport_pc > code, however only code accessing SIO chip via ISA (or LPC) bus was > conditional on it. > > This patch makes SIO chip accesses via PCI bus also dependent on this > config option. > > It should be noted that Super IO support in parport_pc is needed only when > firmware has failed to make parallel port available either via PNP or > on standard I/O ranges and user has one of a few supported SIOs. > > Signed-off-by: Maciej S. Szmigiero <mail@maciej.szmigiero.name> > --- > This is a resend without functional changes. Resend was not required. I had your original patch. Have you tested it on hardware? regards sudip
[toc] | [next] | [standalone]
| From | "Maciej S. Szmigiero" <mail@maciej.szmigiero.name> |
|---|---|
| Date | 2016-04-06 00:30 +0200 |
| Message-ID | <rkHDY-2qu-11@gated-at.bofh.it> |
| In reply to | #1370734 |
Hi Sudip, On 04.04.2016 18:11, Sudip Mukherjee wrote: > On Tue, Mar 29, 2016 at 12:00:56AM +0200, Maciej S. Szmigiero wrote: >> CONFIG_PARPORT_PC_SUPERIO toggles Super IO chip support in parport_pc >> code, however only code accessing SIO chip via ISA (or LPC) bus was >> conditional on it. >> >> This patch makes SIO chip accesses via PCI bus also dependent on this >> config option. >> >> It should be noted that Super IO support in parport_pc is needed only when >> firmware has failed to make parallel port available either via PNP or >> on standard I/O ranges and user has one of a few supported SIOs. >> >> Signed-off-by: Maciej S. Szmigiero <mail@maciej.szmigiero.name> >> --- >> This is a resend without functional changes. > > Resend was not required. I had your original patch. > > Have you tested it on hardware? I've tested it with CONFIG_PARPORT_PC_SUPERIO both set and unset, but on parallel port without one of such SIOs needing special setup. However, considering that this patch only moves PCI SIO access code behind this config option (just like ISA one is currently) and doesn't alter the actual setup functionality I don't expect problems here. > regards > sudip Best regards, Maciej
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web