-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Sayma RTM - version 1 errata & tests #52
Comments
@gkasprow good catch. Are you referring to IC6 and IC7 on the AFE test board? Out of curiosity, would it be easier/more robust to fix this by adding Schottky diodes on the LDO outputs to prevent reverse-bias-induced lockup? This is the standard trick I've used on this kind of dual supply board. |
@hartytp Yes, IC6 and IC7. I'll add diodes and let you know it this solves that issue. |
The BAS281 diode helped. Thanks @hartytp ! |
@gkasprow Great! Glad to hear that fixed it. I've seen quite a few subtle power sequencing problems in dual supply designs in the past -- particularly when the supplies are loaded asymmetrically. But, IME adding diodes always fixes it, so these days I always add them (diodes are cheaper than thinking/debugging time ;) ). We should be aware of this as a potential issue for other Sinara boards and add diodes where appropriate. T |
@gkasprow Regarding discussion of "lockup of analog frontend amplifier" related to the LDO IC6 (ADP7102) on AFE test board. Good catch! Related question... It looks like the LDO has a EN/UVLO pin for delayed startup and user-defined hysteresis intended for sequencing multiple power supplies. Is there a reason this feature wasn't used? |
Great!
I'm not sure I follow you here. Do you mean that when you load all AFE test mezzanines at 50% on the +12V supply (100mA per AFE) the +12V rail is at 11.85V, and when you load all AFE mezzanines with 100% on the +12V supply rail (200mA per AFE) the +12V rail is at +11.99V? |
@jbqubit I don't think that the ADP7102 EN/UVLO will help here. The issue is that the negative supply switches on faster than the positive supply and ends up reverse biasing the positive supply rail (through biasing circuitry/ESD diodes/etc in the ADA4927-1 OpAmp). This causes the regulator to go into lockup. Using the EN/UVLO to delay the positive regulator's startup further will make the problem worse. This is a pretty common issue in this kind of design, and adding diodes to prevent transient reverse bias of the regulators is the standard fix. |
@gkasprow One other question while I'm thinking about RTM power supplies: at the moment, the +-12V supplies for the AFEs come from a switching converter on Sayma RTM, with some LC filtering. The +12V supply is then used on Allaki to bias the pre-amp (and, less importantly, to supply the InAmp). Is that correct? If so, do you think that the noise on this rail will be good enough for the pre-amp if we're aiming for "ultra-low" phase/amplitude noise? We probably talked about this before, so apologies if I've forgotten what was agreed! |
@hartytp I've written that it is below 50mV, which also means close to zero :) 6V rail: -6V rail: -12V rail: 3.3V rail: |
@gkasprow Nice! Thanks for clarifying. Glad to hear that's all working! Out of curiosity, what happens when all rails are at 100% load? |
@hartytp |
@hartytp when I load all rails with 100% , I exceed twice the power foreseen for AMC+RTM boards. |
@gkasprow Thanks for the info. Fluctuations in this voltage rail directly modulate the amp's bias current, so my suspicion is that using the DC/DC output directly will degrade the noise performance somewhat. If we're worried about this/if it turns out to be a problem, we can always add an extra LDO onto Allaki. Maybe @dhslichter has thoughts? In any case, it looks like the RTM works perfectly -- this is something to think about when we do our design review for Allaki AFE.
Fair enough, good point. |
It's not terribly sensitive. At the low frequency (for bias current/power supply fluctuations) end, you get about 0.005 dB more gain for 1% more bias current. |
@jordens True. Depends on what noise floor you're trying to reach though. Anyway, this isn't something I'm particularly worried about as it's easy to test and easy to fix if we do decide it's problematic. |
That's assuming it's linear.... the delta is 0.12 dB for a 20% bias step. |
@hartytp said
Thanks for the explanation. Inclusion of diodes seems like a general solution. I've updated the wiki PCBReviewChecklist to reflect this practice. |
I thought we had an LDO on Allaki for the output amplifier supply -- if not, it would be good to add for the next iteration. The gain of the output amp is basically linear in supply current, so fractional voltage noise is turned directly into fractional gain noise. There is a choke and a capacitor (or at least, there should be) on the supply line which provides some filtering at higher frequencies, but I think an LDO is definitely a good idea here. |
@dhslichter I added the LDO to Allaki schematic review check list. #110 |
@gkasprow Great! |
I'm fighting with HMC7043 SPI. |
Hmm, not sure. Some ideas:
|
@hartytp good catch! |
@hartytp |
The joys of undocumented diagnostic modes... |
Another bizzare thing
So one must make sure that HMC830 SEN goes high first and in our case where clock line is shared between both HMC chips it is critical. |
@enjoy-digital you delivered me 2 bitstreams: one that is supposed to set logic low on SEL output but in fact it sets it randomly. On some boards including yours it is L, but in 3 of mine it is H, while on rest of them it is L. Second bit stream toggles this pin correctly (I tested on problematic board), and I see it on the scope. So it means the hardware is OK, but for some reason your FPGA bitstream (sayma_rtm.bit) disables the clock for HMC chip and I cannot test boards that want to send to UMD. |
@gkasprow: ok, I'll connect that to a register then and see if it's better when controlled by the software. |
@gkasprow: i added that: Here is the bitstream with updated csr.csv (untested): |
@enjoy-digital for some strange reason it works properly with your recent fix. I don't even need to access this register from Python. I replaced the csv file but is says name 'wb' is not defined. |
@gkasprow: maybe another vivado bug... (the difference is that now it has to drive the pin from a register and was maybe doing buggy simplification before??) |
@enjoy-digital I had such issue in the past that compiler synthesized away pins routed to GND/VCC. But in this case the pin was shorten to VCC internally and on some boards to GND. So this may be a bug. Can you check in RTL map what Vivado did ? |
|
@jbqubit please add these issues to the list on top. Whey will get lost in the thread :) |
@gkasprow top post now updated :) |
@gkasprow It would be good to get a price breakdown for Sayma AMC and RTM at some point. What drives the cost of these boards? What can we do to make them a bit cheaper? IME the cost per channel is one of the biggest factors in user adoption! |
@hartytp at the moment we have batch of 10 boards produced. So substantial cost is tooling, FR408 material, connectors. All is expensive because for many components we don't break the first price step. |
Thanks for the info.
|
@jordens If I recall correctly, you felt that Sayma was much more expensive than it should be. Other than getting the cost of the mezzanines right down, do you have any concrete proposals for ways we can save money? There are a few minor things we can do like shaving a mux or two out of the clock tree, but those are barely a rounding error on the overall cost AFAICT... |
I am no expert. The advice to reduce cost should come from those calculating the total. To me the additional runs for AFE and clock mezzanines plus the connector space, cost, risk, time issues are significant. And yes: reduce options that nobody is paying for. You seem to be implying that "a mux or two" is just the cost of s single part. The cost for designing, reviewing, discussing, debugging, tweaking, supporting, coding, and assembling that stuff is orders of magnitude more. E.g. to reduce cost you may want to kill the RF backplane connector and that part of the clocking tree for those who are not using it. |
@jordens these backplane connectors are expensive. We can always mark them as DNP |
@gkasprow can you quantify "expensive" here for the backplane connectors? Of the silicon cost, the ADCs are the dominant contribution IIRC. Certainly the SMP connectors are substantial as well. If people wanted output-only they could have some of these DNP but then there is additional cost associated with the assembly variant because you're not doing it in as big a run. The Sayma boards will need a rather sophisticated functional testing suite after assembly -- what does that contribute to the cost, @gkasprow ? |
@jordens @hartytp IMHO the cost of separate mezzanines is worth it due to the risk mitigation/customizability aspect. However, if there is low-hanging fruit to reduce those costs (e.g. cheaper substrate for Allaki, cheaper board-to-board connectors instead of SMP), I'm certainly in favor, as long as we have tested it to make sure it will work RF-wise for the former and mechanically for the latter. |
Maybe the thing to do is for @gkasprow to produce a list of the cost of the components on Sayma AMC+RTM assuming a sensible batch size like 25. Cost should mean the total cost of getting the components onto the PCB, so both component cost but also assembly etc. Then we can make more of a decision about this.
True, but those are largely one-off costs. While that could have been more of a consideration at the design stage, I'm more concerned with the recurring per-board costs at the moment.
We can take a survey and see how many people plan to use it. If the fraction of users wanting the backplane is small then we can DNP it. However, as @dhslichter points out, one shouldn't underestimate the cost/effort involved in producing and supporting multiple design variations. IME it's generally better to pay a little more and have a single well-supported version (e.g. this makes is far easier for someone to keep them in stock, which is great since the lead time is pretty long).
That's exactly my feeling. These mezzanines are a good thing for a lot of reasons. We just have to make sure that the cost isn't excessive and that we fix the mechanical issues. |
Once M-Labs have Sayma up and running, our top priority should be testing Sayma v1.0, before moving onto v2.0 production. Ideally, I'd like to send v2.0 off to production by the end of January. One thing I really want to avoid is delays due to design discussions (e.g. which connector we should use etc). So, I'd like to get that all out of the way asap, and ideally before we begin testing. Edit: once we begin testing, the only discussions we have should be related to issues that emerge during testing, not longstanding ones. To that end, I've lumped all Sayma-related issues in the v2.0 milestone (some can be moved to a different milestone later but it's nice to have everything related to Sayma in one place for now). Please have a look at these issues and post any comments/open new issues where necessary. I'll assume that anyone who doesn't contribute to those issues over the next couple of weeks is happy with the plans for Sayma v2.0! @jordens @sbourdeauducq IIRC, you had some issues with: the current IPMI/MMC; the Ethernet PHY; the JTAG. I know that some of that has already been posted on the issues, but what I haven't seen yet (apologies if I've missed it) is a clear list of changes you'd like for v2.0. Is there anything we can change to make your life easier on those fronts? IMHO, since you guys are the ones dealing with these hardware components, you should have say in how they're designed. |
@hartytp We have raised lots of issues. Those plus general simplification (i.e. reducing the design). But please file an issue for a Sayma 2.0 system design proposal/change list and assign it to us. |
Done. |
Never mind, this is intended. I was looking at the wrong photo of what the (manually fixed) boards should look like. Should look like this: https://github.com/m-labs/sinara/blob/master/Drawings/Sayma_RTM_rev1.0_front.jpg so this is all as intended for this revision. |
@gkasprow Can you remind me what temp co we used for the DAC bias current resistors? Did we make sure that these are relatively low temp co to avoid degrading the DAC's performance? |
It's 0.1% 4k02 10ppm R0603_4K02_0.1%_0.063W_10PPM GENERIC |
Perfect. |
|
moving mechanical issues to #46 and closing |
Mechanical:
Silkscreen:
BOM:
I2C:
Power supply:
Annotations:
JTAG:
Clocks:
Add negator to one of them or tie HMC RST to GNDgive HMC7043 its own dedicated reset line to avoid noise issues during initialization.DAC:
Test points:
Other:
The text was updated successfully, but these errors were encountered: