* [PATCH 0/5] AM33xx: MMC resources from DT without HWMOD data
@ 2013-06-26 1:23 Joel A Fernandes
[not found] ` <CAD=GYpbdVVrh-tiqOcbHi3mkWRJUBdx0EsEZEFVVbgFgcCWgsw@mail.gmail.com>
0 siblings, 1 reply; 7+ messages in thread
From: Joel A Fernandes @ 2013-06-26 1:23 UTC (permalink / raw)
To: Tony Lindgren, Sekhar Nori, Matt Porter, Grant Likely,
Rob Herring, Vinod Koul, Mark Brown, Benoit Cousson,
Russell King, Rob Landley, Andrew Morton, Jason Kridner,
Koen Kooi, Balaji TK, Chris Ball
Cc: Devicetree Discuss, Linux OMAP List, Linux ARM Kernel List,
Linux DaVinci Kernel List, Linux Kernel Mailing List,
Linux Documentation List, Linux MMC List, Linux SPI Devel List,
Arnd Bergmann
From: Joel A Fernandes <agnel.joel@gmail.com>
This series is fixes to get MMC working on AM33XX without HWMOD data.
On removal of HWMOD data, interrupt and register properties need to be provided
for the driver to function correctly.
Also instead of specifying offset in match->data, I've added bindings for the
same and specify it directly from DT as this is a hardware description.
Reposting Matt's remaining 2 patches on the bindings and dts.
Joel A Fernandes (3):
MMC: omap_hsmmc: Add reg_offset to DT bindings
ARM: dts: am33xx: Add interrupts and memory resources for MMC
mmc: omap_hsmmc: Add reg-offset to bindings documentation
Matt Porter (2):
ARM: dts: add AM33XX MMC support
mmc: omap_hsmmc: add generic DMA request support to the DT binding
.../devicetree/bindings/mmc/ti-omap-hsmmc.txt | 29 +++++++++++++-
arch/arm/boot/dts/am335x-bone.dts | 7 ++++
arch/arm/boot/dts/am335x-evm.dts | 7 ++++
arch/arm/boot/dts/am335x-evmsk.dts | 7 ++++
arch/arm/boot/dts/am33xx.dtsi | 40 ++++++++++++++++++++
drivers/mmc/host/omap_hsmmc.c | 7 +++-
6 files changed, 95 insertions(+), 2 deletions(-)
--
1.7.9.5
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 0/5] AM33xx: MMC resources from DT without HWMOD data
[not found] ` <CAD=GYpbdVVrh-tiqOcbHi3mkWRJUBdx0EsEZEFVVbgFgcCWgsw@mail.gmail.com>
@ 2013-06-27 12:03 ` Benoit Cousson
[not found] ` <CAD=GYpa9LQ1fV6=O9B65sJURVAigsgXCPLWav8cHXTuGQXPc9A@mail.gmail.com>
0 siblings, 1 reply; 7+ messages in thread
From: Benoit Cousson @ 2013-06-27 12:03 UTC (permalink / raw)
To: Joel A Fernandes
Cc: Tony Lindgren, Sekhar Nori, Matt Porter, Grant Likely,
Rob Herring, Vinod Koul, Mark Brown, Russell King, Rob Landley,
Andrew Morton, Jason Kridner, Koen Kooi, Balaji TK, Chris Ball,
Devicetree Discuss, Linux OMAP List, Linux ARM Kernel List,
Linux DaVinci Kernel List, Linux Kernel Mailing List,
Linux Documentation List, Linux MMC List, Linux SPI Devel List,
Arnd Bergmann
Hi Joel,
On 06/26/2013 05:28 AM, Joel A Fernandes wrote:
> On Tue, Jun 25, 2013 at 8:23 PM, Joel A Fernandes <joelagnel@ti.com> wrote:
>> From: Joel A Fernandes <agnel.joel@gmail.com>
>>
>> This series is fixes to get MMC working on AM33XX without HWMOD data.
>> On removal of HWMOD data, interrupt and register properties need to be provided
>> for the driver to function correctly.
>
> Please don't pull patches authored by me in this series. I have posted
> them with my wrong email address by mistake. I will correct my email
> during the next repost, after a round of review. Matt's patches in the
> series are still good though.
In that case, please repost these in two series to avoid pulling the DTS
files in the MMC tree.
Thanks,
Benoit
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 0/5] AM33xx: MMC resources from DT without HWMOD data
[not found] ` <CAD=GYpa9LQ1fV6=O9B65sJURVAigsgXCPLWav8cHXTuGQXPc9A@mail.gmail.com>
@ 2013-06-27 16:26 ` Balaji T K
2013-06-27 18:08 ` Fernandes, Joel
2013-06-27 18:10 ` Fernandes, Joel
0 siblings, 2 replies; 7+ messages in thread
From: Balaji T K @ 2013-06-27 16:26 UTC (permalink / raw)
To: Benoit Cousson, joelf
Cc: lars, Tony Lindgren, Sekhar Nori, Matt Porter, Grant Likely,
Rob Herring, Vinod Koul, Mark Brown, Russell King, Rob Landley,
Andrew Morton, Jason Kridner, Koen Kooi, Chris Ball,
Devicetree Discuss, Linux OMAP List, Linux ARM Kernel List,
Linux DaVinci Kernel List, Linux Kernel Mailing List,
Linux MMC List, Arnd Bergmann
On Thursday 27 June 2013 07:56 PM, Joel A Fernandes wrote:
> Hi Benoit,
>
> On Thu, Jun 27, 2013 at 7:03 AM, Benoit Cousson <b-cousson@ti.com> wrote:
>> Hi Joel,
>>
>> On 06/26/2013 05:28 AM, Joel A Fernandes wrote:
>>> On Tue, Jun 25, 2013 at 8:23 PM, Joel A Fernandes <joelagnel@ti.com> wrote:
>>>> From: Joel A Fernandes <agnel.joel@gmail.com>
>>>>
>>>> This series is fixes to get MMC working on AM33XX without HWMOD data.
>>>> On removal of HWMOD data, interrupt and register properties need to be provided
>>>> for the driver to function correctly.
>>>
>>> Please don't pull patches authored by me in this series. I have posted
>>> them with my wrong email address by mistake. I will correct my email
>>> during the next repost, after a round of review. Matt's patches in the
>>> series are still good though.
>>
>> In that case, please repost these in two series to avoid pulling the DTS
>> files in the MMC tree.
>
> The series has been replaced with the single patch:
> "[PATCH v2] ARM: dts: add AM33XX MMC support"
> http://marc.info/?l=linux-omap&m=137230403815943&w=2
>
> Some patches were squashed and others dropped in the series resulting
> in the single
> patch above. This patch should be good to apply
Hi Joel,
Before pushing mmc dts support for am335x, Can you please let me
know if below 2 patches [1] are merged/accepted,
IIRC these patches are needed for MMC to work properly.
I suggest to hold the dts patch until slave_sg_limit gets merged
otherwise it will cause regression in am335x mmc.
[PATCH v4 1/3] dmaengine: add dma_get_slave_sg_limits()
[PATCH v4 2/3] dma: edma: add device_slave_sg_limits() support
[1]
https://groups.google.com/forum/#!topic/linux.kernel/khTTcPtQpag
>
> Thanks,
>
> -Joel
>
^ permalink raw reply [flat|nested] 7+ messages in thread
* RE: [PATCH 0/5] AM33xx: MMC resources from DT without HWMOD data
2013-06-27 16:26 ` Balaji T K
@ 2013-06-27 18:08 ` Fernandes, Joel
2013-06-28 15:48 ` Balaji T K
2013-06-27 18:10 ` Fernandes, Joel
1 sibling, 1 reply; 7+ messages in thread
From: Fernandes, Joel @ 2013-06-27 18:08 UTC (permalink / raw)
To: Krishnamoorthy, Balaji T, Cousson, Benoit
Cc: lars, Tony Lindgren, Nori, Sekhar, Matt Porter, Grant Likely,
Rob Herring, Vinod Koul, Mark Brown, Russell King, Rob Landley,
Andrew Morton, Jason Kridner, Koen Kooi, Chris Ball,
Devicetree Discuss, Linux OMAP List, Linux ARM Kernel List,
Linux DaVinci Kernel List, Linux Kernel Mailing List,
Linux MMC List, Arnd Bergmann, lars
Hi Balaji,
> -----Original Message-----
> From: Krishnamoorthy, Balaji T
> Sent: Thursday, June 27, 2013 11:27 AM
> To: Cousson, Benoit; Fernandes, Joel
> Cc: lars@metafoo.de; Tony Lindgren; Nori, Sekhar; Matt Porter; Grant Likely;
> Rob Herring; Vinod Koul; Mark Brown; Russell King; Rob Landley; Andrew
> Morton; Jason Kridner; Koen Kooi; Chris Ball; Devicetree Discuss; Linux OMAP
> List; Linux ARM Kernel List; Linux DaVinci Kernel List; Linux Kernel Mailing List;
> Linux MMC List; Arnd Bergmann
> Subject: Re: [PATCH 0/5] AM33xx: MMC resources from DT without HWMOD
> data
>
> On Thursday 27 June 2013 07:56 PM, Joel A Fernandes wrote:
> > Hi Benoit,
> >
> > On Thu, Jun 27, 2013 at 7:03 AM, Benoit Cousson <b-cousson@ti.com> wrote:
> >> Hi Joel,
> >>
> >> On 06/26/2013 05:28 AM, Joel A Fernandes wrote:
> >>> On Tue, Jun 25, 2013 at 8:23 PM, Joel A Fernandes <joelagnel@ti.com>
> wrote:
> >>>> From: Joel A Fernandes <agnel.joel@gmail.com>
> >>>>
> >>>> This series is fixes to get MMC working on AM33XX without HWMOD
> data.
> >>>> On removal of HWMOD data, interrupt and register properties need to
> >>>> be provided for the driver to function correctly.
> >>>
> >>> Please don't pull patches authored by me in this series. I have
> >>> posted them with my wrong email address by mistake. I will correct
> >>> my email during the next repost, after a round of review. Matt's
> >>> patches in the series are still good though.
> >>
> >> In that case, please repost these in two series to avoid pulling the
> >> DTS files in the MMC tree.
> >
> > The series has been replaced with the single patch:
> > "[PATCH v2] ARM: dts: add AM33XX MMC support"
> > http://marc.info/?l=linux-omap&m=137230403815943&w=2
> >
> > Some patches were squashed and others dropped in the series resulting
> > in the single patch above. This patch should be good to apply
> Hi Joel,
>
> Before pushing mmc dts support for am335x, Can you please let me
> know if below 2 patches [1] are merged/accepted,
> IIRC these patches are needed for MMC to work properly.
>
> I suggest to hold the dts patch until slave_sg_limit gets merged
> otherwise it will cause regression in am335x mmc.
>
> [PATCH v4 1/3] dmaengine: add dma_get_slave_sg_limits()
> [PATCH v4 2/3] dma: edma: add device_slave_sg_limits() support
Actually this is an independent regression and is currently worked on by Lars-Peter Clausen [1].
Currently not having the MMC DTS merged is also a regression, so I suggest lets take it one
step at a time and get the DTS merged in first. Also there will be overhead of reposting this
DTS patch later again which will not change. What do you say?
Thanks,
-Joel
[1] https://patchwork.kernel.org/patch/2228041/
^ permalink raw reply [flat|nested] 7+ messages in thread
* RE: [PATCH 0/5] AM33xx: MMC resources from DT without HWMOD data
2013-06-27 16:26 ` Balaji T K
2013-06-27 18:08 ` Fernandes, Joel
@ 2013-06-27 18:10 ` Fernandes, Joel
1 sibling, 0 replies; 7+ messages in thread
From: Fernandes, Joel @ 2013-06-27 18:10 UTC (permalink / raw)
To: Krishnamoorthy, Balaji T, Cousson, Benoit, benoit.cousson
Cc: lars, Tony Lindgren, Nori, Sekhar, Matt Porter, Grant Likely,
Rob Herring, Vinod Koul, Mark Brown, Russell King, Rob Landley,
Andrew Morton, Jason Kridner, Koen Kooi, Chris Ball,
Devicetree Discuss, Linux OMAP List, Linux ARM Kernel List,
Linux DaVinci Kernel List, Linux Kernel Mailing List,
Linux MMC List, Arnd Bergmann
Adding Benoit's other email as current one is marked Away. Thanks.
> -----Original Message-----
> From: Krishnamoorthy, Balaji T
> Sent: Thursday, June 27, 2013 11:27 AM
> To: Cousson, Benoit; Fernandes, Joel
> Cc: lars@metafoo.de; Tony Lindgren; Nori, Sekhar; Matt Porter; Grant Likely;
> Rob Herring; Vinod Koul; Mark Brown; Russell King; Rob Landley; Andrew
> Morton; Jason Kridner; Koen Kooi; Chris Ball; Devicetree Discuss; Linux OMAP
> List; Linux ARM Kernel List; Linux DaVinci Kernel List; Linux Kernel Mailing List;
> Linux MMC List; Arnd Bergmann
> Subject: Re: [PATCH 0/5] AM33xx: MMC resources from DT without HWMOD
> data
>
> On Thursday 27 June 2013 07:56 PM, Joel A Fernandes wrote:
> > Hi Benoit,
> >
> > On Thu, Jun 27, 2013 at 7:03 AM, Benoit Cousson <b-cousson@ti.com> wrote:
> >> Hi Joel,
> >>
> >> On 06/26/2013 05:28 AM, Joel A Fernandes wrote:
> >>> On Tue, Jun 25, 2013 at 8:23 PM, Joel A Fernandes <joelagnel@ti.com>
> wrote:
> >>>> From: Joel A Fernandes <agnel.joel@gmail.com>
> >>>>
> >>>> This series is fixes to get MMC working on AM33XX without HWMOD
> data.
> >>>> On removal of HWMOD data, interrupt and register properties need to
> >>>> be provided for the driver to function correctly.
> >>>
> >>> Please don't pull patches authored by me in this series. I have
> >>> posted them with my wrong email address by mistake. I will correct
> >>> my email during the next repost, after a round of review. Matt's
> >>> patches in the series are still good though.
> >>
> >> In that case, please repost these in two series to avoid pulling the
> >> DTS files in the MMC tree.
> >
> > The series has been replaced with the single patch:
> > "[PATCH v2] ARM: dts: add AM33XX MMC support"
> > http://marc.info/?l=linux-omap&m=137230403815943&w=2
> >
> > Some patches were squashed and others dropped in the series resulting
> > in the single patch above. This patch should be good to apply
> Hi Joel,
>
> Before pushing mmc dts support for am335x, Can you please let me
> know if below 2 patches [1] are merged/accepted,
> IIRC these patches are needed for MMC to work properly.
>
> I suggest to hold the dts patch until slave_sg_limit gets merged
> otherwise it will cause regression in am335x mmc.
>
> [PATCH v4 1/3] dmaengine: add dma_get_slave_sg_limits()
> [PATCH v4 2/3] dma: edma: add device_slave_sg_limits() support
>
> [1]
> https://groups.google.com/forum/#!topic/linux.kernel/khTTcPtQpag
>
> >
> > Thanks,
> >
> > -Joel
> >
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 0/5] AM33xx: MMC resources from DT without HWMOD data
2013-06-27 18:08 ` Fernandes, Joel
@ 2013-06-28 15:48 ` Balaji T K
2013-06-28 16:56 ` Fernandes, Joel
0 siblings, 1 reply; 7+ messages in thread
From: Balaji T K @ 2013-06-28 15:48 UTC (permalink / raw)
To: Fernandes, Joel
Cc: benoit.cousson, lars, Tony Lindgren, Nori, Sekhar, Matt Porter,
Grant Likely, Rob Herring, Vinod Koul, Mark Brown, Russell King,
Rob Landley, Andrew Morton, Jason Kridner, Koen Kooi, Chris Ball,
Devicetree Discuss, Linux OMAP List, Linux ARM Kernel List,
Linux DaVinci Kernel List, Linux Kernel Mailing List,
Linux MMC List, Arnd Bergmann
On Thursday 27 June 2013 11:38 PM, Fernandes, Joel wrote:
> Hi Balaji,
>
<snip>
>>> Some patches were squashed and others dropped in the series resulting
>>> in the single patch above. This patch should be good to apply
>> Hi Joel,
>>
>> Before pushing mmc dts support for am335x, Can you please let me
>> know if below 2 patches [1] are merged/accepted,
>> IIRC these patches are needed for MMC to work properly.
>>
>> I suggest to hold the dts patch until slave_sg_limit gets merged
>> otherwise it will cause regression in am335x mmc.
>>
>> [PATCH v4 1/3] dmaengine: add dma_get_slave_sg_limits()
>> [PATCH v4 2/3] dma: edma: add device_slave_sg_limits() support
>
> Actually this is an independent regression and is currently worked on by Lars-Peter Clausen [1].
>
> Currently not having the MMC DTS merged is also a regression,
No, Right now mmc support is not added to am335x, so it is not really a regression
> so I suggest lets take it one
> step at a time and get the DTS merged in first. Also there will be overhead of reposting this
> DTS patch later again which will not change. What do you say?
>
If it gets enabled via dts, without slave sg limit support,
file read/write will fail, possibly corrupting file system.
> Thanks,
>
> -Joel
>
> [1] https://patchwork.kernel.org/patch/2228041/
>
^ permalink raw reply [flat|nested] 7+ messages in thread
* RE: [PATCH 0/5] AM33xx: MMC resources from DT without HWMOD data
2013-06-28 15:48 ` Balaji T K
@ 2013-06-28 16:56 ` Fernandes, Joel
0 siblings, 0 replies; 7+ messages in thread
From: Fernandes, Joel @ 2013-06-28 16:56 UTC (permalink / raw)
To: Krishnamoorthy, Balaji T
Cc: benoit.cousson, lars, Tony Lindgren, Nori, Sekhar, Matt Porter,
Grant Likely, Rob Herring, Vinod Koul, Mark Brown, Russell King,
Rob Landley, Andrew Morton, Jason Kridner, Koen Kooi, Chris Ball,
Devicetree Discuss, Linux OMAP List, Linux ARM Kernel List,
Linux DaVinci Kernel List, Linux Kernel Mailing List,
Linux MMC List, Arnd Bergmann
> -----Original Message-----
> From: Krishnamoorthy, Balaji T
> Sent: Friday, June 28, 2013 10:49 AM
> To: Fernandes, Joel
> Cc: benoit.cousson@gmail.com; lars@metafoo.de; Tony Lindgren; Nori, Sekhar;
> Matt Porter; Grant Likely; Rob Herring; Vinod Koul; Mark Brown; Russell King;
> Rob Landley; Andrew Morton; Jason Kridner; Koen Kooi; Chris Ball; Devicetree
> Discuss; Linux OMAP List; Linux ARM Kernel List; Linux DaVinci Kernel List;
> Linux Kernel Mailing List; Linux MMC List; Arnd Bergmann
> Subject: Re: [PATCH 0/5] AM33xx: MMC resources from DT without HWMOD
> data
>
> On Thursday 27 June 2013 11:38 PM, Fernandes, Joel wrote:
> > Hi Balaji,
> >
>
> <snip>
>
> >>> Some patches were squashed and others dropped in the series
> >>> resulting in the single patch above. This patch should be good to
> >>> apply
> >> Hi Joel,
> >>
> >> Before pushing mmc dts support for am335x, Can you please let me know
> >> if below 2 patches [1] are merged/accepted, IIRC these patches are
> >> needed for MMC to work properly.
> >>
> >> I suggest to hold the dts patch until slave_sg_limit gets merged
> >> otherwise it will cause regression in am335x mmc.
> >>
> >> [PATCH v4 1/3] dmaengine: add dma_get_slave_sg_limits() [PATCH v4
> >> 2/3] dma: edma: add device_slave_sg_limits() support
> >
> > Actually this is an independent regression and is currently worked on by Lars-
> Peter Clausen [1].
> >
> > Currently not having the MMC DTS merged is also a regression,
>
> No, Right now mmc support is not added to am335x, so it is not really a
> regression.
>
> > so I suggest lets take it one
> > step at a time and get the DTS merged in first. Also there will be
> > overhead of reposting this DTS patch later again which will not change. What
> do you say?
> >
>
> If it gets enabled via dts, without slave sg limit support, file read/write will fail,
> possibly corrupting file system.
I agree with this. Thank you.
Tony, Benoit, I guess let's just not apply the DTS yet till the slave-sg limits support
is merged as well. Sorry for the noise. Thanks.
-Joel
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2013-06-28 16:56 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-06-26 1:23 [PATCH 0/5] AM33xx: MMC resources from DT without HWMOD data Joel A Fernandes
[not found] ` <CAD=GYpbdVVrh-tiqOcbHi3mkWRJUBdx0EsEZEFVVbgFgcCWgsw@mail.gmail.com>
2013-06-27 12:03 ` Benoit Cousson
[not found] ` <CAD=GYpa9LQ1fV6=O9B65sJURVAigsgXCPLWav8cHXTuGQXPc9A@mail.gmail.com>
2013-06-27 16:26 ` Balaji T K
2013-06-27 18:08 ` Fernandes, Joel
2013-06-28 15:48 ` Balaji T K
2013-06-28 16:56 ` Fernandes, Joel
2013-06-27 18:10 ` Fernandes, Joel
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).