2016-03-18 12:49:51

by Joao Pinto

[permalink] [raw]
Subject: UFS V11 patch-set

Hi!

Could you please check the following patch-set in order to evaluate if it is
ready for v4.6?

[PATCH v11 0/6] add support for DWC UFS Controller
http://www.gossamer-threads.com/lists/linux/kernel/2396797?#2396797

[PATCH v11 1/6] fixed typo in ufshcd-pltfrm
http://www.gossamer-threads.com/lists/linux/kernel/2396790?#2396790

[PATCH v11 2/6] added UFS 2.0 capabilities
http://www.gossamer-threads.com/lists/linux/kernel/2396791?#2396791

[PATCH v11 3/6] added support for DesignWare Controller
http://www.gossamer-threads.com/lists/linux/kernel/2396795?#2396795

[PATCH v11 4/6] added support for Synopsys G210 Test Chip
http://www.gossamer-threads.com/lists/linux/kernel/2396792?#2396792

[PATCH v11 5/6] add TC G210 platform driver
http://www.gossamer-threads.com/lists/linux/kernel/2396794?#2396794

[PATCH v11 6/6] add-TC-G210-pci-driver
http://www.gossamer-threads.com/lists/linux/kernel/2396793?#2396793

Thank you very much!

Regards,
Joao


2016-03-18 12:52:21

by Joao Pinto

[permalink] [raw]
Subject: UFS V11 patch-set

Hi!

Could you please check the following patch-set in order to evaluate if it is
ready for v4.6?

[PATCH v11 0/6] add support for DWC UFS Controller
http://www.gossamer-threads.com/lists/linux/kernel/2396797?#2396797

[PATCH v11 1/6] fixed typo in ufshcd-pltfrm
http://www.gossamer-threads.com/lists/linux/kernel/2396790?#2396790

[PATCH v11 2/6] added UFS 2.0 capabilities
http://www.gossamer-threads.com/lists/linux/kernel/2396791?#2396791

[PATCH v11 3/6] added support for DesignWare Controller
http://www.gossamer-threads.com/lists/linux/kernel/2396795?#2396795

[PATCH v11 4/6] added support for Synopsys G210 Test Chip
http://www.gossamer-threads.com/lists/linux/kernel/2396792?#2396792

[PATCH v11 5/6] add TC G210 platform driver
http://www.gossamer-threads.com/lists/linux/kernel/2396794?#2396794

[PATCH v11 6/6] add-TC-G210-pci-driver
http://www.gossamer-threads.com/lists/linux/kernel/2396793?#2396793

Thank you very much!

Regards,
Joao

PS: Changed JBottomley e-mail address.

2016-03-18 14:50:32

by Arnd Bergmann

[permalink] [raw]
Subject: Re: UFS V11 patch-set

On Friday 18 March 2016 12:52:13 Joao Pinto wrote:
> Hi!
>
> Could you please check the following patch-set in order to evaluate if it is
> ready for v4.6?
>

I think the code is ok now, but the timing apparently didn't work for 4.6.
I'd suggest you resend as soon as 4.6-rc1 is out so it can get merged into 4.7.

Arnd

2016-03-18 14:53:50

by Joao Pinto

[permalink] [raw]
Subject: Re: UFS V11 patch-set

Hi!

On 3/18/2016 2:49 PM, Arnd Bergmann wrote:
> On Friday 18 March 2016 12:52:13 Joao Pinto wrote:
>> Hi!
>>
>> Could you please check the following patch-set in order to evaluate if it is
>> ready for v4.6?
>>
>
> I think the code is ok now, but the timing apparently didn't work for 4.6.
> I'd suggest you resend as soon as 4.6-rc1 is out so it can get merged into 4.7.

Isn't it possible to be merged into a ufs-dwc branch in the SCSI Git tree that
could be pulled later, like Bjorn did in the PCIe RC?

>
> Arnd
>

Joao

2016-03-21 10:28:27

by Joao Pinto

[permalink] [raw]
Subject: Re: UFS V11 patch-set

Hi Rafael, James, Mark and Christoph,

I check in the scsi git tree that you were the last to merge code to the ufs
repository.

Is it possible to merge this code into a ufs-dwc branch in SCSI repository, in
order to be merged in v4.6-rc1?

[PATCH v11 0/6] add support for DWC UFS Controller
http://www.gossamer-threads.com/lists/linux/kernel/2396797?#2396797

[PATCH v11 1/6] fixed typo in ufshcd-pltfrm
http://www.gossamer-threads.com/lists/linux/kernel/2396790?#2396790

[PATCH v11 2/6] added UFS 2.0 capabilities
http://www.gossamer-threads.com/lists/linux/kernel/2396791?#2396791

[PATCH v11 3/6] added support for DesignWare Controller
http://www.gossamer-threads.com/lists/linux/kernel/2396795?#2396795

[PATCH v11 4/6] added support for Synopsys G210 Test Chip
http://www.gossamer-threads.com/lists/linux/kernel/2396792?#2396792

[PATCH v11 5/6] add TC G210 platform driver
http://www.gossamer-threads.com/lists/linux/kernel/2396794?#2396794

[PATCH v11 6/6] add-TC-G210-pci-driver
http://www.gossamer-threads.com/lists/linux/kernel/2396793?#2396793

On 3/18/2016 2:53 PM, Joao Pinto wrote:
> Hi!
>
> On 3/18/2016 2:49 PM, Arnd Bergmann wrote:
>> On Friday 18 March 2016 12:52:13 Joao Pinto wrote:
>>> Hi!
>>>
>>> Could you please check the following patch-set in order to evaluate if it is
>>> ready for v4.6?
>>>
>>
>> I think the code is ok now, but the timing apparently didn't work for 4.6.
>> I'd suggest you resend as soon as 4.6-rc1 is out so it can get merged into 4.7.
>
>
>>
>> Arnd

Thanks,

Joao

2016-03-22 10:31:45

by Joao Pinto

[permalink] [raw]
Subject: Re: UFS V11 patch-set

Hi,

The following patch-set was approved by Arnd Bergmann and Rob Herring and I
would appreciate that someone from SCSI / UFS gave a final checkout in order to
evaluate the possibility of still merging it to v4.6 or putting it in a branch
in SCSI to be merged to v4.7.

[PATCH v11 0/6] add support for DWC UFS Controller
http://www.gossamer-threads.com/lists/linux/kernel/2396797?#2396797

[PATCH v11 1/6] fixed typo in ufshcd-pltfrm
http://www.gossamer-threads.com/lists/linux/kernel/2396790?#2396790

[PATCH v11 2/6] added UFS 2.0 capabilities
http://www.gossamer-threads.com/lists/linux/kernel/2396791?#2396791

[PATCH v11 3/6] added support for DesignWare Controller
http://www.gossamer-threads.com/lists/linux/kernel/2396795?#2396795

[PATCH v11 4/6] added support for Synopsys G210 Test Chip
http://www.gossamer-threads.com/lists/linux/kernel/2396792?#2396792

[PATCH v11 5/6] add TC G210 platform driver
http://www.gossamer-threads.com/lists/linux/kernel/2396794?#2396794

[PATCH v11 6/6] add-TC-G210-pci-driver
http://www.gossamer-threads.com/lists/linux/kernel/2396793?#2396793


On 3/21/2016 10:28 AM, Joao Pinto wrote:
>> Hi!
>>
>> On 3/18/2016 2:49 PM, Arnd Bergmann wrote:
>>> On Friday 18 March 2016 12:52:13 Joao Pinto wrote:
>>>> Hi!
>>>>
>>>> Could you please check the following patch-set in order to evaluate if it is
>>>> ready for v4.6?
>>>>
>>>
>>> I think the code is ok now, but the timing apparently didn't work for 4.6.
>>> I'd suggest you resend as soon as 4.6-rc1 is out so it can get merged into 4.7.
>>
>>
>>>
>>> Arnd
>

Thanks,

Joao

2016-03-22 11:00:37

by Mark Brown

[permalink] [raw]
Subject: Re: UFS V11 patch-set

On Tue, Mar 22, 2016 at 10:31:28AM +0000, Joao Pinto wrote:

> The following patch-set was approved by Arnd Bergmann and Rob Herring and I
> would appreciate that someone from SCSI / UFS gave a final checkout in order to
> evaluate the possibility of still merging it to v4.6 or putting it in a branch
> in SCSI to be merged to v4.7.

I've no idea why you're sending me this stuff but please stop - you sent
a similar mail yesterday and I see there were a bunch of similar mails
before you started CCing me into the thread - but sending daily top
posted content free pings is just going to annoy people (or at least
it's annoying me) and sending them to random other maintainers isn't
likely to improve things. This is especially the case when we're more
than half way through the merge window and the code apparently isn't
even in -next yet.

Please stop this and follow Arnd's advice (which you quoted from the
message you're replying to):

> >> On 3/18/2016 2:49 PM, Arnd Bergmann wrote:

> >>> I think the code is ok now, but the timing apparently didn't work for 4.6.
> >>> I'd suggest you resend as soon as 4.6-rc1 is out so it can get merged into 4.7.

If there's some reason to break the process you need to articulate what
it is and give people a chance to respond but that's *very* rare, unless
there is an unusually strong reason people are going to use the normal
development workflow.


Attachments:
(No filename) (1.37 kB)
signature.asc (473.00 B)
Download all attachments

2016-03-22 11:04:50

by Joao Pinto

[permalink] [raw]
Subject: Re: UFS V11 patch-set

Hi Mark,

I have no intend to break the rules. I am just trying to communicate with UFS
maintainers that so far gave me no response. Merging in v4.7 is fine, but I
would like that UFS maintainers give me some feedback about the next steps, but
so far didn't have luck.

Thanks,
Joao


On 3/22/2016 10:58 AM, Mark Brown wrote:
> On Tue, Mar 22, 2016 at 10:31:28AM +0000, Joao Pinto wrote:
>
>> The following patch-set was approved by Arnd Bergmann and Rob Herring and I
>> would appreciate that someone from SCSI / UFS gave a final checkout in order to
>> evaluate the possibility of still merging it to v4.6 or putting it in a branch
>> in SCSI to be merged to v4.7.
>
> I've no idea why you're sending me this stuff but please stop - you sent
> a similar mail yesterday and I see there were a bunch of similar mails
> before you started CCing me into the thread - but sending daily top
> posted content free pings is just going to annoy people (or at least
> it's annoying me) and sending them to random other maintainers isn't
> likely to improve things. This is especially the case when we're more
> than half way through the merge window and the code apparently isn't
> even in -next yet.
>
> Please stop this and follow Arnd's advice (which you quoted from the
> message you're replying to):
>
>>>> On 3/18/2016 2:49 PM, Arnd Bergmann wrote:
>
>>>>> I think the code is ok now, but the timing apparently didn't work for 4.6.
>>>>> I'd suggest you resend as soon as 4.6-rc1 is out so it can get merged into 4.7.
>
> If there's some reason to break the process you need to articulate what
> it is and give people a chance to respond but that's *very* rare, unless
> there is an unusually strong reason people are going to use the normal
> development workflow.
>

2016-03-22 15:31:52

by Mark Brown

[permalink] [raw]
Subject: Re: UFS V11 patch-set

On Tue, Mar 22, 2016 at 11:04:25AM +0000, Joao Pinto wrote:

Please don't top post, reply in line with needed context. This allows
readers to readily follow the flow of conversation and understand what
you are talking about and also helps ensure that everything in the
discussion is being addressed.

> I have no intend to break the rules. I am just trying to communicate with UFS
> maintainers that so far gave me no response. Merging in v4.7 is fine, but I
> would like that UFS maintainers give me some feedback about the next steps, but
> so far didn't have luck.

Like I say the next steps are those Arnd gave you: repost after -rc1.
Then give people at least a week or so to respond, people do get busy
and other people are also sending patches.


Attachments:
(No filename) (753.00 B)
signature.asc (473.00 B)
Download all attachments

2016-03-22 15:40:10

by Joao Pinto

[permalink] [raw]
Subject: Re: UFS V11 patch-set

On 3/22/2016 3:31 PM, Mark Brown wrote:
> On Tue, Mar 22, 2016 at 11:04:25AM +0000, Joao Pinto wrote:
>
> Please don't top post, reply in line with needed context. This allows
> readers to readily follow the flow of conversation and understand what
> you are talking about and also helps ensure that everything in the
> discussion is being addressed.
>
>> I have no intend to break the rules. I am just trying to communicate with UFS
>> maintainers that so far gave me no response. Merging in v4.7 is fine, but I
>> would like that UFS maintainers give me some feedback about the next steps, but
>> so far didn't have luck.
>
> Like I say the next steps are those Arnd gave you: repost after -rc1.
> Then give people at least a week or so to respond, people do get busy
> and other people are also sending patches.
>

Understood, I will repost this patch set after -rc1.
I suggest that the UFS maintenance should be revised because the e-mail of
ufshcd.c authors are no longer valid and the current maintainer (Vinayak
Holikatti) is having a career shift (has you can check in LinkedIn) and probably
is not available at the moment to do patch reviews.

Thanks,
Joao


2016-03-22 15:44:32

by Mark Brown

[permalink] [raw]
Subject: Re: UFS V11 patch-set

On Tue, Mar 22, 2016 at 03:31:25PM +0000, Mark Brown wrote:
> On Tue, Mar 22, 2016 at 11:04:25AM +0000, Joao Pinto wrote:

> > I have no intend to break the rules. I am just trying to communicate with UFS
> > maintainers that so far gave me no response. Merging in v4.7 is fine, but I
> > would like that UFS maintainers give me some feedback about the next steps, but
> > so far didn't have luck.

> Like I say the next steps are those Arnd gave you: repost after -rc1.
> Then give people at least a week or so to respond, people do get busy
> and other people are also sending patches.

I'd also suggest checking James Bottomley's e-mail address, the one
you're using here bounces and isn't the one advertised in MAINTAINERS
(you should check that for other maintainers too).


Attachments:
(No filename) (778.00 B)
signature.asc (473.00 B)
Download all attachments

2016-03-22 15:53:31

by Joao Pinto

[permalink] [raw]
Subject: Re: UFS V11 patch-set

On 3/22/2016 3:43 PM, Mark Brown wrote:
> On Tue, Mar 22, 2016 at 03:31:25PM +0000, Mark Brown wrote:
>> On Tue, Mar 22, 2016 at 11:04:25AM +0000, Joao Pinto wrote:
>
>>> I have no intend to break the rules. I am just trying to communicate with UFS
>>> maintainers that so far gave me no response. Merging in v4.7 is fine, but I
>>> would like that UFS maintainers give me some feedback about the next steps, but
>>> so far didn't have luck.
>
>> Like I say the next steps are those Arnd gave you: repost after -rc1.
>> Then give people at least a week or so to respond, people do get busy
>> and other people are also sending patches.
>
> I'd also suggest checking James Bottomley's e-mail address, the one
> you're using here bounces and isn't the one advertised in MAINTAINERS
> (you should check that for other maintainers too).
>

I took James's email from commit logs in UFS. Thanks for the info.

Joao

2016-03-23 21:17:09

by Martin K. Petersen

[permalink] [raw]
Subject: Re: UFS V11 patch-set

>>>>> "Joao" == Joao Pinto <[email protected]> writes:

Joao,

Joao> Isn't it possible to be merged into a ufs-dwc branch in the SCSI
Joao> Git tree that could be pulled later, like Bjorn did in the PCIe
Joao> RC?

I just started 4.7/scsi-queue so now is a good time to resubmit.

Thanks!

--
Martin K. Petersen Oracle Linux Engineering

2016-03-24 10:16:32

by Joao Pinto

[permalink] [raw]
Subject: Re: UFS V11 patch-set

On 3/23/2016 9:16 PM, Martin K. Petersen wrote:
>>>>>> "Joao" == Joao Pinto <[email protected]> writes:
>
> Joao,
>
> Joao> Isn't it possible to be merged into a ufs-dwc branch in the SCSI
> Joao> Git tree that could be pulled later, like Bjorn did in the PCIe
> Joao> RC?
>
> I just started 4.7/scsi-queue so now is a good time to resubmit.
>
> Thanks!
>

Hi Martin!
Thanks, I will do that in a minute.

Joao