Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

modifie the 4052-NAND-add-easybox904-bbt.patch and add "customized-sa… #26

Open
wants to merge 1 commit into
base: master-lede
Choose a base branch
from

Conversation

Plonkbong
Copy link
Contributor

…msung-K9F4G08U0x;" to DTS

This patch was breaking all other devices with NAND-flash in past, because the different BBT-patern-matching will disable/enable by #define CUSTOMIZED_BBT 1
Now it ask first for the customized-samsung-K9F4G08U0x field from devicetree:
...
nand@1 {
customized-samsung-K9F4G08U0x;
...

Signed-off-by: Plonkbong [email protected]

Thanks for your contribution to OpenWrt!

To help keep the codebase consistent and readable,
and to help people review your contribution,
we ask you to follow the rules you find in the wiki at this link
https://openwrt.org/submitting-patches

Please remove this message before posting the pull request.

This patch was breaking all other devices with NAND-flash in past,
because the different BBT-patern-matching will disable/enable by
a define makro only (#define CUSTOMIZED_BBT 1)
Now it ask first for the customized-samsung-K9F4G08U0x
field from devicetree:
...
nand@1 {
customized-samsung-K9F4G08U0x;
...

Signed-off-by: Jonas Albrecht <[email protected]>
@Plonkbong
Copy link
Contributor Author

Hi i makes my commit message openwrt compatible.

For testing i add a lot of comments
4052-NAND-add-easybox904-bbt-byDTS-with-a-lot-of-comments.patch = https://pastebin.com/5frveXVk
Here the resulting nand_bbt.c = https://pastebin.com/sFVZCEzi

The dmesg output looks like it should be
full = https://pastebin.com/YJ899XSE
filtered = https://pastebin.com/C8wxth7x

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant