On 19/12/18 19:18, Tony Lindgren wrote: > * Mark Brown <broonie@xxxxxxxxxx> [181219 17:07]: >> On Wed, Dec 19, 2018 at 04:48:52PM +0100, David Lechner wrote: >>> On 12/19/18 4:43 PM, Roger Quadros wrote: >> >>>> Did I do something wrong or we just need to enhance regmap_debugfs.c? >> >>> Do you assign the name field in pru_regmap_config and >>> pru_debug_regmap_config? >> >> Yeah, you'll at least need to override the name since the default is to >> use the name of the device and that'll result in two duplicates. I hadn't set the name field and this was the issue. It is resolved now. Thanks :) > > Also combining IO regions often has a serious issue that the > separate IO regions may not be in the same device (or same > interconnect target module). > > This means that flushing a posted write with a read-back for > an IO region will only flush it for a single device and not > the other(s). And this leads into hard to find bugs :) Agree. cheers, -roger -- Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki
