Received-SPF: softfail (google.com: domain of transitioning linux-kernel-owner@vger.kernel.org does not designate 23.128.96.19 as permitted sender) client-ip=23.128.96.19;
Message-ID: <14eeab94-8344-2c8d-9437-ddc8e367cfde@linaro.org>
Date:   Mon, 6 Jun 2022 09:26:50 +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] dt-bindings: i2c: qcom,i2c-qup: convert to dtschema
Content-Language: en-US
To:     Robert Marko <robimarko@gmail.com>, Rob Herring <robh@kernel.org>
Cc:     Andy Gross <agross@kernel.org>,
        Bjorn Andersson <bjorn.andersson@linaro.org>,
        krzysztof.kozlowski+dt@linaro.org,
        linux-arm-msm <linux-arm-msm@vger.kernel.org>,
        linux-i2c@vger.kernel.org,
        Devicetree List <devicetree@vger.kernel.org>,
        open list <linux-kernel@vger.kernel.org>
References: <20220604164653.79284-1-robimarko@gmail.com>
 <20220605140744.GA3416078-robh@kernel.org>
 <CAOX2RU7PHpzKUNvuv=-MyqGtgcz30qKkvx2MHNV=ehtCZGBYEA@mail.gmail.com>
From:   Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
In-Reply-To: <CAOX2RU7PHpzKUNvuv=-MyqGtgcz30qKkvx2MHNV=ehtCZGBYEA@mail.gmail.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: bulk

On 05/06/2022 18:09, Robert Marko wrote:
> On Sun, 5 Jun 2022 at 16:07, Rob Herring <robh@kernel.org> wrote:
>>
>> On Sat, Jun 04, 2022 at 06:46:53PM +0200, Robert Marko wrote:
>>> Convert DT bindings for Qualcomm QUP I2C controller to DT schema format.
>>>
>>> Old text bindings were missing usage of DMA so that was documented, as
>>> well as the max clock-frequency.
>>>
>>> Signed-off-by: Robert Marko <robimarko@gmail.com>
>>> ---
>>>  .../devicetree/bindings/i2c/qcom,i2c-qup.txt  | 40 ---------
>>>  .../devicetree/bindings/i2c/qcom,i2c-qup.yaml | 83 +++++++++++++++++++
>>>  2 files changed, 83 insertions(+), 40 deletions(-)
>>>  delete mode 100644 Documentation/devicetree/bindings/i2c/qcom,i2c-qup.txt
>>>  create mode 100644 Documentation/devicetree/bindings/i2c/qcom,i2c-qup.yaml
>>
>> This one is already done.
> 
> Yeah, I did not check linux-next before doing the conversion.
> Sorry for the noise.

Hi Robert,

Thanks for the patch. Several people are working on the bindings, so the
best is to:
1. rebase always on linux-next,
2. check if anyone else posted the patches thus the work is in progress
- use dfn keyword, like:
https://lore.kernel.org/all/?q=dfn%3Aqcom%2Ci2c-qup.txt


Best regards,
Krzysztof