Re: [OpenWrt-Devel] [PATCH] lantiq: remove read-only flag on two partitions on BTHOMEHUBV3A
On Mon, Apr 25, 2016 at 5:38 PM, Ben Mulvihill wrote: >> @Ben: one probably has to enable caldata swapping in the .dts. > > I saw your comment on openwrt.ebilan.co.uk that the "ath,eep-swap;" > flag should be added to the dts in order to use unmodified calibration > data, but actually this does not appear to be the case. I have just > installed a board using unmodified calibration data and unmodified dts > (apart from read-only flag) and it works fine. I then tried adding > "ath,eep-swap;" just to see what would happen and it no longer worked. probably because the values are swapped twice then, the first time using the "wrong" 32bit swap method which results in the same problem that existed before those ath9k patches. great to hear that it's working for you :-) ___ openwrt-devel mailing list openwrt-devel@lists.openwrt.org https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel
Re: [OpenWrt-Devel] [PATCH] lantiq: remove read-only flag on two partitions on BTHOMEHUBV3A
On 25/04/2016 17:38, Ben Mulvihill wrote: > On Mon, 2016-04-25 at 16:29 +0200, Martin Blumenstingl wrote: >> On Mon, Apr 25, 2016 at 3:00 PM, John Crispin wrote: >>> is this not a bit risky ? normally we try to keep them read-only so that >>> users cannot accidentally kill them. is the calib data normally not >>> located in the art partition or why do you need to copy it after flashing ? > > @John: As supplied from the manufacturer, the caldata is in a file > in a jffs partition, which we overwrite anyway during installation. > The art partition is specifically for openwrt. Up till now we have > been copying the calibration data into it manually from the uboot > command line. The trouble is that the only way to access the uboot > command line involves some very fiddly soldering on minuscule pads. > I am working on a way to install openwrt without a serial console, > but to do it the user will need to be able to write to the art > partition from openwrt. > fine, i'll merge it in that case John >> I think the caldata partition is made writable because of some bug in >> ath9k which required HH3A users to manually modify the ath9k caldata. >> This is not necessary anymore since my patch (+ Felix' fix on top of >> it) is now upstream: [0] > > @Martin. No, as you say, thanks to you and Felix, manual modification > of the calibration data is no longer necessary. Brilliant! See above > for explanation of why I am suggesting this change. > >> >> @Ben: one probably has to enable caldata swapping in the .dts. > > I saw your comment on openwrt.ebilan.co.uk that the "ath,eep-swap;" > flag should be added to the dts in order to use unmodified calibration > data, but actually this does not appear to be the case. I have just > installed a board using unmodified calibration data and unmodified dts > (apart from read-only flag) and it works fine. I then tried adding > "ath,eep-swap;" just to see what would happen and it no longer worked. > > I haven't investigated why "ath,eep-swap;" is no longer needed. Maybe > the result of Felix's latest changes? In any case so much the better! > >> Then we have to figure out what to do with existing HH3A installations. >> > > We'll have to do that anyway. > >> Martin >> >> [0] >> https://github.com/torvalds/linux/commit/6fa658fd5ab26a769ca7df7bfdd53c212a0ba852 > > Thanks, > > Ben > > > > ___ openwrt-devel mailing list openwrt-devel@lists.openwrt.org https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel
Re: [OpenWrt-Devel] [PATCH] lantiq: remove read-only flag on two partitions on BTHOMEHUBV3A
On Mon, 2016-04-25 at 16:29 +0200, Martin Blumenstingl wrote: > On Mon, Apr 25, 2016 at 3:00 PM, John Crispin wrote: > > is this not a bit risky ? normally we try to keep them read-only so that > > users cannot accidentally kill them. is the calib data normally not > > located in the art partition or why do you need to copy it after flashing ? @John: As supplied from the manufacturer, the caldata is in a file in a jffs partition, which we overwrite anyway during installation. The art partition is specifically for openwrt. Up till now we have been copying the calibration data into it manually from the uboot command line. The trouble is that the only way to access the uboot command line involves some very fiddly soldering on minuscule pads. I am working on a way to install openwrt without a serial console, but to do it the user will need to be able to write to the art partition from openwrt. > I think the caldata partition is made writable because of some bug in > ath9k which required HH3A users to manually modify the ath9k caldata. > This is not necessary anymore since my patch (+ Felix' fix on top of > it) is now upstream: [0] @Martin. No, as you say, thanks to you and Felix, manual modification of the calibration data is no longer necessary. Brilliant! See above for explanation of why I am suggesting this change. > > @Ben: one probably has to enable caldata swapping in the .dts. I saw your comment on openwrt.ebilan.co.uk that the "ath,eep-swap;" flag should be added to the dts in order to use unmodified calibration data, but actually this does not appear to be the case. I have just installed a board using unmodified calibration data and unmodified dts (apart from read-only flag) and it works fine. I then tried adding "ath,eep-swap;" just to see what would happen and it no longer worked. I haven't investigated why "ath,eep-swap;" is no longer needed. Maybe the result of Felix's latest changes? In any case so much the better! > Then we have to figure out what to do with existing HH3A installations. > We'll have to do that anyway. > Martin > > [0] > https://github.com/torvalds/linux/commit/6fa658fd5ab26a769ca7df7bfdd53c212a0ba852 Thanks, Ben ___ openwrt-devel mailing list openwrt-devel@lists.openwrt.org https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel
Re: [OpenWrt-Devel] [PATCH] lantiq: remove read-only flag on two partitions on BTHOMEHUBV3A
On Mon, Apr 25, 2016 at 3:00 PM, John Crispin wrote: > is this not a bit risky ? normally we try to keep them read-only so that > users cannot accidentally kill them. is the calib data normally not > located in the art partition or why do you need to copy it after flashing ? I think the caldata partition is made writable because of some bug in ath9k which required HH3A users to manually modify the ath9k caldata. This is not necessary anymore since my patch (+ Felix' fix on top of it) is now upstream: [0] @Ben: one probably has to enable caldata swapping in the .dts. Then we have to figure out what to do with existing HH3A installations. Martin [0] https://github.com/torvalds/linux/commit/6fa658fd5ab26a769ca7df7bfdd53c212a0ba852 ___ openwrt-devel mailing list openwrt-devel@lists.openwrt.org https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel
Re: [OpenWrt-Devel] [PATCH] lantiq: remove read-only flag on two partitions on BTHOMEHUBV3A
On 24/04/2016 21:29, Ben Mulvihill wrote: > Remove read-only flag on two partitions on BTHOMEHUBV3A: > uboot-config - otherwise fw_setenv command cannot be used. > ath9k-cal- so that ath9k calibration data can be copied > to the partition on a newly installed board. > > Signed-off-by: Ben Mulvihill > --- > --- a/target/linux/lantiq/dts/BTHOMEHUBV3A.dts2016-04-24 > 17:30:44.117072962 +0200 > +++ b/target/linux/lantiq/dts/BTHOMEHUBV3A.dts2016-04-24 > 17:30:08.483732762 +0200 > @@ -61,12 +61,10 @@ > partition@64000 { > label = "uboot-config"; > reg = <0x64000 0x004000>; > - read-only; > }; > ath9k_cal: partition@68000 { > label = "art-copy"; > reg = <0x68000 0x004000>; > - read-only; > }; is this not a bit risky ? normally we try to keep them read-only so that users cannot accidentally kill them. is the calib data normally not located in the art partition or why do you need to copy it after flashing ? > partition@6c000 { > label = "kernel"; > ___ > openwrt-devel mailing list > openwrt-devel@lists.openwrt.org > https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel > ___ openwrt-devel mailing list openwrt-devel@lists.openwrt.org https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel
[OpenWrt-Devel] [PATCH] lantiq: remove read-only flag on two partitions on BTHOMEHUBV3A
Remove read-only flag on two partitions on BTHOMEHUBV3A: uboot-config - otherwise fw_setenv command cannot be used. ath9k-cal- so that ath9k calibration data can be copied to the partition on a newly installed board. Signed-off-by: Ben Mulvihill --- --- a/target/linux/lantiq/dts/BTHOMEHUBV3A.dts 2016-04-24 17:30:44.117072962 +0200 +++ b/target/linux/lantiq/dts/BTHOMEHUBV3A.dts 2016-04-24 17:30:08.483732762 +0200 @@ -61,12 +61,10 @@ partition@64000 { label = "uboot-config"; reg = <0x64000 0x004000>; - read-only; }; ath9k_cal: partition@68000 { label = "art-copy"; reg = <0x68000 0x004000>; - read-only; }; partition@6c000 { label = "kernel"; ___ openwrt-devel mailing list openwrt-devel@lists.openwrt.org https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel