| Message ID | 20240224205654.1546744-17-dario.binacchi@amarulasolutions.com |
|---|---|
| State | Changes Requested, archived |
| Headers | show |
| Series | Add support for AM62x-SK HS-FS devices | expand |
Hello Dario, Le 24/02/2024 à 21:56, Dario Binacchi a écrit : > Tested on SK-AM62B-P1. You could complete your series by updating the ti_am64x_sk_defconfig too and remove ti-k3-image-gen. Except for minor improvement, this series is ok for me. Best regards, Romain > > Signed-off-by: Dario Binacchi <dario.binacchi@amarulasolutions.com> > > --- > > Changes in v5: > - Bump to 6.6.17 from 6.6.1 > - Update commit message > > Changes in v4: > - Add BR2_PACKAGE_HOST_LINUX_HEADERS_CUSTOM_6_6 to ti_am62x_sk_defconfig. > > no changes since v1 > > configs/ti_am62x_sk_defconfig | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/configs/ti_am62x_sk_defconfig b/configs/ti_am62x_sk_defconfig > index 484f9feb8c61..8c962b4a9494 100644 > --- a/configs/ti_am62x_sk_defconfig > +++ b/configs/ti_am62x_sk_defconfig > @@ -1,11 +1,11 @@ > BR2_aarch64=y > -BR2_PACKAGE_HOST_LINUX_HEADERS_CUSTOM_6_4=y > +BR2_PACKAGE_HOST_LINUX_HEADERS_CUSTOM_6_6=y > BR2_ROOTFS_POST_BUILD_SCRIPT="board/ti/am62x-sk/post-build.sh" > BR2_ROOTFS_POST_IMAGE_SCRIPT="board/ti/am62x-sk/post-image.sh" > BR2_ROOTFS_POST_SCRIPT_ARGS="-c ttyS2,115200n8 -r /dev/mmcblk1p2 -x earlycon=ns16550a,mmio32,0x02800000" > BR2_LINUX_KERNEL=y > BR2_LINUX_KERNEL_CUSTOM_VERSION=y > -BR2_LINUX_KERNEL_CUSTOM_VERSION_VALUE="6.4.16" > +BR2_LINUX_KERNEL_CUSTOM_VERSION_VALUE="6.6.17" > BR2_LINUX_KERNEL_USE_ARCH_DEFAULT_CONFIG=y > BR2_LINUX_KERNEL_DTS_SUPPORT=y > BR2_LINUX_KERNEL_INTREE_DTS_NAME="ti/k3-am625-sk"
Hi Romain, On Sun, Feb 25, 2024 at 5:43 PM Romain Naour <romain.naour@smile.fr> wrote: > > Hello Dario, > > Le 24/02/2024 à 21:56, Dario Binacchi a écrit : > > Tested on SK-AM62B-P1. > > You could complete your series by updating the ti_am64x_sk_defconfig too and > remove ti-k3-image-gen. I preferred to submit version 7 without the patches for updating the ti_am64x_sk_defconfig and the removal of the ti-k3-image-gen package. This is because I do not have the am64x_sk board to test the changes on. However, I am available to submit a separate series for applying the changes to ti_am64x_sk_defconfig if there is someone who can test the configuration for me. Please let me know. Thanks and regards, Dario > > Except for minor improvement, this series is ok for me. > > Best regards, > Romain > > > > > Signed-off-by: Dario Binacchi <dario.binacchi@amarulasolutions.com> > > > > --- > > > > Changes in v5: > > - Bump to 6.6.17 from 6.6.1 > > - Update commit message > > > > Changes in v4: > > - Add BR2_PACKAGE_HOST_LINUX_HEADERS_CUSTOM_6_6 to ti_am62x_sk_defconfig. > > > > no changes since v1 > > > > configs/ti_am62x_sk_defconfig | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/configs/ti_am62x_sk_defconfig b/configs/ti_am62x_sk_defconfig > > index 484f9feb8c61..8c962b4a9494 100644 > > --- a/configs/ti_am62x_sk_defconfig > > +++ b/configs/ti_am62x_sk_defconfig > > @@ -1,11 +1,11 @@ > > BR2_aarch64=y > > -BR2_PACKAGE_HOST_LINUX_HEADERS_CUSTOM_6_4=y > > +BR2_PACKAGE_HOST_LINUX_HEADERS_CUSTOM_6_6=y > > BR2_ROOTFS_POST_BUILD_SCRIPT="board/ti/am62x-sk/post-build.sh" > > BR2_ROOTFS_POST_IMAGE_SCRIPT="board/ti/am62x-sk/post-image.sh" > > BR2_ROOTFS_POST_SCRIPT_ARGS="-c ttyS2,115200n8 -r /dev/mmcblk1p2 -x earlycon=ns16550a,mmio32,0x02800000" > > BR2_LINUX_KERNEL=y > > BR2_LINUX_KERNEL_CUSTOM_VERSION=y > > -BR2_LINUX_KERNEL_CUSTOM_VERSION_VALUE="6.4.16" > > +BR2_LINUX_KERNEL_CUSTOM_VERSION_VALUE="6.6.17" > > BR2_LINUX_KERNEL_USE_ARCH_DEFAULT_CONFIG=y > > BR2_LINUX_KERNEL_DTS_SUPPORT=y > > BR2_LINUX_KERNEL_INTREE_DTS_NAME="ti/k3-am625-sk" >
Hi Dario, Le 27/02/2024 à 17:40, Dario Binacchi a écrit : > Hi Romain, > > On Sun, Feb 25, 2024 at 5:43 PM Romain Naour <romain.naour@smile.fr> wrote: >> >> Hello Dario, >> >> Le 24/02/2024 à 21:56, Dario Binacchi a écrit : >>> Tested on SK-AM62B-P1. >> >> You could complete your series by updating the ti_am64x_sk_defconfig too and >> remove ti-k3-image-gen. > > I preferred to submit version 7 without the patches for updating the > ti_am64x_sk_defconfig and the removal of the ti-k3-image-gen package. > This is because I do not have the am64x_sk board to test the changes > on. However, I am available to submit a separate series for applying > the changes to ti_am64x_sk_defconfig if there is someone who can test > the configuration for me. > Please let me know. Maybe you can ask to Andreas Dannenberg for the am64x_sk? > Maybe if you can also test on AM64x just to be sure the defconfig is working. I'm planning on testing the entire series here pretty soon and will provide feedback (I have access to any and all boards:) http://lists.busybox.net/pipermail/buildroot/2024-February/685836.html Best regards, Romain > > Thanks and regards, > Dario > >> >> Except for minor improvement, this series is ok for me. >> >> Best regards, >> Romain >> >>> >>> Signed-off-by: Dario Binacchi <dario.binacchi@amarulasolutions.com> >>> >>> --- >>> >>> Changes in v5: >>> - Bump to 6.6.17 from 6.6.1 >>> - Update commit message >>> >>> Changes in v4: >>> - Add BR2_PACKAGE_HOST_LINUX_HEADERS_CUSTOM_6_6 to ti_am62x_sk_defconfig. >>> >>> no changes since v1 >>> >>> configs/ti_am62x_sk_defconfig | 4 ++-- >>> 1 file changed, 2 insertions(+), 2 deletions(-) >>> >>> diff --git a/configs/ti_am62x_sk_defconfig b/configs/ti_am62x_sk_defconfig >>> index 484f9feb8c61..8c962b4a9494 100644 >>> --- a/configs/ti_am62x_sk_defconfig >>> +++ b/configs/ti_am62x_sk_defconfig >>> @@ -1,11 +1,11 @@ >>> BR2_aarch64=y >>> -BR2_PACKAGE_HOST_LINUX_HEADERS_CUSTOM_6_4=y >>> +BR2_PACKAGE_HOST_LINUX_HEADERS_CUSTOM_6_6=y >>> BR2_ROOTFS_POST_BUILD_SCRIPT="board/ti/am62x-sk/post-build.sh" >>> BR2_ROOTFS_POST_IMAGE_SCRIPT="board/ti/am62x-sk/post-image.sh" >>> BR2_ROOTFS_POST_SCRIPT_ARGS="-c ttyS2,115200n8 -r /dev/mmcblk1p2 -x earlycon=ns16550a,mmio32,0x02800000" >>> BR2_LINUX_KERNEL=y >>> BR2_LINUX_KERNEL_CUSTOM_VERSION=y >>> -BR2_LINUX_KERNEL_CUSTOM_VERSION_VALUE="6.4.16" >>> +BR2_LINUX_KERNEL_CUSTOM_VERSION_VALUE="6.6.17" >>> BR2_LINUX_KERNEL_USE_ARCH_DEFAULT_CONFIG=y >>> BR2_LINUX_KERNEL_DTS_SUPPORT=y >>> BR2_LINUX_KERNEL_INTREE_DTS_NAME="ti/k3-am625-sk" >> > >
Hi Romain, On Tue, Feb 27, 2024 at 11:23 PM Romain Naour <romain.naour@smile.fr> wrote: > > Hi Dario, > > Le 27/02/2024 à 17:40, Dario Binacchi a écrit : > > Hi Romain, > > > > On Sun, Feb 25, 2024 at 5:43 PM Romain Naour <romain.naour@smile.fr> wrote: > >> > >> Hello Dario, > >> > >> Le 24/02/2024 à 21:56, Dario Binacchi a écrit : > >>> Tested on SK-AM62B-P1. > >> > >> You could complete your series by updating the ti_am64x_sk_defconfig too and > >> remove ti-k3-image-gen. > > > > I preferred to submit version 7 without the patches for updating the > > ti_am64x_sk_defconfig and the removal of the ti-k3-image-gen package. > > This is because I do not have the am64x_sk board to test the changes > > on. However, I am available to submit a separate series for applying > > the changes to ti_am64x_sk_defconfig if there is someone who can test > > the configuration for me. > > Please let me know. > > Maybe you can ask to Andreas Dannenberg for the am64x_sk? > > > Maybe if you can also test on AM64x just to be sure the defconfig is working. > Ok, I'll get to work and try to modify the configuration based on my experience with am62x. The idea is to submit the changes over the weekend. Thanks and regards, Dario > I'm planning on testing the entire series here pretty soon and will > provide feedback (I have access to any and all boards:) > > http://lists.busybox.net/pipermail/buildroot/2024-February/685836.html > > Best regards, > Romain > > > > > > Thanks and regards, > > Dario > > > >> > >> Except for minor improvement, this series is ok for me. > >> > >> Best regards, > >> Romain > >> > >>> > >>> Signed-off-by: Dario Binacchi <dario.binacchi@amarulasolutions.com> > >>> > >>> --- > >>> > >>> Changes in v5: > >>> - Bump to 6.6.17 from 6.6.1 > >>> - Update commit message > >>> > >>> Changes in v4: > >>> - Add BR2_PACKAGE_HOST_LINUX_HEADERS_CUSTOM_6_6 to ti_am62x_sk_defconfig. > >>> > >>> no changes since v1 > >>> > >>> configs/ti_am62x_sk_defconfig | 4 ++-- > >>> 1 file changed, 2 insertions(+), 2 deletions(-) > >>> > >>> diff --git a/configs/ti_am62x_sk_defconfig b/configs/ti_am62x_sk_defconfig > >>> index 484f9feb8c61..8c962b4a9494 100644 > >>> --- a/configs/ti_am62x_sk_defconfig > >>> +++ b/configs/ti_am62x_sk_defconfig > >>> @@ -1,11 +1,11 @@ > >>> BR2_aarch64=y > >>> -BR2_PACKAGE_HOST_LINUX_HEADERS_CUSTOM_6_4=y > >>> +BR2_PACKAGE_HOST_LINUX_HEADERS_CUSTOM_6_6=y > >>> BR2_ROOTFS_POST_BUILD_SCRIPT="board/ti/am62x-sk/post-build.sh" > >>> BR2_ROOTFS_POST_IMAGE_SCRIPT="board/ti/am62x-sk/post-image.sh" > >>> BR2_ROOTFS_POST_SCRIPT_ARGS="-c ttyS2,115200n8 -r /dev/mmcblk1p2 -x earlycon=ns16550a,mmio32,0x02800000" > >>> BR2_LINUX_KERNEL=y > >>> BR2_LINUX_KERNEL_CUSTOM_VERSION=y > >>> -BR2_LINUX_KERNEL_CUSTOM_VERSION_VALUE="6.4.16" > >>> +BR2_LINUX_KERNEL_CUSTOM_VERSION_VALUE="6.6.17" > >>> BR2_LINUX_KERNEL_USE_ARCH_DEFAULT_CONFIG=y > >>> BR2_LINUX_KERNEL_DTS_SUPPORT=y > >>> BR2_LINUX_KERNEL_INTREE_DTS_NAME="ti/k3-am625-sk" > >> > > > > >
diff --git a/configs/ti_am62x_sk_defconfig b/configs/ti_am62x_sk_defconfig index 484f9feb8c61..8c962b4a9494 100644 --- a/configs/ti_am62x_sk_defconfig +++ b/configs/ti_am62x_sk_defconfig @@ -1,11 +1,11 @@ BR2_aarch64=y -BR2_PACKAGE_HOST_LINUX_HEADERS_CUSTOM_6_4=y +BR2_PACKAGE_HOST_LINUX_HEADERS_CUSTOM_6_6=y BR2_ROOTFS_POST_BUILD_SCRIPT="board/ti/am62x-sk/post-build.sh" BR2_ROOTFS_POST_IMAGE_SCRIPT="board/ti/am62x-sk/post-image.sh" BR2_ROOTFS_POST_SCRIPT_ARGS="-c ttyS2,115200n8 -r /dev/mmcblk1p2 -x earlycon=ns16550a,mmio32,0x02800000" BR2_LINUX_KERNEL=y BR2_LINUX_KERNEL_CUSTOM_VERSION=y -BR2_LINUX_KERNEL_CUSTOM_VERSION_VALUE="6.4.16" +BR2_LINUX_KERNEL_CUSTOM_VERSION_VALUE="6.6.17" BR2_LINUX_KERNEL_USE_ARCH_DEFAULT_CONFIG=y BR2_LINUX_KERNEL_DTS_SUPPORT=y BR2_LINUX_KERNEL_INTREE_DTS_NAME="ti/k3-am625-sk"
Tested on SK-AM62B-P1. Signed-off-by: Dario Binacchi <dario.binacchi@amarulasolutions.com> --- Changes in v5: - Bump to 6.6.17 from 6.6.1 - Update commit message Changes in v4: - Add BR2_PACKAGE_HOST_LINUX_HEADERS_CUSTOM_6_6 to ti_am62x_sk_defconfig. no changes since v1 configs/ti_am62x_sk_defconfig | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)