Hi, Just a quick ping to follow up on this patch, if this is OK to merge or if anything else is still needed.
Best, Juuso ________________________________ From: Stefan Roese <[email protected]> Sent: Friday, 8 May 2026 14.49 To: Juuso Rinta (Nokia) <[email protected]>; [email protected] <[email protected]> Cc: Tom Rini <[email protected]>; Bharat Bhushan <[email protected]>; Aaro Koskinen (Nokia) <[email protected]> Subject: Re: [PATCH RESEND] watchdog: octeontx_wdt: fix DT matches to Marvell compatibles [Some people who received this message don't often get email from [email protected]. Learn why this is important at https://aka.ms/LearnAboutSenderIdentification ] CAUTION: This is an external email. Please be very careful when clicking links or opening attachments. See the URL nok.it/ext for additional information. On 5/4/26 11:36, Juuso Rinta wrote: > The OcteonTX watchdog driver currently matches arm,sbsa-gwdt. > On systems with multiple watchdog devices this can be ambiguous since > arm,sbsa-gwdt is a generic SBSA binding. > > Replace the SBSA match with SoC-specific Marvell compatibles > marvell,cn10624-wdt and marvell,cn9670-wdt. > > These compatibles align with the upstream Linux device tree binding: > https://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgit.kernel.org%2Fpub%2Fscm%2Flinux%2Fkernel%2Fgit%2Ftorvalds%2Flinux.git%2Ftree%2FDocumentation%2Fdevicetree%2Fbindings%2Fwatchdog%2Fmarvell%2Ccn10624-wdt.yaml&data=05%7C02%7Cjuuso.rinta%40nokia.com%7Cdc8481225a664c6ec61308deacf7e7c2%7C5d4717519675428d917b70f44f9630b0%7C0%7C0%7C639138377910570533%7CUnknown%7CTWFpbGZsb3d8eyJFbXB0eU1hcGkiOnRydWUsIlYiOiIwLjAuMDAwMCIsIlAiOiJXaW4zMiIsIkFOIjoiTWFpbCIsIldUIjoyfQ%3D%3D%7C0%7C%7C%7C&sdata=IV4hBdwPl%2BFe43cjTR28jLpdmPpZt2BX%2B8XyidREgpA%3D&reserved=0<https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/devicetree/bindings/watchdog/marvell,cn10624-wdt.yaml> > > Reviewed-by: Aaro Koskinen <[email protected]> > Signed-off-by: Juuso Rinta <[email protected]> Reviewed-by: Stefan Roese <[email protected]> Thanks, Stefan > --- > drivers/watchdog/octeontx_wdt.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/watchdog/octeontx_wdt.c b/drivers/watchdog/octeontx_wdt.c > index c79d9539c13..7299a9f9739 100644 > --- a/drivers/watchdog/octeontx_wdt.c > +++ b/drivers/watchdog/octeontx_wdt.c > @@ -159,7 +159,8 @@ static const struct octeontx_wdt_data octeon_data = { > }; > > static const struct udevice_id octeontx_wdt_ids[] = { > - { .compatible = "arm,sbsa-gwdt", .data = (ulong)&octeontx_data }, > + { .compatible = "marvell,cn10624-wdt", .data = (ulong)&octeontx_data }, > + { .compatible = "marvell,cn9670-wdt", .data = (ulong)&octeontx_data }, > { .compatible = "cavium,octeon-7890-ciu3", .data = (ulong)&octeon_data > }, > {} > }; > > --- > base-commit: 4433253ecf2041f9362a763bb6cb79960921ac7e > change-id: 20260428-rinta-octeontx-wdt-compatible-0ca7b83e634e > > Best regards, > -- > Juuso Rinta <[email protected]>
