Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756904AbdLWE5e (ORCPT ); Fri, 22 Dec 2017 23:57:34 -0500 Received: from mail-wm0-f67.google.com ([74.125.82.67]:38878 "EHLO mail-wm0-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752448AbdLWE5b (ORCPT ); Fri, 22 Dec 2017 23:57:31 -0500 X-Google-Smtp-Source: ACJfBots3IEs+VlbqQ98AEQ2nkYkToix+ywqgWhhyQ/W0urAfZS+F/RIc9yd1k7FJgPP7RjO/MaN2MhOui/WFni1oew= MIME-Version: 1.0 In-Reply-To: <20171205154701.27730-3-georgi.djakov@linaro.org> References: <20171205154701.27730-1-georgi.djakov@linaro.org> <20171205154701.27730-3-georgi.djakov@linaro.org> From: Jassi Brar Date: Sat, 23 Dec 2017 10:27:28 +0530 Message-ID: Subject: Re: [PATCH v11 2/6] mailbox: qcom: Create APCS child device for clock controller To: Georgi Djakov Cc: Stephen Boyd , Bjorn Andersson , Michael Turquette , Rob Herring , linux-clk@vger.kernel.org, Linux Kernel Mailing List , linux-arm-msm@vger.kernel.org, Devicetree List Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 517 Lines: 12 On Tue, Dec 5, 2017 at 9:16 PM, Georgi Djakov wrote: > There is a clock controller functionality provided by the APCS hardware > block of msm8916 devices. The device-tree would represent an APCS node > with both mailbox and clock provider properties. > The spec might depict a 'clock' box and 'mailbox' box inside the bigger APCS box. However, from the code I see in this patchset, they are orthogonal and can & should be represented as independent DT nodes. Patch-1 is ok, though. Thanks