State Framework and dtb

[Çağlar Kilimci]

Hey,

2017-04-04 11:04 GMT+03:00 Jan Remmet <J.Remmet at phytec.de>:
> On Mon, Apr 03, 2017 at 11:59:50PM +0300, Çağlar Kilimci wrote:
>> Hello,
>>
>> 2017-03-31 16:00 GMT+03:00 Sascha Hauer <s.hauer at pengutronix.de>:
>> > On Fri, Mar 31, 2017 at 02:41:19PM +0300, Çağlar Kilimci wrote:
>> >> Hey,
>> >>
>> >>
>> >> I tried but got the same result and then I would like to apply your
>> >> serious patch series of the state framework but release that our
>> >> working branch is 2016.07 so could not apply patches. Let me update
>> >> barebox and apply those patches. Are those based on master branch
>> >> right?  Or, which branch do you recommend to work on?
>> >
>> > The patches are based on master, yes. Anyway, you seem to have a problem
>> > in getting your changes in the dts file to the running barebox. Please
>> > try to add some nodes/properties to your dts file and verify that a
>> > of_dump shows these nodes. Before that is the case it's not worth to
>> > look any further.
>>
>> Finally, I updated and patched the code and now I can see state at 0
>> device in the of_dump:
>>         state at 0 {
>>                 magic = <0x27031977>;
>>                 compatible = "barebox,state";
>>                 backend-type = "raw";
>>                 backend = <0x42>;
>>                 foo {
>>                         reg = <0x0 0x4>;
>>                         type = "uint32";
>>                         default = <0x0>;
>>                 };
>>                 bar {
>>                         reg = <0x10 0x4>;
>>                         type = "enum32";
>>                         names = "baz", "qux";
>>                         default = <0x1>;
>>                 };
>>         };
>
> Here a working example for barebox 2016.11.0 so there may be changes with the
> actual state cleanup patches
>
> +       state2: state_socket {
> +               magic = <0x456ef363>;
> +               compatible = "barebox,state";
> +               backend-type = "raw";
> +               backend = &eeprom;

I changed to:
backend = <&eeprom>;
then it works :)
barebox at Phytec phyCORE AM335x:/ state
registered state instances:
state_socket         (backend: raw, path: /dev/eeprom0)

>
> Maybe the missing backend-storage-type is a problem?

As a documentation, it should not be.

Thank you all.

-- 
Çağlar Kilimci