Hi Marek, On Sun, Apr 17, 2016 at 11:47:23AM +0200, Marek Vasut wrote: > On 04/16/2016 04:55 PM, Nikita Kiryanov wrote: > > This series introduces eeprom layout aware capabilities to the existing > > eeprom > > command, and then enables this feature on Compulab boards. It is an import > > of > > the Compulab eeprom-utility (https://github.com/compulab/eeprom-util), and > > it > > introduces 2 new options to the eeprom command: > > > > eeprom print [-l <layout_version>] bus_num device_addr > > The bus number starts to become a problem, esp. when used with DM and > where the bus number can change between boots.
Are you referring to the fact that the command allows the user to not specify the bus number (relying on a default value instead)? If so, I agree with you that it is problematic. If I had my way I would've made it a mendatory parameter along with the device address, but that will hurt backwards compatibility. I'll be more than happy to redo the last 3 patches if we agree that we are willing to pay that price. > > Best regards, > Marek Vasut > _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
