On Tue, 23 Mar 2021, Mark Brown wrote: > On Tue, Mar 23, 2021 at 05:20:02PM +0000, Lee Jones wrote: > > On Tue, 23 Mar 2021, Mark Brown wrote: > > > > > On Mon, Mar 22, 2021 at 02:59:25PM +0000, Lee Jones wrote: > > > > On Thu, 11 Mar 2021, Mark Brown wrote: > > > > > > If there were more users it'd be better to have this in the core so > > > > > that > > > > > problems we can detect like cache stuff if that's used but if it's > > > > > just > > > > > one broken device it's probably not worth it, this seems like > > > > > something > > > > > you'd have to try to end up with and which is going to cause timeout > > > > > problems with a lot of I2C controllers which would tank performance > > > > > enough that people would notice. > > > > > So Yoda, is this to go into the core, or stay where it is? > > > > Well, nobody's sent me any patches. > > > Code is still in the driver in v4. > > > My question is; should these functions really live in the SS? > > Perhaps we could avoid using that particular abbreviation.
Too soon? > Like I say it depends on how common this is - are we seeing other > devices with the same problem? I'm not witness to any. If this is your first encounter too then maybe just leave it where it is. At least for the moment. -- Lee Jones [李琼斯] Senior Technical Lead - Developer Services Linaro.org │ Open source software for Arm SoCs Follow Linaro: Facebook | Twitter | Blog
