diff mbox

[flasher,2/2] Add config files for Venice2 board

Message ID 52337F1F.9070608@wwwdotorg.org
State Accepted, archived
Headers show

Commit Message

Stephen Warren Sept. 13, 2013, 9:09 p.m. UTC 
On 09/13/2013 10:19 AM, Stephen Warren wrote:
> On 09/13/2013 07:22 AM, Peter De Schrijver wrote:
>> On Thu, Sep 12, 2013 at 01:51:06AM +0200, Stephen Warren wrote:
>>> On 09/11/2013 05:48 PM, Stephen Warren wrote:
>>>> This board is currently disabled by default due to lack of U-Boot support
>>>> in the mainline repository. If you wish to test this, do as follows:
>>>>
>>>> cd u-boot
>>>> git checkout some-branch-with-venice2-support
>>>> cd ../scripts
>>>> ./build --boards venice2 build
>>>
>>> Peter, do you have a Venice2 yet? If so, could I trouble you to apply
>>> the tegrarcm Tegra124 support patches from Allen, the cbootimage-configs
>>> and tegra-uboot-flasher-scripts patches from me, grab a downstream
>>> Venice2 U-Boot branch, and validate that it all works?
>>>
>>
>> It builds all fine, but flashing as such fails with:
>>
>> U-Boot SPL 2013.04-g4e07053 (Sep 13 2013 - 16:11:40)
...
>> SF: Unsupported manufacturer fc
>> Failed to initialize SPI flash at 0:0
>> No SPI flash selected. Please run `sf probe'
>> No SPI flash selected. Please run `sf probe'

Does the following solve that?

     return flashcmd

--
To unsubscribe from this list: send the line "unsubscribe linux-tegra" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Comments

Stephen Warren Sept. 18, 2013, 10:12 p.m. UTC | #1 
On 09/13/2013 03:09 PM, Stephen Warren wrote:
> On 09/13/2013 10:19 AM, Stephen Warren wrote:
>> On 09/13/2013 07:22 AM, Peter De Schrijver wrote:
>>> On Thu, Sep 12, 2013 at 01:51:06AM +0200, Stephen Warren wrote:
>>>> On 09/11/2013 05:48 PM, Stephen Warren wrote:
>>>>> This board is currently disabled by default due to lack of U-Boot support
>>>>> in the mainline repository. If you wish to test this, do as follows:
>>>>>
>>>>> cd u-boot
>>>>> git checkout some-branch-with-venice2-support
>>>>> cd ../scripts
>>>>> ./build --boards venice2 build
...
>>> It builds all fine, but flashing as such fails with:
...
> Does the following solve that?
> 
> diff --git a/configs/venice2.config b/configs/venice2.config
> index 15d0b67..4316a1a 100644
> --- a/configs/venice2.config
> +++ b/configs/venice2.config
> @@ -5,4 +5,5 @@ config = {
>      'flash-erase-size': 1048576,
>      'flash-image': 'venice2-spi.img',
>      'flash-type': 'spi',
> +    'flash-id-uboot', '1:0',

I have applied/pushed the original two patches, with this fix squashed
in, plus fixed the syntax error in the line above; the first comma
should be a colon.
--
To unsubscribe from this list: send the line "unsubscribe linux-tegra" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/configs/venice2.config b/configs/venice2.config
index 15d0b67..4316a1a 100644
--- a/configs/venice2.config
+++ b/configs/venice2.config
@@ -5,4 +5,5 @@  config = {
     'flash-erase-size': 1048576,
     'flash-image': 'venice2-spi.img',
     'flash-type': 'spi',
+    'flash-id-uboot', '1:0',
 }
diff --git a/tegra-uboot-flasher b/tegra-uboot-flasher
index f1abe54..41879c3 100755
--- a/tegra-uboot-flasher
+++ b/tegra-uboot-flasher
@@ -64,7 +64,8 @@  def gen_flashcmd_nand(flash_image_addr, flash_img_size):
     return flashcmd

 def gen_flashcmd_spi(flash_image_addr, flash_img_size):
-    flashcmd = 'sf probe 0 ; '
+    flash_id = config.get('flash-id-uboot', '0')
+    flashcmd = 'sf probe %s ; ' % flash_id
     flashcmd += 'sf erase 0 0x%08x ; ' % config['flash-erase-size']
     flashcmd += 'sf write 0x%08x 0 0x%08x ; ' % (flash_image_addr,
flash_img_size)