Update qcpon(4) compatibles to match on qcom,pmk8350-pon, which is the updated
authorpatrick <patrick@openbsd.org>
Mon, 24 Apr 2023 14:34:13 +0000 (14:34 +0000)
committerpatrick <patrick@openbsd.org>
Mon, 24 Apr 2023 14:34:13 +0000 (14:34 +0000)
binding in Linux 6.3 device trees.  While there, remove retrieving the register
address, as the new binding has an additional register with doesn't work with
a single OF_getpropint(), and since we're not using it we don't need to keep it
around.

sys/dev/fdt/qcpon.c

index 293b454..8268d70 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: qcpon.c,v 1.3 2022/12/21 23:26:54 patrick Exp $       */
+/*     $OpenBSD: qcpon.c,v 1.4 2023/04/24 14:34:13 patrick Exp $       */
 /*
  * Copyright (c) 2022 Patrick Wildt <patrick@blueri.se>
  *
@@ -63,7 +63,8 @@ qcpon_match(struct device *parent, void *match, void *aux)
 {
        struct spmi_attach_args *saa = aux;
 
-       return OF_is_compatible(saa->sa_node, "qcom,pm8998-pon");
+       return (OF_is_compatible(saa->sa_node, "qcom,pm8998-pon") ||
+           OF_is_compatible(saa->sa_node, "qcom,pmk8350-pon"));
 }
 
 void
@@ -71,13 +72,7 @@ qcpon_attach(struct device *parent, struct device *self, void *aux)
 {
        struct spmi_attach_args *saa = aux;
        struct qcpon_softc *sc = (struct qcpon_softc *)self;
-       int node, reg;
-
-       reg = OF_getpropint(saa->sa_node, "reg", -1);
-       if (reg < 0) {
-               printf(": can't find registers\n");
-               return;
-       }
+       int node;
 
        sc->sc_node = saa->sa_node;
        sc->sc_tag = saa->sa_tag;