Re: [IGNORE][PATCH v4 01/11] dt-bindings: Document common device controller bindings

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





On 8/11/23 14:00, Gatien CHEVALLIER wrote:


On 8/11/23 12:16, Greg KH wrote:
On Fri, Aug 11, 2023 at 12:07:21PM +0200, Gatien Chevallier wrote:
From: Oleksii Moisieiev <Oleksii_Moisieiev@xxxxxxxx>

Introducing of the common device controller bindings for the controller
provider and consumer devices. Those bindings are intended to allow
divided system on chip into multiple domains, that can be used to
configure hardware permissions.

Signed-off-by: Oleksii Moisieiev <oleksii_moisieiev@xxxxxxxx>
[Gatien: Fix typos and YAML error]
Signed-off-by: Gatien Chevallier <gatien.chevallier@xxxxxxxxxxx>
---

Changes in V4:
    Corrected typos and YAML errors

Why are we supposed to ignore the first patch in this series, but pay
attention to the 10 after this that depend on it?

totally confused,

greg k-h

Hello Greg,

I'm sorry that this tag troubles your review. It was first suggested
in [1]. The "IGNORE" means ignore review on this thread, as it is still
under review in another thread (Link in the cover letter). It does not
mean that the content should be ignored for the series. I will change
this to something else as this is obviously confusing the review.

@Oleksii, can we imagine integrating this patch to this series or do
you prefer to keep it apart?


Hi,

after a discussion with Oleksii: I'll rename the binding to narrow
down its scope and integrate the patch in my series. This way, I'll drop
the [IGNORE] tag.

=> I'll stick with the generic binding for V5 (Sorry for the misleading
   information in the previous mail)

Best regards,
Gatien

Should I consider a resend with another tag if Oleksii prefers to keep
this patch apart?

[1] https://lore.kernel.org/all/1e498b93-d3bd-bd12-e991-e3f4bedf632d@xxxxxxxxxx/

Best regards,
Gatien



[Index of Archives]     [Linux GPIO]     [Linux SPI]     [Linux Hardward Monitoring]     [LM Sensors]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux