All of lore.kernel.org
 help / color / mirror / Atom feed
From: Roger Quadros <rogerq@ti.com>
To: Tony Lindgren <tony@atomide.com>, Ladislav Michl <ladis@linux-mips.org>
Cc: Boris Brezillon <boris.brezillon@free-electrons.com>,
	Kyungmin Park <kyungmin.park@samsung.com>,
	linux-omap@vger.kernel.org, linux-mtd@lists.infradead.org
Subject: Re: [PATCH v3 1/7] memory: omap-gpmc: Refactor OneNAND support
Date: Fri, 10 Nov 2017 10:12:22 +0200	[thread overview]
Message-ID: <b59f9975-bc59-97c4-ddb2-57637de12e2c@ti.com> (raw)
In-Reply-To: <20171109184828.GF28152@atomide.com>

On 09/11/17 20:48, Tony Lindgren wrote:
> * Ladislav Michl <ladis@linux-mips.org> [171109 18:35]:
>> On Thu, Nov 09, 2017 at 10:26:45AM -0800, Tony Lindgren wrote:
>>> Well we should have the dependencies merged first to avoid breaking
>>
>> Yes, that's what cover letter says :-)
>> Also, I still count on your suggestion to merge it via mtd tree.
> 
> Well only after it has been tested :) At this point I'd say
> we're best off waiting for v4.16 merge window on the clean
> up too as the merge window is about to open.
> 
>>> git bisect. After applying this and the first patch I see:
>>>
>>> omap-gpmc 6e000000.gpmc: /ocp@68000000/gpmc@6e000000/onenand@0,0 has no 'bank-width' property
>>> omap-gpmc 6e000000.gpmc: failed to probe DT child 'onenand': -22
>>>
>>> So seems like more dts changes are needed to test this.
>>
>> Argh... You are right, I should add this into serie:
>> https://patchwork.kernel.org/patch/10043259/

You don't need to resend that patch as it has been already
reviewed and in my v4.16 queue. Just indicate in the cover letter that this
series depends on that patch.

> 
> Well don't we still need the related dts changes posted
> and merged first> 
> Regards,
> 
> Tony
> 

-- 
cheers,
-roger

Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki

______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/

WARNING: multiple messages have this Message-ID (diff)
From: Roger Quadros <rogerq@ti.com>
To: Tony Lindgren <tony@atomide.com>, Ladislav Michl <ladis@linux-mips.org>
Cc: <linux-mtd@lists.infradead.org>, <linux-omap@vger.kernel.org>,
	Boris Brezillon <boris.brezillon@free-electrons.com>,
	Kyungmin Park <kyungmin.park@samsung.com>
Subject: Re: [PATCH v3 1/7] memory: omap-gpmc: Refactor OneNAND support
Date: Fri, 10 Nov 2017 10:12:22 +0200	[thread overview]
Message-ID: <b59f9975-bc59-97c4-ddb2-57637de12e2c@ti.com> (raw)
In-Reply-To: <20171109184828.GF28152@atomide.com>

On 09/11/17 20:48, Tony Lindgren wrote:
> * Ladislav Michl <ladis@linux-mips.org> [171109 18:35]:
>> On Thu, Nov 09, 2017 at 10:26:45AM -0800, Tony Lindgren wrote:
>>> Well we should have the dependencies merged first to avoid breaking
>>
>> Yes, that's what cover letter says :-)
>> Also, I still count on your suggestion to merge it via mtd tree.
> 
> Well only after it has been tested :) At this point I'd say
> we're best off waiting for v4.16 merge window on the clean
> up too as the merge window is about to open.
> 
>>> git bisect. After applying this and the first patch I see:
>>>
>>> omap-gpmc 6e000000.gpmc: /ocp@68000000/gpmc@6e000000/onenand@0,0 has no 'bank-width' property
>>> omap-gpmc 6e000000.gpmc: failed to probe DT child 'onenand': -22
>>>
>>> So seems like more dts changes are needed to test this.
>>
>> Argh... You are right, I should add this into serie:
>> https://patchwork.kernel.org/patch/10043259/

You don't need to resend that patch as it has been already
reviewed and in my v4.16 queue. Just indicate in the cover letter that this
series depends on that patch.

> 
> Well don't we still need the related dts changes posted
> and merged first> 
> Regards,
> 
> Tony
> 

-- 
cheers,
-roger

Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki

  parent reply	other threads:[~2017-11-10  8:12 UTC|newest]

Thread overview: 92+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-11-09  9:11 [PATCH v3 0/7] OMAP2+ OneNAND driver update Ladislav Michl
2017-11-09  9:11 ` Ladislav Michl
2017-11-09  9:12 ` [PATCH v3 1/7] memory: omap-gpmc: Refactor OneNAND support Ladislav Michl
2017-11-09  9:12   ` Ladislav Michl
2017-11-09 17:56   ` Tony Lindgren
2017-11-09 17:56     ` Tony Lindgren
2017-11-09 18:10     ` Ladislav Michl
2017-11-09 18:10       ` Ladislav Michl
2017-11-09 18:26       ` Tony Lindgren
2017-11-09 18:26         ` Tony Lindgren
2017-11-09 18:34         ` Ladislav Michl
2017-11-09 18:34           ` Ladislav Michl
2017-11-09 18:48           ` Tony Lindgren
2017-11-09 18:48             ` Tony Lindgren
2017-11-09 19:10             ` Ladislav Michl
2017-11-09 19:10               ` Ladislav Michl
2017-11-09 21:59               ` Tony Lindgren
2017-11-09 21:59                 ` Tony Lindgren
2017-11-09 22:26                 ` Ladislav Michl
2017-11-09 22:26                   ` Ladislav Michl
2017-11-10  8:12             ` Roger Quadros [this message]
2017-11-10  8:12               ` Roger Quadros
2017-11-09  9:13 ` [PATCH v3 2/7] mtd: onenand: omap2: Remove regulator support Ladislav Michl
2017-11-09  9:13   ` Ladislav Michl
2017-11-10  8:17   ` Roger Quadros
2017-11-10  8:17     ` Roger Quadros
2017-11-09  9:14 ` [PATCH v3 3/7] mtd: onenand: omap2: Remove skip initial unlocking support Ladislav Michl
2017-11-09  9:14   ` Ladislav Michl
2017-11-10  8:18   ` Roger Quadros
2017-11-10  8:18     ` Roger Quadros
2017-11-09  9:14 ` [PATCH v3 4/7] mtd: onenand: omap2: Remove partitioning support from platform data Ladislav Michl
2017-11-09  9:14   ` Ladislav Michl
2017-11-10  8:19   ` Roger Quadros
2017-11-10  8:19     ` Roger Quadros
2017-11-10  9:48     ` Ladislav Michl
2017-11-10  9:48       ` Ladislav Michl
2017-11-09  9:15 ` [PATCH v3 5/7] mtd: onenand: omap2: Unify OMAP2 and OMAP3 DMA implementation Ladislav Michl
2017-11-09  9:15   ` Ladislav Michl
2017-11-10  8:21   ` Roger Quadros
2017-11-10  8:21     ` Roger Quadros
2017-11-10  9:51     ` Ladislav Michl
2017-11-10  9:51       ` Ladislav Michl
2017-11-10 15:26       ` Tony Lindgren
2017-11-10 15:26         ` Tony Lindgren
2017-11-10 18:19         ` Ladislav Michl
2017-11-10 18:19           ` Ladislav Michl
2017-11-10 18:29           ` Tony Lindgren
2017-11-10 18:29             ` Tony Lindgren
2017-11-11 12:50             ` Ladislav Michl
2017-11-11 12:50               ` Ladislav Michl
2017-11-13 20:10               ` Peter Ujfalusi
2017-11-13 20:10                 ` Peter Ujfalusi
2017-11-14 14:47               ` Roger Quadros
2017-11-14 14:47                 ` Roger Quadros
2017-11-14 15:03                 ` Ladislav Michl
2017-11-14 15:03                   ` Ladislav Michl
2017-11-10  8:25   ` Peter Ujfalusi
2017-11-10  8:25     ` Peter Ujfalusi
2017-11-10 10:04     ` Ladislav Michl
2017-11-10 10:04       ` Ladislav Michl
2017-11-10 15:24       ` Tony Lindgren
2017-11-10 15:24         ` Tony Lindgren
2017-11-10 18:26         ` Ladislav Michl
2017-11-10 18:26           ` Ladislav Michl
2017-11-10 18:48           ` Tony Lindgren
2017-11-10 18:48             ` Tony Lindgren
2017-11-10 21:39         ` Ladislav Michl
2017-11-10 21:39           ` Ladislav Michl
2017-11-14 21:53           ` Tony Lindgren
2017-11-14 21:53             ` Tony Lindgren
2017-11-14 22:32             ` Ladislav Michl
2017-11-14 22:32               ` Ladislav Michl
2017-11-15  2:11               ` Tony Lindgren
2017-11-15  2:11                 ` Tony Lindgren
2017-11-13  8:22         ` Peter Ujfalusi
2017-11-13  8:22           ` Peter Ujfalusi
2017-11-13 12:15           ` Ladislav Michl
2017-11-13 12:15             ` Ladislav Michl
2017-11-13 14:36             ` Peter Ujfalusi
2017-11-13 14:36               ` Peter Ujfalusi
2017-11-13 15:23               ` Tony Lindgren
2017-11-13 15:23                 ` Tony Lindgren
2017-11-13 15:27               ` Ladislav Michl
2017-11-13 15:27                 ` Ladislav Michl
2017-11-14 15:05                 ` Roger Quadros
2017-11-14 15:05                   ` Roger Quadros
2017-11-14 15:22                   ` Ladislav Michl
2017-11-14 15:22                     ` Ladislav Michl
2017-11-09  9:16 ` [PATCH v3 6/7] mtd: onenand: omap2: Do not make delay for GPIO OMAP3 specific Ladislav Michl
2017-11-09  9:16   ` Ladislav Michl
2017-11-09  9:17 ` [PATCH v3 7/7] mtd: onenand: omap2: Configure driver from DT Ladislav Michl
2017-11-09  9:17   ` Ladislav Michl

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=b59f9975-bc59-97c4-ddb2-57637de12e2c@ti.com \
    --to=rogerq@ti.com \
    --cc=boris.brezillon@free-electrons.com \
    --cc=kyungmin.park@samsung.com \
    --cc=ladis@linux-mips.org \
    --cc=linux-mtd@lists.infradead.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=tony@atomide.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.