[PATCH v3 0/8] ARM: NOMMU: MPU updates

[Russell King - ARM Linux]

On Tue, Oct 17, 2017 at 03:13:36PM +0100, Vladimir Murzin wrote:
> On 17/10/17 15:05, Benjamin Gaignard wrote:
> > 2017-09-25 11:05 GMT+02:00 Vladimir Murzin <vladimir.murzin at arm.com>:
> >> Hi,
> >>
> >> This is an update of the existent MPU code which consist set of
> >> clean-ups and enhancements. The most significant changes are in the
> >> way how MPU settings sync-up for secondaries (PATCH 3/8), more
> >> flexible layout for memory (PATCH 7/8) and support for XIP (PATCH
> >> 8/8). Since there is no in-tree user of MPU, support for M-class has
> >> been added (PATCH 6/8).
> >>
> >> There is also practical benefit of patch set:
> >>
> >> "These MPU patches introduce a nice feature that it simplifies the
> >> bootloader: do not need to setup the MPU, for example for coherent DMA
> >> region, which is described in device-tree too." András Szemző
> >>
> >> Changelog:
> >>            v2 -> v3
> >>               - fixed support for XIP (PATCH 8/8)
> >>               - rebased on 4.14-rc2
> >>
> >>            v1 -> v2
> >>               - support for XIP (PATCH 8/8)
> >>               - rebased on 4.13-rc1
> >>
> >>         RFC v2 -> v1
> >>                - dropped RFC tag
> >>                - Tested-by from András
> >>                - rebased on 4.12-rc5
> >>
> >>            RFC -> RFC v2
> >>                - fixed MPU enable for v7m (PATCH 6/7)
> >>
> >> Thanks!
> > 
> > for the whole serie on stm32f4 and stm32f7:
> > 
> > Tested-by: Benjamin Gaignard <benjamin.gaignard at linaro.org>
> 
> Thanks for giving it a try!
> 
> The series is already in Russell's patch system, so I dunno
> how to add your Tested-by, maybe Russell can made an update
> when he pull patches in his tree...

Applied, along with the tested-by, and with the fixup patch folded.

However, I notice that with "Szemz■ András" name, the ■ character has
been replaced by a "?" somewhere along the lines (and my VT console
can't even display it, so (a) I've no idea what character it's
supposed to be, and (b) I can't cut'n'paste it either - as illustrated
by my pasting above making it come out as a rectangular block.)

The problem I have is that if I push these patches out, as soon as I
do, I can't fix this up, because it's the "devel-stable" branch - and
the promise I make to everyone is that branch is never rebased (which
is why it has the -stable suffix.)  Once commits are published in that
branch, they're set in stone.

>From what I can tell, the patches were emailed to the patch system,
which means it would've gone through perl's Unicode::Map8 - internally
the patch system uses UTF-8 to store the summary and description,
while preserving the patch as a byte stream.  Your emails to the list
claim to be UTF-8, so it should've been a no-op.  So, don't know what
has happened, and I can't fix it at the moment, not even in the actual
commits.

So, I won't be pushing this out, nor will I be marking the patches as
applied in the patch system until I have some kind of solution.

-- 
RMK's Patch system: http://www.armlinux.org.uk/developer/patches/
FTTC broadband for 0.8mile line in suburbia: sync at 8.8Mbps down 630kbps up
According to speedtest.net: 8.21Mbps down 510kbps up