Re: [PATCH v10 10/10] mtd: maps: gpio-addr-flash: Add support for device-tree devices

[Ricardo Ribalda Delgado]

Hi again Boris


On Fri, Oct 5, 2018 at 2:10 PM Boris Brezillon
<boris.brezillon@xxxxxxxxxxx> wrote:
>
> On Fri, 5 Oct 2018 14:04:52 +0200
> Ricardo Ribalda Delgado <ricardo.ribalda@xxxxxxxxx> wrote:
>
> > Hi Boris
> > On Fri, Oct 5, 2018 at 12:12 PM Boris Brezillon
> > <boris.brezillon@xxxxxxxxxxx> wrote:
> > >
> > > On Fri, 5 Oct 2018 11:54:18 +0200
> > > Ricardo Ribalda Delgado <ricardo.ribalda@xxxxxxxxx> wrote:
> > >
> > > > Hi Boris
> > > >
> > > > Just seen that  you already did the rebase at
> > > > https://github.com/bbrezillon/linux-0day/commits/mtd/physmap-cleanup
> > > >
> > > > Thanks for that.
> > > >
> > > > I am about to test it in real hw (unless you want me wait)
> > >
> > > Sure, go ahead and test it.
> > >
> > > Thanks,
> > >
> > > Boris
> > I had to change this on your patchset to have it working on hw:
> > https://pastebin.com/78A7yhJ9
> >
> > If you send the patchset to the mailing list I can review it patch by patch.
> >
> > Also
> > mtd: maps: Prepare merging of physmap and physmap_of
> >
> > I do not think that can be bisected. (Not sure, I have to test it)
>
> Okay, I'll have a look.
>
> >
> > I add the diff to the mail, but gmail will probably scramble the
> > lines(yes I know I have to use other mail client)
>
> The diff looks good, I'll fix that an send a push a new version.

Also fix on physmap_flash_remove

physmap_data->exit(dev); must be called BEFORE
map_destroy(info->mtds[i]);

otherwise OOPS

Cheers!

[  162.700421] Unable to handle kernel paging request at virtual
address 7974696e6966a6
[  162.705461] Mem abort info:
[  162.713171]   Exception class = DABT (current EL), IL = 32 bits
[  162.715678]   SET = 0, FnV = 0
[  162.721588]   EA = 0, S1PTW = 0
[  162.724703] Data abort info:
[  162.727767]   ISV = 0, ISS = 0x00000004
[  162.730879]   CM = 0, WnR = 0
[  162.734436] [007974696e6966a6] address between user and kernel address ranges
[  162.737567] Internal error: Oops: 96000004 [#1] PREEMPT SMP
[  162.744674] Modules linked in: qtec_m43 qtec_mem qt5023_video
qtec_white qtec_cmosis arc4 snd_soc_hdmi_codec ath10k_pci ath10k_core
venus_dec venus_enc ath mac80211 q6asm_dai cfi_cmdset_0002 q6routing
q6afe_dai cfi_probe q6adm cfi_util q6asm gen_probe q6afe q6dsp_common
q6core physmap gpio_xilinx qtec_pcie qtec_clk apr qrtr_smd rpmsg_char
imx214 ad5820 v4l2_common joydev cfg80211 msm venus_core hci_uart
qt5023 qcom_camss btqca v4l2_mem2mem drm_kms_helper videobuf2_dma_sg
v4l2_fwnode videobuf2_memops videobuf2_v4l2 bluetooth drm
videobuf2_core videodev ecdh_generic i2c_qcom_cci media rtc_pm8xxx
leds_qcom_lpg qrtr crc32_ce qcom_adsp_pil crct10dif_ce qcom_common
snd_soc_apq8096 qcom_glink_smem qcom_sysmon remoteproc qmi_helpers
rmtfs_mem mdt_loader sch_fq_codel
[  162.795188] CPU: 2 PID: 1 Comm: systemd-shutdow Not tainted
4.14.53-qtec-linaro #34
[  162.817417] Hardware name: Qtechnology QT5506 (DT)
[  162.824793] task: ffff8000baac8000 task.stack: ffff000008058000
[  162.829671] PC is at cfi_amdstd_reset+0xb4/0x188 [cfi_cmdset_0002]
[  162.835474] LR is at cfi_amdstd_reset+0xb4/0x188 [cfi_cmdset_0002]
[  162.841721] pc : [<ffff00000107fe44>] lr : [<ffff00000107fe44>]
pstate: 40000145
[  162.847885] sp : ffff00000805bc80
[  162.855433] x29: ffff00000805bc80 x28: ffff8000baac8000
[  162.858648] x27: ffff000008b51000 x26: 000000000000008e
[  162.864030] x25: 0000000000000124 x24: 0000000000000014
[  162.869324] x23: 0000000000000000 x22: 0000000000000000
[  162.874619] x21: 5f7974696e696666 x20: ffff8000b3178a18
[  162.879915] x19: ffff8000b5215800 x18: 0000000000000000
[  162.885209] x17: 0000ffffa46d1768 x16: ffff0000080f9960
[  162.890504] x15: 0000000000000010 x14: ffffffffffffffff
[  162.895800] x13: ffff0000896355e7 x12: ffff0000096355ef
[  162.901094] x11: ffff00000944a000 x10: ffff00000805b9b0
[  162.906389] x9 : 0000000000000039 x8 : 000000000000000d
[  162.911686] x7 : ffff00000805ba2c x6 : 0000000000000491
[  162.916980] x5 : 0000000000000000 x4 : 0000000000000000
[  162.922275] x3 : ffffffffffffffff x2 : ffff00000944a278
[  162.927570] x1 : ffff8000baac8000 x0 : 0000000000000028
[  162.932867] Process systemd-shutdow (pid: 1, stack limit =
0xffff000008058000)
[  162.938164] Call trace:
[  162.945191] Exception stack(0xffff00000805bb40 to 0xffff00000805bc80)
[  162.947538] bb40: 0000000000000028 ffff8000baac8000
ffff00000944a278 ffffffffffffffff
[  162.954136] bb60: 0000000000000000 0000000000000000
0000000000000491 ffff00000805ba2c
[  162.961949] bb80: 000000000000000d 0000000000000039
ffff00000805b9b0 ffff00000944a000
[  162.969762] bba0: ffff0000096355ef ffff0000896355e7
ffffffffffffffff 0000000000000010
[  162.977574] bbc0: ffff0000080f9960 0000ffffa46d1768
0000000000000000 ffff8000b5215800
[  162.985386] bbe0: ffff8000b3178a18 5f7974696e696666
0000000000000000 0000000000000000
[  162.993200] bc00: 0000000000000014 0000000000000124
000000000000008e ffff000008b51000
[  163.001011] bc20: ffff8000baac8000 ffff00000805bc80
ffff00000107fe44 ffff00000805bc80
[  163.008823] bc40: ffff00000107fe44 0000000040000145
0000000000000b5b ffffffffffffffff
[  163.016637] bc60: 0000ffffffffffff 0000000000000000
ffff00000805bc80 ffff00000107fe44
[  163.024454] [<ffff00000107fe44>] cfi_amdstd_reset+0xb4/0x188
[cfi_cmdset_0002]
[  163.032264] [<ffff00000107ff78>] cfi_amdstd_reboot+0x10/0x98
[cfi_cmdset_0002]
[  163.039386] [<ffff0000080f7d0c>] notifier_call_chain+0x54/0x90
[  163.046583] [<ffff0000080f85d4>] blocking_notifier_call_chain+0x54/0x78
[  163.052399] [<ffff0000080f96ac>] kernel_restart_prepare+0x1c/0x40
[  163.058910] [<ffff0000080f97a4>] kernel_restart+0x14/0x60
[  163.065158] [<ffff0000080f9b68>] SyS_reboot+0x208/0x230


>
> Thanks,
>
> Boris



-- 
Ricardo Ribalda