mbox

[GIT,PULL] Qualcomm Driver updates for 4.13

Message ID 1497383309-14226-4-git-send-email-andy.gross@linaro.org
State New
Headers show

Pull-request

git://git.kernel.org/pub/scm/linux/kernel/git/agross/linux.git tags/qcom-drivers-for-4.13

Message

Andy Gross June 13, 2017, 7:48 p.m. UTC
The following changes since commit 2ea659a9ef488125eb46da6eb571de5eae5c43f6:

  Linux 4.12-rc1 (2017-05-13 13:19:49 -0700)

are available in the git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/agross/linux.git tags/qcom-drivers-for-4.13

for you to fetch changes up to d4348b43bbcadb6d5faee36024245944b6600d25:

  firmware: qcom_scm: Fix to allow COMPILE_TEST-ing (2017-06-09 16:15:10 -0500)

----------------------------------------------------------------
Qualcomm ARM Based Driver Updates for v4.13

* Fix COMPILE_TEST flag for QCOM SCM
* Improve QCOM SMSM error handling

----------------------------------------------------------------
Jonathan Neuschäfer (1):
      soc: qcom: smsm: Improve error handling, quiesce probe deferral

Stanimir Varbanov (1):
      firmware: qcom_scm: Fix to allow COMPILE_TEST-ing

 drivers/firmware/Kconfig    |   2 +-
 drivers/firmware/qcom_scm.h | 122 ++++++++++++++++++++++++++++++++++++--------
 drivers/soc/qcom/smsm.c     |   9 ++--
 include/linux/qcom_scm.h    |  32 ------------
 4 files changed, 106 insertions(+), 59 deletions(-)

Comments

Olof Johansson June 19, 2017, 5:37 a.m. UTC | #1
Hi Andy,

On Tue, Jun 13, 2017 at 03:48:27PM -0400, Andy Gross wrote:
> The following changes since commit 2ea659a9ef488125eb46da6eb571de5eae5c43f6:
> 
>   Linux 4.12-rc1 (2017-05-13 13:19:49 -0700)
> 
> are available in the git repository at:
> 
>   git://git.kernel.org/pub/scm/linux/kernel/git/agross/linux.git tags/qcom-drivers-for-4.13
> 
> for you to fetch changes up to d4348b43bbcadb6d5faee36024245944b6600d25:
> 
>   firmware: qcom_scm: Fix to allow COMPILE_TEST-ing (2017-06-09 16:15:10 -0500)
> 
> ----------------------------------------------------------------
> Qualcomm ARM Based Driver Updates for v4.13
> 
> * Fix COMPILE_TEST flag for QCOM SCM
> * Improve QCOM SMSM error handling
> 
> ----------------------------------------------------------------
> Jonathan Neusch??fer (1):
>       soc: qcom: smsm: Improve error handling, quiesce probe deferral
> 
> Stanimir Varbanov (1):
>       firmware: qcom_scm: Fix to allow COMPILE_TEST-ing

This patch doesn't seem like it's doing the right thing -- I've replied to it
on the list with comments.

Not merged. Let me know if you want me to cherry-pick the other patch (or send
a new pull request with just that one).


-Olof
Andy Gross June 20, 2017, 2:48 p.m. UTC | #2
On Sun, Jun 18, 2017 at 10:37:07PM -0700, Olof Johansson wrote:
> Hi Andy,
> 
> On Tue, Jun 13, 2017 at 03:48:27PM -0400, Andy Gross wrote:
> > The following changes since commit 2ea659a9ef488125eb46da6eb571de5eae5c43f6:
> > 
> >   Linux 4.12-rc1 (2017-05-13 13:19:49 -0700)
> > 
> > are available in the git repository at:
> > 
> >   git://git.kernel.org/pub/scm/linux/kernel/git/agross/linux.git tags/qcom-drivers-for-4.13
> > 
> > for you to fetch changes up to d4348b43bbcadb6d5faee36024245944b6600d25:
> > 
> >   firmware: qcom_scm: Fix to allow COMPILE_TEST-ing (2017-06-09 16:15:10 -0500)
> > 
> > ----------------------------------------------------------------
> > Qualcomm ARM Based Driver Updates for v4.13
> > 
> > * Fix COMPILE_TEST flag for QCOM SCM
> > * Improve QCOM SMSM error handling
> > 
> > ----------------------------------------------------------------
> > Jonathan Neusch??fer (1):
> >       soc: qcom: smsm: Improve error handling, quiesce probe deferral
> > 
> > Stanimir Varbanov (1):
> >       firmware: qcom_scm: Fix to allow COMPILE_TEST-ing
> 
> This patch doesn't seem like it's doing the right thing -- I've replied to it
> on the list with comments.

For some reason I thought it was a rehash of a previous patch and it looked
right to me.  Dropping this looks like the right answer for now.

> 
> Not merged. Let me know if you want me to cherry-pick the other patch (or send
> a new pull request with just that one).

I'll send another pull request today.


Regards,

Andy
Stanimir Varbanov June 21, 2017, 8:35 a.m. UTC | #3
Hi,

On 06/20/2017 05:48 PM, Andy Gross wrote:
> On Sun, Jun 18, 2017 at 10:37:07PM -0700, Olof Johansson wrote:
>> Hi Andy,
>>
>> On Tue, Jun 13, 2017 at 03:48:27PM -0400, Andy Gross wrote:
>>> The following changes since commit 2ea659a9ef488125eb46da6eb571de5eae5c43f6:
>>>
>>>   Linux 4.12-rc1 (2017-05-13 13:19:49 -0700)
>>>
>>> are available in the git repository at:
>>>
>>>   git://git.kernel.org/pub/scm/linux/kernel/git/agross/linux.git tags/qcom-drivers-for-4.13
>>>
>>> for you to fetch changes up to d4348b43bbcadb6d5faee36024245944b6600d25:
>>>
>>>   firmware: qcom_scm: Fix to allow COMPILE_TEST-ing (2017-06-09 16:15:10 -0500)
>>>
>>> ----------------------------------------------------------------
>>> Qualcomm ARM Based Driver Updates for v4.13
>>>
>>> * Fix COMPILE_TEST flag for QCOM SCM
>>> * Improve QCOM SMSM error handling
>>>
>>> ----------------------------------------------------------------
>>> Jonathan Neusch??fer (1):
>>>       soc: qcom: smsm: Improve error handling, quiesce probe deferral
>>>
>>> Stanimir Varbanov (1):
>>>       firmware: qcom_scm: Fix to allow COMPILE_TEST-ing
>>
>> This patch doesn't seem like it's doing the right thing -- I've replied to it
>> on the list with comments.
> 
> For some reason I thought it was a rehash of a previous patch and it looked
> right to me.  Dropping this looks like the right answer for now.

yes, please drop this patch and I'm sorry for the inconvenience. It will
need more work than I thought initially.

> 
>>
>> Not merged. Let me know if you want me to cherry-pick the other patch (or send
>> a new pull request with just that one).
> 
> I'll send another pull request today.

Andy could you also drop the patch from your tree because it breaks
linux-next which already has alternative approach to compile test qcom
video codec driver.
Andy Gross June 21, 2017, 4:37 p.m. UTC | #4
On Wed, Jun 21, 2017 at 11:35:02AM +0300, Stanimir Varbanov wrote:
> Hi,
> 
> On 06/20/2017 05:48 PM, Andy Gross wrote:
> > On Sun, Jun 18, 2017 at 10:37:07PM -0700, Olof Johansson wrote:
> >> Hi Andy,
> >>
> >> On Tue, Jun 13, 2017 at 03:48:27PM -0400, Andy Gross wrote:
> >>> The following changes since commit 2ea659a9ef488125eb46da6eb571de5eae5c43f6:
> >>>
> >>>   Linux 4.12-rc1 (2017-05-13 13:19:49 -0700)
> >>>
> >>> are available in the git repository at:
> >>>
> >>>   git://git.kernel.org/pub/scm/linux/kernel/git/agross/linux.git tags/qcom-drivers-for-4.13
> >>>
> >>> for you to fetch changes up to d4348b43bbcadb6d5faee36024245944b6600d25:
> >>>
> >>>   firmware: qcom_scm: Fix to allow COMPILE_TEST-ing (2017-06-09 16:15:10 -0500)
> >>>
> >>> ----------------------------------------------------------------
> >>> Qualcomm ARM Based Driver Updates for v4.13
> >>>
> >>> * Fix COMPILE_TEST flag for QCOM SCM
> >>> * Improve QCOM SMSM error handling
> >>>
> >>> ----------------------------------------------------------------
> >>> Jonathan Neusch??fer (1):
> >>>       soc: qcom: smsm: Improve error handling, quiesce probe deferral
> >>>
> >>> Stanimir Varbanov (1):
> >>>       firmware: qcom_scm: Fix to allow COMPILE_TEST-ing
> >>
> >> This patch doesn't seem like it's doing the right thing -- I've replied to it
> >> on the list with comments.
> > 
> > For some reason I thought it was a rehash of a previous patch and it looked
> > right to me.  Dropping this looks like the right answer for now.
> 
> yes, please drop this patch and I'm sorry for the inconvenience. It will
> need more work than I thought initially.
> 
> > 
> >>
> >> Not merged. Let me know if you want me to cherry-pick the other patch (or send
> >> a new pull request with just that one).
> > 
> > I'll send another pull request today.
> 
> Andy could you also drop the patch from your tree because it breaks
> linux-next which already has alternative approach to compile test qcom
> video codec driver.

Done.  I pushed a revised tag and also sent a revised pull request.


Regards,

Andy