mbox

[GIT,PULL] ARM: OMAP2+: some hwmod data patches for v3.16

Message ID alpine.DEB.2.02.1405160444200.29738@utopia.booyaka.com
State New
Headers show

Pull-request

git://git.kernel.org/pub/scm/linux/kernel/git/pjw/omap-pending.git for-v3.16/hwmod-a

Message

Paul Walmsley May 16, 2014, 4:45 a.m. UTC
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Hi Tony

The following changes since commit d1db0eea852497762cab43b905b879dfcd3b8987:

  Linux 3.15-rc3 (2014-04-27 19:29:27 -0700)

are available in the git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/pjw/omap-pending.git for-v3.16/hwmod-a

for you to fetch changes up to 433480707967187a74ced38bd38edba749998013:

  ARM: OMAP2+: hwmod: OMAP5 DSS hwmod data (2014-05-14 12:26:10 -0600)

- ----------------------------------------------------------------
First (and possibly last) set of hwmod data changes for v3.16.  This
set should clean up some obsolete OMAP4 hwmod data, and add OMAP5 DSS
support.

Basic build, boot, and PM logs are available here:

http://www.pwsan.com/omap/testlogs/hwmod-a-v3.16/20140515210013/

- ----------------------------------------------------------------
Archit Taneja (1):
      ARM: OMAP2+: hwmod: OMAP5 DSS hwmod data

Peter Ujfalusi (1):
      ARM: omap4: hwmod_data: Clean up audio related structures (remove/merge them)

 arch/arm/mach-omap2/omap_hwmod_44xx_data.c |  99 +---------
 arch/arm/mach-omap2/omap_hwmod_54xx_data.c | 283 +++++++++++++++++++++++++++++
 2 files changed, 292 insertions(+), 90 deletions(-)
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1

iQIcBAEBAgAGBQJTdZeVAAoJEMePsQ0LvSpLlPsP/32nvu4wLWKSRfHAZFSTO3cf
MAPWwxsej9FfkOJN92Y/z6RVt8UNaIGmNTZgJr4tQMhvNuIxTM39jkl2NSB0ggJU
NlQdJkgV6M5cGhKkAC3u7+Fz7/lxsr8ml2MpBzifHGVf9zZHGrNRhwMs3r34fVz0
pGu3XGQI/LBkASW5TLfEqWrYfYCBR1dsEVLlFuUfLc+DIvpFU7z07DM577BHLlzg
17KByqWHFW3dn15JSWYgOEFicYGnD5absjSXayAw5B9gjCKrNycPEk1Tm3FePX8r
lH8cqHGJ7aiffhj2h/HkX8jUHKXNCyWkyEOzey59V8Rde1G5Rhm3dIxrlBldBS9e
Wxa16MiqMktcVf4iUbuesTPEldtqkCc39zuiRwqQ6YRotFxmCPZlI+gNKkfmcqSD
zRn+q+XoeNsh5Y9UekqfK4Cm1+NacLxmS8nxtjxaVPvkzLLX0lYubG8WRedg/501
7wCd/ubsRGeH+tSt49IItmoEYZgBysareZDxAL8oSX/7zjklaNJcR4k1uGZ8xDVU
r9MNzxBl0a/wvCz6zGY8gdpy1hbCt9uM9iXH36nopG6oXOFayTQKZlv0g/xOlyzs
gNy7ELwtVq7AVCnAPaeBy3kLbV5nOFWaTSR/jdaCfNx6oaGOVvFEzsDEU820Sd2G
WXtp0giThLDS7xBdsQc/
=seX1
-----END PGP SIGNATURE-----

Comments

Tomi Valkeinen May 16, 2014, 8:38 a.m. UTC | #1
Hi Paul,

On 16/05/14 07:45, Paul Walmsley wrote:
> Hi Tony
> 
> The following changes since commit d1db0eea852497762cab43b905b879dfcd3b8987:
> 
>   Linux 3.15-rc3 (2014-04-27 19:29:27 -0700)
> 
> are available in the git repository at:
> 
>   git://git.kernel.org/pub/scm/linux/kernel/git/pjw/omap-pending.git for-v3.16/hwmod-a
> 
> for you to fetch changes up to 433480707967187a74ced38bd38edba749998013:
> 
>   ARM: OMAP2+: hwmod: OMAP5 DSS hwmod data (2014-05-14 12:26:10 -0600)
> 
> ----------------------------------------------------------------
> First (and possibly last) set of hwmod data changes for v3.16.  This
> set should clean up some obsolete OMAP4 hwmod data, and add OMAP5 DSS
> support.

What about the AM43xx hwmod data I sent in the same series:

http://article.gmane.org/gmane.linux.ports.arm.omap/114192

 Tomi
Tony Lindgren May 16, 2014, 9:23 p.m. UTC | #2
* Tomi Valkeinen <tomi.valkeinen@ti.com> [140516 01:39]:
> Hi Paul,
> 
> On 16/05/14 07:45, Paul Walmsley wrote:
> > Hi Tony
> > 
> > The following changes since commit d1db0eea852497762cab43b905b879dfcd3b8987:
> > 
> >   Linux 3.15-rc3 (2014-04-27 19:29:27 -0700)
> > 
> > are available in the git repository at:
> > 
> >   git://git.kernel.org/pub/scm/linux/kernel/git/pjw/omap-pending.git for-v3.16/hwmod-a
> > 
> > for you to fetch changes up to 433480707967187a74ced38bd38edba749998013:
> > 
> >   ARM: OMAP2+: hwmod: OMAP5 DSS hwmod data (2014-05-14 12:26:10 -0600)
> > 
> > ----------------------------------------------------------------
> > First (and possibly last) set of hwmod data changes for v3.16.  This
> > set should clean up some obsolete OMAP4 hwmod data, and add OMAP5 DSS
> > support.
> 
> What about the AM43xx hwmod data I sent in the same series:
> 
> http://article.gmane.org/gmane.linux.ports.arm.omap/114192

I guess the issue there is that there's no way for Paul or much
anybody else to check it against any documentation or hardware :(

Is this just cut and paste code, or has you verified it against
the documentation and the hardare?

Anyways, pulling this set into omap-for-v3.16/soc thanks.

Regards,

Tony
Tomi Valkeinen May 19, 2014, 9:01 a.m. UTC | #3
On 17/05/14 00:23, Tony Lindgren wrote:
> * Tomi Valkeinen <tomi.valkeinen@ti.com> [140516 01:39]:
>> Hi Paul,
>>
>> On 16/05/14 07:45, Paul Walmsley wrote:
>>> Hi Tony
>>>
>>> The following changes since commit d1db0eea852497762cab43b905b879dfcd3b8987:
>>>
>>>   Linux 3.15-rc3 (2014-04-27 19:29:27 -0700)
>>>
>>> are available in the git repository at:
>>>
>>>   git://git.kernel.org/pub/scm/linux/kernel/git/pjw/omap-pending.git for-v3.16/hwmod-a
>>>
>>> for you to fetch changes up to 433480707967187a74ced38bd38edba749998013:
>>>
>>>   ARM: OMAP2+: hwmod: OMAP5 DSS hwmod data (2014-05-14 12:26:10 -0600)
>>>
>>> ----------------------------------------------------------------
>>> First (and possibly last) set of hwmod data changes for v3.16.  This
>>> set should clean up some obsolete OMAP4 hwmod data, and add OMAP5 DSS
>>> support.
>>
>> What about the AM43xx hwmod data I sent in the same series:
>>
>> http://article.gmane.org/gmane.linux.ports.arm.omap/114192
> 
> I guess the issue there is that there's no way for Paul or much
> anybody else to check it against any documentation or hardware :(
> 
> Is this just cut and paste code, or has you verified it against
> the documentation and the hardare?

It's a cut and paste from TI's kernel, I did not write it. I have tested
it on two different AM4xxx boards, and reviewed it, although I have not
checked every individual bit of information there as it's a bit
difficult to find information of all the details.

The AM43xx's DSS is more or less a subset of the DSS in OMAP3, so while
a proper bit-for-bit review cannot be done without AM43xx documentation,
a  "looks sane" review can be done against OMAP3.

 Tomi
Tony Lindgren May 19, 2014, 4:09 p.m. UTC | #4
* Tomi Valkeinen <tomi.valkeinen@ti.com> [140519 02:02]:
> On 17/05/14 00:23, Tony Lindgren wrote:
> > * Tomi Valkeinen <tomi.valkeinen@ti.com> [140516 01:39]:
> >> Hi Paul,
> >>
> >> On 16/05/14 07:45, Paul Walmsley wrote:
> >>> Hi Tony
> >>>
> >>> The following changes since commit d1db0eea852497762cab43b905b879dfcd3b8987:
> >>>
> >>>   Linux 3.15-rc3 (2014-04-27 19:29:27 -0700)
> >>>
> >>> are available in the git repository at:
> >>>
> >>>   git://git.kernel.org/pub/scm/linux/kernel/git/pjw/omap-pending.git for-v3.16/hwmod-a
> >>>
> >>> for you to fetch changes up to 433480707967187a74ced38bd38edba749998013:
> >>>
> >>>   ARM: OMAP2+: hwmod: OMAP5 DSS hwmod data (2014-05-14 12:26:10 -0600)
> >>>
> >>> ----------------------------------------------------------------
> >>> First (and possibly last) set of hwmod data changes for v3.16.  This
> >>> set should clean up some obsolete OMAP4 hwmod data, and add OMAP5 DSS
> >>> support.
> >>
> >> What about the AM43xx hwmod data I sent in the same series:
> >>
> >> http://article.gmane.org/gmane.linux.ports.arm.omap/114192
> > 
> > I guess the issue there is that there's no way for Paul or much
> > anybody else to check it against any documentation or hardware :(
> > 
> > Is this just cut and paste code, or has you verified it against
> > the documentation and the hardare?
> 
> It's a cut and paste from TI's kernel, I did not write it. I have tested
> it on two different AM4xxx boards, and reviewed it, although I have not
> checked every individual bit of information there as it's a bit
> difficult to find information of all the details.
> 
> The AM43xx's DSS is more or less a subset of the DSS in OMAP3, so while
> a proper bit-for-bit review cannot be done without AM43xx documentation,
> a  "looks sane" review can be done against OMAP3.

OK thanks for reviewing it. Looks like Rajendra found few issues, so
I'm personally fine with your and Rajendra's acks on it. Paul may
have still other comments though.

Regards,

Tony