On Friday 15 November 2013, Loc Ho wrote: > >> +- CTLE0 : PHY override parameters for channel 0 register REG1 > >> + field CTLE_EQ. First value for Gen1, second value > >> + for Gen2, and third value for Gen3. Default is 0x2. > >> +- CTLE1 : PHY override parameters for channel 1 register REG1 > >> + field CTLE_EQ. First value for Gen1, second value > >> + for Gen2, and third value for Gen3. Default is 0x2. > >> +- PQ0 : PHY override parameters for channel 0 register REG125 > >> + field PQ_REG. First value for Gen1, second value > >> + for Gen2, and third value for Gen3. Default is 0xA. > >> +- PQ1 : PHY override parameters for channel 1 register REG125 > >> + field PQ_REG. First value for Gen1, second value > >> + for Gen2, and third value for Gen3. Default is 0xA. > > > > As mentioned before, I don't think putting register-level information into the binding > > is the right approach here. > > > [Loc Ho] > I don't want to change the driver for every possible customer or APM > boards out there. In general, if the board designer follows the board > design guideline, this isn't necessary. Unfortunately, I can not > control what customer do. If these setting is NOT driven by DTS, then > I or others may have to change the driver every time there is an new > board. Another option is pass them as module parameters. Module parameters are certainly not a solution for this, that would make the problem worse. Would it be possible to read out the values that were set by the firmware at startup and reuse those during run-time? I assume that the devices are already brought up when the kernel starts. If that s not possible, change the properties to be named after what the setting does and make the values be units that are sensible to a board designer, e.g. (specifics are totally made up here) - polarity: If present, override the polarity of the signal line. zero for negative polarity, one for positive polarity. If absent, use the firmware default. - signal-clock-frequency: If present, override the signal clock frequency. This value is defined in HZ - x-y-time: If present, override the timing setting for the X/Y signal time, in nanoseconds. This way, you can reuse the properties for an updated version of the device that has similar settings but a different register layout. Arnd -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
