Messages in this thread | | | Subject | Re: [PATCH v3 09/22] i2c: tegra: Clean up probe function | From | Dmitry Osipenko <> | Date | Thu, 3 Sep 2020 18:56:24 +0300 |
| |
03.09.2020 17:02, Andy Shevchenko пишет: > On Thu, Sep 3, 2020 at 4:48 PM Dmitry Osipenko <digetx@gmail.com> wrote: >> 03.09.2020 14:17, Andy Shevchenko пишет: >>> On Thu, Sep 3, 2020 at 3:54 AM Dmitry Osipenko <digetx@gmail.com> wrote: > > ... > >>>> + of_property_read_u32(np, "clock-frequency", &bus_clk_rate); >>>> + i2c_dev->bus_clk_rate = bus_clk_rate; >>> >>> Hmm... I dunno if Wolfram is going to implement a special helper >>> exactly for this. I remember we discussed that with him during v5.8 >>> (?) times. >> >> I now see that there is a i2c_parse_fw_timings() which parses >> "clock-frequency" and other common properties. I could switch to use >> that helper, but not sure whether it would be really worthwhile because >> only one property is needed to be parsed. I'll consider this change for >> v4, thank you for the suggestion! > > That's exactly why I was wondering about the current state of affairs > with the discussed helper which should only parse the clock-frequency > property. >
Alright, I see that many other I2C drivers are in the same position, so perhaps should better to keep the current variant as-is until a common solution will become available.
| |