On 4 April 2018 at 04:45, Neil Armstrong <narmstr...@baylibre.com> wrote:
> On 03/04/2018 19:53, Simon Glass wrote:
>> On 3 April 2018 at 17:40, Neil Armstrong <narmstr...@baylibre.com> wrote:
>>> This patch adds the bulk reset API tests for the sandbox test suite.
>>> Unlike the main test, it also check the "other" reset signal using the bulk
>>> and checks if the resets are correctly asserted/deasserted.
>>> To allow the bulk API to work, and avoid changing the DT, the number of
>>> of the sandbox reset controller has been bumped to 101 for the "other" reset
>>> line to be valid.
>> Does it need to be 101, or would, say, 5 be enough?
> It could stay at 3, but a dts change should be needed since it declares :
> resets = <&resetc 100>, <&resetc 2>;
> reset-names = "other", "test";
Well that is fine if you want to change test.dts - you just need to
make sure you keep the intent.
But you've explained the reason, and it's fine to keep this patch as
is, if you like.
U-Boot mailing list