Quoting Tomi Valkeinen (2022-11-17 15:49:36) > On 17/11/2022 17:46, Kieran Bingham wrote: > >> + if (ret) > >> + return ret; > >> + > >> + if (setup_info->hsfreq <= MHZ(1000)) > >> + ret = WRITE_PHTW(0x01020100, 0x01910170, 0x01020171, > >> + 0x01110172); > > The <=1Gbps reads: > > > > <=1Gbps > > > > Set PHTW=H’01020100 > > Set PHTW=H’01010172 > > Set PHTW=H’01570170 > > Set PHTW=H’01060171 > > Set PHTW=H’01110172 > > > > Is it clear why theres a difference here? > > What doc is that? Mine (R19UH0172EJ0054 Rev.0.54) says: Hrm ... mine is R19UH0172EJ0051 ... So I guess I'm on an outdated datasheet. Which leaves me pondering if I should just wave my hand over my eyes and add a tag ... (Or see if I can dig up the later datasheet) ;-) -- Kieran > > Set PHTW=H’01020100 > Set PHTW=H’01910170 > Set PHTW=H’01020171 > Set PHTW=H’01110172 > > Tomi >