NXP / i3c-slave-design

MIPI I3C Basic v1.0 communication Slave source code in Verilog with BSD license to support use in sensors and other devices.
Other
114 stars 35 forks source link

the difference between PIN_MODEL == `PINM_COMBO and PINM_REG #38

Open wh0xjjx opened 1 year ago

wh0xjjx commented 1 year ago

Hi Paul, could you explain the different between PIN_MODEL == `PINM_COMBO and PINM_REG, i am a little confused, thank you!

pkimelman-nxp commented 1 year ago

Pin combo generates the pad Controls live. Ends up with smaller logic since can be spread over path. But need to be careful in timing closure. Works well with older process but newer/smaller process is too variant often. Registered means the controls are registered a half clock in advance.

Sent from my iPhone

On Dec 23, 2022, at 2:10 AM, wh0xjjx @.***> wrote:



Caution: EXT Email

Hi Paul, could you explain the different between PIN_MODEL == `PINM_COMBO and PINM_REG, i am a little confused, thank you!

— Reply to this email directly, view it on GitHubhttps://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FNXP%2Fi3c-slave-design%2Fissues%2F38&data=05%7C01%7Cpaul.kimelman%40nxp.com%7C6286331da53d4766dec308dae4cde05d%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C638073870123372386%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=zkT1Pu%2B%2Bf7auYCNwcf6cVoY8uKNbRNz8NLIarnDaTto%3D&reserved=0, or unsubscribehttps://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAFNELX5ID6PVWG5BHR6NOD3WOV3AFANCNFSM6AAAAAATHSYHYA&data=05%7C01%7Cpaul.kimelman%40nxp.com%7C6286331da53d4766dec308dae4cde05d%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C638073870123372386%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=DUuvsgYIXb2lq55L5TMNiUsC6AK0AHGO%2FSqeS%2B0S188%3D&reserved=0. You are receiving this because you are subscribed to this thread.Message ID: @.***>

wh0xjjx commented 1 year ago

Thank you for your reply. I test combo and reg . the pin_SDA_out and pin_SDA_oena are almost same. but when state == R9TH, the pin_SDA_oena seems fall down too early(in the posedge of SCL) and the parity value on pin_SDA_out should be high, while it is low, a little wired

wh0xjjx commented 1 year ago

The result is in here! PIN_COMBO PIN_REGS