[PATCH v5 01/12] dt-bindings: PCI: Cleanup of brcmstb YAML and add 7712 SoC
Jim Quinlan
james.quinlan at broadcom.com
Wed Aug 14 10:35:32 PDT 2024
On Tue, Aug 13, 2024 at 4:27 AM Krzysztof Kozlowski <krzk at kernel.org> wrote:
>
> On 13/08/2024 00:07, Jim Quinlan wrote:
> > On Fri, Aug 2, 2024 at 2:43 AM Krzysztof Kozlowski <krzk at kernel.org> wrote:
> >>
> >> On 01/08/2024 00:28, Jim Quinlan wrote:
> >>> o Change order of the compatible strings to be alphabetical
> >>> o Use "maxItems" where needed.
> >>
> >> I asked at v3 and then in v4 about splitting this. You never responded
> >> to that comment, so sorry I won't be repeating the same thing in v5.
> >
> > I'm sorry Krzyszof, but I just reviewed your responses in V3 and V4
> > and I can't find you saying anything about splitting off the above two
> > bullet points. Perhaps I am somehow losing email responses but all I
> > see is this in V3 is the following, where you ask me to do a squash,
> > not a commit:
> >
> > [JQ] o Change order of the compatible strings to be alphabetical
> > []KK] That's a cleanup. You can squash it with a previous patch.
> >
> > Now you did say in V3
> >
> > [JQ] o Describe resets/reset-names before using them in rules
> > [KK] That's a new commit.
> >
> > but this bullet item does not relate to the bullet points you have
> > highlighted in this email. As for your responses to V4, I don't see
>
> It exactly relates to the quoted part. The comment is ALWAYS under exact
> part of your patch being questioned/commented.
>
> So first I said one part is cleanup and should be moved away. Then I
> explained that this part is a NEW COMMIT. New, so one more, different.
>
> I understand that this was not clear, but you never came with a question
> what did I mean.
Hello Krzysztof,
Sorry to bother you again but I want this to be clear. This commit
contains the changes:
1. alphabetical order of compatible strings
2. maxItems
3. maintainer change
Do you want me to split off 3 since it is not a "cleanup"? If not,
please specify.
Regards,
Jim Quinlan
Broadcom STB/CM
>
> Best regards,
> Krzysztof
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 4210 bytes
Desc: S/MIME Cryptographic Signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20240814/6a33c245/attachment.p7s>
More information about the linux-arm-kernel
mailing list