From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 2C3E9C433EF for ; Thu, 2 Jun 2022 10:46:01 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232729AbiFBKp7 (ORCPT ); Thu, 2 Jun 2022 06:45:59 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:47528 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230270AbiFBKp7 (ORCPT ); Thu, 2 Jun 2022 06:45:59 -0400 Received: from mail-ed1-x52f.google.com (mail-ed1-x52f.google.com [IPv6:2a00:1450:4864:20::52f]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 008916368 for ; Thu, 2 Jun 2022 03:45:57 -0700 (PDT) Received: by mail-ed1-x52f.google.com with SMTP id h19so5759293edj.0 for ; Thu, 02 Jun 2022 03:45:57 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=message-id:date:mime-version:user-agent:subject:content-language:to :cc:references:from:in-reply-to:content-transfer-encoding; bh=nw6pxzn4DckQbSIrMyIKvslcoovKd9JNNcEV3M74aJU=; b=FVtc2GMHaMVFwBTB9n2bmkPgrxq0S/3I/jF+Z5+VjhYWNvNhdvd9FduqFd8H21CXnp 5CoJBlRBbspU+PTFXH1VuLP74dEZNgdRy0pabuto/4CifAKSH60UJrIP/u0wDj1LjPpZ pwN9n0KWujb8LNQ1GokLI6aRvWoP/kuIV4CibHVrjYuCB1sg9UWKqq9nlBBb3+IcyRxM oG+8X0jviUubYCuFv0wHF4I2MgJIe0fOxMsZb88UZBXiUgyGXO2IBR03XgFDZDgYBVZj nmSUJqmu0Thut+grCufPZRhNpL1Wm/mKbJQEtEjOiUCn2fVSqMR7B0gk0OB/P3jwjSYP Vw9w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:message-id:date:mime-version:user-agent:subject :content-language:to:cc:references:from:in-reply-to :content-transfer-encoding; bh=nw6pxzn4DckQbSIrMyIKvslcoovKd9JNNcEV3M74aJU=; b=ItvROHASuQGW8FQJHzrjlLRr6Gyj4LySPpPx3aKxmpTCVPmG14i7nt6m/uc0h2IsVJ SO4O8pQOuQBj/+J4zcnWVOwWTJUIt9WegrlFuZRhYTs3tkL6k5gdGoUHmwyq7geSSE/W dzW0FmUODyjX2bFV51MyDvrVwyq38qVXrue0EUfSaqIGA4OHxTWfTnILFV7KCQVc1R+U 0zlxagAjOIccd0Mxb/I14W3ELAtfh2BZcsHqG60pGy7wXY0TZ0SHOWIB1dM3wcMpQ84T NSSCz7lSTz0hCzeBe/+tapUak/h+ViBeWAgsAU61tom5G+xig4Di488DXTBek6hGSti/ ScUA== X-Gm-Message-State: AOAM531PaH2hb0+abcLVlv+2HWdG6i/ovqgr8UpS+GxhUg+OGICtc+fe oAMTQkMLfLudT+gQ7BGrDJKLhg== X-Google-Smtp-Source: ABdhPJyHBWmqVU4LKyl/2v3+gW1nAjEOcRCOkljscEs8ZrboVaNdsEU+GlslnGTpUxxoIj+RBNBgJw== X-Received: by 2002:a05:6402:3901:b0:42d:f38f:a5c6 with SMTP id fe1-20020a056402390100b0042df38fa5c6mr4551192edb.321.1654166756552; Thu, 02 Jun 2022 03:45:56 -0700 (PDT) Received: from [192.168.0.181] (xdsl-188-155-176-92.adslplus.ch. [188.155.176.92]) by smtp.gmail.com with ESMTPSA id q2-20020a17090609a200b006f3ef214e63sm1600576eje.201.2022.06.02.03.45.55 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 02 Jun 2022 03:45:55 -0700 (PDT) Message-ID: <9e3f5586-59fa-42cc-770c-b8694b4f2bf3@linaro.org> Date: Thu, 2 Jun 2022 12:45:54 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.10.0 Subject: Re: [PATCH v1] dt-bindings: dsp: mediatek: add mt8186 dsp document Content-Language: en-US To: Tinghan Shen , Rob Herring , Krzysztof Kozlowski , Matthias Brugger Cc: devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org, linux-kernel@vger.kernel.org, Project_Global_Chrome_Upstream_Group@mediatek.com References: <20220422071534.15653-1-tinghan.shen@mediatek.com> <591767ee-e349-7a17-a9e9-b95d0500c7c1@linaro.org> <774c075ca4ad815c88be755cfb51889a171e835d.camel@mediatek.com> From: Krzysztof Kozlowski In-Reply-To: <774c075ca4ad815c88be755cfb51889a171e835d.camel@mediatek.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org On 02/06/2022 12:19, Tinghan Shen wrote: > Hi Krzysztof, > > On Thu, 2022-06-02 at 09:40 +0200, Krzysztof Kozlowski wrote: >> On 02/06/2022 08:44, Tinghan Shen wrote: >>>>> + mbox-names: >>>>> + items: >>>>> + - const: mbox0 >>>>> + - const: mbox1 >>>> >>>> These should be rather some meaningful names, e.g. "rx" and "tx". >>> >>> The mbox name has to align with the adsp ipc driver. >>> The adsp ipc driver is using 'mbox%d' for mailbox channels. >>> >>> >>> > https://urldefense.com/v3/__https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git/commit/?id=9db69df4bdd37eb1f65b6931ee067fb15b9a4d5c__;!!CTRNKA9wMg0ARbw!1TmempNkQhC5QuLBhyfWo_AC97MoLuWipsGV-LPaW9RKNPheU7Bgc-eboNi1JA1nC5I$ >>> >>> >>> chan_name = kasprintf(GFP_KERNEL, "mbox%d", i); >>> >>> /* ...snip... */ >>> >>> adsp_chan->ch = mbox_request_channel_byname(cl, chan_name); >>> >>> Is it ok to continue using these names? >> >> It is a bit confusing... how did that driver got merged recently without >> bindings? Why bindings are separate? >> >> The bindings always come together in one patchset with the driver >> implementing them. Bindings are though a separate patch, yet still >> followed by the driver which uses them. >> >> I do not see any compatibles in that driver, which suggests there is no >> other binding using it. If that's correct, then you need to change the >> driver. >> > > The mtk-adsp-ipc driver's sole function is to encapsulate the operations > of mailbox framework from adsp ipc users. The mtk-adsp-ipc is not defined > in the dts file and we don't need it to be defined. The creation of mtk-adsp-ipc > device is requested by adsp ipc users via the use of 'platform_device_register_data'[1]. > > the driver implemented the mailbox framework is 'mtk-adsp-mailbox'[2]. it has > corresponding hardwares and a yaml file[3] to describe it. I don't understand how is this related. We talk here about the mbox-names for this bindings file. You replied, that these bindings are already used by something, but now you say that they are not? So why do you need to change anything in any driver? Simple question - do the bindings here "add mt8186 dsp document" are used by any specific Linux driver already? Best regards, Krzysztof