On Fri, May 12, 2017 at 5:29 PM, Mathias Kresin <d...@kresin.me> wrote:
> 12.05.2017 03:37, kyson lok:
>>
>> On Fri, May 12, 2017 at 6:18 AM, L. D. Pinney <ldpin...@gmail.com> wrote:
>>>>
>>>> +&spi0 {
>>>> +       status = "okay";
>>>> +
>>>> +       m25p80@0 {
>>>> +               #address-cells = <1>;
>>>> +               #size-cells = <1>;
>>>> +               compatible = "jedec,spi-nor";
>>>> +               reg = <0>;
>>>> +               spi-max-frequency = <10000000>;
>>>> +               m25p,chunked-io = <32>;
>>>> +
>>>> +               partition@0 {
>>>> +                       label = "u-boot";
>>>> +                       reg = <0x0 0x30000>;
>>>> +                       read-only;
>>>> +               };
>>>> +
>>>> +               partition@30000 {
>>>> +                       label = "u-boot-env";
>>>> +                       reg = <0x30000 0x10000>;
>>>> +                       read-only;
>>>
>>>
>>> Is there a reason that users can not or should not write to the
>>> uboot-env partition?
>
>
> Yes, to prevent the user to shout them self into the foot. If it ain't broke
> don't fix it.

"Unix was not designed to stop you from doing stupid things, because
that would also stop you from doing clever things."
 Doug Gwyn

In this case using the uboot-envtools package...

>
>>>
>>>                partition@30000 {
>>>                       label = "u-boot-env";
>>>                       reg = <0x30000 0x10000>;
>>>                       read-only;     <---- remove this line IF it is
>>> OK for user to write here.
>>
>>
>> I don't think user can write to uboot-env, other vendor does not.
>>
>>>
>>>> +               };
>>>> +
>>>> +               factory: partition@40000 {
>>>> +                       label = "factory";
>>>> +                       reg = <0x40000 0x10000>;
>>>> +                       read-only;
>>>> +               };
>>>> +
>>>> +               partition@50000 {
>>>> +                       label = "firmware";
>>>> +                       reg = <0x50000 0xf00000>;
>>>
>>>
>>> Is this correct? other mt76x8 devices with 16MB SPI Flash use :
>>>
>>> partition@50000 {
>>> label = "firmware";
>>> reg = <0x50000 0xfb0000>;
>>>
>>
>> I think it doesn't matter. I only use 15MB for firmware.
>
>
> But why don't you use all available flash space? As far as I can see, there
> isn't anything in the last 704 KB of the flash. If possible expand the
> firmware partition to use all of the remaining flash space.
>
> Please update the IMAGE_SIZE in the build code to the value set here.
>
> Mathias

_______________________________________________
Lede-dev mailing list
Lede-dev@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/lede-dev

Reply via email to