From: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
To: Frank Li <Frank.Li@nxp.com>
Cc: bhelgaas@google.com, conor+dt@kernel.org,
devicetree@vger.kernel.org, festevam@gmail.com,
helgaas@kernel.org, hongxing.zhu@nxp.com, imx@lists.linux.dev,
kernel@pengutronix.de, krzysztof.kozlowski+dt@linaro.org,
kw@linux.com, l.stach@pengutronix.de,
linux-arm-kernel@lists.infradead.org, linux-imx@nxp.com,
linux-kernel@vger.kernel.org, linux-pci@vger.kernel.org,
lpieralisi@kernel.org, manivannan.sadhasivam@linaro.org,
robh@kernel.org, s.hauer@pengutronix.de, shawnguo@kernel.org
Subject: Re: [PATCH v6 09/16] dt-bindings: imx6q-pcie: Clean up irrationality clocks check
Date: Mon, 25 Dec 2023 20:09:51 +0100 [thread overview]
Message-ID: <cf5fd42f-6ca8-4109-a314-3ede9810a5ed@linaro.org> (raw)
In-Reply-To: <63d93850-e88b-4311-accb-e7fa9a89d44f@linaro.org>
On 25/12/2023 20:05, Krzysztof Kozlowski wrote:
> On 24/12/2023 19:32, Frank Li wrote:
>> There are clocks and clock-names restriction for difference compatible
>> string. So needn't irrationality check again for clock's miniItems and
>> maxItems.
>>
>> In fsl,imx6q-pcie-ep.yaml
>
> It is enough to say that bindings referencing this file already define
> these constraints for each of the variants, so the if:else: is redundant.
>
> Two lines instead of ~20.
>
>> ...
>>
>> Acked-by: Rob Herring <robh@kernel.org>
>
> How did you get it? Please point me to the lore link. This patch
> appeared in v5 and there was no response from Rob, AFAIK.
Ah, this patch was added at v4 and Rob acked that one. It's fine then,
but you still could narrow the commit msg, if there is going to be a
resend. Please write informative but concise messages.
Best regards,
Krzysztof
next prev parent reply other threads:[~2023-12-25 19:09 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-12-24 18:32 [PATCH v6 00/16] PCI: imx6: Clean up and add imx95 pci support Frank Li
2023-12-24 18:32 ` [PATCH v6 01/16] PCI: imx6: Simplify clock handling by using bulk_clk_*() function Frank Li
2023-12-24 18:32 ` [PATCH v6 02/16] PCI: imx6: Simplify phy handling by using by using IMX6_PCIE_FLAG_HAS_PHY Frank Li
2023-12-24 18:32 ` [PATCH v6 03/16] PCI: imx6: Simplify reset handling by using by using *_FLAG_HAS_*_RESET Frank Li
2023-12-24 18:32 ` [PATCH v6 04/16] dt-bindings: imx6q-pcie: Add linux,pci-domain as required for iMX8MQ Frank Li
2023-12-24 18:32 ` [PATCH v6 05/16] PCI: imx6: Using "linux,pci-domain" as slot ID Frank Li
2023-12-24 18:32 ` [PATCH v6 06/16] PCI: imx6: Simplify ltssm_enable() by using ltssm_off and ltssm_mask Frank Li
2023-12-24 18:32 ` [PATCH v6 07/16] PCI: imx6: Simplify configure_type() by using mode_off and mode_mask Frank Li
2023-12-24 18:32 ` [PATCH v6 08/16] PCI: imx6: Simplify switch-case logic by involve init_phy callback Frank Li
2023-12-24 18:32 ` [PATCH v6 09/16] dt-bindings: imx6q-pcie: Clean up irrationality clocks check Frank Li
2023-12-25 19:05 ` Krzysztof Kozlowski
2023-12-25 19:09 ` Krzysztof Kozlowski [this message]
2023-12-24 18:32 ` [PATCH v6 10/16] dt-bindings: imx6q-pcie: restruct reg and reg-name Frank Li
2023-12-25 19:11 ` Krzysztof Kozlowski
2023-12-24 18:32 ` [PATCH v6 11/16] dt-bindings: imx6q-pcie: Add imx95 pcie compatible string Frank Li
2023-12-25 19:13 ` Krzysztof Kozlowski
2023-12-24 18:32 ` [PATCH v6 12/16] PCI: imx6: Add iMX95 PCIe support Frank Li
2023-12-24 18:32 ` [PATCH v6 13/16] PCI: imx6: Clean up get addr_space code Frank Li
2023-12-24 18:32 ` [PATCH v6 14/16] PCI: imx6: Add epc_features in imx6_pcie_drvdata Frank Li
2023-12-24 18:32 ` [PATCH v6 15/16] dt-bindings: imx6q-pcie: Add iMX95 pcie endpoint compatible string Frank Li
2023-12-25 19:16 ` Krzysztof Kozlowski
2023-12-26 16:12 ` Frank Li
2023-12-26 19:01 ` Krzysztof Kozlowski
2023-12-26 21:53 ` Frank Li
2023-12-27 12:00 ` Krzysztof Kozlowski
2023-12-24 18:32 ` [PATCH v6 16/16] PCI: imx6: Add iMX95 Endpoint (EP) function support Frank Li
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=cf5fd42f-6ca8-4109-a314-3ede9810a5ed@linaro.org \
--to=krzysztof.kozlowski@linaro.org \
--cc=Frank.Li@nxp.com \
--cc=bhelgaas@google.com \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=festevam@gmail.com \
--cc=helgaas@kernel.org \
--cc=hongxing.zhu@nxp.com \
--cc=imx@lists.linux.dev \
--cc=kernel@pengutronix.de \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=kw@linux.com \
--cc=l.stach@pengutronix.de \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-imx@nxp.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pci@vger.kernel.org \
--cc=lpieralisi@kernel.org \
--cc=manivannan.sadhasivam@linaro.org \
--cc=robh@kernel.org \
--cc=s.hauer@pengutronix.de \
--cc=shawnguo@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).