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 X-Spam-Level: X-Spam-Status: No, score=-0.7 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,FORGED_MUA_MOZILLA,HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,MSGID_FROM_MTA_HEADER,SPF_HELO_NONE,SPF_PASS, USER_AGENT_SANE_1 autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id DA7ADC433DF for ; Thu, 9 Jul 2020 13:29:58 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id A8B752065D for ; Thu, 9 Jul 2020 13:29:58 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="e3R/D/GT" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org A8B752065D Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=linux.intel.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=merlin.20170209; h=Sender:Content-Type: Content-Transfer-Encoding:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date:From:References:To: Subject:Message-Id:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=lYP7oS9/zAYe1FzeiBqhFcDahpYKcAm+R7NDj5S6cFI=; b=e3R/D/GTV6O2WsK2u/ZHbs2OH +IU3kOLW0jPOC5+rxRfh09kY556191ESt/1FGTdMHoWnGMEIByuDETjgtJ7/Ht/WxDDPn28xmxqBI 7yzDP3URiRyzfhs/c2+StozDxr93C3S8JPMHukdZjefwXdTVZIasUPJ4quvARmDRqcEjCDu4GqTti wCPQDk+mReRaWOiFY2hCw7ltxU5Sc6ve5LIfDCHxgMZcPC4NrnSP8SvYw4NRrMQqwFBX2l5XUpgwM o6nlfoxF+95IuPcy753vvOSQhnZo8iBPTSYU6bSJ3wzLJ5fDl2rFYKwoI/RMfYdw4N7yeMRZPqssC 7h7cBbD1A==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1jtWb7-0004xZ-2m; Thu, 09 Jul 2020 13:28:21 +0000 Received: from mga03.intel.com ([134.134.136.65]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1jtWb4-0004wQ-2f for linux-arm-kernel@lists.infradead.org; Thu, 09 Jul 2020 13:28:19 +0000 IronPort-SDR: /4dAsLbvlsgH3/1rixfml263RFT0dbcbu/lKx82kCO/qxr+xHClemoFScwVVQAIanQFtLHxJwn UI+3GdBTaGnw== X-IronPort-AV: E=McAfee;i="6000,8403,9676"; a="147984869" X-IronPort-AV: E=Sophos;i="5.75,331,1589266800"; d="scan'208";a="147984869" X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga004.jf.intel.com ([10.7.209.38]) by orsmga103.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 09 Jul 2020 06:28:15 -0700 Message-Id: <537a4c$cobr4j@orsmga004-auth.jf.intel.com> IronPort-SDR: lbGftqsvxRyXOACOGBHSm1In1Skp/KaIfrSFiKY6OurNhCBCCg5UvS81aWet1lIxbq8nWff6dl iJLteg35tBRA== X-IronPort-AV: E=Sophos;i="5.75,331,1589266800"; d="scan'208";a="428207258" Received: from paulmurp-mobl.ger.corp.intel.com (HELO [192.168.153.164]) ([10.213.225.132]) by orsmga004-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 09 Jul 2020 06:28:12 -0700 Subject: Re: [PATCH 2/7] mailbox: keembay-scmi-mailbox: Add support for Keem Bay mailbox To: Daniele Alessandrelli , Sudeep Holla References: <20200616155613.121242-1-daniele.alessandrelli@linux.intel.com> <20200616155613.121242-3-daniele.alessandrelli@linux.intel.com> <20200708203428.GB31671@bogus> <491f994c6f87a542e49fe37c8a75ab46f1b3712b.camel@linux.intel.com> From: Paul Murphy Date: Thu, 9 Jul 2020 14:28:10 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.8.0 MIME-Version: 1.0 In-Reply-To: <491f994c6f87a542e49fe37c8a75ab46f1b3712b.camel@linux.intel.com> Content-Language: en-US X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20200709_092818_289549_81A1F568 X-CRM114-Status: GOOD ( 23.28 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: devicetree@vger.kernel.org, Arnd Bergmann , Catalin Marinas , Jassi Brar , linux-kernel@vger.kernel.org, Dinh Nguyen , Rob Herring , Olof Johansson , Paul Murphy , Will Deacon , linux-arm-kernel@lists.infradead.org Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 7/9/20 13:23, Daniele Alessandrelli wrote: > Hi Sudeep, > > Thanks for your review. > > On Wed, 2020-07-08 at 21:34 +0100, Sudeep Holla wrote: >> On Tue, Jun 16, 2020 at 04:56:08PM +0100, Daniele Alessandrelli >> wrote: >>> From: Paul Murphy >>> >>> Keem Bay SoC has a ARM trusted firmware-based secure monitor which >>> acts >>> as the SCP for the purposes of power management over SCMI. >>> >>> This driver implements the transport layer for SCMI to function. >>> >> Please use the smc transport support in >> driver/firmware/arm_scmi/smc.c >> for this. You don't need mailbox support for SMC/HVC. Basically you >> don't need this driver at all and you have everything you need to >> support >> what you want. >> >> Let me know if you face issues. >> > Sorry, we didn't know about the SMC transport support for SCMI. Looks > like it was added only recently, while our driver was already developed > and waiting to be upstreamed. > > I agree that we can drop this driver and switch to the SMC transport as > you suggested, but I think we'll have to modify our bootloader SiP > service slightly. Paul, can you elaborate? > Just one question. In our patch, we pass the shared memory address as the second argument of the SiP service, as it means we don't have to hardcode that in our firmware. Sudeep, do you know if it was intentional in smc_send_message() to leave that out? If we leave it out, we are requiring the secure monitor to hardcode the shared memory address. Regards, Paul _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel