From owner-svn-src-all@freebsd.org Tue Jul 2 08:48:20 2019 Return-Path: Delivered-To: svn-src-all@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 8E87715CCD36; Tue, 2 Jul 2019 08:48:20 +0000 (UTC) (envelope-from ganbold@gmail.com) Received: from mail-qt1-x830.google.com (mail-qt1-x830.google.com [IPv6:2607:f8b0:4864:20::830]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) server-signature RSA-PSS (4096 bits) client-signature RSA-PSS (2048 bits) client-digest SHA256) (Client CN "smtp.gmail.com", Issuer "GTS CA 1O1" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 2E28874BE7; Tue, 2 Jul 2019 08:48:20 +0000 (UTC) (envelope-from ganbold@gmail.com) Received: by mail-qt1-x830.google.com with SMTP id d23so17596779qto.2; Tue, 02 Jul 2019 01:48:20 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=03j4ISGRIC2PNl3m68h/RxrzUihtJGBIKtZf8H+NsLQ=; b=AUAa6CpcCbz2YYpC6KCYX9IltV3Avqu6vlFHng3M/JI/W7LNpBqwNz9cizy1zaLlpG NgNVpvdCCM8f2KXevpuWlJ3gW/qgEq6FPNpJ7T9l2JEpnA1+EBps00TABhYSzsOMDxns lkyGikfmTJrlxG3wRB72jFQAjwPacEmGBjuROCs5lB/Z5+dRX7XdHPgYzz7UoXK0O0yp XO5300oD++v+/ysx7lgcJuPba6Q6MUS0YRzrSQujhQcMfRuqu0pmqbI+GZed1CCUs5A4 E0kkUsNOG4UkdXFON7n0AOzhszW8XsEO62jXCDCLcvwUkozGF2DcbwUH85cHJY5ygRaM MwUA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=03j4ISGRIC2PNl3m68h/RxrzUihtJGBIKtZf8H+NsLQ=; b=GmedZCUaYIFkW6EqvPXE0FwKhe3zON3IcDHbKqxgwqZENYQs+6JBeiP3Zzj2ocFw6R S0TRvcbdQas1CZMxKo2KdQYyz61SXo30eMKX1rJRIvPKLnTNYl1Y7+XgL1shbXu3zWmX HIVZd9tzrnsZOMFEzj4IqZ0IMNU0fnJAyNNfBbVgJhN5EURfsdm1+584jAtLb/ZQVRYU SDXgmbzs5BJUmEZrPQn1H0Da8KfOgUwFwz+ZELfLQ+N9LfIKkQyDuJ4cI7jbJhdX9ZiK jkDEnEzXLh8FP9q7FbwtchJXS6nzeKtcW3zG4AhTxU9HiOveaBfLzaAYe8kmC9xkACIq GJpQ== X-Gm-Message-State: APjAAAWlXHHaJiU+aKKt1r+0CUtBVPPMvmCI0Yf4E0jfN+sK9RPynEX8 153gjb1uVJ8v74LtshvN3wQD/B2A8binnMoNp2DmN7IpghY= X-Google-Smtp-Source: APXvYqz8NFSo/WKlnF+PemIKrh0UIlWonjAaSVvt+w9SiOI1kS5Aqrs18YfTRrfas9oywQJZF7CiqlpKLZhX8mpeApQ= X-Received: by 2002:ac8:c0e:: with SMTP id k14mr23874756qti.72.1562057299543; Tue, 02 Jul 2019 01:48:19 -0700 (PDT) MIME-Version: 1.0 References: <201907020847.x628lIAW066141@repo.freebsd.org> In-Reply-To: <201907020847.x628lIAW066141@repo.freebsd.org> From: Ganbold Tsagaankhuu Date: Tue, 2 Jul 2019 16:48:08 +0800 Message-ID: Subject: Re: svn commit: r349596 - head/sys/dev/fdt To: Ganbold Tsagaankhuu Cc: src-committers , svn-src-all , svn-src-head X-Rspamd-Queue-Id: 2E28874BE7 X-Spamd-Bar: ------ Authentication-Results: mx1.freebsd.org X-Spamd-Result: default: False [-6.97 / 15.00]; NEURAL_HAM_MEDIUM(-1.00)[-0.999,0]; NEURAL_HAM_SHORT(-0.97)[-0.967,0]; REPLY(-4.00)[]; NEURAL_HAM_LONG(-1.00)[-1.000,0] Content-Type: text/plain; charset="UTF-8" X-Content-Filtered-By: Mailman/MimeDel 2.1.29 X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 02 Jul 2019 08:48:20 -0000 On Tue, Jul 2, 2019 at 4:47 PM Ganbold Tsagaankhuu wrote: > Author: ganbold > Date: Tue Jul 2 08:47:18 2019 > New Revision: 349596 > URL: https://svnweb.freebsd.org/changeset/base/349596 > > Log: > Extend simple_mfd driver to expose a syscon interface if > that node is also compatible with syscon. For instance, > Rockchip RK3399's GRF (General Register Files) is compatible > with simple-mfd as well as syscon and has devices like > usb2-phy, emmc-phy and pcie-phy etc. under it. > Forgot to include: https://reviews.freebsd.org/D20647 thanks, Ganbold > > Reviewed by: manu > > Added: > head/sys/dev/fdt/simple_mfd.h (contents, props changed) > Modified: > head/sys/dev/fdt/simple_mfd.c > > Modified: head/sys/dev/fdt/simple_mfd.c > > ============================================================================== > --- head/sys/dev/fdt/simple_mfd.c Tue Jul 2 07:47:11 2019 > (r349595) > +++ head/sys/dev/fdt/simple_mfd.c Tue Jul 2 08:47:18 2019 > (r349596) > @@ -33,7 +33,10 @@ __FBSDID("$FreeBSD$"); > #include > #include > #include > +#include > #include > +#include > +#include > > #include > > @@ -42,15 +45,88 @@ __FBSDID("$FreeBSD$"); > #include > #include > > -struct simple_mfd_softc { > - struct simplebus_softc sc; > -}; > +#include > > device_t simple_mfd_add_device(device_t dev, phandle_t node, u_int order, > const char *name, int unit, struct simplebus_devinfo *di); > struct simplebus_devinfo *simple_mfd_setup_dinfo(device_t dev, phandle_t > node, struct simplebus_devinfo *di); > > +#include "syscon_if.h" > +#include > + > +MALLOC_DECLARE(M_SYSCON); > + > +static uint32_t simple_mfd_syscon_read_4(struct syscon *syscon, > bus_size_t offset); > +static int simple_mfd_syscon_write_4(struct syscon *syscon, bus_size_t > offset, > + uint32_t val); > +static int simple_mfd_syscon_modify_4(struct syscon *syscon, bus_size_t > offset, > + uint32_t clear_bits, uint32_t set_bits); > + > +#define SYSCON_LOCK(_sc) mtx_lock_spin(&(_sc)->mtx) > +#define SYSCON_UNLOCK(_sc) > mtx_unlock_spin(&(_sc)->mtx) > +#define SYSCON_LOCK_INIT(_sc) mtx_init(&(_sc)->mtx, \ > + device_get_nameunit((_sc)->dev), "syscon", MTX_SPIN) > +#define SYSCON_LOCK_DESTROY(_sc) mtx_destroy(&(_sc)->mtx); > +#define SYSCON_ASSERT_LOCKED(_sc) mtx_assert(&(_sc)->mtx, > MA_OWNED); > +#define SYSCON_ASSERT_UNLOCKED(_sc) mtx_assert(&(_sc)->mtx, > MA_NOTOWNED); > + > +static syscon_method_t simple_mfd_syscon_methods[] = { > + SYSCONMETHOD(syscon_read_4, simple_mfd_syscon_read_4), > + SYSCONMETHOD(syscon_write_4, simple_mfd_syscon_write_4), > + SYSCONMETHOD(syscon_modify_4, simple_mfd_syscon_modify_4), > + > + SYSCONMETHOD_END > +}; > +DEFINE_CLASS_1(simple_mfd_syscon, simple_mfd_syscon_class, > + simple_mfd_syscon_methods, 0, syscon_class); > + > +static uint32_t > +simple_mfd_syscon_read_4(struct syscon *syscon, bus_size_t offset) > +{ > + struct simple_mfd_softc *sc; > + uint32_t val; > + > + sc = device_get_softc(syscon->pdev); > + > + SYSCON_LOCK(sc); > + val = bus_read_4(sc->mem_res, offset); > + SYSCON_UNLOCK(sc); > + return (val); > +} > + > static int > +simple_mfd_syscon_write_4(struct syscon *syscon, bus_size_t offset, > + uint32_t val) > +{ > + struct simple_mfd_softc *sc; > + > + sc = device_get_softc(syscon->pdev); > + > + SYSCON_LOCK(sc); > + bus_write_4(sc->mem_res, offset, val); > + SYSCON_UNLOCK(sc); > + return (0); > +} > + > +static int > +simple_mfd_syscon_modify_4(struct syscon *syscon, bus_size_t offset, > + uint32_t clear_bits, uint32_t set_bits) > +{ > + struct simple_mfd_softc *sc; > + uint32_t val; > + > + sc = device_get_softc(syscon->pdev); > + > + SYSCON_LOCK(sc); > + val = bus_read_4(sc->mem_res, offset); > + val &= ~clear_bits; > + val |= set_bits; > + bus_write_4(sc->mem_res, offset, val); > + SYSCON_UNLOCK(sc); > + return (0); > +} > + > +static int > simple_mfd_probe(device_t dev) > { > > @@ -70,10 +146,14 @@ simple_mfd_attach(device_t dev) > struct simple_mfd_softc *sc; > phandle_t node, child; > device_t cdev; > + int rid; > > sc = device_get_softc(dev); > node = ofw_bus_get_node(dev); > > + sc->dev = dev; > + rid = 0; > + > /* Parse address-cells and size-cells from the parent node as a > fallback */ > if (OF_getencprop(node, "#address-cells", &sc->sc.acells, > sizeof(sc->sc.acells)) == -1) { > @@ -105,9 +185,48 @@ simple_mfd_attach(device_t dev) > device_probe_and_attach(cdev); > } > > + if (ofw_bus_is_compatible(dev, "syscon")) { > + sc->mem_res = bus_alloc_resource_any(dev, SYS_RES_MEMORY, > &rid, > + RF_ACTIVE); > + if (sc->mem_res == NULL) { > + device_printf(dev, > + "Cannot allocate memory resource\n"); > + return (ENXIO); > + } > + > + SYSCON_LOCK_INIT(sc); > + sc->syscon = syscon_create_ofw_node(dev, > + &simple_mfd_syscon_class, ofw_bus_get_node(dev)); > + if (sc->syscon == NULL) { > + device_printf(dev, > + "Failed to create/register syscon\n"); > + return (ENXIO); > + } > + } > return (bus_generic_attach(dev)); > } > > +static int > +simple_mfd_detach(device_t dev) > +{ > + struct simple_mfd_softc *sc; > + > + sc = device_get_softc(dev); > + if (ofw_bus_is_compatible(dev, "syscon")) { > + if (sc->syscon != NULL) { > + syscon_unregister(sc->syscon); > + free(sc->syscon, M_SYSCON); > + } > + > + SYSCON_LOCK_DESTROY(sc); > + > + if (sc->mem_res != NULL) > + bus_release_resource(dev, SYS_RES_MEMORY, 0, > + sc->mem_res); > + } > + return (0); > +} > + > struct simplebus_devinfo * > simple_mfd_setup_dinfo(device_t dev, phandle_t node, > struct simplebus_devinfo *di) > @@ -162,6 +281,7 @@ static device_method_t simple_mfd_methods[] = { > /* Device interface */ > DEVMETHOD(device_probe, simple_mfd_probe), > DEVMETHOD(device_attach, simple_mfd_attach), > + DEVMETHOD(device_detach, simple_mfd_detach), > > DEVMETHOD_END > }; > > Added: head/sys/dev/fdt/simple_mfd.h > > ============================================================================== > --- /dev/null 00:00:00 1970 (empty, because file is newly added) > +++ head/sys/dev/fdt/simple_mfd.h Tue Jul 2 08:47:18 2019 > (r349596) > @@ -0,0 +1,44 @@ > +/*- > + * SPDX-License-Identifier: BSD-2-Clause-FreeBSD > + * > + * Copyright (c) 2019 Ganbold Tsagaankhuu > + * > + * Redistribution and use in source and binary forms, with or without > + * modification, are permitted provided that the following conditions > + * are met: > + * 1. Redistributions of source code must retain the above copyright > + * notice, this list of conditions and the following disclaimer. > + * 2. Redistributions in binary form must reproduce the above copyright > + * notice, this list of conditions and the following disclaimer in the > + * documentation and/or other materials provided with the distribution. > + * > + * THIS SOFTWARE IS PROVIDED BY THE AUTHOR ``AS IS'' AND ANY EXPRESS OR > + * IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED > WARRANTIES > + * OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED. > + * IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR ANY DIRECT, INDIRECT, > + * INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, > BUT > + * NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF > USE, > + * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY > + * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT > + * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE > OF > + * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. > + * > + * $FreeBSD$ > + */ > + > +#ifndef DEV_SIMPLE_MFD_H > +#define DEV_SIMPLE_MFD_H > + > +#include > + > +struct simple_mfd_softc { > + struct simplebus_softc sc; > + device_t dev; > + struct syscon *syscon; > + struct resource *mem_res; > + struct mtx mtx; > +}; > + > +DECLARE_CLASS(simple_mfd_driver); > + > +#endif /* DEV_SIMPLE_MFD_H */ > >