Received: by 2002:a05:6a10:206:0:0:0:0 with SMTP id 6csp4201107pxj; Mon, 21 Jun 2021 16:20:00 -0700 (PDT) X-Google-Smtp-Source: ABdhPJyPZQz2lQCl3V1kMUgjIxBlvICTaFfbk8MXnaUjEBRnEkss6TNpKoV0gXpVGO3T6HSDLcls X-Received: by 2002:a92:6b06:: with SMTP id g6mr487528ilc.270.1624317600658; Mon, 21 Jun 2021 16:20:00 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1624317600; cv=none; d=google.com; s=arc-20160816; b=kLkodf2wzvtyvBbOIftRtkbhNAYUr72+/eXz3YBBhPY0wzTYOJjoud4d8pXUzRvGna AZkq0cf6B8Oi616hpxF1jht8JmA5dq3sJ8XIW6mcnUN3yA/hfqFraYfbKdL4vxTA1tN0 eoXSPQXVRNw3O3cKarVmxSCXcN0NRKvuEsI6wUzj/PHsKCm5GVxxP3xF301iRxO5lLHI ZZbaB12drRBFNAPT5Yz+Yac+5QE/BcCLnEqVDXjgES6rTJB9W+W8sMlutM+ajspgFHcm /k+JjHuL/+2JwEOEuKQlM3z99MdZqAORj9ehlrN4KEc98bjqWsXBM8GM1NWQVkR0I4tu MNQg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:cc:to:subject:message-id:date:from:in-reply-to :references:mime-version:dkim-signature; bh=sgvN1H001LgWjs+xhV+6wZXYPL50psHYe1z6+Ci9990=; b=LnBZvBSfSZTo/peYjmLO9PneZwyqk24d5Htj/f6S/MvrOCkeSkLSDTFiUV4HTly8q4 n9ytOHVRvx/jArCKT2IFChqIKqGwb2W6B0XklLbAiPvEOU4k+hIMoBAAgx5U1BfVh4K0 4WSKsOt5JwicA3GcCboKra4eK2KQJxWeBts+1+KZO8hX6RZBcxdZU57rKnXqt74GxB6o VRMTdCq5YBoWnBVyPgET2Y55qJXCkEQCYSCraq31iAr1ZxYEmpnlH92hjYrZE6uE2245 c4ctDo6Uf4LvslC9Yg9mTiedIbYa+3yNVAo0piheocXjsVZo1eUx6qNQ65vylqhIF+eV NNyQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=EALJRLLp; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id t1si13748640ile.141.2021.06.21.16.19.48; Mon, 21 Jun 2021 16:20:00 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=EALJRLLp; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231950AbhFUXVc (ORCPT + 99 others); Mon, 21 Jun 2021 19:21:32 -0400 Received: from mail.kernel.org ([198.145.29.99]:34958 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232064AbhFUXV1 (ORCPT ); Mon, 21 Jun 2021 19:21:27 -0400 Received: by mail.kernel.org (Postfix) with ESMTPSA id 2D8756128E; Mon, 21 Jun 2021 23:19:13 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1624317553; bh=EYSEiTRNCJeziu2InDx+K6VGSnjU1txVEGEaZ6iYlE8=; h=References:In-Reply-To:From:Date:Subject:To:Cc:From; b=EALJRLLpGbQ0ysosvYNsYGhLN94XGwgW99Tyxah/VjbhGto+G8K0xtHfJo85MpjI9 paLLIdrZ0Sx6/WI5qL0xasZuhUblrr5+xrTeRO4EBTmZF0Uiyn9xFCreS5y4KbfLZz Ebx3uYHhMxvYRodwqRDRIOYwVTmgZPDuQunGJVyLNwYrPBUYTBjRtjUtL5hcKMUB/k 9QIeAhMk64hSJdVlS9+TiEKlQ+n0qG8vb5jt5MFpqPluO02UbkXswBavQ44x+U2NQ+ iMqfjlTD0BG/VkFAU/nnqm1xY+wFb1R1fnEs8wmVVpYvd/EuckxisM3U8D/fMc8Ec6 mStAmAlNR/YQg== Received: by mail-ed1-f53.google.com with SMTP id df12so18712555edb.2; Mon, 21 Jun 2021 16:19:13 -0700 (PDT) X-Gm-Message-State: AOAM530psCsXxZaCHMB0tmOlI0BO9CL2muXpWmWFrRAR7wWz1Ck2U5MM 0hEnlLAZQZQ3turdgEHfdRxN+Fb/SecNrMfkqQ== X-Received: by 2002:aa7:ccca:: with SMTP id y10mr1021381edt.258.1624317551768; Mon, 21 Jun 2021 16:19:11 -0700 (PDT) MIME-Version: 1.0 References: <20210612094631.89980-1-martin.botka@somainline.org> <20210612094631.89980-3-martin.botka@somainline.org> In-Reply-To: From: Rob Herring Date: Mon, 21 Jun 2021 17:19:00 -0600 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH V3 3/3] mailbox: qcom-apcs: Add SM6125 compatible To: Jassi Brar Cc: Martin Botka , ~postmarketos/upstreaming@lists.sr.ht, Konrad Dybcio , AngeloGioacchino Del Regno , Marijn Suijten , jamipkettunen@somainline.org, Andy Gross , Bjorn Andersson , linux-arm-msm , Linux Kernel Mailing List Content-Type: text/plain; charset="UTF-8" Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Jun 21, 2021 at 5:10 PM Jassi Brar wrote: > > On Mon, Jun 21, 2021 at 2:46 PM Rob Herring wrote: > > > > On Sun, Jun 20, 2021 at 10:03 PM Jassi Brar wrote: > > > > > > On Sat, Jun 12, 2021 at 4:46 AM Martin Botka > > > wrote: > > > > > > > > This commit adds compatible for the SM6125 SoC > > > > > > > > Signed-off-by: Martin Botka > > > > --- > > > > Changes in V2: > > > > None > > > > Changes in V3: > > > > Change compatible to apcs-hmss-global > > > > drivers/mailbox/qcom-apcs-ipc-mailbox.c | 5 +++++ > > > > 1 file changed, 5 insertions(+) > > > > > > > > diff --git a/drivers/mailbox/qcom-apcs-ipc-mailbox.c b/drivers/mailbox/qcom-apcs-ipc-mailbox.c > > > > index f25324d03842..f24c5ad8d658 100644 > > > > --- a/drivers/mailbox/qcom-apcs-ipc-mailbox.c > > > > +++ b/drivers/mailbox/qcom-apcs-ipc-mailbox.c > > > > @@ -57,6 +57,10 @@ static const struct qcom_apcs_ipc_data sdm660_apcs_data = { > > > > .offset = 8, .clk_name = NULL > > > > }; > > > > > > > > +static const struct qcom_apcs_ipc_data sm6125_apcs_data = { > > > > + .offset = 8, .clk_name = NULL > > > > +}; > > > > + > > > > static const struct qcom_apcs_ipc_data apps_shared_apcs_data = { > > > > .offset = 12, .clk_name = NULL > > > > }; > > > > @@ -166,6 +170,7 @@ static const struct of_device_id qcom_apcs_ipc_of_match[] = { > > > > { .compatible = "qcom,sc8180x-apss-shared", .data = &apps_shared_apcs_data }, > > > > { .compatible = "qcom,sdm660-apcs-hmss-global", .data = &sdm660_apcs_data }, > > > > { .compatible = "qcom,sdm845-apss-shared", .data = &apps_shared_apcs_data }, > > > > + { .compatible = "qcom,sm6125-apcs-hmss-global", .data = &sm6125_apcs_data }, > > > > { .compatible = "qcom,sm8150-apss-shared", .data = &apps_shared_apcs_data }, > > > > { .compatible = "qcom,sdx55-apcs-gcc", .data = &sdx55_apcs_data }, > > > > {} > > > > > > > These all are basically different names for the same controller. > > > The 'offset' is a configuration parameter and the 'clock', when NULL, > > > is basically some "always-on" clock. > > > I am sure we wouldn't be doing it, if the controller was third-party. > > > > If newer implementations are 'the same', then they should have a > > fallback compatible to the existing one that is the same and no driver > > change is needed. If the differences are board or instance (within an > > SoC) specific, then a DT property would be appropriate. > > > The controllers (13 now) only differ by the 'offset' where the > registers are mapped. Clock-name is a pure s/w artifact. > So, maybe we could push all these in DT. Why is 'reg' not used for the offset? In any case, we can't really get rid of the first 13 instances though... Rob