From mboxrd@z Thu Jan 1 00:00:00 1970 From: Simon Glass Date: Thu, 14 Feb 2019 17:24:38 +0100 Subject: [U-Boot] [U-Boot, v4, 1/1] avb: add support for named persistent values In-Reply-To: References: <1548599645-9252-1-git-send-email-igor.opaniuk@linaro.org> <20190209125037.GF16814@bill-the-cat> <20190211153028.GF16814@bill-the-cat> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Igor, On Thu, 14 Feb 2019 at 17:19, Igor Opaniuk wrote: > > Hi Simon, > > I've fixed failing test on sandbox_flattree by keeping all defined > global variables in a driver-private data struct(as you suggested), > which > permits to probe multiple tee sandbox devices (sent v5, kept you R-b > tag, please let me know if there are any objections from your side). > > But the way how dm test framework is invoked and why dm_root is simply > re-allocated (and what happens with the previous one) is still an open > question to me. > If you can confirm that this a bug, I'll try to think about how this > can be fixed. I don't think it is a bug, actually. When the dm_root goes away the devices go away too since they are no-longer accessible. Of course someone might still have a pointer to a device that was active before the root went away. But I don't think we should worry about devices that are implemented with static data. That is not how things are supposed to be implemented. Regards, Simon