On Sun, Jul 01, 2018 at 02:15:04PM +0200, Mark Kettenis wrote:
> Diff below makes it possible to attach ahci(4) at acpi(4) as required by
> arm64 machines like the MACCHIATOBin and Overdrive 1000.
> 
> ok?
> 

ok mlarkin

> 
> Index: dev/acpi/ahci_acpi.c
> ===================================================================
> RCS file: dev/acpi/ahci_acpi.c
> diff -N dev/acpi/ahci_acpi.c
> --- /dev/null 1 Jan 1970 00:00:00 -0000
> +++ dev/acpi/ahci_acpi.c      1 Jul 2018 12:13:03 -0000
> @@ -0,0 +1,145 @@
> +/*   $OpenBSD$       */
> +/*
> + * Copyright (c) 2018 Mark Kettenis
> + *
> + * Permission to use, copy, modify, and distribute this software for any
> + * purpose with or without fee is hereby granted, provided that the above
> + * copyright notice and this permission notice appear in all copies.
> + *
> + * THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL WARRANTIES
> + * WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF
> + * MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR
> + * ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES
> + * WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN
> + * ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF
> + * OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
> + */
> +
> +#include <sys/param.h>
> +#include <sys/malloc.h>
> +#include <sys/systm.h>
> +
> +#include <dev/acpi/acpireg.h>
> +#include <dev/acpi/acpivar.h>
> +#include <dev/acpi/acpidev.h>
> +#include <dev/acpi/amltypes.h>
> +#include <dev/acpi/dsdt.h>
> +
> +#undef DEVNAME
> +#include <dev/ic/ahcireg.h>
> +#include <dev/ic/ahcivar.h>
> +
> +struct ahci_acpi_softc {
> +     struct ahci_softc sc;
> +     struct acpi_softc *sc_acpi;
> +     struct aml_node *sc_node;
> +
> +     bus_addr_t sc_addr;
> +     bus_size_t sc_size;
> +
> +     int sc_irq;
> +     int sc_irq_flags;
> +     void *sc_ih;
> +};
> +
> +int  ahci_acpi_match(struct device *, void *, void *);
> +void ahci_acpi_attach(struct device *, struct device *, void *);
> +
> +struct cfattach ahci_acpi_ca = {
> +     sizeof(struct ahci_acpi_softc), ahci_acpi_match, ahci_acpi_attach
> +};
> +
> +const char *ahci_hids[] = {
> +     "AMDI0600",
> +     "LNRO001E",
> +     NULL
> +};
> +
> +int  ahci_acpi_parse_resources(int, union acpi_resource *, void *);
> +
> +int
> +ahci_acpi_match(struct device *parent, void *match, void *aux)
> +{
> +     struct acpi_attach_args *aaa = aux;
> +     struct cfdata *cf = match;
> +
> +     return acpi_matchhids(aaa, ahci_hids, cf->cf_driver->cd_name);
> +}
> +
> +void
> +ahci_acpi_attach(struct device *parent, struct device *self, void *aux)
> +{
> +     struct acpi_attach_args *aaa = aux;
> +     struct ahci_acpi_softc *sc = (struct ahci_acpi_softc *)self;
> +     struct aml_value res;
> +
> +     sc->sc_acpi = (struct acpi_softc *)parent;
> +     sc->sc_node = aaa->aaa_node;
> +     printf(" %s", sc->sc_node->name);
> +
> +     if (aml_evalname(sc->sc_acpi, sc->sc_node, "_CRS", 0, NULL, &res)) {
> +             printf(": can't find registers\n");
> +             return;
> +     }
> +
> +     aml_parse_resource(&res, ahci_acpi_parse_resources, sc);
> +     printf(" addr 0x%lx/0x%lx", sc->sc_addr, sc->sc_size);
> +     if (sc->sc_addr == 0 || sc->sc_size == 0) {
> +             printf("\n");
> +             return;
> +     }
> +
> +     printf(" irq %d", sc->sc_irq);
> +
> +     sc->sc.sc_iot = aaa->aaa_memt;
> +     sc->sc.sc_ios = sc->sc_size;
> +     sc->sc.sc_dmat = aaa->aaa_dmat;
> +
> +     if (bus_space_map(sc->sc.sc_iot, sc->sc_addr, sc->sc_size, 0,
> +         &sc->sc.sc_ioh)) {
> +             printf(": can't map registers\n");
> +             return;
> +     }
> +
> +     sc->sc_ih = acpi_intr_establish(sc->sc_irq, sc->sc_irq_flags, IPL_BIO,
> +         ahci_intr, sc, sc->sc.sc_dev.dv_xname);
> +     if (sc->sc_ih == NULL) {
> +             printf(": can't establish interrupt\n");
> +             return;
> +     }
> +
> +     printf(":");
> +
> +     if (ahci_attach(&sc->sc) != 0) {
> +             /* error printed by ahci_attach */
> +             goto irq;
> +     }
> +
> +     return;
> +
> +irq:
> +     return;
> +}
> +
> +int
> +ahci_acpi_parse_resources(int crsidx, union acpi_resource *crs, void *arg)
> +{
> +     struct ahci_acpi_softc *sc = arg;
> +     int type = AML_CRSTYPE(crs);
> +
> +     switch (type) {
> +     case LR_MEM32FIXED:
> +             /* AHCI registers are specified by the first resource. */
> +             if (sc->sc_size == 0) {
> +                     sc->sc_addr = crs->lr_m32fixed._bas;
> +                     sc->sc_size = crs->lr_m32fixed._len;
> +             }
> +             break;
> +     case LR_EXTIRQ:
> +             sc->sc_irq = crs->lr_extirq.irq[0];
> +             sc->sc_irq_flags = crs->lr_extirq.flags;
> +             break;
> +     }
> +
> +     return 0;
> +}
> Index: dev/acpi/files.acpi
> ===================================================================
> RCS file: /cvs/src/sys/dev/acpi/files.acpi,v
> retrieving revision 1.45
> diff -u -p -r1.45 files.acpi
> --- dev/acpi/files.acpi       1 Jul 2018 10:29:30 -0000       1.45
> +++ dev/acpi/files.acpi       1 Jul 2018 12:13:03 -0000
> @@ -137,6 +137,10 @@ device   ccpmic
>  attach       ccpmic at i2c
>  file dev/acpi/ccpmic.c               ccpmic
>  
> +# AHCI
> +attach       ahci at acpi with ahci_acpi
> +file dev/acpi/ahci_acpi.c            ahci_acpi
> +
>  # NS16550 compatible UART
>  attach       com at acpi with com_acpi
>  file dev/acpi/com_acpi.c             com_acpi
> 

  • ahci@acpi Mark Kettenis
    • Re: ahci@acpi Mike Larkin

Reply via email to