Re: [GIT PULL] ARM: tegra: Changes for v2023.10-rc1

2023-08-22 Thread Svyatoslav Ryhel
22 серпня 2023 р. 16:13:05 GMT+03:00, Tom Rini  
написав(-ла):
>On Tue, Aug 22, 2023 at 09:40:44AM +0200, Thierry Reding wrote:
>> On Tue, Aug 22, 2023 at 08:16:29AM +0300, Svyatoslav Ryhel wrote:
>> > 
>> > 
>> > 18 серпня 2023 р. 20:04:47 GMT+03:00, Tom Rini  
>> > написав(-ла):
>> > >On Fri, Aug 18, 2023 at 08:02:30PM +0300, Svyatoslav Ryhel wrote:
>> > >> 
>> > >> 
>> > >> 18 серпня 2023 р. 19:49:43 GMT+03:00, Tom Rini  
>> > >> написав(-ла):
>> > >> >On Fri, Aug 18, 2023 at 03:39:22PM +0200, Thierry Reding wrote:
>> > >> >
>> > >> >> From: Thierry Reding 
>> > >> >> 
>> > >> >> Hi Tom,
>> > >> >> 
>> > >> >> The following changes since commit 
>> > >> >> 68c07fc5fdf34f0926cf06fc0c4ebd6f2f3afe19:
>> > >> >> 
>> > >> >>   Merge https://source.denx.de/u-boot/custodians/u-boot-usb 
>> > >> >> (2023-06-21 14:42:50 -0400)
>> > >> >> 
>> > >> >> are available in the Git repository at:
>> > >> >> 
>> > >> >>   g...@source.denx.de:u-boot/custodians/u-boot-tegra.git 
>> > >> >> tags/tegra-for-2023.10-rc1
>> > >> >
>> > >> >FYI, in your .git/config you can do:
>> > >> >   url = https://source.denx.de/u-boot/custodians/u-boot-tegra.git
>> > >> >   pushurl = g...@source.denx.de:u-boot/custodians/u-boot-tegra.git
>> > >> >
>> > >> >And this part looks "nicer".  Not a big deal, just an FYI.
>> > >> >
>> > >> >> for you to fetch changes up to 
>> > >> >> bdf9dead86f06c7d6980c399a4a6339430b531ec:
>> > >> >> 
>> > >> >>   board: htc: endeavoru: add One X support (2023-06-30 15:20:37 
>> > >> >> +0200)
>> > >> >> 
>> > >> >> This passes CI successfully:
>> > >> >> 
>> > >> >>   
>> > >> >> https://source.denx.de/u-boot/custodians/u-boot-tegra/-/pipelines/17411
>> > >> >> 
>> > >> >> Thanks,
>> > >> >> Thierry
>> > >> >> 
>> > >> >> 
>> > >> >> ARM: tegra: Changes for v2023.10-rc1
>> > >> >> 
>> > >> >> This adds support for various new Tegra30 boards (ASUS, LG and HTC) 
>> > >> >> and
>> > >> >> has some other minor enhancements, such as enabling the poweroff 
>> > >> >> command
>> > >> >> on several Tegra210 and Tegra186 boards.
>> > >> >> 
>> > >> >> 
>> > >> >> Jonas Schwöbel (1):
>> > >> >>   configs: tegra-common-post: make PXE and DHCP boot targets 
>> > >> >> optional
>> > >> >> 
>> > >> >> Svyatoslav Ryhel (6):
>> > >> >>   configs: tegra-common-post: add GPIO keyboard as STDIN device
>> > >> >>   ARM: tegra: add SoC UID calculation function
>> > >> >>   board: asus: transformer: add ASUS Transformer T30 family 
>> > >> >> support
>> > >> >>   board: asus: grouper: add Google Nexus 7 (2012) support
>> > >> >>   board: lg: x3: add Optimus 4X HD and Optimus Vu support
>> > >> >>   board: htc: endeavoru: add One X support
>> > >> >> 
>> > >> >> Thierry Reding (2):
>> > >> >>   ARM: tegra: Enable poweroff command on Jetson TX1 and Jetson 
>> > >> >> Nano
>> > >> >>   ARM: tegra: Enable poweroff command on Jetson TX2
>> > >> >> 
>> > >> >> Tomasz Maciej Nowak (1):
>> > >> >>   ARM: dts: trimslice: sync SPI node with Linux dts
>> > >> >
>> > >> >Since I'm not sure if this is v8 or v9, given when v9 was posted, I've
>> > >> >applied this PR now, as I had said I wanted this in.  I do have two
>> > >> >requests for you Svyatoslav, however.  First, if there's changes that
>> > >> >are missing in master, please post those ASAP and we'll get them in, 
>> > >> >for
>> > >> >v2023.10. 
>> > >> 
>> > >> Patches Thierry applied are v8, while v9 has significant improvements.
>> > >
>> > >OK, I've updated patchwork to try and reflect this too.
>> > >
>> > 
>> > Merged patches are not the most recent I have sent. How to fix this?
>> 
>> Usually once something has been merged it's difficult to back it out. So
>> unless Tom can (and is willing to) somehow redo the merge with the
>> newest patches, you're going to have to rebase what you have and send
>> out as follow-up patches.
>
>I would hope that locally, doing a git diff between master with this PR
>applied and a branch with v9 on it instead would yield a set of changes
>that are easy to turn in to a follow up patch or series of patches. I
>really don't want to revert the merge and bring in a new PR instead.
>

Alright, done.


Re: [GIT PULL] ARM: tegra: Changes for v2023.10-rc1

2023-08-22 Thread Tom Rini
On Tue, Aug 22, 2023 at 09:40:44AM +0200, Thierry Reding wrote:
> On Tue, Aug 22, 2023 at 08:16:29AM +0300, Svyatoslav Ryhel wrote:
> > 
> > 
> > 18 серпня 2023 р. 20:04:47 GMT+03:00, Tom Rini  
> > написав(-ла):
> > >On Fri, Aug 18, 2023 at 08:02:30PM +0300, Svyatoslav Ryhel wrote:
> > >> 
> > >> 
> > >> 18 серпня 2023 р. 19:49:43 GMT+03:00, Tom Rini  
> > >> написав(-ла):
> > >> >On Fri, Aug 18, 2023 at 03:39:22PM +0200, Thierry Reding wrote:
> > >> >
> > >> >> From: Thierry Reding 
> > >> >> 
> > >> >> Hi Tom,
> > >> >> 
> > >> >> The following changes since commit 
> > >> >> 68c07fc5fdf34f0926cf06fc0c4ebd6f2f3afe19:
> > >> >> 
> > >> >>   Merge https://source.denx.de/u-boot/custodians/u-boot-usb 
> > >> >> (2023-06-21 14:42:50 -0400)
> > >> >> 
> > >> >> are available in the Git repository at:
> > >> >> 
> > >> >>   g...@source.denx.de:u-boot/custodians/u-boot-tegra.git 
> > >> >> tags/tegra-for-2023.10-rc1
> > >> >
> > >> >FYI, in your .git/config you can do:
> > >> >url = https://source.denx.de/u-boot/custodians/u-boot-tegra.git
> > >> >pushurl = g...@source.denx.de:u-boot/custodians/u-boot-tegra.git
> > >> >
> > >> >And this part looks "nicer".  Not a big deal, just an FYI.
> > >> >
> > >> >> for you to fetch changes up to 
> > >> >> bdf9dead86f06c7d6980c399a4a6339430b531ec:
> > >> >> 
> > >> >>   board: htc: endeavoru: add One X support (2023-06-30 15:20:37 +0200)
> > >> >> 
> > >> >> This passes CI successfully:
> > >> >> 
> > >> >>   
> > >> >> https://source.denx.de/u-boot/custodians/u-boot-tegra/-/pipelines/17411
> > >> >> 
> > >> >> Thanks,
> > >> >> Thierry
> > >> >> 
> > >> >> 
> > >> >> ARM: tegra: Changes for v2023.10-rc1
> > >> >> 
> > >> >> This adds support for various new Tegra30 boards (ASUS, LG and HTC) 
> > >> >> and
> > >> >> has some other minor enhancements, such as enabling the poweroff 
> > >> >> command
> > >> >> on several Tegra210 and Tegra186 boards.
> > >> >> 
> > >> >> 
> > >> >> Jonas Schwöbel (1):
> > >> >>   configs: tegra-common-post: make PXE and DHCP boot targets 
> > >> >> optional
> > >> >> 
> > >> >> Svyatoslav Ryhel (6):
> > >> >>   configs: tegra-common-post: add GPIO keyboard as STDIN device
> > >> >>   ARM: tegra: add SoC UID calculation function
> > >> >>   board: asus: transformer: add ASUS Transformer T30 family 
> > >> >> support
> > >> >>   board: asus: grouper: add Google Nexus 7 (2012) support
> > >> >>   board: lg: x3: add Optimus 4X HD and Optimus Vu support
> > >> >>   board: htc: endeavoru: add One X support
> > >> >> 
> > >> >> Thierry Reding (2):
> > >> >>   ARM: tegra: Enable poweroff command on Jetson TX1 and Jetson 
> > >> >> Nano
> > >> >>   ARM: tegra: Enable poweroff command on Jetson TX2
> > >> >> 
> > >> >> Tomasz Maciej Nowak (1):
> > >> >>   ARM: dts: trimslice: sync SPI node with Linux dts
> > >> >
> > >> >Since I'm not sure if this is v8 or v9, given when v9 was posted, I've
> > >> >applied this PR now, as I had said I wanted this in.  I do have two
> > >> >requests for you Svyatoslav, however.  First, if there's changes that
> > >> >are missing in master, please post those ASAP and we'll get them in, for
> > >> >v2023.10. 
> > >> 
> > >> Patches Thierry applied are v8, while v9 has significant improvements.
> > >
> > >OK, I've updated patchwork to try and reflect this too.
> > >
> > 
> > Merged patches are not the most recent I have sent. How to fix this?
> 
> Usually once something has been merged it's difficult to back it out. So
> unless Tom can (and is willing to) somehow redo the merge with the
> newest patches, you're going to have to rebase what you have and send
> out as follow-up patches.

I would hope that locally, doing a git diff between master with this PR
applied and a branch with v9 on it instead would yield a set of changes
that are easy to turn in to a follow up patch or series of patches. I
really don't want to revert the merge and bring in a new PR instead.

-- 
Tom


signature.asc
Description: PGP signature


Re: [GIT PULL] ARM: tegra: Changes for v2023.10-rc1

2023-08-22 Thread Thierry Reding
On Tue, Aug 22, 2023 at 08:16:29AM +0300, Svyatoslav Ryhel wrote:
> 
> 
> 18 серпня 2023 р. 20:04:47 GMT+03:00, Tom Rini  
> написав(-ла):
> >On Fri, Aug 18, 2023 at 08:02:30PM +0300, Svyatoslav Ryhel wrote:
> >> 
> >> 
> >> 18 серпня 2023 р. 19:49:43 GMT+03:00, Tom Rini  
> >> написав(-ла):
> >> >On Fri, Aug 18, 2023 at 03:39:22PM +0200, Thierry Reding wrote:
> >> >
> >> >> From: Thierry Reding 
> >> >> 
> >> >> Hi Tom,
> >> >> 
> >> >> The following changes since commit 
> >> >> 68c07fc5fdf34f0926cf06fc0c4ebd6f2f3afe19:
> >> >> 
> >> >>   Merge https://source.denx.de/u-boot/custodians/u-boot-usb (2023-06-21 
> >> >> 14:42:50 -0400)
> >> >> 
> >> >> are available in the Git repository at:
> >> >> 
> >> >>   g...@source.denx.de:u-boot/custodians/u-boot-tegra.git 
> >> >> tags/tegra-for-2023.10-rc1
> >> >
> >> >FYI, in your .git/config you can do:
> >> >  url = https://source.denx.de/u-boot/custodians/u-boot-tegra.git
> >> >  pushurl = g...@source.denx.de:u-boot/custodians/u-boot-tegra.git
> >> >
> >> >And this part looks "nicer".  Not a big deal, just an FYI.
> >> >
> >> >> for you to fetch changes up to bdf9dead86f06c7d6980c399a4a6339430b531ec:
> >> >> 
> >> >>   board: htc: endeavoru: add One X support (2023-06-30 15:20:37 +0200)
> >> >> 
> >> >> This passes CI successfully:
> >> >> 
> >> >>   
> >> >> https://source.denx.de/u-boot/custodians/u-boot-tegra/-/pipelines/17411
> >> >> 
> >> >> Thanks,
> >> >> Thierry
> >> >> 
> >> >> 
> >> >> ARM: tegra: Changes for v2023.10-rc1
> >> >> 
> >> >> This adds support for various new Tegra30 boards (ASUS, LG and HTC) and
> >> >> has some other minor enhancements, such as enabling the poweroff command
> >> >> on several Tegra210 and Tegra186 boards.
> >> >> 
> >> >> 
> >> >> Jonas Schwöbel (1):
> >> >>   configs: tegra-common-post: make PXE and DHCP boot targets 
> >> >> optional
> >> >> 
> >> >> Svyatoslav Ryhel (6):
> >> >>   configs: tegra-common-post: add GPIO keyboard as STDIN device
> >> >>   ARM: tegra: add SoC UID calculation function
> >> >>   board: asus: transformer: add ASUS Transformer T30 family support
> >> >>   board: asus: grouper: add Google Nexus 7 (2012) support
> >> >>   board: lg: x3: add Optimus 4X HD and Optimus Vu support
> >> >>   board: htc: endeavoru: add One X support
> >> >> 
> >> >> Thierry Reding (2):
> >> >>   ARM: tegra: Enable poweroff command on Jetson TX1 and Jetson Nano
> >> >>   ARM: tegra: Enable poweroff command on Jetson TX2
> >> >> 
> >> >> Tomasz Maciej Nowak (1):
> >> >>   ARM: dts: trimslice: sync SPI node with Linux dts
> >> >
> >> >Since I'm not sure if this is v8 or v9, given when v9 was posted, I've
> >> >applied this PR now, as I had said I wanted this in.  I do have two
> >> >requests for you Svyatoslav, however.  First, if there's changes that
> >> >are missing in master, please post those ASAP and we'll get them in, for
> >> >v2023.10. 
> >> 
> >> Patches Thierry applied are v8, while v9 has significant improvements.
> >
> >OK, I've updated patchwork to try and reflect this too.
> >
> 
> Merged patches are not the most recent I have sent. How to fix this?

Usually once something has been merged it's difficult to back it out. So
unless Tom can (and is willing to) somehow redo the merge with the
newest patches, you're going to have to rebase what you have and send
out as follow-up patches.

That's one of the reasons why you shouldn't keep sending new versions of
patches once things have settled down and there haven't been any more
review comments on the list. The assumption is that when you send
patches out you think they are all done. They may still need to be
updated based on review comments and that's fine because there's
communication on the mailing list to indicate so. Once discussions have
stopped it is assumed that the patches are done. This doesn't
necessarily mean that they get picked up right away, because people
might be busy.

The prudent thing to do in such cases, in my experience, is to leave the
patches as-is and do any subsequent changes in follow-up patches. If you
quickly notice that there will need to be more changes, then let people
know as soon as possible, otherwise they may apply patches and run tests
on them, all of which will have to be redone if you repost a new version
later on.

In this particular case I was in the process of preparing the pull
request to Tom and didn't see v9 until I had sent out the PR.

Depending on who you're sending patches to the "point of no return"
could be a lot sooner. Once patches are applied to a custodian tree and
other patches have been applied on top, switching out new versions of
patches can become really messy. Once they hit the mainline it often
becomes impossible.

Thierry


signature.asc
Description: PGP signature


Re: [GIT PULL] ARM: tegra: Changes for v2023.10-rc1

2023-08-21 Thread Svyatoslav Ryhel



18 серпня 2023 р. 20:04:47 GMT+03:00, Tom Rini  
написав(-ла):
>On Fri, Aug 18, 2023 at 08:02:30PM +0300, Svyatoslav Ryhel wrote:
>> 
>> 
>> 18 серпня 2023 р. 19:49:43 GMT+03:00, Tom Rini  
>> написав(-ла):
>> >On Fri, Aug 18, 2023 at 03:39:22PM +0200, Thierry Reding wrote:
>> >
>> >> From: Thierry Reding 
>> >> 
>> >> Hi Tom,
>> >> 
>> >> The following changes since commit 
>> >> 68c07fc5fdf34f0926cf06fc0c4ebd6f2f3afe19:
>> >> 
>> >>   Merge https://source.denx.de/u-boot/custodians/u-boot-usb (2023-06-21 
>> >> 14:42:50 -0400)
>> >> 
>> >> are available in the Git repository at:
>> >> 
>> >>   g...@source.denx.de:u-boot/custodians/u-boot-tegra.git 
>> >> tags/tegra-for-2023.10-rc1
>> >
>> >FYI, in your .git/config you can do:
>> >url = https://source.denx.de/u-boot/custodians/u-boot-tegra.git
>> >pushurl = g...@source.denx.de:u-boot/custodians/u-boot-tegra.git
>> >
>> >And this part looks "nicer".  Not a big deal, just an FYI.
>> >
>> >> for you to fetch changes up to bdf9dead86f06c7d6980c399a4a6339430b531ec:
>> >> 
>> >>   board: htc: endeavoru: add One X support (2023-06-30 15:20:37 +0200)
>> >> 
>> >> This passes CI successfully:
>> >> 
>> >>   https://source.denx.de/u-boot/custodians/u-boot-tegra/-/pipelines/17411
>> >> 
>> >> Thanks,
>> >> Thierry
>> >> 
>> >> 
>> >> ARM: tegra: Changes for v2023.10-rc1
>> >> 
>> >> This adds support for various new Tegra30 boards (ASUS, LG and HTC) and
>> >> has some other minor enhancements, such as enabling the poweroff command
>> >> on several Tegra210 and Tegra186 boards.
>> >> 
>> >> 
>> >> Jonas Schwöbel (1):
>> >>   configs: tegra-common-post: make PXE and DHCP boot targets optional
>> >> 
>> >> Svyatoslav Ryhel (6):
>> >>   configs: tegra-common-post: add GPIO keyboard as STDIN device
>> >>   ARM: tegra: add SoC UID calculation function
>> >>   board: asus: transformer: add ASUS Transformer T30 family support
>> >>   board: asus: grouper: add Google Nexus 7 (2012) support
>> >>   board: lg: x3: add Optimus 4X HD and Optimus Vu support
>> >>   board: htc: endeavoru: add One X support
>> >> 
>> >> Thierry Reding (2):
>> >>   ARM: tegra: Enable poweroff command on Jetson TX1 and Jetson Nano
>> >>   ARM: tegra: Enable poweroff command on Jetson TX2
>> >> 
>> >> Tomasz Maciej Nowak (1):
>> >>   ARM: dts: trimslice: sync SPI node with Linux dts
>> >
>> >Since I'm not sure if this is v8 or v9, given when v9 was posted, I've
>> >applied this PR now, as I had said I wanted this in.  I do have two
>> >requests for you Svyatoslav, however.  First, if there's changes that
>> >are missing in master, please post those ASAP and we'll get them in, for
>> >v2023.10. 
>> 
>> Patches Thierry applied are v8, while v9 has significant improvements.
>
>OK, I've updated patchwork to try and reflect this too.
>

Merged patches are not the most recent I have sent. How to fix this?

>> Second, on Monday after I release -rc3 and merge that in to
>> >next, in the next branch we allow for (and encourage!) config fragments
>> 
>> I will apply fragments move in the next patchset for t20/t30 devices
>> which will be send once this pathes are applied. This is acceptable
>> for you?
>
>Yes, I think this will be fine, thanks!
>


Re: [GIT PULL] ARM: tegra: Changes for v2023.10-rc1

2023-08-21 Thread Svyatoslav Ryhel



21 серпня 2023 р. 16:59:58 GMT+03:00, Thierry Reding  
написав(-ла):
>On Fri, Aug 18, 2023 at 08:02:30PM +0300, Svyatoslav Ryhel wrote:
>> 
>> 
>> 18 серпня 2023 р. 19:49:43 GMT+03:00, Tom Rini  
>> написав(-ла):
>> >On Fri, Aug 18, 2023 at 03:39:22PM +0200, Thierry Reding wrote:
>> >
>> >> From: Thierry Reding 
>> >> 
>> >> Hi Tom,
>> >> 
>> >> The following changes since commit 
>> >> 68c07fc5fdf34f0926cf06fc0c4ebd6f2f3afe19:
>> >> 
>> >>   Merge https://source.denx.de/u-boot/custodians/u-boot-usb (2023-06-21 
>> >> 14:42:50 -0400)
>> >> 
>> >> are available in the Git repository at:
>> >> 
>> >>   g...@source.denx.de:u-boot/custodians/u-boot-tegra.git 
>> >> tags/tegra-for-2023.10-rc1
>> >
>> >FYI, in your .git/config you can do:
>> >url = https://source.denx.de/u-boot/custodians/u-boot-tegra.git
>> >pushurl = g...@source.denx.de:u-boot/custodians/u-boot-tegra.git
>> >
>> >And this part looks "nicer".  Not a big deal, just an FYI.
>> >
>> >> for you to fetch changes up to bdf9dead86f06c7d6980c399a4a6339430b531ec:
>> >> 
>> >>   board: htc: endeavoru: add One X support (2023-06-30 15:20:37 +0200)
>> >> 
>> >> This passes CI successfully:
>> >> 
>> >>   https://source.denx.de/u-boot/custodians/u-boot-tegra/-/pipelines/17411
>> >> 
>> >> Thanks,
>> >> Thierry
>> >> 
>> >> 
>> >> ARM: tegra: Changes for v2023.10-rc1
>> >> 
>> >> This adds support for various new Tegra30 boards (ASUS, LG and HTC) and
>> >> has some other minor enhancements, such as enabling the poweroff command
>> >> on several Tegra210 and Tegra186 boards.
>> >> 
>> >> 
>> >> Jonas Schwöbel (1):
>> >>   configs: tegra-common-post: make PXE and DHCP boot targets optional
>> >> 
>> >> Svyatoslav Ryhel (6):
>> >>   configs: tegra-common-post: add GPIO keyboard as STDIN device
>> >>   ARM: tegra: add SoC UID calculation function
>> >>   board: asus: transformer: add ASUS Transformer T30 family support
>> >>   board: asus: grouper: add Google Nexus 7 (2012) support
>> >>   board: lg: x3: add Optimus 4X HD and Optimus Vu support
>> >>   board: htc: endeavoru: add One X support
>> >> 
>> >> Thierry Reding (2):
>> >>   ARM: tegra: Enable poweroff command on Jetson TX1 and Jetson Nano
>> >>   ARM: tegra: Enable poweroff command on Jetson TX2
>> >> 
>> >> Tomasz Maciej Nowak (1):
>> >>   ARM: dts: trimslice: sync SPI node with Linux dts
>> >
>> >Since I'm not sure if this is v8 or v9, given when v9 was posted, I've
>> >applied this PR now, as I had said I wanted this in.  I do have two
>> >requests for you Svyatoslav, however.  First, if there's changes that
>> >are missing in master, please post those ASAP and we'll get them in, for
>> >v2023.10. 
>> 
>> Patches Thierry applied are v8, while v9 has significant improvements.
>
>I'm a little surprised by how this series developed. You've been urging
>me since v6 to apply this, so I assumed this was mostly done. Having
>another 3 versions with significant improvements come in after that
>point indicates the opposite was true.

View changes, same as code. I am adapting and changing this to my needs and if 
you have hold it more, your would get even more iterations. Only way to stop is 
to upstream.

>Thierry


Re: [GIT PULL] ARM: tegra: Changes for v2023.10-rc1

2023-08-21 Thread Thierry Reding
On Fri, Aug 18, 2023 at 08:02:30PM +0300, Svyatoslav Ryhel wrote:
> 
> 
> 18 серпня 2023 р. 19:49:43 GMT+03:00, Tom Rini  
> написав(-ла):
> >On Fri, Aug 18, 2023 at 03:39:22PM +0200, Thierry Reding wrote:
> >
> >> From: Thierry Reding 
> >> 
> >> Hi Tom,
> >> 
> >> The following changes since commit 
> >> 68c07fc5fdf34f0926cf06fc0c4ebd6f2f3afe19:
> >> 
> >>   Merge https://source.denx.de/u-boot/custodians/u-boot-usb (2023-06-21 
> >> 14:42:50 -0400)
> >> 
> >> are available in the Git repository at:
> >> 
> >>   g...@source.denx.de:u-boot/custodians/u-boot-tegra.git 
> >> tags/tegra-for-2023.10-rc1
> >
> >FYI, in your .git/config you can do:
> > url = https://source.denx.de/u-boot/custodians/u-boot-tegra.git
> > pushurl = g...@source.denx.de:u-boot/custodians/u-boot-tegra.git
> >
> >And this part looks "nicer".  Not a big deal, just an FYI.
> >
> >> for you to fetch changes up to bdf9dead86f06c7d6980c399a4a6339430b531ec:
> >> 
> >>   board: htc: endeavoru: add One X support (2023-06-30 15:20:37 +0200)
> >> 
> >> This passes CI successfully:
> >> 
> >>   https://source.denx.de/u-boot/custodians/u-boot-tegra/-/pipelines/17411
> >> 
> >> Thanks,
> >> Thierry
> >> 
> >> 
> >> ARM: tegra: Changes for v2023.10-rc1
> >> 
> >> This adds support for various new Tegra30 boards (ASUS, LG and HTC) and
> >> has some other minor enhancements, such as enabling the poweroff command
> >> on several Tegra210 and Tegra186 boards.
> >> 
> >> 
> >> Jonas Schwöbel (1):
> >>   configs: tegra-common-post: make PXE and DHCP boot targets optional
> >> 
> >> Svyatoslav Ryhel (6):
> >>   configs: tegra-common-post: add GPIO keyboard as STDIN device
> >>   ARM: tegra: add SoC UID calculation function
> >>   board: asus: transformer: add ASUS Transformer T30 family support
> >>   board: asus: grouper: add Google Nexus 7 (2012) support
> >>   board: lg: x3: add Optimus 4X HD and Optimus Vu support
> >>   board: htc: endeavoru: add One X support
> >> 
> >> Thierry Reding (2):
> >>   ARM: tegra: Enable poweroff command on Jetson TX1 and Jetson Nano
> >>   ARM: tegra: Enable poweroff command on Jetson TX2
> >> 
> >> Tomasz Maciej Nowak (1):
> >>   ARM: dts: trimslice: sync SPI node with Linux dts
> >
> >Since I'm not sure if this is v8 or v9, given when v9 was posted, I've
> >applied this PR now, as I had said I wanted this in.  I do have two
> >requests for you Svyatoslav, however.  First, if there's changes that
> >are missing in master, please post those ASAP and we'll get them in, for
> >v2023.10. 
> 
> Patches Thierry applied are v8, while v9 has significant improvements.

I'm a little surprised by how this series developed. You've been urging
me since v6 to apply this, so I assumed this was mostly done. Having
another 3 versions with significant improvements come in after that
point indicates the opposite was true.

Thierry


signature.asc
Description: PGP signature


Re: [GIT PULL] ARM: tegra: Changes for v2023.10-rc1

2023-08-21 Thread Thierry Reding
On Fri, Aug 18, 2023 at 12:49:43PM -0400, Tom Rini wrote:
> On Fri, Aug 18, 2023 at 03:39:22PM +0200, Thierry Reding wrote:
> 
> > From: Thierry Reding 
> > 
> > Hi Tom,
> > 
> > The following changes since commit 68c07fc5fdf34f0926cf06fc0c4ebd6f2f3afe19:
> > 
> >   Merge https://source.denx.de/u-boot/custodians/u-boot-usb (2023-06-21 
> > 14:42:50 -0400)
> > 
> > are available in the Git repository at:
> > 
> >   g...@source.denx.de:u-boot/custodians/u-boot-tegra.git 
> > tags/tegra-for-2023.10-rc1
> 
> FYI, in your .git/config you can do:
>   url = https://source.denx.de/u-boot/custodians/u-boot-tegra.git
>   pushurl = g...@source.denx.de:u-boot/custodians/u-boot-tegra.git
> 
> And this part looks "nicer".  Not a big deal, just an FYI.

This actually comes from a script that doesn't use the git config but
takes this from gitlab directly using the REST API. This is primarily
because the script also gets the link to the CI pipeline from there so
it was convenient.

Anyway, I wasn't sure if you preferred the HTTPS link or the SSH link,
but I can update the script now that I know. =)

Thierry


signature.asc
Description: PGP signature


Re: [GIT PULL] ARM: tegra: Changes for v2023.10-rc1

2023-08-18 Thread Tom Rini
On Fri, Aug 18, 2023 at 08:02:30PM +0300, Svyatoslav Ryhel wrote:
> 
> 
> 18 серпня 2023 р. 19:49:43 GMT+03:00, Tom Rini  
> написав(-ла):
> >On Fri, Aug 18, 2023 at 03:39:22PM +0200, Thierry Reding wrote:
> >
> >> From: Thierry Reding 
> >> 
> >> Hi Tom,
> >> 
> >> The following changes since commit 
> >> 68c07fc5fdf34f0926cf06fc0c4ebd6f2f3afe19:
> >> 
> >>   Merge https://source.denx.de/u-boot/custodians/u-boot-usb (2023-06-21 
> >> 14:42:50 -0400)
> >> 
> >> are available in the Git repository at:
> >> 
> >>   g...@source.denx.de:u-boot/custodians/u-boot-tegra.git 
> >> tags/tegra-for-2023.10-rc1
> >
> >FYI, in your .git/config you can do:
> > url = https://source.denx.de/u-boot/custodians/u-boot-tegra.git
> > pushurl = g...@source.denx.de:u-boot/custodians/u-boot-tegra.git
> >
> >And this part looks "nicer".  Not a big deal, just an FYI.
> >
> >> for you to fetch changes up to bdf9dead86f06c7d6980c399a4a6339430b531ec:
> >> 
> >>   board: htc: endeavoru: add One X support (2023-06-30 15:20:37 +0200)
> >> 
> >> This passes CI successfully:
> >> 
> >>   https://source.denx.de/u-boot/custodians/u-boot-tegra/-/pipelines/17411
> >> 
> >> Thanks,
> >> Thierry
> >> 
> >> 
> >> ARM: tegra: Changes for v2023.10-rc1
> >> 
> >> This adds support for various new Tegra30 boards (ASUS, LG and HTC) and
> >> has some other minor enhancements, such as enabling the poweroff command
> >> on several Tegra210 and Tegra186 boards.
> >> 
> >> 
> >> Jonas Schwöbel (1):
> >>   configs: tegra-common-post: make PXE and DHCP boot targets optional
> >> 
> >> Svyatoslav Ryhel (6):
> >>   configs: tegra-common-post: add GPIO keyboard as STDIN device
> >>   ARM: tegra: add SoC UID calculation function
> >>   board: asus: transformer: add ASUS Transformer T30 family support
> >>   board: asus: grouper: add Google Nexus 7 (2012) support
> >>   board: lg: x3: add Optimus 4X HD and Optimus Vu support
> >>   board: htc: endeavoru: add One X support
> >> 
> >> Thierry Reding (2):
> >>   ARM: tegra: Enable poweroff command on Jetson TX1 and Jetson Nano
> >>   ARM: tegra: Enable poweroff command on Jetson TX2
> >> 
> >> Tomasz Maciej Nowak (1):
> >>   ARM: dts: trimslice: sync SPI node with Linux dts
> >
> >Since I'm not sure if this is v8 or v9, given when v9 was posted, I've
> >applied this PR now, as I had said I wanted this in.  I do have two
> >requests for you Svyatoslav, however.  First, if there's changes that
> >are missing in master, please post those ASAP and we'll get them in, for
> >v2023.10. 
> 
> Patches Thierry applied are v8, while v9 has significant improvements.

OK, I've updated patchwork to try and reflect this too.

> Second, on Monday after I release -rc3 and merge that in to
> >next, in the next branch we allow for (and encourage!) config fragments
> 
> I will apply fragments move in the next patchset for t20/t30 devices
> which will be send once this pathes are applied. This is acceptable
> for you?

Yes, I think this will be fine, thanks!

-- 
Tom


signature.asc
Description: PGP signature


Re: [GIT PULL] ARM: tegra: Changes for v2023.10-rc1

2023-08-18 Thread Svyatoslav Ryhel



18 серпня 2023 р. 19:49:43 GMT+03:00, Tom Rini  
написав(-ла):
>On Fri, Aug 18, 2023 at 03:39:22PM +0200, Thierry Reding wrote:
>
>> From: Thierry Reding 
>> 
>> Hi Tom,
>> 
>> The following changes since commit 68c07fc5fdf34f0926cf06fc0c4ebd6f2f3afe19:
>> 
>>   Merge https://source.denx.de/u-boot/custodians/u-boot-usb (2023-06-21 
>> 14:42:50 -0400)
>> 
>> are available in the Git repository at:
>> 
>>   g...@source.denx.de:u-boot/custodians/u-boot-tegra.git 
>> tags/tegra-for-2023.10-rc1
>
>FYI, in your .git/config you can do:
>   url = https://source.denx.de/u-boot/custodians/u-boot-tegra.git
>   pushurl = g...@source.denx.de:u-boot/custodians/u-boot-tegra.git
>
>And this part looks "nicer".  Not a big deal, just an FYI.
>
>> for you to fetch changes up to bdf9dead86f06c7d6980c399a4a6339430b531ec:
>> 
>>   board: htc: endeavoru: add One X support (2023-06-30 15:20:37 +0200)
>> 
>> This passes CI successfully:
>> 
>>   https://source.denx.de/u-boot/custodians/u-boot-tegra/-/pipelines/17411
>> 
>> Thanks,
>> Thierry
>> 
>> 
>> ARM: tegra: Changes for v2023.10-rc1
>> 
>> This adds support for various new Tegra30 boards (ASUS, LG and HTC) and
>> has some other minor enhancements, such as enabling the poweroff command
>> on several Tegra210 and Tegra186 boards.
>> 
>> 
>> Jonas Schwöbel (1):
>>   configs: tegra-common-post: make PXE and DHCP boot targets optional
>> 
>> Svyatoslav Ryhel (6):
>>   configs: tegra-common-post: add GPIO keyboard as STDIN device
>>   ARM: tegra: add SoC UID calculation function
>>   board: asus: transformer: add ASUS Transformer T30 family support
>>   board: asus: grouper: add Google Nexus 7 (2012) support
>>   board: lg: x3: add Optimus 4X HD and Optimus Vu support
>>   board: htc: endeavoru: add One X support
>> 
>> Thierry Reding (2):
>>   ARM: tegra: Enable poweroff command on Jetson TX1 and Jetson Nano
>>   ARM: tegra: Enable poweroff command on Jetson TX2
>> 
>> Tomasz Maciej Nowak (1):
>>   ARM: dts: trimslice: sync SPI node with Linux dts
>
>Since I'm not sure if this is v8 or v9, given when v9 was posted, I've
>applied this PR now, as I had said I wanted this in.  I do have two
>requests for you Svyatoslav, however.  First, if there's changes that
>are missing in master, please post those ASAP and we'll get them in, for
>v2023.10. 

Patches Thierry applied are v8, while v9 has significant improvements.

Second, on Monday after I release -rc3 and merge that in to
>next, in the next branch we allow for (and encourage!) config fragments

I will apply fragments move in the next patchset for t20/t30 devices which will 
be send once this pathes are applied. This is acceptable for you?

Best regards,
Svyatoslav R.

>to be in the board directory rather than top-level configs/ directory.
>Please make a patch on top of -next that moves the fragments and updates
>the MAINTAINERS entries.  Thanks all!
>


Re: [GIT PULL] ARM: tegra: Changes for v2023.10-rc1

2023-08-18 Thread Tom Rini
On Fri, Aug 18, 2023 at 03:39:22PM +0200, Thierry Reding wrote:

> From: Thierry Reding 
> 
> Hi Tom,
> 
> The following changes since commit 68c07fc5fdf34f0926cf06fc0c4ebd6f2f3afe19:
> 
>   Merge https://source.denx.de/u-boot/custodians/u-boot-usb (2023-06-21 
> 14:42:50 -0400)
> 
> are available in the Git repository at:
> 
>   g...@source.denx.de:u-boot/custodians/u-boot-tegra.git 
> tags/tegra-for-2023.10-rc1

FYI, in your .git/config you can do:
url = https://source.denx.de/u-boot/custodians/u-boot-tegra.git
pushurl = g...@source.denx.de:u-boot/custodians/u-boot-tegra.git

And this part looks "nicer".  Not a big deal, just an FYI.

> for you to fetch changes up to bdf9dead86f06c7d6980c399a4a6339430b531ec:
> 
>   board: htc: endeavoru: add One X support (2023-06-30 15:20:37 +0200)
> 
> This passes CI successfully:
> 
>   https://source.denx.de/u-boot/custodians/u-boot-tegra/-/pipelines/17411
> 
> Thanks,
> Thierry
> 
> 
> ARM: tegra: Changes for v2023.10-rc1
> 
> This adds support for various new Tegra30 boards (ASUS, LG and HTC) and
> has some other minor enhancements, such as enabling the poweroff command
> on several Tegra210 and Tegra186 boards.
> 
> 
> Jonas Schwöbel (1):
>   configs: tegra-common-post: make PXE and DHCP boot targets optional
> 
> Svyatoslav Ryhel (6):
>   configs: tegra-common-post: add GPIO keyboard as STDIN device
>   ARM: tegra: add SoC UID calculation function
>   board: asus: transformer: add ASUS Transformer T30 family support
>   board: asus: grouper: add Google Nexus 7 (2012) support
>   board: lg: x3: add Optimus 4X HD and Optimus Vu support
>   board: htc: endeavoru: add One X support
> 
> Thierry Reding (2):
>   ARM: tegra: Enable poweroff command on Jetson TX1 and Jetson Nano
>   ARM: tegra: Enable poweroff command on Jetson TX2
> 
> Tomasz Maciej Nowak (1):
>   ARM: dts: trimslice: sync SPI node with Linux dts

Since I'm not sure if this is v8 or v9, given when v9 was posted, I've
applied this PR now, as I had said I wanted this in.  I do have two
requests for you Svyatoslav, however.  First, if there's changes that
are missing in master, please post those ASAP and we'll get them in, for
v2023.10.  Second, on Monday after I release -rc3 and merge that in to
next, in the next branch we allow for (and encourage!) config fragments
to be in the board directory rather than top-level configs/ directory.
Please make a patch on top of -next that moves the fragments and updates
the MAINTAINERS entries.  Thanks all!

-- 
Tom


signature.asc
Description: PGP signature


[GIT PULL] ARM: tegra: Changes for v2023.10-rc1

2023-08-18 Thread Thierry Reding
From: Thierry Reding 

Hi Tom,

The following changes since commit 68c07fc5fdf34f0926cf06fc0c4ebd6f2f3afe19:

  Merge https://source.denx.de/u-boot/custodians/u-boot-usb (2023-06-21 
14:42:50 -0400)

are available in the Git repository at:

  g...@source.denx.de:u-boot/custodians/u-boot-tegra.git 
tags/tegra-for-2023.10-rc1

for you to fetch changes up to bdf9dead86f06c7d6980c399a4a6339430b531ec:

  board: htc: endeavoru: add One X support (2023-06-30 15:20:37 +0200)

This passes CI successfully:

  https://source.denx.de/u-boot/custodians/u-boot-tegra/-/pipelines/17411

Thanks,
Thierry


ARM: tegra: Changes for v2023.10-rc1

This adds support for various new Tegra30 boards (ASUS, LG and HTC) and
has some other minor enhancements, such as enabling the poweroff command
on several Tegra210 and Tegra186 boards.


Jonas Schwöbel (1):
  configs: tegra-common-post: make PXE and DHCP boot targets optional

Svyatoslav Ryhel (6):
  configs: tegra-common-post: add GPIO keyboard as STDIN device
  ARM: tegra: add SoC UID calculation function
  board: asus: transformer: add ASUS Transformer T30 family support
  board: asus: grouper: add Google Nexus 7 (2012) support
  board: lg: x3: add Optimus 4X HD and Optimus Vu support
  board: htc: endeavoru: add One X support

Thierry Reding (2):
  ARM: tegra: Enable poweroff command on Jetson TX1 and Jetson Nano
  ARM: tegra: Enable poweroff command on Jetson TX2

Tomasz Maciej Nowak (1):
  ARM: dts: trimslice: sync SPI node with Linux dts

 arch/arm/dts/Makefile  |  13 +
 arch/arm/dts/tegra20-trimslice.dts |   8 +-
 arch/arm/dts/tegra30-asus-grouper-common.dtsi  | 157 +++
 arch/arm/dts/tegra30-asus-nexus7-grouper-E1565.dts |  43 ++
 arch/arm/dts/tegra30-asus-nexus7-grouper-PM269.dts |  36 ++
 arch/arm/dts/tegra30-asus-nexus7-tilapia-E1565.dts |  62 +++
 arch/arm/dts/tegra30-asus-p1801-t.dts  |  18 +
 arch/arm/dts/tegra30-asus-tf201.dts|   9 +
 arch/arm/dts/tegra30-asus-tf300t.dts   |  18 +
 arch/arm/dts/tegra30-asus-tf300tg.dts  |   9 +
 arch/arm/dts/tegra30-asus-tf300tl.dts  |   9 +
 arch/arm/dts/tegra30-asus-tf600t.dts   |  89 
 arch/arm/dts/tegra30-asus-tf700t.dts   |  13 +
 arch/arm/dts/tegra30-asus-transformer.dtsi | 211 ++
 arch/arm/dts/tegra30-htc-endeavoru.dts | 166 
 arch/arm/dts/tegra30-lg-p880.dts   |  40 ++
 arch/arm/dts/tegra30-lg-p895.dts   |  50 +++
 arch/arm/dts/tegra30-lg-x3.dtsi| 180 +
 arch/arm/include/asm/arch-tegra/fuse.h |   7 +
 arch/arm/mach-tegra/Makefile   |   4 +
 arch/arm/mach-tegra/fuse.c | 151 +++
 arch/arm/mach-tegra/tegra30/Kconfig|  20 +
 board/asus/grouper/Kconfig |  22 +
 board/asus/grouper/MAINTAINERS |  10 +
 board/asus/grouper/Makefile|  14 +
 board/asus/grouper/grouper-spl-max.c   |  45 +++
 board/asus/grouper/grouper-spl-ti.c|  41 ++
 board/asus/grouper/grouper.c   | 202 +
 board/asus/grouper/pinmux-config-grouper.h | 362 +
 board/asus/transformer-t30/Kconfig |  23 ++
 board/asus/transformer-t30/MAINTAINERS |  15 +
 board/asus/transformer-t30/Makefile|  11 +
 .../transformer-t30/pinmux-config-transformer.h| 365 +
 board/asus/transformer-t30/transformer-t30-spl.c   |  41 ++
 board/asus/transformer-t30/transformer-t30.c   | 201 +
 board/htc/endeavoru/Kconfig|  12 +
 board/htc/endeavoru/MAINTAINERS|   7 +
 board/htc/endeavoru/Makefile   |  11 +
 board/htc/endeavoru/endeavoru-spl.c|  47 +++
 board/htc/endeavoru/endeavoru.c| 116 ++
 board/htc/endeavoru/pinmux-config-endeavoru.h  | 362 +
 board/lg/x3-t30/Kconfig|  26 ++
 board/lg/x3-t30/MAINTAINERS|   9 +
 board/lg/x3-t30/Makefile   |  11 +
 board/lg/x3-t30/pinmux-config-x3.h | 449 +
 board/lg/x3-t30/x3-t30-spl.c   |  48 +++
 board/lg/x3-t30/x3-t30.c   | 176 
 configs/endeavoru_defconfig|  84 
 configs/grouper_E1565.config   |   2 +
 configs/grouper_PM269.config   |   2 +
 configs/grouper_common_defconfig   |  84 
 configs/p1801-t.config |   2 +
 configs/p2371-2180_defconfig   |   1 +