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