Re: [PATCH v2 1/3] nvmem: core: introduce cells parser

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





On 20/09/2021 11:24, Vadym Kochan wrote:

Probably this might be fixed by lookup nvmem device too ?
Honestly, this approach is a total hack to get it working.

This is what Am thinking which should look like:

## DT ##
eeprom_at24: at24@56 {
	compatible = "atmel,24c32";
	/* Some way to identify that this is a TLV data */
	nvmem-cell-parser-name = "onie-tlv-cells";
	reg = <0x56>;

	mac_address: mac-address {
		/* THIS REG is updated once TLV is parsed */
		reg = <0 0x6>
	};
I assume these cell nodes should be marked with some property that they
should be evaluated later, so the cell will be not read
in case it was not parsed because it may
exist in nvmem device optionally.
No, we should hit that use case to start with.

nvmem cells are parsed in the core during register path.
Parser should parse the cells before this to keep it simple.


Or, treat cells with length "0" in a special way and allow to update
cell info later.you can update irrespective of the length, as long as this is done
before register.



};

some_dev_node {
	compatible = "xxx";
	nvmem-cells = <&mac_address>;
	nvmem-cell-names = "mac-address";
};

== CODE ==
ret = of_get_mac_address(dev->of_node, base_mac);
==========


If you notice the mac_address node reg is 0.
This node "reg" property should be updated ( using of_update_property())
by nvmem-provider driver while tlv parsing and by matching the node name
with tlv name.

I assume parser driver can just invoke add_cell_table (with may be
by adding 'bool update' field to the cell_info struct) and core.c will just
update existing cells parsed from OF.


Lets keep the core code clean for now, I would expect the provider driver along with parser function to do update the nodes.

--srini

That way rest of the code will work as usual.

If this work for you then we can ask Rob if he foresee any issues in
this approach. I already see similar usage in reserved-memory case.



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux