Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1304521

Re: [PATCH v1 1/3] i2c: rk3x: add calc_divs ops for new version

From "David.Wu" <wdc@rock-chips.com>
Newsgroups linux.kernel
Subject Re: [PATCH v1 1/3] i2c: rk3x: add calc_divs ops for new version
Date 2016-01-08 14:20 +0100
Message-ID <qOF7r-7eQ-9@gated-at.bofh.it> (permalink)
References <qEx1v-5JU-1@gated-at.bofh.it> <qFLOO-56v-27@gated-at.bofh.it> <qHOpc-2qP-17@gated-at.bofh.it> <qNjiH-7Q7-67@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


Hi Wolfram, Andy

Thanks for your suggestion.
I will  use i2c_parse_fw_timings() function to replace it.

在 2016/1/5 3:40, Wolfram Sang 写道:
>>>> +static int rk3x_i2c_v0_calc_divs(unsigned long clk_rate, unsigned long
>>>> scl_rate,
>>>> +                                unsigned long scl_rise_ns,
>>>> +                                unsigned long scl_fall_ns,
>>>> +                                unsigned long sda_fall_ns,
>> Wolfram did some sturct to assign the parameters from device properties.
>> It might be re-used here.
> Yes, I think it makes sense to convert the driver first to use
> the new i2c_parse_fw_timings() function and see if it fits (or if we need
> to extend it perhaps).
>
> David, does this make sense to you?
>

Back to linux.kernel | Previous | NextPrevious in thread | Find similar | Unroll thread


Thread

Re: [PATCH v1 1/3] i2c: rk3x: add calc_divs ops for new version Wolfram Sang <wsa@the-dreams.de> - 2016-01-04 20:50 +0100
  Re: [PATCH v1 1/3] i2c: rk3x: add calc_divs ops for new version "David.Wu" <wdc@rock-chips.com> - 2016-01-08 14:20 +0100

csiph-web