<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Sun, Jun 18, 2017 at 7:51 PM, Matthew McClintock <span dir="ltr"><<a href="mailto:msm-oss@mcclintock.net" target="_blank">msm-oss@mcclintock.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div class="gmail-HOEnZb"><div class="gmail-h5">On Sat, Jun 17, 2017 at 11:41 AM, Khem Raj <<a href="mailto:raj.khem@gmail.com">raj.khem@gmail.com</a>> wrote:<br>
> On Sat, Jun 17, 2017 at 8:20 AM, Tom Rini <<a href="mailto:trini@konsulko.com">trini@konsulko.com</a>> wrote:<br>
>> On Fri, Jun 16, 2017 at 06:05:07PM -0700, Khem Raj wrote:<br>
>>> On Fri, Jun 16, 2017 at 12:12 PM, Matthew McClintock<br>
>>> <<a href="mailto:msm-oss@mcclintock.net">msm-oss@mcclintock.net</a>> wrote:<br>
>>> > This is useful to update the bootloader/vfat partition from u-boot when<br>
>>> > you don't want to update everything:<br>
>>> ><br>
>>> > U-Boot> tftpboot 0x1000000 tmp/0VXje<br>
>>> > Waiting for Ethernet connection... done.<br>
>>> > Using sms0 device<br>
>>> > TFTP from server 192.168.0.1; our IP address is 192.168.0.26<br>
>>> > Filename 'image.vfat'.<br>
>>> > Load address: 0x1000000<br>
>>> > Loading: ##############################<wbr>####################  40 MiB<br>
>>> >          2.1 MiB/s<br>
>>> > done<br>
>>> > Bytes transferred = 41943040 (2800000 hex)<br>
>>> > U-Boot> mmc part<br>
>>> ><br>
>>> > Partition Map for MMC device 0  --   Partition Type: DOS<br>
>>> ><br>
>>> > Part    Start Sector    Num Sectors     UUID            Type<br>
>>> >   1     8192            81920           a63a4fbc-01     0c Boot<br>
>>> >   2     90112           163840          a63a4fbc-02     83<br>
>>> > U-Boot> mmc erase 0x2000 0x14000<br>
>>> ><br>
>>> > MMC erase: dev # 0, block # 8192, count 81920 ... 81920 blocks erased:<br>
>>> > OK<br>
>>> > U-Boot> mmc write 0x1000000 0x2000 0x14000<br>
>>> ><br>
>>> > MMC write: dev # 0, block # 8192, count 81920 ... 81920 blocks written:<br>
>>> > OK<br>
>>> > U-Boot><br>
>>> ><br>
>>> > Signed-off-by: Matthew McClintock <<a href="mailto:msm-oss@mcclintock.net">msm-oss@mcclintock.net</a>><br>
>>> > ---<br>
>>> >  classes/sdcard_image-rpi.<wbr>bbclass | 8 ++++++++<br>
>>> >  1 file changed, 8 insertions(+)<br>
>>> ><br>
>>> > diff --git a/classes/sdcard_image-rpi.<wbr>bbclass b/classes/sdcard_image-rpi.<wbr>bbclass<br>
>>> > index af3e807..27a0dfc 100644<br>
>>> > --- a/classes/sdcard_image-rpi.<wbr>bbclass<br>
>>> > +++ b/classes/sdcard_image-rpi.<wbr>bbclass<br>
>>> > @@ -72,6 +72,10 @@ SDIMG = "${IMGDEPLOYDIR}/${IMAGE_NAME}<wbr>.rootfs.rpi-sdimg"<br>
>>> >  # Additional files and/or directories to be copied into the vfat partition from the IMAGE_ROOTFS.<br>
>>> >  FATPAYLOAD ?= ""<br>
>>> ><br>
>>> > +# SD card vfat partition image name<br>
>>> > +SDIMG_VFAT = "${IMGDEPLOYDIR}/${IMAGE_NAME}<wbr>.vfat"<br>
>>> > +SDIMG_LINK_VFAT = "${IMGDEPLOYDIR}/${IMAGE_LINK_<wbr>NAME}.vfat"<br>
>>> > +<br>
>>> >  IMAGE_CMD_rpi-sdimg () {<br>
>>> ><br>
>>> >         # Align partitions<br>
>>> > @@ -145,6 +149,10 @@ IMAGE_CMD_rpi-sdimg () {<br>
>>> >         echo "${IMAGE_NAME}" > ${WORKDIR}/image-version-info<br>
>>> >         mcopy -i ${WORKDIR}/boot.img -v ${WORKDIR}/image-version-info ::<br>
>>> ><br>
>>> > +        # Deploy vfat partition<br>
>>> > +        cp ${WORKDIR}/boot.img ${SDIMG_VFAT}<br>
>>> > +        ln -sf ${SDIMG_VFAT} ${SDIMG_LINK_VFAT}<br>
>>> > +<br>
>>><br>
>>> it is of use if I am not using u-boot ? is there any penalty ?<br>
<br>
</div></div>Other than just having another thing copied into tmp there is no<br>
effect, even if you're not using u-boot.<br>
<span class="gmail-"><br>
>><br>
>> The stock firmware also uses a vfat partition, so this could just as<br>
>> easily hold those contents.<br>
><br>
> yes it does, can it do the same operations like u-boot ?<br></span></blockquote><div><br></div><div>I think what Khem is trying to say is if we don't need it (because I don't think there is any use for it when using proprietary bootloader) can we deploy it only in the case of uboot bootloader section?</div><div><br></div><div>P.S.: Would really be helpful to push a PR to github too so I can merge it easier. </div><div><br></div><div><div><div class="gmail_signature"><div dir="ltr"><div dir="ltr"><div>--</div>Andrei Gherzan</div></div></div></div></div><div> </div></div></div></div>