On Wed, May 10, 2017 at 3:59 PM, Bartlomiej Zolnierkiewicz <b.zolnierkie@xxxxxxxxxxx> wrote: > On Monday, May 08, 2017 10:26:49 PM Linus Walleij wrote: >> On Mon, May 8, 2017 at 12:47 PM, Bartlomiej Zolnierkiewicz >> <b.zolnierkie@xxxxxxxxxxx> wrote: >> So depending on whether they use an FPGA or an ASIC the values are >> different, no matter what frequency (50 or 66 MHz) is used. So it is not >> derived from frequency. >> >> So I think it makes most sense to have it in the device tree as we don't >> know what designs are out there. > > I still would prefer to keep timing values private to the driver and just > select the controller type (FPGA/ASIC) using the device tree. OK I'll revert to static timings in the driver, no problem. Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html