From patchwork Thu Dec 17 12:29:26 2015 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Auger Eric X-Patchwork-Id: 58588 Delivered-To: patches@linaro.org Received: by 10.112.89.199 with SMTP id bq7csp332407lbb; Thu, 17 Dec 2015 04:29:39 -0800 (PST) X-Received: by 10.28.139.206 with SMTP id n197mr3629917wmd.20.1450355379417; Thu, 17 Dec 2015 04:29:39 -0800 (PST) Return-Path: Received: from mail-wm0-x22d.google.com (mail-wm0-x22d.google.com. [2a00:1450:400c:c09::22d]) by mx.google.com with ESMTPS id h7si17119403wjy.46.2015.12.17.04.29.39 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Thu, 17 Dec 2015 04:29:39 -0800 (PST) Received-SPF: pass (google.com: domain of eric.auger@linaro.org designates 2a00:1450:400c:c09::22d as permitted sender) client-ip=2a00:1450:400c:c09::22d; Authentication-Results: mx.google.com; spf=pass (google.com: domain of eric.auger@linaro.org designates 2a00:1450:400c:c09::22d as permitted sender) smtp.mailfrom=eric.auger@linaro.org; dkim=pass header.i=@linaro.org Received: by mail-wm0-x22d.google.com with SMTP id p187so18898898wmp.1 for ; Thu, 17 Dec 2015 04:29:39 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=2PmgiVaScu2+F1i0uZ42xvV0aHvEIYtuDLVXAyu1IzQ=; b=bL7pVep42cWkx7luEGilCoSdLN4InvaLuP7UfNZ0P8fkyDQJBxMrMIDRw5a7AENF2R J+PPNFomoS4oeGRzrOP1GrXehvfa2PT2qNl4bepY1E2zDbJhjfBwY6Vq8bE+EsKIMgRM V8/r+prK782k/fzT31O8SGMU9SJccAHac4byo= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=2PmgiVaScu2+F1i0uZ42xvV0aHvEIYtuDLVXAyu1IzQ=; b=djs9uToB8VsHSedbkmGoZNSbf/+j4sysPGtiVEh+gR4Kh/vbjI4FeT0u8wQ3U8fnzu Bqcr60yoh4pxW54lU56mpBg1FAHnKuPC/39dSeLtRO8JOBWRiHV7M6D0KRtv6oHzkHh3 WjJMBKXuaqPlb+sG6MHxqi23Jtt8Pqx3poT9M8UKRIotO0m3gG4o4tGHEdNoERG1NgQD XEVM86OiwFlB4tgjPrMDu7YqBh522xi7H2LppfcvJenGdEq60J1TPUdUZUuXO4cdCLcR 0UNyHksCBUJ/SHenWZ0cklGM68XfKKw/qnvL2mG0vTmoqLDmdmXl8rKvXdpAexNi2w+O OLGg== X-Gm-Message-State: ALoCoQkSDNhHEt/AjSiO5ecpe+06j+a8Uo2B+mHO+n78tRxhVlwjPTnrHHOJYWuNnxxVKHM7xjeSTvBSw2bQ+XfvbvUVBfx6Ew== X-Received: by 10.28.8.15 with SMTP id 15mr3952144wmi.50.1450355379192; Thu, 17 Dec 2015 04:29:39 -0800 (PST) Return-Path: Received: from new-host-17.home (LMontsouris-657-1-37-90.w80-11.abo.wanadoo.fr. [80.11.198.90]) by smtp.gmail.com with ESMTPSA id bh6sm10313963wjb.0.2015.12.17.04.29.36 (version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Thu, 17 Dec 2015 04:29:37 -0800 (PST) From: Eric Auger To: eric.auger@st.com, eric.auger@linaro.org, qemu-devel@nongnu.org, peter.maydell@linaro.org, david@gibson.dropbear.id.au, alex.williamson@redhat.com Cc: alex.bennee@linaro.org, thuth@redhat.com, crosthwaitepeter@gmail.com, patches@linaro.org, christoffer.dall@linaro.org, pbonzini@redhat.com, b.reynal@virtualopensystems.com, suravee.suthikulpanit@amd.com, thomas.lendacky@amd.com Subject: [PATCH 5/6] hw/arm/sysbus-fdt: helpers for clock node generation Date: Thu, 17 Dec 2015 12:29:26 +0000 Message-Id: <1450355367-14818-6-git-send-email-eric.auger@linaro.org> X-Mailer: git-send-email 1.9.1 In-Reply-To: <1450355367-14818-1-git-send-email-eric.auger@linaro.org> References: <1450355367-14818-1-git-send-email-eric.auger@linaro.org> Some passthrough'ed devices depend on clock nodes. Those need to be generated in the guest device tree. This patch introduces some helpers to build a clock node from information retrieved in the host device tree. - inherit_properties copies properties from a host device tree node to a guest device tree node - fdt_build_clock_node builds a guest clock node and checks the host fellow clock is a fixed one. fdt_build_clock_node will become static as soon as it gets used. A dummy pre-declaration is needed for compilation of this patch. Signed-off-by: Eric Auger --- RFC -> v1: - use the new proto of qemu_fdt_getprop - remove newline in error_report - fix some style issues --- hw/arm/sysbus-fdt.c | 111 ++++++++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 111 insertions(+) -- 1.9.1 diff --git a/hw/arm/sysbus-fdt.c b/hw/arm/sysbus-fdt.c index 9d28797..c2d813b 100644 --- a/hw/arm/sysbus-fdt.c +++ b/hw/arm/sysbus-fdt.c @@ -21,6 +21,7 @@ * */ +#include #include "hw/arm/sysbus-fdt.h" #include "qemu/error-report.h" #include "sysemu/device_tree.h" @@ -56,6 +57,116 @@ typedef struct NodeCreationPair { int (*add_fdt_node_fn)(SysBusDevice *sbdev, void *opaque); } NodeCreationPair; +/* helpers */ + +struct HostProperty { + const char *name; + bool optional; +}; + +typedef struct HostProperty HostProperty; + +/** + * inherit_properties + * + * copies properties listed in an array from host device tree to + * guest device tree. If a non optional property is not found, the + * function self-asserts. An optional property is ignored if not found + * in the host device tree. + * @props: array of HostProperty to copy + * @nb_props: number of properties in the array + * @host_dt: host device tree blob + * @guest_dt: guest device tree blob + * @node_path: host dt node path where the property is supposed to be + found + * @nodename: guest node name the properties should be added to + */ +static void inherit_properties(HostProperty *props, int nb_props, + void *host_fdt, void *guest_fdt, + char *node_path, char *nodename) +{ + int i, prop_len; + const void *r; + + for (i = 0; i < nb_props; i++) { + r = qemu_fdt_getprop(host_fdt, node_path, + props[i].name, + &prop_len, + props[i].optional ? NULL : &error_fatal); + if (r) { + qemu_fdt_setprop(guest_fdt, nodename, + props[i].name, r, prop_len); + } + } +} + +/* clock properties whose values are copied/pasted from host */ +static HostProperty clock_inherited_properties[] = { + {"compatible", 0}, + {"#clock-cells", 0}, + {"clock-frequency", 1}, + {"clock-output-names", 1}, +}; + +/** + * fdt_build_clock_node + * + * Build a guest clock node, used as a dependency from a passthrough'ed + * device. Most information are retrieved from the host clock node. + * Also check the host clock is a fixed one. + * + * @host_fdt: host device tree blob from which info are retrieved + * @guest_fdt: guest device tree blob where the clock node is added + * @host_phandle: phandle of the clock in host device tree + * @guest_phandle: phandle to assign to the guest node + */ +int fdt_build_clock_node(void *host_fdt, void *guest_fdt, + uint32_t host_phandle, + uint32_t guest_phandle); +int fdt_build_clock_node(void *host_fdt, void *guest_fdt, + uint32_t host_phandle, + uint32_t guest_phandle) +{ + char node_path[256]; + char *nodename; + const void *r; + int ret, prop_len; + + ret = fdt_node_offset_by_phandle(host_fdt, host_phandle); + if (ret <= 0) { + error_report("not able to locate clock handle %d in host device tree", + host_phandle); + goto out; + } + ret = fdt_get_path(host_fdt, ret, node_path, 256); + if (ret < 0) { + error_report("not able to retrieve node path for clock handle %d", + host_phandle); + goto out; + } + + r = qemu_fdt_getprop(host_fdt, node_path, "compatible", &prop_len, + &error_fatal); + if (strcmp(r, "fixed-clock")) { + error_report("clock handle %d is not a fixed clock", host_phandle); + ret = -1; + goto out; + } + + nodename = strrchr(node_path, '/'); + qemu_fdt_add_subnode(guest_fdt, nodename); + + inherit_properties(clock_inherited_properties, + ARRAY_SIZE(clock_inherited_properties), + host_fdt, guest_fdt, + node_path, nodename); + + qemu_fdt_setprop_cell(guest_fdt, nodename, "phandle", guest_phandle); + +out: + return ret; +} + /* Device Specific Code */ /**