On 06.12.2021 20:43, Florian Fainelli wrote:
On 12/6/21 11:37 AM, Guenter Roeck wrote:
On 12/6/21 11:13 AM, Florian Fainelli wrote:
There is simply no rush in getting the bcm7038-wdt driver to support
4908 *just now*, so why don't you just take the bcm63xx-wdt series that
I posed, and Rafal posts an updated series that adds support for the
4908 watchdog for the 5.18 cycle?
Your series includes the patch discussed here, and it is the first patch
of your series. The second patch in your series depends on it. Are you
telling me that I should drop those two patches from your series ?
No, quite the contrary, I want you to keep the entire 7 patches that
converted the bcm7038-wdt binding to YAML and get rid of the bcm63xx-wdt
changes, the branch that you have right now is good in that regard.
I don't see why you should be creating an immutable branch for Lee and
not simply merge Rafal's "[PATCH V4 RESEND 2/2] dt-bindings: mfd: add
Broadcom's Timer-Watchdog block" patch with Lee's ack directly. This is
a new file, so I don't see how it would create conflicts as long as we
don't pile up changes on top.
I'd like that pretty much as I wouldn't need to wait few extra months.
Lee: would that be OK for you to simply ack 2/2? So Guenter can pick my
patch without the whole immutable branch & PR thing?
