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 smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 70D06C7EE24 for ; Tue, 16 May 2023 07:36:05 +0000 (UTC) Received: by smtp.kernel.org (Postfix) id 3A577C433A1; Tue, 16 May 2023 07:36:05 +0000 (UTC) Received: from szxga02-in.huawei.com (szxga02-in.huawei.com [45.249.212.188]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.kernel.org (Postfix) with ESMTPS id 79A35C433EF; Tue, 16 May 2023 07:35:59 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 smtp.kernel.org 79A35C433EF Authentication-Results: smtp.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=huawei.com Authentication-Results: smtp.kernel.org; spf=pass smtp.mailfrom=huawei.com Received: from kwepemm600004.china.huawei.com (unknown [172.30.72.53]) by szxga02-in.huawei.com (SkyGuard) with ESMTP id 4QL7Fn2fNmzTkhh; Tue, 16 May 2023 15:31:09 +0800 (CST) Received: from [10.67.103.231] (10.67.103.231) by kwepemm600004.china.huawei.com (7.193.23.242) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.23; Tue, 16 May 2023 15:35:55 +0800 Message-ID: Date: Tue, 16 May 2023 15:35:54 +0800 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101 Thunderbird/91.2.0 Subject: Re: [PATCH] soc: hisilicon: Support HCCS driver on Kunpeng SoC To: Sudeep Holla List-Id: CC: Arnd Bergmann , Bjorn Andersson , Matthias Brugger , AngeloGioacchino Del Regno , Shawn Guo , , , , , , , , Len Brown , "Rafael J. Wysocki" , , Rob Herring , Frank Rowand , Krzysztof Kozlowski References: <20230424073020.4039-1-lihuisong@huawei.com> <20230425103040.znv66k364ant6klq@bogus> <20230425131918.5tf5vot4h7jf54xk@bogus> <20230515130807.pdvx7bxwjkfdsmsr@bogus> From: "lihuisong (C)" In-Reply-To: <20230515130807.pdvx7bxwjkfdsmsr@bogus> Content-Type: text/plain; charset="UTF-8"; format=flowed Content-Transfer-Encoding: 8bit X-Originating-IP: [10.67.103.231] X-ClientProxiedBy: dggems703-chm.china.huawei.com (10.3.19.180) To kwepemm600004.china.huawei.com (7.193.23.242) X-CFilter-Loop: Reflected 在 2023/5/15 21:08, Sudeep Holla 写道: > On Thu, May 04, 2023 at 09:16:16PM +0800, lihuisong (C) wrote: >> I'm tring to use CRS with GAS to report PCC channel ID and get other >> informations driver need by address. > OK you had pcc-chan-id pcc-type and device-flags in the DSD style bindings > to begin with. I haven't understood device-flags here so can't comment on > that. We want to use the 'device-flags' to report some information by bit. Currently, this driver requests PCC channel and use type2 to communicate with firmware. But, if some platform support type3 and PCC Operation Region, driver can choice this method to communicate with firmware. So firmware and driver have to use this flag to make compatibility. > >> I found a way to obtain the generic register information according to >> "Referencing the PCC address space" in ACPI spec. >> And driver also get the PCC generic register information successfully. >> > Can you elaborate ? I assume by that you must be able to get pcc-chan-id Yes,driver can get pcc-chan-id by below register. Register (PCC, RegisterBitWidth, RegisterBitOffset, RegisterAddress, AccessSize) > right ? You must not need pcc-type as the pcc mailbox driver must handle > the type for you. If not, we may need to fix or add any missing support. Yes, PCC driver doesn't support it currently. And aother patch [1] we've been talking about does it. If it is applied to kernel, we can drop this pcc-type here. [1] https://patchwork.kernel.org/project/linux-acpi/patch/20230423110335.2679-2-lihuisong@huawei.com/ > >> But I don't know how to set and use the address in PCC register. > It must be same as what you would have specified in you new bindings > under "pcc-chan-id". I am confused as you say you were able to get the > PCC generic register information successfully but you still claim you > don't know how to set or use the address. My confusion about this address is mentioned below. >> Where should this address come from? >> It seems that ACPI spec is not very detailed about this. >> Do you have any suggestions? >> > I am afraid, I don't have any as I am failing to understand the exact issue > you are facing. > > Let me try to ask the question explicity here: > > If you are just referring to just the in > > Register (PCC, RegisterBitWidth, RegisterBitOffset, RegisterAddress, AccessSize) Yeah, this is what I'm using. > > then, > > RegisterAddress is usually the offset in the comms address associated with Communication subspace in share memory of PCC subspace? > the PCC subspace ID specified in AccessSize. Yes the use of AccessSize for > the PCC subspace ID is bit confusing though. > > You can either list all the registers with _CRS individually or the driver List all the registers as following way? Name (_CRS, ResourceTemplate ()  // _CRS: Current Resource Settings {     QWordMemory (ResourceProducer, PosDecode, MinFixed, MaxFixed, NonCacheable, ReadWrite,         0x0000000000000000, // Granularity         0x0000000098190000, // Range Minimum         0x000000009819FFFF, // Range Maximum         0x0000000000000000, // Translation Offset         0x0000000000010000, // Length         ,, , AddressRangeMemory, TypeStatic) }) > can just use the PCC subspace ID in AccessSize and keep RegisterAddress = 0 > but access individual offset based on its own knowledge. I haven't seen the Following words come from ACPI spec. --> As an example, the following resource template refers to the feld occupying bits 8 through 15 at address 0x30 in PCC subspace 9: ResourceTemplate() { Register (     PCC, //AddressSpaceKeyword     8, //RegisterBitWidth     8, //RegisterBitOffset     0x30, //RegisterAddress     9 //AccessSize (subspace ID)     ) } If the width of the address is 32bit, set RegisterAddress to 0, RegisterBitOffset to 0 and set RegisterBitWidth to 64 here. Driver can access to the ((void __iomem *)pcc_comm_addr + 0x8 + 0) and ((void __iomem *)pcc_comm_addr + 0x8 + 4) address,right? (This virtual address = pcc mapped address + header size + offset within PCC subspace.) > full driver yet but I assuming that's how you would have used if you went with > your DSD pcc-chan-id proposal. > >> On the other hand, we think that System Memory space + method can also >> achieve above goal. What do you think of that? > Again I don't understand what you mean by that. Sorry, here is what I want to say. --> OperationRegion (CCS0, SystemMemory, 0x00000002081000CC, 0x04) Field (CCS0, DWordAcc, NoLock, Preserve) {     HAU1,   32 } OperationRegion (CCS1, SystemMemory, 0x0000000201070410, 0x04) Field (CCS1, DWordAcc, NoLock, Preserve) {     HCGE,   32 } Method (_DSM, 2, Serialized)  // _DSM: Device-Specific Method {     If ((Arg0 == ToUUID ("b06b81ab-0134-4a45-9b0c-483447b95fa7")))     {         If ((Arg1 == One))         {             Return (HAU1)         }         Return (HCGE)     } } Driver can call _DSM method to get some information, such as pcc_chan_id and device_flags. >