Re: State Framework and dtb

[Ian Abbott <abbotti@mev.co.uk>]

On 04/04/17 11:27, Çağlar Kilimci wrote:
> Hey,
>
> 2017-04-04 11:04 GMT+03:00 Jan Remmet <J.Remmet@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@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@0
>>> device in the of_dump:
>>>         state@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@Phytec phyCORE AM335x:/ state
> registered state instances:
> state_socket         (backend: raw, path: /dev/eeprom0)

Won't that use the whole EEPROM rather than a partition thereof? 
According to the documentation, you can use:

		backend = <&eeprom>, "partname:state";

assuming the partition has `label = "state";`.

-- 
-=( Ian Abbott @ MEV Ltd.    E-mail: <abbotti@mev.co.uk> )=-
-=(                          Web: http://www.mev.co.uk/  )=-

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox