[PATCH 0/2] reset: fix reset_get_by_index_nodev index handling

[Tom Rini]

On Tue, Apr 27, 2021 at 09:37:11AM +0200, Neil Armstrong wrote:
> Hi Tom, Simon,
> 
> On 20/04/2021 10:42, Neil Armstrong wrote:
> > A regression weas detected on Amlogic G12A/G12B SoCs, where HDMI output was disable
> > even when Linux was booting.
> > 
> > Bisect reports 139e4a1cbe ("drivers: reset: Add a managed API to get reset controllers from the DT")
> > as the offending commit.
> > 
> > But the error is in ea9dc35aab ("reset: Get the RESET by index without device") where a spurius "> 0"
> > was added to the index handling.
> > 
> > But the dm_test_reset_base() test did not catch it.
> > 
> > The first commit extends the test to catch the regression, and the second patch fixes the regression.
> 
> So you have any comment on the patch order here ? Should I push the fixed test after the fix ?

I'll grab these shortly, thanks for the reminder.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20210427/f223429e/attachment.sig>