On 23/11/2020 00:35, Jonathan Cameron wrote:
Having a software interface to assert the reset would be nice.
If there's no comments against the change, I'll submit a patch in the next day or so.
Hi Phil,
So I've fixed the binding example up to have the sense as ACTIVE_LOW as discussed.
If you have time to flip the sense round in the driver to match that it
would be great and I'd propose we then mark that as suitable for stable
so that we get it backported into existing trees. We could also spin
a patch to the txt binding but perhaps that is more effort than is needed
here. If I get time before you do I might send a proposed fix patch.
Hopefully anyone then using it will get the sense right and we won't
accidentally break anyone.
At some point soon I'll also try and audit similar cases and hopefully we
will get those fixed up as well. Obviously if anyone else wants to take
a look at that it would be much appreciated!
G'day Jonathan,
Sorry this had slipped a bit on my list.
I'll send a patch in next day or so.
--
Regards
Phil Reid