lkml.org 
[lkml]   [2019]   [Aug]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    SubjectRe: [PATCH v2 1/4] i2c/busses: Add i2c-icy for I2C on m68k/Amiga
    From
    Date
    Hi Wolfram,

    Thanks for your input!
    Replies below.


    On 08/14/2019 09:47 PM, Wolfram Sang wrote:
    >> +static int clock = 0x1c;
    >> +module_param(clock, int, 0444);
    >
    > 'clock' determines the bus speed?

    Yes, but it also determines the frequency of the oscillator driving the PCF8584. It doesn't usually need to be touched, the current setting drives at the maximum I2C bus speed, with the maximum oscillator (12 MHz).
    I kept it in because i2c-elektor also exposes it in the same way, and it seems useful.


    >> + i2c->adapter.class = I2C_CLASS_DEPRECATED;
    >
    > This is only needed for drivers which used to have a class and decided
    > to drop it. You can leave it empty.

    Thanks, I'll drop it.


    >> + algo_data->data = (void *)i2c;
    >
    > You don't need the cast, do you?

    True. Dropped, thanks.


    >> +MODULE_LICENSE("GPL");
    >
    > Your SPDX header says GPL 2.0 only.

    Thanks! I'll change it to "GPL v2", but unfortunately there is no option for "only" in include/linux/module.h.


    Cheers

    Max

    \
     
     \ /
      Last update: 2019-08-15 00:35    [W:4.305 / U:0.212 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site