Re: [PATCH v4 2/3] i2c: tegra: make reset an optional property

[Akhil R <akhilrajeev@nvidia.com>]

On Thu, 12 Jun 2025 21:43:56 +0300, Andy Shevchenko wrote:

>> >> >     if (handle)
>> >> >             err = acpi_evaluate_object(handle, "_RST", NULL, NULL);
>> >> > -   else
>> >> > +   else if (i2c_dev->rst)
>> >> >             err = reset_control_reset(i2c_dev->rst);
>> >> > +   else
>> >> > +           err = tegra_i2c_master_reset(i2c_dev);
>> >>
>> >> Can you please take a look here? Should the reset happen in ACPI?
>> >
>> > This is a good question. Without seeing all the implementations of _RST method
>> > for the platforms based on this SoC it's hard to say. Ideally the _RST (which
>> > is called above) must handle it properly, but firmwares have bugs...
>> >
>> > TL;DR: I think the approach is correct, and if any bug in ACPI will be found,
>> > the workaround (quirk) needs to be added here later on.
>> 
>> As in Thierry's comment, I was in thought of updating the code as below.
>> Does it make sense or would it be better keep what it is there now?
>> 
>> if (handle && acpi_has_method(handle, "_RST"))
>> 	err = acpi_evaluate_object(handle, "_RST", NULL, NULL);
>> else if (i2c_dev->rst)
>> 	err = reset_control_reset(i2c_dev->rst);
>> else
>> 	err = tegra_i2c_master_reset(i2c_dev);
>
> This will change current behaviour for the ACPI based platforms that do not
> have an _RST method. At bare minumum this has to be elaborated in the commit
> message with an explanation why it's not a probnlem.
>

This sequence is hit only at boot and on any error. It should be good to reset
the controller internally at least for those cases. We are reconfiguring the I2C
anyway after this and hence should not cause any problem.
Will add these in the commit message as well.

Thanks & Regards,
Akhil