Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1602844

Re: [PATCH] pata_pcmcia: add EBSA110's PCMCIA slot support

From Tejun Heo <tj@kernel.org>
Newsgroups linux.kernel
Subject Re: [PATCH] pata_pcmcia: add EBSA110's PCMCIA slot support
Date 2017-03-16 22:30 +0100
Message-ID <tlL85-4dF-5@gated-at.bofh.it> (permalink)
References <tkYTL-3Be-3@gated-at.bofh.it> <tkYTL-3Be-1@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


Hello,

On Tue, Mar 14, 2017 at 06:50:43PM +0100, Bartlomiej Zolnierkiewicz wrote:
> +static struct ata_port_operations pcmcia_ebsa110_port_ops = {
> +	.inherits		= &ata_sff_port_ops,
> +	.sff_dev_select		= pmcmia_ebsa110_dev_select,
                                  ^^^^^^
> +	.sff_set_devctl         = pcmcia_ebsa110_set_devctl,
> +	.sff_check_status	= pcmcia_ebsa110_check_status,
> +	.sff_check_altstatus    = pcmcia_ebsa110_check_altstatus,
> +	.sff_tf_load		= pcmcia_ebsa110_tf_load,
> +	.sff_tf_read		= pcmcia_ebsa110_tf_read,
> +	.sff_exec_command	= pcmcia_ebsa110_exec_command,
> +	.sff_data_xfer		= ata_sff_data_xfer_noirq,
> +	.softreset		= pata_pcmcia_ebsa110_softreset,
> +	.cable_detect		= ata_cable_40wire,
> +	.set_mode		= pcmcia_set_mode,
> +};

Heh, that's a fat driver for a sff device.  I suppose this is mostly
copied from the matching ide driver but it'd be nice to explain why it
needs duplicate most standard functions.  Is it because PCMCIA
assigned address doesn't fall under the usual read/io boundary on the
arch?

Thanks.

-- 
tejun

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH] pata_pcmcia: add EBSA110's PCMCIA slot support Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com> - 2017-03-14 19:00 +0100
  Re: [PATCH] pata_pcmcia: add EBSA110's PCMCIA slot support Tejun Heo <tj@kernel.org> - 2017-03-16 22:30 +0100
    Re: [PATCH] pata_pcmcia: add EBSA110's PCMCIA slot support Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com> - 2017-03-17 13:10 +0100
      Re: [PATCH] pata_pcmcia: add EBSA110's PCMCIA slot support Tejun Heo <tj@kernel.org> - 2017-03-17 14:50 +0100
        Re: [PATCH] pata_pcmcia: add EBSA110's PCMCIA slot support Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com> - 2017-03-17 15:20 +0100
          Re: [PATCH] pata_pcmcia: add EBSA110's PCMCIA slot support Tejun Heo <tj@kernel.org> - 2017-03-17 15:50 +0100
            Re: [PATCH] pata_pcmcia: add EBSA110's PCMCIA slot support Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com> - 2017-03-17 17:10 +0100
              Re: [PATCH] pata_pcmcia: add EBSA110's PCMCIA slot support Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com> - 2017-03-20 14:20 +0100

csiph-web