Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754379AbbG0RoI (ORCPT ); Mon, 27 Jul 2015 13:44:08 -0400 Received: from mail-by2on0093.outbound.protection.outlook.com ([207.46.100.93]:58640 "EHLO na01-by2-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1753734AbbG0Rni (ORCPT ); Mon, 27 Jul 2015 13:43:38 -0400 Authentication-Results: spf=fail (sender IP is 66.35.236.227) smtp.mailfrom=opensource.altera.com; infradead.org; dkim=none (message not signed) header.d=none; Authentication-Results: st.com; dkim=none (message not signed) header.d=none; Message-ID: <55B66CEF.7030100@opensource.altera.com> Date: Mon, 27 Jul 2015 12:39:59 -0500 From: Graham Moore User-Agent: Mozilla/5.0 (X11; Linux i686; rv:17.0) Gecko/20130329 Thunderbird/17.0.5 MIME-Version: 1.0 To: vikasm CC: Marek Vasut , "linux-mtd@lists.infradead.org" , David Woodhouse , Brian Norris , "linux-kernel@vger.kernel.org" , Alan Tull , Dinh Nguyen , Yves Vandervennet Subject: Re: [PATCH V5 2/2] mtd: spi-nor: Add driver for Cadence Quad SPI Flash Controller. References: <1437758259-28299-1-git-send-email-grmoore@opensource.altera.com> <1437758259-28299-2-git-send-email-grmoore@opensource.altera.com> <55B2D1F9.9040709@st.com> In-Reply-To: <55B2D1F9.9040709@st.com> Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit X-Originating-IP: [64.129.157.38] X-ClientProxiedBy: BLUPR01CA053.prod.exchangelabs.com (25.160.23.43) To BL2PR03MB434.namprd03.prod.outlook.com (10.141.92.22) X-Microsoft-Exchange-Diagnostics-untrusted: 1;BL2PR03MB434;2:93jdjx8RDjDqq6jcWw+27+raehXPSpmDpWBiMB0LK0TqQUXDlZdSzAG+T2TssxGFaeekEiEvy/MAFwoPUiXhEqEWFHIwRqG/UqRdIMGkMky1pmYMvOQi9jhfNqADKZfIH7mwTj0IFdzYfhdBP9osW0E2/sytXysL/rGUbLFSbhg=;3:daLuIUKz5kYdb8/E83OLlRLPp0mUwyMuKv80BSHnn1FOgi8r4CDuCJtizhyMGH+us6FABlXxHqDt34hxai+kIPraQ8L3v2MUADDzQSAvRcCME+C6OKGeY7bpAX1oLNrFnCvojCJoUICc9TF+UIVU6w==;25:7SIBLNfT8Bg5LQZKpL+RLuvtx4CGSuQNnchj6DEezEsWjs41V9sj4PhNBz994KG7b4PJmlkgfhAn9g4Nz53rhUvxdfP75KbLoX0auK0aGOC742mCoBP5HlDblabjv+2Y0qke31i9AzFS+o1HEiYjuDBGSh+VBnnjLBVdqe6At3fkTPrEjYz6HxO5cx6WWTVUW7JLpepUSEPCCSDHFS9eyy+GnBAKEeMHTzDA4fGXfKgFxNkC9GY1mXBZoNAS7+4Su3Wx2bOANKXOq5lw3aI69Q==;20:Cc2id5EG4y00Acqdn5zU0+NH6uHoPPORSt42fk+EuDPnf+OVzVcwNvJnT5rfOc+tyo9tbIdaVrcwRwUms8GKJP6oBMBq49BxFnCI/R6eKV5EcsDL42JQWZyh2zhBGVowYfOROtOEQs8UEbfDP9azHHXe7xbCZwAEK0UmgwLWiV0= X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:BL2PR03MB434;UriScan:;BCL:0;PCL:0;RULEID:;SRVR:BY1PR03MB1498;UriScan:;BCL:0;PCL:0;RULEID:;SRVR:BY1PR03MB1419; X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:;UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(601004)(5005006)(3002001);SRVR:BL2PR03MB434;BCL:0;PCL:0;RULEID:;SRVR:BL2PR03MB434;BCL:0;PCL:0;RULEID:(601004)(5005006)(3002001);SRVR:BY1PR03MB1498;BCL:0;PCL:0;RULEID:;SRVR:BY1PR03MB1498; X-Microsoft-Exchange-Diagnostics-untrusted: 1;BL2PR03MB434;4:U/0lh/ze2Ml5CjWT438kLtXA9srCvRnccDIJPD3TjTEL7ebwEcz5oDGIph1x3qQuRgr0ZvQ8bLd1pFDKkFiGzmvxRL/j1gB4qALfZOVpFp5tCtGhwM2UKE3XAup3G6legLc1Jl1z4c7RT2FJP5xGLWoLvxg3lfUTe/l1OyRQfBBXG57vYHEnhNRc7xaY/Y+MRG6v8meDYfo0hsJm/0FTbgIwwrOy+Cd0Exd2YAb8MAz4B9u+BlVu5clysuZ27HC4GGWwuFsOquMb4iZwImEozkejEdofk9BIV9KKGFyewQA= X-Forefront-PRVS: 0650714AAA X-Forefront-Antispam-Report-Untrusted: SFV:NSPM;SFS:(10009020)(6049001)(6009001)(479174004)(377454003)(189998001)(50466002)(83506001)(19580405001)(19580395003)(77096005)(76176999)(87266999)(42186005)(54356999)(40100003)(122386002)(5001920100001)(33656002)(86362001)(62966003)(65956001)(66066001)(65806001)(64126003)(50986999)(107886002)(110136002)(77156002)(47776003)(92566002)(87976001)(46102003)(2950100001)(59896002)(4001350100001)(4001430100001);DIR:OUT;SFP:1101;SCL:1;SRVR:BL2PR03MB434;H:[137.57.160.195];FPR:;SPF:None;MLV:sfv;LANG:en; X-Microsoft-Exchange-Diagnostics-untrusted: =?Windows-1252?Q?1;BL2PR03MB434;23:mjsI4j8GViqFGd6RVfHZQ6At7LJbjiTyvY5TwN?= =?Windows-1252?Q?a2XQZciwePcvXLAAijNPkOaECwebUAoLNfwyIzgrDa1eO2F4npt5ANjP?= =?Windows-1252?Q?FXrC1brr3Ys6fyoQWY0rb7ZaE2d1YpK2wkIPwYh85R+rZG+ojeagckHk?= =?Windows-1252?Q?6NYRg0fx6MGv0zBmVTVDoPKaRBr2Tq/IofGzV/42BaH/bwXYFHM/VoqC?= =?Windows-1252?Q?lHYPgCNdrbmNoL9uDRc5EfMBECsIl57c66Qdv3NL6tpShV6Oibvbig9t?= =?Windows-1252?Q?lvBKn0ooy6N0ef9ScO89R+ODa2At+W55HQ8KLeJl5PziCQ1I9vA7i42x?= =?Windows-1252?Q?ufyRYhJXwNLQD4eHSb0WQRRiqyfj593LuCVt6YFEVwDsgxi4rTZlNWuu?= =?Windows-1252?Q?oDV7KGis6LMltsL9LtsSr+6cl3jCZ/8oIzi2vdvMV9+0p3AQVCHg3Ybd?= =?Windows-1252?Q?5dqH+lzWswV6xgHKS3ww9vx3D9W7Z+LOvvYnTyrzFtORJO7QqC9chyao?= =?Windows-1252?Q?ZdQzIXZmSKVgGPSzrcvvvDCFaHht6z693bl6/VUbHuzr5jRy1Ej92Ey4?= =?Windows-1252?Q?iW7zy0dEOnhsKPveu+hVlVw6JwT/ffgoPv2xMEVUAAvpbVg705PgD6pP?= =?Windows-1252?Q?SWM2CEqRrDY7f9slGkTZKtObs/NCm3lVqM7TESfWxO/rl7dW2PZFRfgy?= =?Windows-1252?Q?3kJ/qAn6VSXv0zRrrJgu0fnyNbT702HWhQAIxB/z/5fQ+sEmD84lY+K1?= =?Windows-1252?Q?Zep9X6WE5QolMTUVg+wTvxyYULlRc0CCeIfhQC1XWXVQdKPOchpIUv66?= =?Windows-1252?Q?cvHUp3URF69+3Zdr8ork9NTBzkv4f5I9IzM6Mnddc5OzqguMZJ8QbuKr?= =?Windows-1252?Q?OBPOOc2hPeJX8XBM2dtr1e5P01xza14YGQk0G6m3P1ldkA0ipAAcWIy5?= =?Windows-1252?Q?ZX6aIlhwzUenZyA1xf3sbbwaRXGHhbKbPmfatHez54GaN0hR5rCXmnY7?= =?Windows-1252?Q?mL01AfbivOpZ+6jl+MDTcrZoNY+CFtS7uxAG30cTiEUVvG5FvifptD6W?= =?Windows-1252?Q?WqJCNi1ujb0zA=3D?= X-Microsoft-Exchange-Diagnostics-untrusted: 1;BL2PR03MB434;5:N9gzuVol1Tq6tICInxb3sRXiTjhyD7Ha9OWgOW+h3thUocjwrBH5wQXooHLl1eLpp/YDg2DvF6fQ/T9HcwyP5aUHxqCLW5YhaQxU1lXLPt4/VejeaFUwohjIb4wiHPv30lkJziI81tVU9yxtlXiufg==;24:doXx3uqDftMjQ9GvPHtSaXWQKJWMN6F1/nCo0yy4Ri6XTE2UxDoSpLYKMpFZo/YQolHaSu3aigGpyvzSX5MJ4h4jXxnUkugGSc6jOJd0CQc=;20:Pqtys99qosmfC/ygBF1udC6LXrYrVqs2ybNMBl86kMgydQZ4Pr41ONLbi3r80m4QKFeVvrZ0aj4GpBMMX6OMLgF9ZtRjRSPx+1HS1oNUR/uB0Tw/o4ASxW3xDB23hVWAahMTlaCwmQBE+n20VDK8LW6Nk7GlJW2q/kkEoPWxa1M= X-MS-Exchange-Transport-CrossTenantHeadersStamped: BL2PR03MB434 X-EOPAttributedMessage: 0 X-MS-Exchange-Transport-CrossTenantHeadersStripped: BY2FFO11FD016.protection.gbl X-Microsoft-Exchange-Diagnostics: 1;BY2FFO11FD016;1:bw+2QoprD51u9QWl44JiRb2PMlVLcb8PjW86lsOCkMLKn6dK2zrF5VxqAMkxfaAUwdKAk/BOKPNxb9JHxz7WsJizqIEazg3XizpmzYD1YopsEPOT1cDqi+2jxmHmXK2555V+7mPxV1ZB4vmN3OiqNGKHbYluT5LTPdWj1z/bDEXHSeQ3L1QvIqcKt7eDNLeEfK4Irvh3fQ2hF9lWVzBol7zlRSQjhYDfgFjf0k55iLKF4rapKIUs4GyYhbOBnpiWaEZ5pAdd+kEtG6f53tGi5SIQ2X+ShhWMozaCfHoejJbEZEwBdXcRqoYTtznb8x5+clUbSPG9dsrZzaKYS/l9Mw== X-Forefront-Antispam-Report: CIP:66.35.236.227;CTRY:US;IPV:NLI;EFV:NLI;SFV:NSPM;SFS:(10009020)(6009001)(2980300002)(339900001)(377454003)(189002)(199003)(479174004)(87266999)(66066001)(92566002)(122386002)(87936001)(86362001)(110136002)(33656002)(85426001)(65956001)(62966003)(189998001)(83506001)(19580395003)(4001350100001)(64126003)(65806001)(47776003)(19580405001)(107886002)(77096005)(2950100001)(40100003)(59896002)(6806004)(76176999)(50466002)(50986999)(54356999)(46102003)(16796002)(106466001)(77156002)(7099028)(4001430100001);DIR:OUT;SFP:1101;SCL:1;SRVR:BY1PR03MB1498;H:sj-itexedge03.altera.priv.altera.com;FPR:;SPF:Fail;MLV:sfv;MX:1;A:0;LANG:en; X-Microsoft-Exchange-Diagnostics: 1;BY1PR03MB1498;2:ziU5r48+B/+LNycvc5x1UM+U1McYJOQZ+QaiDGeTX223fKrPhtlhv3dkT5RaG3D5xH4GXGcn63q/EUnpGMXwA9gLEt8Tef4uCr8rIPidewRPyUM+jl6whBBn+JCTsp9+CjCFSxK/ORY4AyqGiCe7E3frqPXsym0haB7P6t9tYVY=;3:rIWkjerNhHkU7qBuDwyHVFfJfR6d6oll7Jyb9xX7k+ZdPuORk/rRegUE8S7pE4o3m/+3xsO0mZ+PMIovBdVfJ5wAwyf6F1H7G6xqWY1+YKt3zhq0rmYt+rRu0Qa9S0fEr0g6xVmT+nw2tYw5s6yXkFQBNrU+s7q8o3qsoaWp0tPhkAdV+sZBwtjYyk7H4v9Crc7x7zrDkSZjv+dGH0o0NMRriA5rJKLMKPPvGJ3gpwk=;25:xgCrBG/6k5dLWdHQ9w/U10799Fb9t5NfofN/+K2WVEGpLtBrUhSzpHm+dLg8QGOVI573x/oTC/QcX2n4TFtLgzIP9592goGPNYca9DHtK5jWr5mwf251Km6cI4yEQFjtmWSuOUWQTfdulo37/iXyQEJl7LtQkOLvOW9uQaoCECU+KJsdPUoNtjIo6r2Mn1ruWUVf7VnTwDgBmhd2ToAV3t2FJNJ+66yn39KnbwF/xfx8jb7qZVZ2HTCKFiGuEzi8vNgVNrNZhr7RBchhyMLvwA==;20:AFqn/Pi5ru04Imb6XctQ6r+JrZ4Bnl9It5LXVkM01IXCXYESUkcqKCKYyxPz4cEqs+a70b24JtYWTP58Ey6HsfvKjdNmFK8rT5QXN3lUInP+g9Jq97NJ+mF81TNOZylfZJzps9HJMEKlLebRsNdUQCZOY+m/r1k4f0GjbTaJatM= BY1PR03MB1498: X-MS-Exchange-Organization-RulesExecuted X-Microsoft-Exchange-Diagnostics: 1;BY1PR03MB1498;4:YRGCqwvlmPWLvk2rUvh5zzk5HfT9FVymnXnHnlajRLrWZ5E+nvemuWQXZqZJ11x3XfcUTyL98wcXer3PVukDTEZUkJCPnHkUgQ/H6tXa4QvIucgc9YjtFiq//RqCF4TIeOeDpXtH8wfizYV6vb1SPA6cz18+b8QL9rpWu2mEQkA5GZLio13G1InmqA7Jw1TwGEmpLlr3QjZabEgqfweTRdIKoPGbHT++q9z6n4HHgf8mVHXWY+PFeQi8BhsxrnvUVDsYGN//Rpk9brbFupSiKGnV6wOCd0yJk4cxQzcgxL8= X-Forefront-PRVS: 0650714AAA X-Microsoft-Exchange-Diagnostics: =?Windows-1252?Q?1;BY1PR03MB1498;23:6a6yabd85GdW4sPN0mL1ULoIGLSFzOxuOeuUa?= =?Windows-1252?Q?WqAVc9o7UwgxZl0Y7846hJEM6GC6MpBth8tfdxx+QnYj6Oa+m4zIb5R/?= =?Windows-1252?Q?XiGrnPzBIeTTi4zoAByrj0gF0A3omMypNV2uH+MNrF7nmvRXItkrp3UJ?= =?Windows-1252?Q?oET0yRZAj48EDkoHrEGBgD5obKs9UtU2GXN+ghF/TErfV59vO/gDzhze?= =?Windows-1252?Q?byjGBG6OahBK1wJaaiwJQKAU0wrOfTQwMRVBZodo1nTSqpDj77ZeVFcz?= =?Windows-1252?Q?ek7XZieRUq0CxP6gWZHxysKM6csQIlYWxGfQ2OU2pLicpqFrJKl5Ifoe?= =?Windows-1252?Q?uD1fRa3NWbMHUhlN8Yys94rHYLludB9tFbrIzVfXm6M9EQUJs7ENVDbQ?= =?Windows-1252?Q?Ia7btdoIXAkr5qlq8n7+lqZwKauAaDZUlfCEfTu7Bb+FMJi3UpnrYcFg?= =?Windows-1252?Q?zr6Rue7IuYhuPHkyNfzQp4tOFlmEXqst89QwPBOu9c5luhOBzfbiI0UF?= =?Windows-1252?Q?wKfLD9t9mU+odJqd39HUiskYpFSwASal9C9ib+Shb5UHrDuNFNU9oh+G?= =?Windows-1252?Q?XTD4gkvmogi+/jNoBu2vBvh+Zwt7QKa1eu9s6VDRj/5sq5QLuViCRCUo?= =?Windows-1252?Q?hlUl737ePb0+wjvOyCGBEv7oXmFcpZ9a7qE8j2IvlES8HDMj/Id/S0Fk?= =?Windows-1252?Q?Artz9sFL57cP8NJFmMSctXxB5KYPe7aBehOfkTs48WN9IGvM44hTv8H6?= =?Windows-1252?Q?9GvqaLktLW4mYiU7c/O8ETcTFeSSJrQMeD4eDJVfeneGVsEhpmTdRKXm?= =?Windows-1252?Q?PgawZCk0DDnA0gZQo93MTI+EGF1uKnihnbz95TlQW8Dycr5LQvccpa1j?= =?Windows-1252?Q?jw3crZDUzR48I18DOT2GQOjVo409xEzEeaEE9h75cnLAJP811iCpmOij?= =?Windows-1252?Q?21AAg1llJlBOhA6wZuKqcnWvNORZVZMzZf+iLJIS1mXLMgDnYx4Jo5By?= =?Windows-1252?Q?VRIoB5O5GydaX3yOtJjiGdfjuLW9UFbRsonOxm5aFIR1Gxs8f47gMVgT?= =?Windows-1252?Q?1Rd9S/d72cOWRzr0/gwVAwPrIirKjmdJpSNYBg9dtSrvVl4SAbgATp8Y?= =?Windows-1252?Q?j0voPziVujLNYtOeCveWrszIRF2+7ePzVlJqQErKP0UhHT0ayIeZE3nB?= =?Windows-1252?Q?kCu8h0fpg=3D=3D?= X-Microsoft-Exchange-Diagnostics: 1;BY1PR03MB1498;5:T5Z+HonQ5iEd6jppjbCKVpOhnngxhOimzeEIyPfpZhTgLH8fQbqh32jmiw/atD0On5quyGto4wQv6s0bh1h6ciYUeoZwy6g9HAFM9RgDf3R5LPe4vroJicaTbGUBvAKkkOYmrJTU8g7jmT4MjnQwzQ==;24:ERGWwsldAynEFiR5nU+rB6f5ZRP9cgHin6vPxI9MHbagiK/uPWh2hQYsyrJzTFsFop2qwzmdnPx1Nt6HTWO1jkkVi698SNshluszY3XPRDM=;20:DcSgoSmthv0KbP7iDyFTzrbWCz8nbIcepww+e/JvAqbsfePs3QQMV2jZgjwqf8ttqT3D0GQM9oZlcAwqFKk7tqey+a8SH7cC8Y/X1YUlkWrMrrui2awk+7y+mXRylffPLI901x5hrNLgGTR99VRME1J6kJUquSRtey+y+kgeZpE= X-MS-Exchange-CrossTenant-OriginalArrivalTime: 27 Jul 2015 17:43:35.5565 (UTC) X-MS-Exchange-CrossTenant-Id: fbd72e03-d4a5-4110-adce-614d51f2077a X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=fbd72e03-d4a5-4110-adce-614d51f2077a;Ip=[66.35.236.227];Helo=[sj-itexedge03.altera.priv.altera.com] X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: BY1PR03MB1498 X-Microsoft-Exchange-Diagnostics: 1;BY1PR03MB1419;2:kwHmZG2ftyqo5w/CUtghkXHSHrbgRdpw+CgRa8eYCYg2tyKcb4JI6YyYrn5Sbxl8hxsZbcutRMkZ7TTBHBFbE8jlTGrRI2Ayp1WrlteJxsquEELT46o+BGPBWJX3DmbHSObYk2bt3W1thIGpWukzoFFyDEjfaTfUPr8foFmxpDA=;3:bEKoTayJ86p8FYL9NPpuGUWOBKf9+IpfXCv4syrkAD+T/abvzoPcCGvy55VRG/og3ZoMww/EfNehoi7kjro/m4uJ2Jwho3dZLDgBBWSln3fMJySDr6TCefdYkCSOHCmlNe3taUhIneZPBTGwWUKcIGXK5RFcYG5LfraG0bQRb0BddtyEkvHcNdYqkNJlO7nBP67hzD7fNSJ6wW3DLchX+PhKzM18jBMs2XZC/9xabbY=;25:GSSgrIOZzSjY9A4oewFoKG05dRLHObTal1U61zWfsf2zQqdPaJ5YtjDdlNApuCPOeKek/WGMPQI+erNX2fqwmEpQPMDi7XHJvsry70NKvfU3QzGS4bu0ivy5rLHoyYz31E21/beb+NRW6hv2UP8LfO4BzDElTA6gWOa0uTGCSS3BA4Mw0QcSZc3RqeblHyAGJpHyP5p61jnDVXG1Ykd95zJ0LoS/PoNg8P2ur3iT3abwBa3JlrUVP6gmyRyvFzF9RgSTWyRThrdSVNixv6z3Sg==;23:fFWiuqTum8szl+aJx2fq4vkDMpt1gTEuWwcLPkvoKQva5ZZ0b62y4NY3lRITzednK5U7JoW2IgUxTwsegZD4endS3qRitSWxv1Cxbcf8Idl2tqymtHIWLEHRhUF8Qg0RfRrx4VDcVs6MS/EthlMrqkKbiSwX5ELErgcU/FWuT/F/KmmDZk17szmi2BTwyW+J33ccrpyW5OTudCDoF4j53Xlxo+t0jFATDFdOD1BXstKCDag8X/3Q/UMdEfcOQHMA X-OriginatorOrg: opensource.altera.com Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 5413 Lines: 149 Hi Vikas, On 07/24/2015 07:02 PM, vikasm wrote: > Hi Graham, > > On 07/24/2015 10:17 AM, Graham Moore wrote: >> Signed-off-by: Graham Moore >> --- >> V2: use NULL instead of modalias in spi_nor_scan call >> V3: Use existing property is-decoded-cs instead of creating duplicate. >> V4: Support Micron quad mode by snooping command stream for EVCR command >> and subsequently configuring Cadence controller for quad mode. >> V5: Clean up sparse and smatch complaints. Remove snooping of Micron >> quad mode. Add comment on XIP mode bit and dummy clock cycles. Set >> up SRAM partition at 1:1 during init. >> --- >> arch/arm/boot/dts/socfpga.dtsi | 1 + >> arch/arm/boot/dts/socfpga_cyclone5_socdk.dts | 1 - >> drivers/mtd/spi-nor/Kconfig | 6 + >> drivers/mtd/spi-nor/Makefile | 1 + >> drivers/mtd/spi-nor/cadence-quadspi.c | 1261 ++++++++++++++++++++++++++ >> 5 files changed, 1269 insertions(+), 1 deletion(-) >> create mode 100644 drivers/mtd/spi-nor/cadence-quadspi.c >> >> diff --git a/arch/arm/boot/dts/socfpga.dtsi b/arch/arm/boot/dts/socfpga.dtsi >> index c71a705..e9ecdce 100644 >> --- a/arch/arm/boot/dts/socfpga.dtsi >> +++ b/arch/arm/boot/dts/socfpga.dtsi >> @@ -695,6 +695,7 @@ >> is-decoded-cs = <1>; >> fifo-depth = <128>; >> status = "disabled"; >> + m25p,fast-read; > > This patch is not applicable to l2-mtd master or linus master repo, might need to rebase it. > Argh, my mistake, these dts files are not supposed to be in the patch. ... >> +#include >> + >> +#define CQSPI_NAME "cadence-qspi" > > replace space with tabs. > They *are* tabs in my original patch. Not sure how they got changed to spaces... ... >> + >> +#define CQSPI_FIFO_WIDTH 4 > > FIFO width could be 4 or 8 etc depending on the SOC, it would be better to get it from device. > You can refer to u-boot code for the same. > OK ... >> + >> +#define CQSPI_INDIRECTTRIGGER_ADDR_MASK 0xFFFFF > > Mask value of 0xFFFFF is specific to socfpga platform. > Please refer to u-boot patchset for same discussion. > OK ... >> +static void cqspi_fifo_read(void *dest, const void __iomem *src_ahb_addr, >> + unsigned int bytes) >> +{ >> + unsigned int temp; >> + int remaining = bytes; >> + unsigned int *dest_ptr = (unsigned int *)dest; >> + >> + while (remaining >= CQSPI_FIFO_WIDTH) { >> + *dest_ptr = readl(src_ahb_addr); >> + dest_ptr++; >> + remaining -= CQSPI_FIFO_WIDTH; > > this logic only works when fifo width is 4 with "unsigned int" data of 4 bytes. > It has been corrected in mainline u-boot or in the u-boot patches. > OK ... >> +static int cqspi_indirect_read_setup(struct spi_nor *nor, >> + unsigned int from_addr) >> +{ >> + unsigned int reg; >> + unsigned int dummy_clk = 0; >> + struct cqspi_st *cqspi = nor->priv; >> + void __iomem *reg_base = cqspi->iobase; >> + unsigned int ahb_phy_addr = cqspi->ahb_phy_addr; >> + >> + writel((ahb_phy_addr & CQSPI_INDIRECTTRIGGER_ADDR_MASK), >> + reg_base + CQSPI_REG_INDIRECTTRIGGER); >> + writel(from_addr, reg_base + CQSPI_REG_INDIRECTRDSTARTADDR); > > Base trigger register address (0x1c register) corresponds to the address which > should be put on AHB bus to handle indirect transfer triggered before. > > To handle indirect transfer we need to issue addresses from (value of 0x1c) to > (value of 0x1c) + 15*4 ("4" corresponds to size of SRAM location). > There are no obstacles in issuing const address just equal to 0x1c. Important > thing to note is that indirect trigger address has nothing in common with your > physical or mapped NOR Flash address. > > Transfer read/write start addresses should be programmed with the absolute flash address > to be read/written. > OK, I'll add the trigger address to the device tree, etc. You seem to be concerned about the fact that on Altera SoC, the trigger address and the ahb address are different. I'm not seeing any problem with that. The CPU reads/writes the FIFO using an address, but that is not the same address that goes on the bus way down in the interconnect. What's wrong with that? It's the way our SoC works. If we use the trigger address for the trigger address register, and the ahb address for reading/writing the FIFO, then it works fine. ... >> + watermark = cqspi->fifo_depth * CQSPI_FIFO_WIDTH / 2; >> + writel(watermark, reg_base + CQSPI_REG_INDIRECTRDWATERMARK); >> + writel(remaining, reg_base + CQSPI_REG_INDIRECTRDBYTES); >> + writel(cqspi->fifo_depth - CQSPI_REG_SRAM_RESV_WORDS, >> + reg_base + CQSPI_REG_SRAMPARTITION); > > sram partioning is not needed to be changed at every read/write, it should be ok to configure it > in the init like divide half for read & half for write. > OK ... > > Rgds, > Vikas > Best regards, Graham -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/