Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <4535f993-156c-5401-53a8-bcf938bf24b7@quicinc.com>
Date:   Wed, 10 May 2023 11:21:59 +0800
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101
 Thunderbird/102.10.1
Subject: Re: [PATCH v4 2/3] dt-bindings: arm: Add Coresight Dummy Trace
Content-Language: en-US
To:     Leo Yan <leo.yan@linaro.org>,
        Suzuki K Poulose <suzuki.poulose@arm.com>,
        James Clark <james.clark@arm.com>
CC:     Mike Leach <mike.leach@linaro.org>,
        Alexander Shishkin <alexander.shishkin@linux.intel.com>,
        Mathieu Poirier <mathieu.poirier@linaro.org>,
        Konrad Dybcio <konradybcio@gmail.com>,
        "Rob Herring" <robh+dt@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Andy Gross <agross@kernel.org>,
        "Paul Walmsley" <paul.walmsley@sifive.com>,
        Palmer Dabbelt <palmer@dabbelt.com>,
        Albert Ou <aou@eecs.berkeley.edu>,
        Jonathan Corbet <corbet@lwn.net>,
        "Greg Kroah-Hartman" <gregkh@linuxfoundation.org>,
        <coresight@lists.linaro.org>,
        <linux-arm-kernel@lists.infradead.org>,
        <linux-kernel@vger.kernel.org>, <devicetree@vger.kernel.org>,
        Tingwei Zhang <quic_tingweiz@quicinc.com>,
        Jinlong Mao <quic_jinlmao@quicinc.com>,
        Yuanfang Zhang <quic_yuanfang@quicinc.com>,
        Tao Zhang <quic_taozha@quicinc.com>,
        Trilok Soni <quic_tsoni@quicinc.com>,
        <linux-arm-msm@vger.kernel.org>,
        Bjorn Andersson <andersson@kernel.org>,
        <linux-doc@vger.kernel.org>
References: <20230505092422.32217-1-quic_hazha@quicinc.com>
 <20230505092422.32217-3-quic_hazha@quicinc.com>
 <958ae925-dee2-3273-0cd6-b5edc891ba70@arm.com>
 <20230505120529.GB898031@leoy-yangtze.lan>
From:   Hao Zhang <quic_hazha@quicinc.com>
In-Reply-To: <20230505120529.GB898031@leoy-yangtze.lan>
Content-Type: text/plain; charset="UTF-8"; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk

Hi Suzuki, Leo,

On 5/5/2023 8:05 PM, Leo Yan wrote:
> On Fri, May 05, 2023 at 11:54:03AM +0100, Suzuki Kuruppassery Poulose wrote:
> 
> [...]
> 
>>> +title: ARM Coresight Dummy component
>>> +
>>> +description: |
>>> +  Coresight Dummy Trace Module is for the specific devices that kernel
>>> +  don't have permission to access or configure, e.g., CoreSight TPDMs
>>> +  on Qualcomm platforms. So there need driver to register dummy devices
>>> +  as Coresight devices. It may also be used to define components that
>>> +  may not have any programming interfaces (e.g, static links), so that
>>> +  paths can be established in the driver. Provide Coresight API for
>>> +  dummy device operations, such as enabling and disabling dummy devices.
>>> +  Build the Coresight path for dummy sink or dummy source for debugging. > +
>>> +  The primary use case of the coresight dummy is to build path in kernel
>>> +  side for dummy sink and dummy source.
>>> +
>>> +maintainers:
>>> +  - Mao Jinlong <quic_jinlmao@quicinc.com>
>>> +  - Tao Zhang <quic_taozha@quicinc.com>
>>> +  - Hao Zhang <quic_hazha@quicinc.com>
>>> +  - Yuanfang Zhang <quic_yuanfang@quicinc.com>
>>
>> Given this is a generic "CoreSight" component, I would prefer to have the
>> CoreSight subsystem maintainers listed here (too). I don't mind
>> the entries above, but would like to make sure that the subsystem
>> people are aware of the changes happening here. Please use:
>>
>> Mike Leach <mike.leach@linaro.org>
>> Suzuki K Poulose <suzuki.poulose@arm.com>
>> Leo Yan <leo.yan@linaro.org>
>  > Given I am spending little time on CoreSight reviewing, I'd like to
> use James Clark's email address to replace my own; I believe this
> would benefit long term maintenance.
> 
>    James Clark <james.clark@arm.com>
> 
> Thanks!
>
Thanks for your review. I will update the maintainers in the next patch 
series.

Thanks,
Hao

>> With the above:
>>
>> Acked-by: Suzuki K Poulose <suzuki.poulose@arm.com>
>>
>>> +
>>> +properties:
>>> +  compatible:
>>> +    items:
>>> +      - enum:
>>> +          - arm,coresight-dummy-sink
>>> +          - arm,coresight-dummy-source
>>> +
>>> +  out-ports:
>>> +    $ref: /schemas/graph.yaml#/properties/ports
>>> +
>>> +    properties:
>>> +      port:
>>> +        description: Output connection from the source to Coresight
>>> +          Trace bus.
>>> +        $ref: /schemas/graph.yaml#/properties/port
>>> +
>>> +  in-ports:
>>> +    $ref: /schemas/graph.yaml#/properties/ports
>>> +
>>> +    properties:
>>> +      port:
>>> +        description: Input connection from the Coresight Trace bus to
>>> +          dummy sink, such as Embedded USB debugger(EUD).
>>> +        $ref: /schemas/graph.yaml#/properties/port
>>> +
>>> +required:
>>> +  - compatible
>>> +
>>> +if:
>>> +  # If the compatible contains the below value
>>> +  properties:
>>> +    compatible:
>>> +      contains:
>>> +        const: arm,coresight-dummy-sink
>>> +
>>> +then:
>>> +  required:
>>> +    - in-ports
>>> +
>>> +else:
>>> +  required:
>>> +    - out-ports
>>> +
>>> +additionalProperties: false
>>> +
>>> +examples:
>>> +  # Minimum dummy sink definition. Dummy sink connect to coresight replicator.
>>> +  - |
>>> +    sink {
>>> +      compatible = "arm,coresight-dummy-sink";
>>> +
>>> +      in-ports {
>>> +        port {
>>> +          eud_in_replicator_swao: endpoint {
>>> +            remote-endpoint = <&replicator_swao_out_eud>;
>>> +          };
>>> +        };
>>> +      };
>>> +    };
>>> +
>>> +  # Minimum dummy source definition. Dummy source connect to coresight funnel.
>>> +  - |
>>> +    source {
>>> +      compatible = "arm,coresight-dummy-source";
>>> +
>>> +      out-ports {
>>> +        port {
>>> +          dummy_riscv_out_funnel_swao: endpoint {
>>> +            remote-endpoint = <&funnel_swao_in_dummy_riscv>;
>>> +          };
>>> +        };
>>> +      };
>>> +    };
>>> +
>>> +...
>>