Received: by 10.213.65.68 with SMTP id h4csp276546imn; Tue, 13 Mar 2018 04:09:58 -0700 (PDT) X-Google-Smtp-Source: AG47ELttTp+h+e9oMrmtsk59crEoj57lhL9dxjLuy3FExJRFuK7uDp/2giQmiibNXqLxUXAat+TR X-Received: by 2002:a17:902:a614:: with SMTP id u20-v6mr171687plq.249.1520939398132; Tue, 13 Mar 2018 04:09:58 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1520939398; cv=none; d=google.com; s=arc-20160816; b=WSIXS1rl9rQwlGmYZW7w2QlTp93BmzLL5SdDL5xCI8RdULcgpL7qcvJkIDkoGKUSvS 8tyY1mtQMq0WxbR9SE3N1odkh4qqeiLezmyMIw6vCuFS3JE+Z3uK+GAFa/gR0xoTP5Ck HMcuW9kTWV/jRoHjVUMYe3Btjh+JOXLUOraJ0oUw6w9SEwoMuD8/Fov/+Rp2XRsV5rLN 3O9FuHaUevFnEvLKsE1d51z1zMiqYoB/hDKNBgghSSB4/k3DgMOjSY5LOMoHEpLjz2TH H58XFhbbVz7hzWJpuqjgx6Rc38mFJwU/l/rqDkhN4HDi+p/eIKSzHUQtQLXqdtejm0J/ Q7GA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:mime-version:message-id:date:references :in-reply-to:subject:cc:to:from:arc-authentication-results; bh=961Vg1pzPzEc2IXIS5oSgk1gsAPD1MPiuSld/FtshwI=; b=TUitcD5J+/9zZq0i14xRVE7i7WVeB67ovdvGENSP+hnyQ3p26KnjsmUZ5ZxJfZs6je VEp5wTPQOTaIrk/iR0GzZDCLjC9pxGOV25PWcR/WGatOM0HD6fqbkW1oitTo7CN03JeT 0PiP9SszgorxR5sUeo4uru7EvXJHBMMQziDbFamnY4ZKvcKokMliQzYoiBWH+yO9z26Q 4gmqYa+5SFSr7h7awQrFLByW2xLkTtJNqIJOCq33fZnRQHUEnJrBLUmgUhCT2G3Fo99D nHZ0AMT0UGtozkrIyBxZUXcPqgsBdSLgt4O2zoSOz8jpOgrZ6pxmcITIxQqwX73JfXgM JYIQ== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id c66si67708pfc.416.2018.03.13.04.09.42; Tue, 13 Mar 2018 04:09:58 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932841AbeCMLIg (ORCPT + 99 others); Tue, 13 Mar 2018 07:08:36 -0400 Received: from mga07.intel.com ([134.134.136.100]:64240 "EHLO mga07.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752396AbeCMLId (ORCPT ); Tue, 13 Mar 2018 07:08:33 -0400 X-Amp-Result: UNSCANNABLE X-Amp-File-Uploaded: False Received: from orsmga004.jf.intel.com ([10.7.209.38]) by orsmga105.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 13 Mar 2018 04:08:30 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.47,464,1515484800"; d="asc'?scan'208";a="182240708" Received: from pipin.fi.intel.com (HELO localhost) ([10.237.68.37]) by orsmga004.jf.intel.com with ESMTP; 13 Mar 2018 04:08:28 -0700 From: Felipe Balbi To: Manu Gautam , Rob Herring , Andy Gross Cc: linux-arm-msm@vger.kernel.org, linux-usb@vger.kernel.org, Manu Gautam , Greg Kroah-Hartman , open list Subject: Re: [PATCH v1 2/2] usb: dwc3: Add Qualcomm DWC3 glue driver In-Reply-To: <1520937362-28777-2-git-send-email-mgautam@codeaurora.org> References: <1520937362-28777-1-git-send-email-mgautam@codeaurora.org> <1520937362-28777-2-git-send-email-mgautam@codeaurora.org> Date: Tue, 13 Mar 2018 13:08:14 +0200 Message-ID: <878taw8c81.fsf@linux.intel.com> MIME-Version: 1.0 Content-Type: multipart/signed; boundary="=-=-="; micalg=pgp-sha512; protocol="application/pgp-signature" Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --=-=-= Content-Type: text/plain Content-Transfer-Encoding: quoted-printable Hi, +Andy Manu Gautam writes: > DWC3 controller on Qualcomm SOCs has a Qscratch wrapper. > Some of its uses are described below resulting in need to > have a separate glue driver instead of using dwc3-of-simple: > - It exposes register interface to override vbus-override > and lane0-pwr-present signals going to hardware. These > must be updated in peripheral mode for DWC3 if vbus lines > are not connected to hardware block. Otherwise RX termination > in SS mode or DP pull-up is not applied by device controller. right, core needs to know that VBUS is above 4.4V. Why wasn't this a problem when the original glue layer was first published? > - pwr_events_irq_stat support to ensure USB2 PHY is in L2 state > before glue driver can turn-off clocks and suspend PHY. Core manages PHY suspend automatically. Isn't that working for you? Why? > - Support for wakeup interrupts lines that are asserted whenever > there is any wakeup event on USB3 or USB2 bus. ok > - Support to replace pip3 clock going to DWC3 with utmi clock > for hardware configuration where SSPHY is not used with DWC3. Is that SW configurable? Really? In any case seems like this and SESSVLD valid should be handled using Hans' and Heikki's mux support. > Other than above hardware features in Qscratch wrapper there > are some limitations on QCOM SOCs that require special handling > of power management e.g. suspending PHY using GUSB2PHYCFG > register and ensuring PHY enters L2 before turning off clocks etc. > > Signed-off-by: Manu Gautam > --- > drivers/usb/dwc3/Kconfig | 11 + > drivers/usb/dwc3/Makefile | 1 + > drivers/usb/dwc3/dwc3-of-simple.c | 1 - > drivers/usb/dwc3/dwc3-qcom.c | 635 ++++++++++++++++++++++++++++++++= ++++++ > 4 files changed, 647 insertions(+), 1 deletion(-) > create mode 100644 drivers/usb/dwc3/dwc3-qcom.c > > diff --git a/drivers/usb/dwc3/Kconfig b/drivers/usb/dwc3/Kconfig > index ab8c0e0..f5bb4f1 100644 > --- a/drivers/usb/dwc3/Kconfig > +++ b/drivers/usb/dwc3/Kconfig > @@ -106,4 +106,15 @@ config USB_DWC3_ST > inside (i.e. STiH407). > Say 'Y' or 'M' if you have one such device. >=20=20 > +config USB_DWC3_QCOM > + tristate "Qualcomm Platform" > + depends on ARCH_QCOM || COMPILE_TEST > + depends on OF > + default USB_DWC3 > + help > + Some Qualcomm SoCs use DesignWare Core IP for USB2/3 > + functionality. > + > + Say 'Y' or 'M' if you have one such device. > + > endif > diff --git a/drivers/usb/dwc3/Makefile b/drivers/usb/dwc3/Makefile > index 7ac7250..c3ce697 100644 > --- a/drivers/usb/dwc3/Makefile > +++ b/drivers/usb/dwc3/Makefile > @@ -48,3 +48,4 @@ obj-$(CONFIG_USB_DWC3_PCI) +=3D dwc3-pci.o > obj-$(CONFIG_USB_DWC3_KEYSTONE) +=3D dwc3-keystone.o > obj-$(CONFIG_USB_DWC3_OF_SIMPLE) +=3D dwc3-of-simple.o > obj-$(CONFIG_USB_DWC3_ST) +=3D dwc3-st.o > +obj-$(CONFIG_USB_DWC3_QCOM) +=3D dwc3-qcom.o > diff --git a/drivers/usb/dwc3/dwc3-of-simple.c b/drivers/usb/dwc3/dwc3-of= -simple.c > index cb2ee96..0fd0e8e 100644 > --- a/drivers/usb/dwc3/dwc3-of-simple.c > +++ b/drivers/usb/dwc3/dwc3-of-simple.c > @@ -208,7 +208,6 @@ static int dwc3_of_simple_runtime_resume(struct devic= e *dev) > }; >=20=20 > static const struct of_device_id of_dwc3_simple_match[] =3D { > - { .compatible =3D "qcom,dwc3" }, > { .compatible =3D "rockchip,rk3399-dwc3" }, > { .compatible =3D "xlnx,zynqmp-dwc3" }, > { .compatible =3D "cavium,octeon-7130-usb-uctl" }, > diff --git a/drivers/usb/dwc3/dwc3-qcom.c b/drivers/usb/dwc3/dwc3-qcom.c > new file mode 100644 > index 0000000..917199e > --- /dev/null > +++ b/drivers/usb/dwc3/dwc3-qcom.c > @@ -0,0 +1,635 @@ > +// SPDX-License-Identifier: GPL-2.0 > +/* Copyright (c) 2018, The Linux Foundation. All rights reserved. > + * > + * Inspired by dwc3-of-simple.c > + */ > + > +#include > +#include > +#include > +#include > +#include > +#include > +#include > +#include > +#include > +#include > +#include > +#include > +#include > +#include > + > +#include "core.h" > +#include "io.h" you must be kidding, right? There's no way I'm gonna let a glue poke at core registers. > +static void dwc3_qcom_suspend_hsphy(struct dwc3_qcom *qcom) > +{ > + struct dwc3 *dwc =3D platform_get_drvdata(qcom->dwc3); > + int ret; > + u32 val; > + > + /* Clear previous L2 events */ > + dwc3_qcom_setbits(qcom->qscratch_base, PWR_EVNT_IRQ_STAT_REG, > + PWR_EVNT_LPM_IN_L2_MASK | PWR_EVNT_LPM_OUT_L2_MASK); > + > + /* Allow controller to suspend HSPHY */ > + val =3D dwc3_readl(dwc->regs, DWC3_GUSB2PHYCFG(0)); > + val |=3D DWC3_GUSB2PHYCFG_ENBLSLPM | DWC3_GUSB2PHYCFG_SUSPHY; > + dwc3_writel(dwc->regs, DWC3_GUSB2PHYCFG(0), val); core should handle PHY bits. In any case, why don't you let core handle PHY suspend? It handles it automatically for us, there should be no need for SW intervention. > +static int dwc3_qcom_suspend(struct dwc3_qcom *qcom) > +{ > + struct dwc3 *dwc =3D platform_get_drvdata(qcom->dwc3); nope! Glue shouldn't touch dwc3 at all. > + int i; > + > + if (qcom->is_suspended) > + return 0; > + > + if (!dwc) > + return -EBUSY; =2DENODEV? > + dwc3_qcom_suspend_hsphy(qcom); > + > + if (dwc->usb2_generic_phy) > + phy_pm_runtime_put_sync(dwc->usb2_generic_phy); > + if (dwc->usb3_generic_phy) > + phy_pm_runtime_put_sync(dwc->usb3_generic_phy); core.c should do this. > + /* Disable HSPHY auto suspend */ > + val =3D dwc3_readl(dwc->regs, DWC3_GUSB2PHYCFG(0)); > + val &=3D ~(DWC3_GUSB2PHYCFG_ENBLSLPM | DWC3_GUSB2PHYCFG_SUSPHY); > + dwc3_writel(dwc->regs, DWC3_GUSB2PHYCFG(0), val); > + > + qcom->is_suspended =3D false; > + > + dev_dbg(qcom->dev, "DWC3 exited from low power mode\n"); no dev_dbg() or dev_info() in this driver. > +static irqreturn_t qcom_dwc3_resume_irq(int irq, void *data) > +{ > + struct dwc3_qcom *qcom =3D data; > + struct dwc3 *dwc =3D platform_get_drvdata(qcom->dwc3); > + > + /* If pm_suspended then let pm_resume take care of resuming h/w */ > + if (qcom->pm_suspended) > + return IRQ_HANDLED; > + > + dwc3_qcom_resume(qcom); instead, why don't you pm_runtime_resume() here, and let runtime_resume() handle dwc3_qcom_resume() ? > +static int dwc3_qcom_clk_init(struct dwc3_qcom *qcom, int count) > +{ > + struct device *dev =3D qcom->dev; > + struct device_node *np =3D dev->of_node; > + int i; > + > + qcom->num_clocks =3D count; > + > + if (!count) > + return 0; > + > + qcom->clks =3D devm_kcalloc(dev, qcom->num_clocks, > + sizeof(struct clk *), GFP_KERNEL); > + if (!qcom->clks) > + return -ENOMEM; > + > + for (i =3D 0; i < qcom->num_clocks; i++) { > + struct clk *clk; > + int ret; > + > + clk =3D of_clk_get(np, i); > + if (IS_ERR(clk)) { > + while (--i >=3D 0) > + clk_put(qcom->clks[i]); > + return PTR_ERR(clk); > + } > + > + ret =3D clk_prepare_enable(clk); > + if (ret < 0) { > + while (--i >=3D 0) { > + clk_disable_unprepare(qcom->clks[i]); > + clk_put(qcom->clks[i]); > + } > + clk_put(clk); > + > + return ret; > + } > + > + qcom->clks[i] =3D clk; > + } > + > + return 0; > +} > + > +static int dwc3_qcom_probe(struct platform_device *pdev) > +{ > + struct device_node *np =3D pdev->dev.of_node, *dwc3_np; > + struct dwc3_qcom *qcom; > + struct resource *res; > + int irq, ret, i; > + bool ignore_pipe_clk; > + > + qcom =3D devm_kzalloc(&pdev->dev, sizeof(*qcom), GFP_KERNEL); > + if (!qcom) > + return -ENOMEM; > + > + platform_set_drvdata(pdev, qcom); > + qcom->dev =3D &pdev->dev; > + > + qcom->resets =3D of_reset_control_array_get_optional_exclusive(np); > + if (IS_ERR(qcom->resets)) { > + ret =3D PTR_ERR(qcom->resets); > + dev_err(&pdev->dev, "failed to get resets, err=3D%d\n", ret); > + return ret; > + } > + > + ret =3D reset_control_deassert(qcom->resets); > + if (ret) > + goto reset_put; > + > + ret =3D dwc3_qcom_clk_init(qcom, of_clk_get_parent_count(np)); > + if (ret) { > + dev_err(qcom->dev, "failed to get clocks\n"); > + goto reset_assert; > + } > + > + res =3D platform_get_resource_byname(pdev, IORESOURCE_MEM, "qscratch"); > + qcom->qscratch_base =3D devm_ioremap_resource(qcom->dev, res); > + if (IS_ERR(qcom->qscratch_base)) { > + dev_err(qcom->dev, "failed to map qscratch - %d\n", ret); > + ret =3D PTR_ERR(qcom->qscratch_base); > + goto clk_disable; > + } > + > + irq =3D platform_get_irq_byname(pdev, "hs_phy_irq"); > + if (irq > 0) { > + ret =3D devm_request_threaded_irq(qcom->dev, irq, NULL, > + qcom_dwc3_resume_irq, > + IRQF_TRIGGER_HIGH | IRQF_ONESHOT, > + "qcom_dwc3 HS", qcom); > + if (ret) { > + dev_err(qcom->dev, "hs_phy_irq failed: %d\n", ret); > + goto clk_disable; > + } > + } > + > + irq =3D platform_get_irq_byname(pdev, "dp_hs_phy_irq"); > + if (irq > 0) { > + irq_set_status_flags(irq, IRQ_NOAUTOEN); why do you need to set this flag? > + ret =3D devm_request_threaded_irq(qcom->dev, irq, NULL, > + qcom_dwc3_resume_irq, > + IRQF_TRIGGER_HIGH | IRQF_ONESHOT, > + "qcom_dwc3 DP_HS", qcom); this is the same as devm_request_irq() > + if (ret) { > + dev_err(qcom->dev, "dp_hs_phy_irq failed: %d\n", ret); > + goto clk_disable; > + } > + qcom->dp_hs_phy_irq =3D irq; > + } > + > + irq =3D platform_get_irq_byname(pdev, "dm_hs_phy_irq"); > + if (irq > 0) { > + irq_set_status_flags(irq, IRQ_NOAUTOEN); > + ret =3D devm_request_threaded_irq(qcom->dev, irq, NULL, > + qcom_dwc3_resume_irq, > + IRQF_TRIGGER_HIGH | IRQF_ONESHOT, > + "qcom_dwc3 DM_HS", qcom); > + if (ret) { > + dev_err(qcom->dev, "dm_hs_phy_irq failed: %d\n", ret); > + goto clk_disable; > + } > + qcom->dm_hs_phy_irq =3D irq; > + } > + > + irq =3D platform_get_irq_byname(pdev, "ss_phy_irq"); > + if (irq > 0) { > + irq_set_status_flags(irq, IRQ_NOAUTOEN); > + ret =3D devm_request_threaded_irq(qcom->dev, irq, NULL, > + qcom_dwc3_resume_irq, > + IRQF_TRIGGER_HIGH | IRQF_ONESHOT, > + "qcom_dwc3 SS", qcom); > + if (ret) { > + dev_err(qcom->dev, "ss_phy_irq failed: %d\n", ret); > + goto clk_disable; > + } > + qcom->ss_phy_irq =3D irq; > + } > + > + dwc3_np =3D of_get_child_by_name(np, "dwc3"); > + if (!dwc3_np) { > + dev_err(qcom->dev, "failed to find dwc3 core child\n"); > + ret =3D -ENODEV; > + goto clk_disable; > + } > + > + /* > + * Disable pipe_clk requirement if specified. Used when dwc3 > + * operates without SSPHY and only HS/FS/LS modes are supported. > + */ > + ignore_pipe_clk =3D device_property_read_bool(qcom->dev, > + "qcom,select-utmi-as-pipe-clk"); > + if (ignore_pipe_clk) > + dwc3_qcom_select_utmi_clk(qcom); > + > + ret =3D of_platform_populate(np, NULL, NULL, qcom->dev); > + if (ret) { > + dev_err(qcom->dev, "failed to register dwc3 core - %d\n", ret); > + goto clk_disable; > + } > + > + qcom->dwc3 =3D of_find_device_by_node(dwc3_np); > + if (!qcom->dwc3) { > + dev_err(&pdev->dev, "failed to get dwc3 platform device\n"); > + goto depopulate; > + } > + > + qcom->mode =3D usb_get_dr_mode(&qcom->dwc3->dev); > + > + /* register extcon to override vbus later on mode switch */ > + if (qcom->mode =3D=3D USB_DR_MODE_OTG) { > + ret =3D dwc3_qcom_register_extcon(qcom); > + if (ret) > + goto depopulate; > + } else if (qcom->mode =3D=3D USB_DR_MODE_PERIPHERAL) { > + /* enable vbus override for device mode */ > + dwc3_qcom_vbus_overrride_enable(qcom, true); > + } > + > + device_init_wakeup(&pdev->dev, 1); > + qcom->is_suspended =3D false; > + pm_runtime_set_active(qcom->dev); > + pm_runtime_enable(qcom->dev); > + > + return 0; > + > +depopulate: > + of_platform_depopulate(&pdev->dev); > +clk_disable: > + for (i =3D qcom->num_clocks - 1; i >=3D 0; i--) { > + clk_disable_unprepare(qcom->clks[i]); > + clk_put(qcom->clks[i]); > + } > +reset_assert: > + reset_control_assert(qcom->resets); > +reset_put: > + reset_control_put(qcom->resets); > + > + return ret; > +} > + > +static int dwc3_qcom_remove(struct platform_device *pdev) > +{ > + struct dwc3_qcom *qcom =3D platform_get_drvdata(pdev); > + struct device *dev =3D qcom->dev; > + int i; > + > + of_platform_depopulate(&pdev->dev); > + > + for (i =3D qcom->num_clocks - 1; i >=3D 0; i--) { > + clk_disable_unprepare(qcom->clks[i]); > + clk_put(qcom->clks[i]); > + } > + qcom->num_clocks =3D 0; > + > + reset_control_assert(qcom->resets); > + reset_control_put(qcom->resets); > + > + pm_runtime_disable(dev); > + > + return 0; > +} > + > +#ifdef CONFIG_PM_SLEEP > +static int dwc3_qcom_pm_suspend(struct device *dev) > +{ > + struct dwc3_qcom *qcom =3D dev_get_drvdata(dev); > + int ret =3D 0; > + > + dev_err(dev, "dwc3-qcom PM suspend\n"); why an error message here? There's no error. Remove this, it's unnecessary. > + ret =3D dwc3_qcom_suspend(qcom); > + if (!ret) > + qcom->pm_suspended =3D true; > + > + return ret; > +} > + > +static int dwc3_qcom_pm_resume(struct device *dev) > +{ > + struct dwc3_qcom *qcom =3D dev_get_drvdata(dev); > + int ret; > + > + dev_err(dev, "dwc3-qcom PM resume\n"); ditto > + ret =3D dwc3_qcom_resume(qcom); > + if (!ret) > + qcom->pm_suspended =3D false; > + > + return ret; > +} > +#endif > + > +#ifdef CONFIG_PM > +static int dwc3_qcom_runtime_suspend(struct device *dev) > +{ > + struct dwc3_qcom *qcom =3D dev_get_drvdata(dev); > + > + dev_err(dev, "DWC3-qcom runtime suspend\n"); ditto > + return dwc3_qcom_suspend(qcom); > +} > + > +static int dwc3_qcom_runtime_resume(struct device *dev) > +{ > + struct dwc3_qcom *qcom =3D dev_get_drvdata(dev); > + > + dev_err(dev, "DWC3-qcom runtime resume\n"); ditto > + return dwc3_qcom_resume(qcom); > +} > +#endif > + > +static const struct dev_pm_ops dwc3_qcom_dev_pm_ops =3D { > + SET_SYSTEM_SLEEP_PM_OPS(dwc3_qcom_pm_suspend, dwc3_qcom_pm_resume) > + SET_RUNTIME_PM_OPS(dwc3_qcom_runtime_suspend, dwc3_qcom_runtime_resume, > + NULL) why don't you have runtime_idle? =2D-=20 balbi --=-=-= Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAEBCgAdFiEElLzh7wn96CXwjh2IzL64meEamQYFAlqnsR4ACgkQzL64meEa mQaM2hAAtpcjjx2FtDuNsiIx7EMT84EnJjt6o60odgNAvg3GwaxFMR405mpskyUf EbBSAuMI3CGQyBjB/zHigZdco5KTpVnFMu5ksFaAcZVP1JGmqWsGN6JVOkk9KEv9 42HZhawXDOzzc7BlJ1scoegomjx/sByIGfHtLFHGeG3idexSEKbd8oUG87mwwBM2 ldT5RNwKb+VLyIdCssdnbbJoZ0ZXC7ESXMfQ4DbSGzWaD0KTKXHCOh/cJdceuAJq TjC7c1YDQFbTJDSoepk4LkH3h90xEpPVtTokh5pz9MMRmyZDer7AkVW51uzKNi9b J6uRCPmvPjX+CkHtbS7L/vVOEowQy7TxwPA+ZMKyYvdxzrJx/KSCbe3Irx8tSB0E Lkkre3a9/rhtrBvQ5MW5jvkh00Hv1JPxjVh99ZZBUQ0Zb7mPWZYA3j7eSDGofMOz NZml3hi05jGqtrXqQduHU0UwcXQbfbBQY+VThYOyaKlG8cBV8t4fISJXaaAP/2lO C/cBnXAKokmKkOL9+p0YRVUp67KLJZNtoEbvvUjX/sA/MSTW9Y0OPBFganTJTXZ/ UlK5W7AJ7UIvlx0rPipiPmam0AyE1uSgaVWPzfWJeaY5Hu9aHKWiC4PnnZ576viP lo6Z65BK6wEBlvfmA1zdC9rcPQtsqsuLuxnu7t6KIU0l/Kc8UZY= =LJgf -----END PGP SIGNATURE----- --=-=-=--