[RESEND PATCH 2/4] i2c: i2c-bcm-kona: Add support for high-speed mode

Tim Kryger tim.kryger at linaro.org
Thu Nov 7 14:57:57 EST 2013


On Fri, Nov 1, 2013 at 6:08 AM, Wolfram Sang <wsa at the-dreams.de> wrote:
> On Wed, Oct 16, 2013 at 03:01:47PM -0700, Tim Kryger wrote:
>> Add support for I2C high-speed mode (3.4 MHz).
>>
>> Signed-off-by: Tim Kryger <tim.kryger at linaro.org>
>> Reviewed-by: Matt Porter <matt.porter at linaro.org>
>> Reviewed-by: Markus Mayer <markus.mayer at linaro.org>
>
> Mostly good as well:
>
>> +enum hs_bus_speed_index {
>> +     BCM_SPD_3P4MHZ = 0,
>> +};
>
> I just realized that you should update both patches to add the supported
> bus speeds to the binding documentation.

While the bus speed property is really a configuration parameter (and
not a true description of of the hardware) it seems improper to put
driver specific details into the binding document.

Please let me know how you would like me to proceed.

I should mention that I plan to break out the binding into its own
patch as this seems to be the latest best practice just in case that
influences your decision.

Thanks,
Tim Kryger



More information about the linux-arm-kernel mailing list